@suffolklitlab/alkiln 5.16.1-always-html-3 → 5.16.1-always-html-4

package/lib/scope.js

@@ -3116,7 +3116,7 @@ module.exports = {
 
         let err_msg = `Error occurred when ALKiln tried to go to "${ login_url }".`
         if ( error.name === `TimeoutError` ) {
-          console.log(`🖊️ 110`);
+          // console.log(`🖊️ 110`);
           let non_reload_report_msg = `It took too long to load "${ login_url }"`;
           await scope.handle_page_timeout_error( scope, { non_reload_report_data: {
             code: `ALK0159`,
@@ -3124,7 +3124,7 @@ module.exports = {
           }, error });
         } else {
           // Throw any non-timeout error
-          console.log(`🖊️ 111`);
+          // console.log(`🖊️ 111`);
           await scope.take_a_screenshot(scope, {path:`./_alkiln_temp/Temp-4.jpg`});
           reports.addToReport( scope, { type: `error`, code: `ALK0160`, value: err_msg });
           throw error;
@@ -3166,7 +3166,7 @@ module.exports = {
           scope.scenarios.get( scope.scenario_id ).api_keys.push( api_key );
         }
       }
-      console.log(`🖊️ 112`);
+      // console.log(`🖊️ 112`);
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-4.jpg`});
 
       expect( email, email_msg ).to.not.equal( undefined );
@@ -3177,14 +3177,14 @@ module.exports = {
       await scope.page.type( `#password`, password );
       let elem = await scope.page.$( `button[type="submit"]` );
       await scope.guard_against_missing_tap_element(scope, { elem });
-      console.log(`🖊️ 113`);
+      // console.log(`🖊️ 113`);
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-5.jpg`});
 
       // Submit and see what happens
       let winner = await scope.steps.race_sign_in_navigation( scope, { elem, login_url });
-      console.log(`🖊️ 113.5 ${ winner }`);
+      // console.log(`🖊️ 113.5 ${ winner }`);
       // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-6.jpg`});
-      // console.log(`🖊️ 113.6 ${ winner }`);
+      console.log(`🖊️ 113.6 ${ winner }`);
       // Add the result to the report and possibly throw errors
       if ( winner[0] === `success` ) {
         reports.addToReport( scope, {
@@ -3197,7 +3197,7 @@ module.exports = {
           type: `error`, code: `ALK0209`,
           value: `Failed to sign into ${ login_url }. Make sure you followed the instructions at https://assemblyline.suffolklitlab.org/docs/alkiln/writing/#sign-in.`
         });
-        console.log(`🍉 ${ error_msg }`);
+        // console.log(`🍉 ${ error_msg }`);
         throw new Error( error_msg );
       } else if ( winner[0] === `error` ) {
         let error_msg = reports.addToReport( scope, {
@@ -3206,7 +3206,7 @@ module.exports = {
         });
         throw new Error( error_msg );
       }
-      console.log(`🖊️ 114`);
+      // console.log(`🖊️ 114`);
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-7.jpg`});
       if (error_msg) {
         return [ error_msg ];
@@ -3222,15 +3222,15 @@ module.exports = {
 
       // After everything, clean up incomplete promises
       const controller = new AbortController;
-      console.log(`⏳ 200`);
+      // console.log(`⏳ 200`);
 
       // Redirect
       let redirect_promise = scope.page.waitForResponse(async ( response ) => {
-        console.log(`⏳ 300`);
+        // console.log(`⏳ 300`);
         return response.status() === 302;
       }, { signal: controller.signal })
       .catch(async ( catch_info ) => {
-        console.log(`⏳ 201`);
+        // console.log(`⏳ 201`);
         // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp2-1.jpg`});
         log.debug({ code: `ALK0204`, context: `nav`, },
           `302 sign-in wait catch error:`, catch_info.name, catch_info,
@@ -3239,12 +3239,12 @@ module.exports = {
 
       // Invalid credentials, no navigation, no sign in
       let wrong_sign_in_promise = scope.page.waitForResponse(async ( response ) => {
-        console.log(`⏳ 301`);
+        // console.log(`⏳ 301`);
         return response.status() >= 200 && response.status() < 300 && response.url().includes(`/sign-in`);
       }, { signal: controller.signal })
       .catch(async ( catch_info ) => {
         // errors?
-        console.log(`⏳ 202`);
+        // console.log(`⏳ 202`);
         // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp2-2.jpg`});
         log.debug({ code: `ALK0205`, context: `nav`, },
           `200s sign-in wait catch error:`, catch_info.name, catch_info,
@@ -3253,11 +3253,11 @@ module.exports = {
 
       // System error
       let error_promise = scope.page.waitForResponse(async ( response ) => {
-        console.log(`⏳ 302`);
+        // console.log(`⏳ 302`);
         return response.status() >= 500 && response.status() < 600;
       }, { signal: controller.signal })
       .catch(async ( catch_info ) => {
-        console.log(`⏳ 203`);
+        // console.log(`⏳ 203`);
         await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp2-3.jpg`});
         log.debug({ code: `ALK0206`, context: `nav`, },
           `500s sign-in wait catch error:`, catch_info.name, catch_info,
@@ -3269,29 +3269,29 @@ module.exports = {
       let click_promise = elem.click();  // MUST complete
       let nav_promise = scope.nav_race(scope, {}).result;
       let pic_promise = true; // scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp2-10.jpg`});
-      console.log(`⏳ 204`);
+      // console.log(`⏳ 204`);
       const winner = await Promise.race([
         Promise.all([ `success`, click_promise, pic_promise, redirect_promise ]),
         Promise.all([ `failure`, click_promise, pic_promise, wrong_sign_in_promise ]),
         Promise.all([ `error`, click_promise, pic_promise, error_promise ]),
       ]).catch(async ( error ) => {
-        console.log(`⏳ 205`);
+        // console.log(`⏳ 205`);
         // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp2-4.jpg`});
         let error_msg = reports.addToReport( scope, {
           type: `error`, code: `ALK0207`,
           value: `Unknown error waiting for results during sign in at ${ login_url }.`
         });
-        console.log(`⏳ 205.1`, error);
+        // console.log(`⏳ 205.1`, error);
         throw error;
       });
 
-      console.log(`⏳ 206`);
+      // console.log(`⏳ 206`);
       // // await waitForTimeout(500);
       // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp2-5.jpg`});
 
       // Clean up unresolved promises
       controller.abort();
-      console.log(`⏳ 207`);
+      // console.log(`⏳ 207`);
       await nav_promise;
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp3-1.jpg`});
 

package/lib/steps.js

@@ -116,7 +116,7 @@ BeforeAll(async function() {
 
 Before( beforeScenario );
 async function beforeScenario( scenario ) {
-  console.log('👁️ 111 start beforeScenario');
+  // console.log('👁️ 111 start beforeScenario');
   // Start the running "progress bar" for the Scenario
   log.stdout({}, `\nScenario: ${ scenario.pickle.name }: `);
 
@@ -126,20 +126,20 @@ async function beforeScenario( scenario ) {
       // Will only run in the Playground outside of a sandbox. TODO: There's a
       // better way to do this, though it's more complicated. See comments in
       // https://github.com/SuffolkLITLab/ALKiln/issues/661
-      console.log('👁️ 112 no browser, await new local browser');
+      // console.log('👁️ 112 no browser, await new local browser');
       scope.browser = await scope.driver.launch({ args: ['--no-sandbox'] });
     } else {
-      console.log('👁️ 113 no browser, await new remote browser');
+      // console.log('👁️ 113 no browser, await new remote browser');
       scope.browser = await scope.driver.launch({ headless: !session_vars.get_debug(), devtools: session_vars.get_debug() });
     }
   }
-  console.log('👁️ 114 we have ensured browser exists, awaiting pages');
+  // console.log('👁️ 114 we have ensured browser exists, awaiting pages');
   // Clean up all previously existing pages
   for (const page of await scope.browser.pages()) {
-    console.log('👁️ 115 awaiting closing page');
+    // console.log('👁️ 115 awaiting closing page');
     await page.close();
   }
-  console.log('👁️ 116 awaiting opening a new page');
+  // console.log('👁️ 116 awaiting opening a new page');
   // Make a new page
   scope.page = await scope.browser.newPage()
 
@@ -153,10 +153,10 @@ async function beforeScenario( scenario ) {
   scope.server_reload_promise = null;
 
   reports.addReportHeading(scope, {scenario});
-  console.log('👁️ 117 awaiting getting safe scenario name');
+  // console.log('👁️ 117 awaiting getting safe scenario name');
   // Make folder for this Scenario in the all-tests artifacts folder
   scope.base_filename = await scope.getSafeScenarioBaseFilename(scope, {scenario});
-  console.log('👁️ 118 got safe scenario name');
+  // console.log('👁️ 118 got safe scenario name');
   // Add a date for uniqueness in case dev has accidentally copied a Scenario description
   let date = Date.now();
   scope.paths.scenario = `${ scope.paths.artifacts }/${ scope.base_filename }-${ date }`;
@@ -316,7 +316,7 @@ Given(
   // Couldn't test the timeout for tapping the button because there's not
   // enough of a pause between initial navigation and pressing button.
   foo = scope.page
-  console.log(`⭐️ ${ JSON.stringify(scope.page, null, 2) }`)
+  // console.log(`⭐️ ${ JSON.stringify(scope.page, null, 2) }`)
   let result = await scope.steps.sign_in( scope, {
     email_secret_name: email,
     password_secret_name: password,
@@ -324,7 +324,7 @@ Given(
   });
 
 
-  console.log(`🖊️ 115`);
+  // console.log(`🖊️ 115`);
   // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-8.jpg`});
 
   // if ( !result.ok && result.errors.length > 0 ) {
@@ -1344,7 +1344,7 @@ Then(/I fail to delete (\d) detected interview(?:s)? and get no error __internal
 AfterStep(async function({ result }) {
   // TODO: Abstract this
   // await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-13.jpg`});
-  console.log(`🛑 100`);
+  // console.log(`🛑 100`);
   if ( result.status === `PASSED` ) { log.stdout({ records_only: true }, `.`); }
   else if ( result.status === `FAILED` ) { log.stdout({ records_only: true }, `F`); }
   else if ( result.status === `UNDEFINED` ) { log.stdout({ records_only: true }, `U`); }
@@ -1352,12 +1352,12 @@ AfterStep(async function({ result }) {
   else if ( result.status === `PENDING` ) { log.stdout({ records_only: true }, `P`); }
 
   reports.outdent();
-  console.log(`🛑 101`);
+  // console.log(`🛑 101`);
 });
 
 After(afterScenario);
 async function afterScenario(scenario) {
-  console.log(`👀 1 ${ foo == scope.page } ${ JSON.stringify(scope.page, null, 2) }`);
+  // console.log(`👀 1 ${ foo == scope.page } ${ JSON.stringify(scope.page, null, 2) }`);
   await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-30.jpg`});
   // Log errors
   if ( scenario.result.message ) {
@@ -1377,7 +1377,7 @@ async function afterScenario(scenario) {
       value: `Accessibility standards tests failed. See information above.`
     });
   }
-  console.log(`👀 2`);
+  // console.log(`👀 2`);
   await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-9.jpg`});
 
   if ( scope.failed_pdf_compares.length > 0) {
@@ -1400,7 +1400,7 @@ async function afterScenario(scenario) {
     );
 
     if ( scope.page ) {
-      console.log(`👀 3`);
+      // console.log(`👀 3`);
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-10.jpg`});
 
       if ( !!scope.disable_error_screenshot ) {
@@ -1413,7 +1413,7 @@ async function afterScenario(scenario) {
           value: `For security, ALKiln will avoid creating a picture of the page for this error. It's possible a secret is being used on this screen. ALKiln will save the HTML, which omits the values in the fields.`
         });
       }
-      console.log(`👀 4 ${ foo == scope.page } ${ JSON.stringify(scope.page, null, 2) }`);
+      // console.log(`👀 4 ${ foo == scope.page } ${ JSON.stringify(scope.page, null, 2) }`);
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-11.jpg`});
 
       // Save/download a picture of the screen that's showing during the unexpected status
@@ -1425,7 +1425,7 @@ async function afterScenario(scenario) {
         disable_pic: scope.disable_error_screenshot,
         disable_html: false,
       });
-      console.log(`👀 5 ${ foo == scope.page } ${ JSON.stringify(scope.page, null, 2) }`);
+      // console.log(`👀 5 ${ foo == scope.page } ${ JSON.stringify(scope.page, null, 2) }`);
       await scope.take_a_screenshot(scope, {path:`${scope.paths.artifacts}/Temp-12.jpg`});
 
       // Save another copy in the artifact's Scenario folder
@@ -1439,7 +1439,7 @@ async function afterScenario(scenario) {
         disable_pic: scope.disable_error_screenshot,
         disable_html: false,
       });
-      console.log(`👀 6`);
+      // console.log(`👀 6`);
 
     }  // ends if scope.page exists
 
@@ -1447,31 +1447,31 @@ async function afterScenario(scenario) {
     if (changeable_test_status === `FAILED`) {
       reports.addToReport(scope, { type: `outcome failure info`, code: `ALK0096`, value: `**-- Scenario Failed --**` });
     }
-    console.log(`👀 7`);
+    // console.log(`👀 7`);
 
   }  // ends if not passed
-  console.log(`👀 8`);
+  // console.log(`👀 8`);
 
   // Add report text to Scenario folder after everything has been added to the report
   let scenario_report_obj = scope.report.get( scope.scenario_id );
-  console.log(`👀 8.01`);
+  // console.log(`👀 8.01`);
   let report = reports.getPrintableScenario( scenario_report_obj );
   // Save the report as in the Scenario folder
-  console.log(`👀 8.02`);
+  // console.log(`👀 8.02`);
   fs.writeFileSync( scope.paths.scenario_report, report );
-  console.log(`👀 8.03`);
+  // console.log(`👀 8.03`);
   // TODO: Save any cucumber failure message for the Scenario here (instead of waiting till run_cucumber.js)
 
   // ---------------- Check internal test results ----------------
 
   if ( scope.expected_in_report && scope.expected_in_report.length > 0) {
-    console.log(`👀 8.04`);
+    // console.log(`👀 8.04`);
     let data = await scope.reportIncludesAllExpected(
       scope,
       { expected: scope.expected_in_report }
     );
     let report_includes_all_expected_strings = data.all_included;
-    console.log(`👀 8.05`);
+    // console.log(`👀 8.05`);
     if ( !report_includes_all_expected_strings ) {
       changeable_test_status = `FAILED`;
       let msg = `These strings are missing from the report:
@@ -1484,22 +1484,22 @@ async function afterScenario(scenario) {
     scope.expected_in_report = null;
 
   }
-  console.log(`👀 8.06`);
+  // console.log(`👀 8.06`);
 
   if (scope.expected_not_in_report && scope.expected_not_in_report.length > 0) {
-    console.log(`👀 8.07`);
+    // console.log(`👀 8.07`);
     let all_prohibited_strings_were_absent = await scope.reportDoesNotInclude(
       scope,
       { not_expected: scope.expected_not_in_report }
     );
-    console.log(`👀 8.08`);
+    // console.log(`👀 8.08`);
     if ( !all_prohibited_strings_were_absent ) { changeable_test_status = `FAILED`;}
-    console.log(`👀 8.09`);
+    // console.log(`👀 8.09`);
     // Reset report values no matter what so they don't mess up future scenarios
     scope.expected_not_in_report = null;
-    console.log(`👀 8.091`);
+    // console.log(`👀 8.091`);
   }
-  console.log(`👀 9`);
+  // console.log(`👀 9`);
 
   // ---------------- Ends internal results check ----------------
 
@@ -1537,7 +1537,7 @@ async function afterScenario(scenario) {
       );
     }
   }
-  console.log(`👀 10`);
+  // console.log(`👀 10`);
   scope.expected_status = null;  // reset for next Scenario
 
   let need_to_force_failed_status = false;
@@ -1578,15 +1578,15 @@ async function afterScenario(scenario) {
   let signout_error_msg = ``;
   // If there is a page open, then sign out and close it
   if ( scope.page ) {
-    console.log(`👀 11`);
+    // console.log(`👀 11`);
     // Catch a possible reload error.
     try {
       // puppeteer will ensure proper timeout.
-      console.log(`👀 12`);
+      // console.log(`👀 12`);
       await scope.page.goto(`${ session_vars.get_da_server_url() }/user/sign-out`, {waitUntil: `domcontentloaded`});
 
     } catch ( error ) {
-      console.log(`👀 13`);
+      // console.log(`👀 13`);
       signout_succeeded = false;
 
       if ( error.name === `TimeoutError` ) {
@@ -1613,7 +1613,7 @@ async function afterScenario(scenario) {
     }  // ends try/catch
 
   }  // ends if scope.page
-  console.log(`👀 14`);
+  // console.log(`👀 14`);
 
   // Now that we're no longer on the interview page, one way or
   // another, try to delete the interviews created during the test
@@ -1670,7 +1670,7 @@ async function afterScenario(scenario) {
       internal_errors
     );
   }
-  console.log(`👀 15`);
+  // console.log(`👀 15`);
 
   // Find the race condition.
   log.debug({ code: `ALK0100`, context: `scenario`, },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suffolklitlab/alkiln",
-  "version": "5.16.1-always-html-3",
+  "version": "5.16.1-always-html-4",
   "description": "Integrated automated end-to-end testing with docassemble, puppeteer, and cucumber.",
   "main": "lib/index.js",
   "scripts": {