remove debugging junk

.travis.yml

@@ -13,13 +13,9 @@ before_install:
 - sudo apt-get update
 - sudo apt-get install python3
 - npm install --global http-server
-#- python3 start_test_http_server.py &
 install:
 - npm install
+- npm build
 script:
 #- npm test
 - npm run coveralls
-#after_success:
-#- python3 stop_test_http_server.py 
-#after_failure:
-#- python3 stop_test_http_server.py 

html/entry.js

@@ -8,7 +8,7 @@ if (!global.jQuery) {
 
 require("../dist/index");
 
-//jQuery(function(){
+jQuery(function(){
   var $ = jQuery;
   var y = $("#container");
   var z = $('<div>JQuery is working?</div>').appendTo(y);
@@ -18,10 +18,11 @@ require("../dist/index");
     html: "plugin is working",
     italic: true
   });
-  //window.js_loaded = true;
-  ready_for_tests();
-//});
+
+  // make sure jquery is in the global environment so we can use it in tests
+  window.jQuery = jQuery;
+
+  // Flag that we are ready for testing (observed by end-to-end tests).
+  window.jQuery.ready_for_tests = true;
+});
 
-// put jquery in the global environment so we can use it in tests
-window.jQuery = jQuery;
-window.jQuery.ready_for_tests = true;

jest-puppeteer.config.js

@@ -8,8 +8,4 @@ module.exports = {
         timeout: 3000000, //  5 minute timeout
         args: ["--no-sandbox"],   // don't use a sandbox
     },
-    //server: {
-    //  command: 'node server.js',
-    //  port: 4444,
-    //},
 }

jest/globalTeardown.js

@@ -2,6 +2,8 @@ const { teardown: teardownDevServer } = require('jest-dev-server');
 const { teardown: teardownPuppeteer } = require('jest-environment-puppeteer');
 
 module.exports = async function globalTeardown(globalConfig){
+
+  // shut down the testing http server.
   await teardownDevServer();
 
   await teardownPuppeteer(globalConfig);

tests/headless.test.js

@@ -49,10 +49,11 @@ describe("headless browser tests", async () => {
         await page.goto(url, {waitUntil: 'networkidle2'});
         //await jestPuppeteer.debug();
         // sleep a second to let the page execute javascript
-        await sleep(1000);
+        //await sleep(1000);
         //check_truthy("window.test_function");
         //await page.waitForFunction(() => !!(window.test_function));
-        await page.waitForFunction(async () => !!(window.jQuery));
+        // Wait for the test function to become available.
+        await page.waitForFunction(async () => !!(window.test_function));
         var content = await page.evaluate(async () => window.test_function());
         //console.log("function content is: " + content);
         var expected_content = "hi there!";
@@ -68,14 +69,14 @@ describe("headless browser tests", async () => {
         await page.goto(url, {waitUntil: 'networkidle2'});
         //await jestPuppeteer.debug();
         // sleep a second to let the page execute javascript
-        await sleep(1000);
+        //await sleep(1000);
         //check_truthy("window");
-        console.log(" js_loaded before: " + await page.evaluate(async () => !!(window.js_loaded)));
-        console.log(" ... now waiting for window.jQuery ...")
+        //console.log(" js_loaded before: " + await page.evaluate(async () => !!(window.js_loaded)));
+        //console.log(" ... now waiting for window.jQuery ...")
         await page.waitForFunction(async () => !!(window.jQuery));
-        console.log(" ... now awaiting window.jQuery.ready_for_tests")
+        //console.log(" ... now awaiting window.jQuery.ready_for_tests")
         await page.waitForFunction(async () => !!(window.jQuery.ready_for_tests));
-        //await page.waitForFunction(async () => !!(window.js_loaded));
+        // Validate that the plugin code executed.
         var content = await page.evaluate(async () => window.jQuery('#target').text());
         //console.log("target content is: " + content);
         var expected_content = "plugin is working";