@suffolklitlab/alkiln 3.0.9 → 4.0.0-pre-api.4

package/lib/steps.js

@@ -1,10 +1,23 @@
-const { When, Then, And, Given, After, Before, BeforeAll, AfterAll, setDefaultTimeout } = require('cucumber');
+const {
+  When,
+  Then,
+  And,
+  Given,
+  After,
+  Before,
+  BeforeAll,
+  AfterAll,
+  setDefaultTimeout //,
+  // wrapPromiseWithTimeout  // cucumber v8
+} = require('@cucumber/cucumber');
+const wrapPromiseWithTimeout = require('./cucumber_8_shim');
 const fs = require('fs');
 const path = require('path');
 const { expect } = require('chai');
 const puppeteer = require('puppeteer');
 const { v4: uuidv4 } = require('uuid');
 
+// Ours
 const scope = require('./scope');
 const env_vars = require('./utils/env_vars');
 
@@ -57,6 +70,7 @@ BeforeAll(async () => {
   scope.timeout = 30 * 1000;
   scope.showif_timeout = 600;
   scope.report = new Map();
+  fs.writeFileSync('log.txt', "");
 });
 
 Before(async (scenario) => {
@@ -76,7 +90,6 @@ Before(async (scenario) => {
 
   // Reset default timeout
   scope.timeout = 30 * 1000;
-  setDefaultTimeout( 30 * 1000 );
 });
 
 // Add a check for an error page before each step? After each step?
@@ -167,8 +180,9 @@ Given(/I start the interview at "([^"]+)"(?: in lang "([^"]+)")?/, {timeout: -1}
 
     // If response has a file in it that matches the file we are downloading
     if ( response._headers['content-disposition'] && (response._headers['content-disposition']).includes(`attachment;filename=${scope.toDownload}`) ) {
-      // Remove the event listener since we might not be closing the browser
-      scope.page.removeEventListener('response', arguments.callee);
+      // We're chosing avoid removing the event listener in case removal
+      // would prevent downloading other files. The listener should get
+      // destroyed when the page closes anyway.
 
       // Watch event on download folder or file
       await fs.watchFile( docs_dir, function (curr, prev) {
@@ -220,6 +234,24 @@ Given(/I start the interview at "([^"]+)"(?: in lang "([^"]+)")?/, {timeout: -1}
   await custom_aferStep_timeout;
 });
 
+Given(/I (?:sign|log) ?(?:in)?(?:on)?(?:to the server)? with(?: the email)? "([^"]+)" and(?: the password)? "([^"]+)"(?: SECRETs)?/i, {timeout: -1}, async ( email, password ) => {
+  /** Uses the names of environment variables (most often GitHub SECRETs) to
+  *     log into an account on the user's server. Must be SECRETs to stay secure.
+  * 
+  * Variations:
+  * I sign into the server with the email "USER1_EMAIL" and the password "USER1_PASSWORD" SECRETs
+  * I sign on with the email "USER1_EMAIL" and the password "USER1_PASSWORD" SECRETs
+  * I log in with "USER1_EMAIL" and "USER1_PASSWORD"
+  */
+  return wrapPromiseWithTimeout(
+    scope.steps.sign_in( scope, {
+      email_secret_name: email,
+      password_secret_name: password
+    }),
+    scope.timeout
+  );
+});
+
 // I am using a moble/pc
 When(/I am using an? (.*)/, async ( device ) => {
   // Let developer pick mobile device if they want to
@@ -240,7 +272,9 @@ When(/I am using an? (.*)/, async ( device ) => {
 Given(/the max(?:imum)? sec(?:ond)?s for each step (?:in this scenario )?is (\d+)/i, async ( seconds ) => {
   /* At any time, set a custom timeout for the scenario. */
   scope.timeout = parseInt( seconds ) * 1000;
-  setDefaultTimeout( scope.timeout );
+  
+  // Cucumber timeouts are made custom for each step that needs it
+
   // If there is no page yet, we've ensured custom timeout will still be set
   // appropriately in the interview loading step
   if ( scope.page ) { await scope.page.setDefaultTimeout( scope.timeout ); }
@@ -326,7 +360,7 @@ When(/^(?:the user gets)?(?:I get)? to ?(?:the)?(?: ?question)?(?: id)? "([^"]+)
   // Add full used table to report so it can be copied if needed.
   // TODO: Should we/How do we get this in the final scenario report instead of here?
   await scope.addToReport( scope, { type: `note`, value: `\n${ ' '.repeat(2) }Rows that got set:` });
-  await scope.addToReport( scope, { type: `step`, value: `${ ' '.repeat(4) }And I get the question id "${ target_id }" with this data:` });
+  await scope.addToReport( scope, { type: `step`, value: `${ ' '.repeat(4) }And I get to the question id "${ target_id }" with this data:` });
   await scope.addToReport(scope, { type: `row`, value: `${ ' '.repeat(6) }| var | value | trigger |`, });
   let at_least_one_row_was_NOT_used = false;
   for ( let row of sorted_rows ) {
@@ -358,41 +392,57 @@ When(/^(?:the user gets)?(?:I get)? to ?(?:the)?(?: ?question)?(?: id)? "([^"]+)
 //#####################################
 //#####################################
 
-// Need to see if it's possible to remove the need for this on most occasions
-// I wait 1 second
-// I wait .5 seconds
-When(/I wait (\d*\.?\d+) seconds?/, async (seconds) => {  // √
-  // Should this reset the 'navigated' flag?
-  // Shoudl any observational things reset the 'navigated' flag?
-  // Maybe the 'navitaged' flag should only be reset just before trying to
-  // navigate (after pressing a button)
-  await scope.afterStep(scope, {waitForTimeout: (parseFloat(seconds) * 1000)});
+When(/I wait (\d*\.?\d+) seconds?/, { timeout: -1 }, async ( seconds ) => {
+  /** Wait for a specified amount of time. Can help with some race conditions.
+  *    We're working on reducing race conditions, but it'll never be perfect.
+  *
+  * Variations:
+  * - I wait 1 second
+  * - I wait .5 seconds
+  *
+  * Questions:
+  * - Should this really reset the 'navigated' flag? Should the below
+  *     return succeed or fail? e.g:
+  *     I tap to continue; I wait 10 seconds; I arrive at the next page;
+  *     Maybe the 'navitaged' flag should only be reset just before trying to
+  *     navigate (after pressing a button)
+  */
+  return wrapPromiseWithTimeout(
+    scope.afterStep(scope, {waitForTimeout: (parseFloat( seconds ) * 1000)}),
+    scope.timeout
+  );
 });
 
-Then(/I take a screenshot ?(?:named "([^"]+)")?/, async ( name ) => {
-  /* Save/download a screenshot. `name` does not have to be unique as
-  * the filename will be made unique. */
-  name = name || '';
-  let safe_path = await scope.getSafeScenarioFilename( scope, { prefix: `screenshot_${ name }` });
-  await scope.page.screenshot({ path: `./${ safe_path }.jpg`, type: 'jpeg', fullPage: true });
-  await scope.afterStep(scope);
+Then(/I take a screenshot ?(?:named "([^"]+)")?/, { timeout: -1 }, async ( name ) => {
+  /* Download and save a screenshot. `name` does not have to be
+  * unique as the filename will be made unique. */
+  return wrapPromiseWithTimeout(
+    scope.steps.screenshot( scope, name ),
+    scope.timeout
+  );
 });
 
 /* Here to make writing tests more comfortable. */
 When("I do nothing", async () => { await scope.afterStep(scope); });  // √
 
-Then('the question id should be {string}', async (question_id) => {  // √
+Then('the question id should be {string}', { timeout: -1 }, async ( question_id ) => {
   /* Looks for a santized version of the question id as it's written
   *     in the .yml. docassemble's way */
   let { question_has_id, id, id_matches } = await scope.examinePageID( scope, question_id );
 
-  // Trying to make helpful error messages
+  // No question id exists
   let no_id_msg = `Did not find any question id.`;
-  if ( !question_has_id ) { await scope.addToReport( scope, { type: `error`, value: no_id_msg }); }
+  if ( !question_has_id ) {
+    await scope.addToReport( scope, { type: `error`, value: no_id_msg });
+  }
   expect( question_has_id , no_id_msg ).to.be.true;
 
-  let no_match_msg = `The question id was supposed to be "${ question_id }", but it's actually "${ id }".`;
-  if ( !id_matches ) { await scope.addToReport( scope, { type: `error`, value: no_match_msg }); }
+  // Wrong question id
+  let no_match_msg = `The question id was supposed to be "${ question_id }"`
+    + `, but it's actually "${ id }".`;
+  if ( !id_matches ) {
+    await scope.addToReport( scope, { type: `error`, value: no_match_msg });
+  }
   expect( id_matches, no_match_msg ).to.be.true;
 
   await scope.afterStep(scope);
@@ -477,7 +527,7 @@ Then('an element should have the id {string}', async ( id ) => {  // √
 //   await scope.afterStep(scope);
 // });
 
-// This is phrase is confusing. It sounds like it's going to at least try to continue.
+// This phrase is confusing. It sounds like it's going to at least try to continue.
 Then(/I (don't|can't|don’t|can’t|cannot|do not) continue/, async (unused) => {  // √
   /* Tests for detection of url change from button or link tap.
   *    Can other things trigger navigation? Re: other inputs things
@@ -573,6 +623,63 @@ Then(/the "([^"]+)" link opens in (a new window|the same window)/, async (linkTe
   await scope.afterStep(scope);
 });
 
+Then(/(?:the )?text(?: in)?(?: the)?(?: JSON)?(?: var)?(?:iable)?(?: )?"([^"]+)" should be/i, async (var_name, expected_value) => {
+  /* Check that the text string of the page's JSON var value matches
+  *    the given text string. Does not currently accept nested values
+  *    like "foo.bar". The word "text" is required.
+  *
+  * The format for this step is something like this:
+  * ```
+  * Then the text in the JSON variable "var_name" should be
+  * """
+  * A single or
+  * multiline string
+  * """
+  * ```
+  *
+  * Sentence combinations that will work:
+  * `the text in the JSON variable "variable_with_text_value" should be`
+  * `the text in the JSON var "variable_with_text_value" should be`
+  * `the text in "variable_with_text_value" should be`
+  * `text var "variable_with_text_value" should be`
+  */
+
+  // Get JSON variable value
+  let { filename, json } = await scope.savePageJSONData( scope );
+  let actual_value = json.variables[ var_name ];
+
+  // Add warning to report if value is not as expected
+  if ( actual_value !== expected_value ) {
+    await scope.addToReport( scope, {
+      type: `error`,
+      value: `The variable "${ var_name }" was not equal to the expected `
+        + `value on the "${ scope.page_id }" screen. Check the downloads `
+        + `file ${ filename } to see the page's JSON variables.`
+    });
+  }
+
+  expect( actual_value ).to.equal( expected_value );
+
+});
+
+
+Then(/I get the(?: page's)?(?: JSON)? var(?:iable)?s and val(?:ue)?s/i, { timeout: -1 }, async () => {
+  /** Uses  https://docassemble.org/docs/functions.html#js_get_interview_variables
+  *    to get the variable values on the page of the interview and add
+  *    them to the report.
+  * 
+  * Varations:
+  * I get the page's JSON variables and values
+  * I get the json vars and vals
+  * I get the page's vars and vals
+  * I get the vars and vals
+  */
+  return await wrapPromiseWithTimeout(
+    scope.steps.add_json_vars_to_report( scope ),
+    scope.timeout
+  );
+});
+
 //#####################################
 //#####################################
 // Actions
@@ -585,27 +692,20 @@ Then(/the "([^"]+)" link opens in (a new window|the same window)/, async (linkTe
 
 // Consider people wanting to use this for an in-interview page
 // Also consider 'the link url "..." should open...'?
-Then(/the "([^"]+)" link opens a working page/, async (linkText) => {  // √
-  let [link] = await scope.page.$x(`//a[contains(text(), "${linkText}")]`);
-  let prop_obj = await link.getProperty('href');
-  let actual_url = await prop_obj.jsonValue();
-
-  let linkPage = await scope.browser.newPage();
-  let response = await linkPage.goto(actual_url, {waitUntil: 'domcontentloaded'});
-
-  let msg = `The "${ linkText }" link is broken.`;
-  if ( !response.ok() ) { await scope.addToReport(scope, { type: `error`, value: msg }); }
-  expect( response.ok(), msg ).to.be.true;
-
-  linkPage.close()
-
-  await scope.afterStep(scope);
+Then(/the "([^"]+)" link opens a working page/, { timeout: -1 }, async ( linkText ) => {
+  return wrapPromiseWithTimeout(
+    scope.steps.link_works( scope, linkText ),
+    scope.timeout
+  );
 });
 
-When('I tap to continue', async () => {
+When('I tap to continue', { timeout: -1 }, async () => {
   // Any selectors for this seem somewhat precarious
   // No data will cause an automatic continue when `ensure_navigation` is true
-  await scope.setFields( scope, { var_data: [], ensure_navigation: true });
+  return wrapPromiseWithTimeout(
+    scope.setFields( scope, { var_data: [], ensure_navigation: true }),
+    scope.timeout
+  );
 });
 
 
@@ -616,7 +716,7 @@ When('I tap to continue', async () => {
 // -------------------------------------
 // --- VARIABLES-BASED STEPS ---
 
-When(/I set the var(?:iable)? "([^"]+)" to "([^"]+)"/, async ( var_name, set_to ) => {
+When(/I set the var(?:iable)? "([^"]+)" to "([^"]+)"/, { timeout: -1 }, async ( var_name, set_to ) => {
   /* Set any variable (with or without a choice associated with it) to a value. Set:
   *  - Buttons associated with variables
   *  - Dropdowns
@@ -629,60 +729,51 @@ When(/I set the var(?:iable)? "([^"]+)" to "([^"]+)"/, async ( var_name, set_to
   * 1. I set the var "benefits['SSI']" to "false"
   * 1. I set the var "rent_interval" to "weekly"
   */
-  let var_data = await scope.normalizeTable( scope, {
-    var_data: [{ var: var_name, value: set_to, trigger: '' }],
-  });
-  // Don't continue if the variable doesn't navigate and if this variable cannot be set
-  // on this page, trigger an error
-  await scope.setFields( scope, { var_data, ensure_navigation: false, ensure_all_vars_used: true });
+  return wrapPromiseWithTimeout(
+    scope.steps.set_regular_var( scope, var_name, set_to ),
+    scope.timeout
+  );
 });
 
-Then(/I upload "([^"]+)" to "([^"]+)"/, async ( filenames, var_name ) => {
+Then(/I upload "([^"]+)" to "([^"]+)"/, { timeout: -1 }, async ( filenames, var_name ) => {
   /* Uploads sibling file (docx, jpg, etc) to input var. Can actually use
   * 'I set the var "" to ""', but this is more explicit if it's desired. */
-  let var_data = await scope.normalizeTable( scope, {
-    var_data: [{ var: var_name, value: filenames, trigger: '' }],
-  });
-  // Don't continue if the variable doesn't navigate and if this variable cannot be set
-  // on this page, trigger an error
-  await scope.setFields( scope, { var_data, ensure_navigation: false, ensure_all_vars_used: true });
+  return wrapPromiseWithTimeout(
+    scope.steps.set_regular_var( scope, var_name, filenames ),
+    scope.timeout
+  );
 });
 
 
 // -------------------------------------
 // --- OTHER UI STEPS ---
 
-When('I tap the defined text link {string}', async (phrase) => {  // hold
-  /* Not sure what 'defined' means here. Maybe for terms? */
-  const [link] = await scope.page.$x(`//a[contains(text(), "${phrase}")]`);
-
-  let msg = `The term "${ phrase }" seems to be missing.`;
-  if ( !link ) { await scope.addToReport(scope, { type: `error`, value: msg }); }
-  expect( link, msg ).to.exist;
-
-  await link.evaluate( elem => { return elem.click(); });
-
-  await scope.afterStep(scope, {waitForShowIf: true});
+When('I tap the defined text link {string}', { timeout: -1 }, async ( phrase ) => {
+  /** Tap a link that doesn't navigate. Depends on the language of the text. */
+  return wrapPromiseWithTimeout(
+    scope.steps.tap_term( scope, phrase ),
+    scope.timeout
+  );
 });
 
-Then('I sign', async () => {
-  await scope.sign( scope );
-  await scope.afterStep(scope);
+Then('I sign', { timeout: -1 }, async () => {
+  return wrapPromiseWithTimeout(
+    scope.steps.sign( scope ),
+    scope.timeout
+  );
 });
 
-When(/I download "([^"]+)"$/, async (filename) => {
+When(/I download "([^"]+)"$/, { timeout: -1 }, async ( filename ) => {
   /* Taps the link that leads to the given filename to trigger downloading.
-  * and waits till the file has been downloaded before allowing the tests to continue.
-  WARNING: Cannot download the same file twice in a single scenario. */
-  let [elem] = await scope.page.$x(`//a[contains(@href, "${ filename }")]`);
-
-  let msg = `"${ filename }" seems to be missing. Cannot find a link to that document.`;
-  if ( !elem ) { await scope.addToReport(scope, { type: `error`, value: msg }); }
-  expect( elem, msg ).to.exist;
-
-  scope.toDownload = filename;
-  await elem.evaluate( elem => { return elem.click(); });
-  await scope.detectDownloadComplete( scope );
+  *    and waits till the file has been downloaded before allowing the tests to continue.
+  *    WARNING: Cannot download the same file twice in a single scenario.
+  *    WARNING: Must not contain any characters that are totally unique to that
+  *    page load. Just stick to the name of the file.
+  */
+  return wrapPromiseWithTimeout(
+    scope.steps.download( scope, filename ),
+    scope.timeout
+  );
 });
 
 //#####################################
@@ -691,7 +782,7 @@ When(/I download "([^"]+)"$/, async (filename) => {
 //#####################################
 //#####################################
 
-When(/I set the name of ?(?:the var(?:iable)?)? "([^"]+)" to "([^"]+)"/, async (var_base, name_str) => {
+When(/I set the name of ?(?:the var(?:iable)?)? "([^"]+)" to "([^"]+)"/, { timeout: -1 }, async (var_base, name_str) => {
   /* Given a variable name and a space-separated string, fills the
   *    appropriate fields on an Assembly Line name question.
   *    It handles any of:
@@ -708,28 +799,14 @@ When(/I set the name of ?(?:the var(?:iable)?)? "([^"]+)" to "([^"]+)"/, async (
   * 1. I set the name of the var "defendant" to "Dalia Day Defo Jr"
   * 1. I set the name of the variable "defendant" to "Dalia Day Defo Jr"
   */
-  let name_patterns = [
-    [ `text` ],
-    [ `first`, `last` ],
-    [ `first`, `middle`, `last` ],
-    [ `first`, `middle`, `last`, `suffix` ],
-  ];
-
-  let name_parts = name_str.split(' ');
-  let desired_pattern = name_patterns[( name_parts.length - 1 )];
-
-  for ( let parti = 0; parti < name_parts.length; parti++ ) {
-    let var_name = `${ var_base }.name.${ desired_pattern[ parti ] }`
-    let value = name_parts[ parti ];
-    await scope.setVar( scope, var_name, value );
-  }
-
-  // Continue to next page...?
-  await scope.afterStep(scope, {waitForShowIf: true});
+  return wrapPromiseWithTimeout(
+    scope.steps.set_al_name( scope, var_base, name_str ),
+    scope.timeout
+  );
 });
 
 
-When(/I set the address of ?(?:the var(?:iable)?)? "([^"]+)" to "([^"]+)"/, async (var_base, address_str) => {
+When(/I set the address of ?(?:the var(?:iable)?)? "([^"]+)" to "([^"]+)"/, { timeout: -1 }, async (var_base, address_str) => {
   /* Given a variable name and a four-part address formatted as below,
   *    sets the appropriate Assembly Line address fields.
   *
@@ -740,24 +817,10 @@ When(/I set the address of ?(?:the var(?:iable)?)? "([^"]+)" to "([^"]+)"/, asyn
   * 1. When I set the address of the variable "users[0]" to "112 Southampton St., Unit 1, Boston, MA 02118"
   * 1. When I set the address of the var "users[0]" to "112 Southampton St., Unit 1, Boston, MA 02118"
   */
-  // Get all the parts of the address properly separated
-  // First, split apart each line of the address to [address line 1, address line 2, city, state zip]...
-  let split = address_str.split(/,\s*/);
-  // ...then, separate the state and the zip code...
-  let last_parts = (split.pop()).split(/\s+/);
-  // ...and add them back in, to end up with [address line 1, address line 2, city, state, zip].
-  let address_parts = split.concat( last_parts );
-  let start = `${ var_base }.address.`;
-
-  let index = 0;
-  await scope.setVar( scope, `${start}address`, address_parts[0] );
-  // Ensure it's without 'Unit '/'unit '
-  await scope.setVar( scope, `${start}unit`, (address_parts[1].toLowerCase()).replace(/unit\s*/, '') );
-  await scope.setVar( scope, `${start}city`, address_parts[2] );
-  await scope.setVar( scope, `${start}state`, address_parts[3] );
-  await scope.setVar( scope, `${start}zip`, address_parts[4] );
-
-  await scope.afterStep(scope, {waitForShowIf: true});
+  return wrapPromiseWithTimeout(
+    scope.steps.set_al_address( scope, var_base, address_str ),
+    scope.timeout
+  );
 });
 
 // Next macro - download sequence?
@@ -788,7 +851,7 @@ Given(/the final Scenario status should be "(passed|failed)"/i, async ( expected
   /* WARNING: If a Step will fail, this Step must come first.
   * 
   * Prepare to test that the Scenario completes with the correct status. */
-  scope.expected_status = expected_status;
+  scope.expected_status = expected_status.toUpperCase();
 });
 
 
@@ -803,18 +866,23 @@ After(async function(scenario) {
   // Record status before internal report tests can interfere with values
   await scope.setReportScenarioStatus( scope, { status: scenario.result.status });
 
-  if (scenario.result.status !== `passed`) {
+  if (scenario.result.status !== `PASSED`) {
     if ( env_vars.DEBUG ) {
-      // When debugging locally, pause to allow dev to examine the page
-      console.log( `Error occurred while running test` );
-      await scope.page.waitFor( 60 * 1000 );
+      console.log( `Non-passed status occurred while running test:`, scenario.result.status );
+      // If there's a page visible, give us time to examine it
+      if ( scope.page ) { await scope.page.waitForTimeout( 60 * 1000 ); }
     }
 
     await scope.addToReport(scope, { type: `outcome`, value: `**-- Scenario Failed --**` });
-    // Save/download a picture of the screen during the error
-    let safe_filename = await scope.getSafeScenarioFilename( scope, { prefix: `error` });
-    await scope.page.screenshot({ path: `${ safe_filename }.jpg`, type: `jpeg`, fullPage: true });
+    if ( scope.page ) {
+      // Save/download a picture of the screen that's showing during the unexpected status
+      let safe_filename = await scope.getSafeScenarioFilename( scope, { prefix: `error` });
+      await scope.page.screenshot({ path: `${ safe_filename }.jpg`, type: `jpeg`, fullPage: true });
+    }
+  }  // Ends if not passed
 
+  if (scenario.result.status === `FAILED`) {
+    await scope.addToReport(scope, { type: `outcome`, value: `**-- Scenario Failed --**` });
   }
 
   let original_scenario_status = scenario.result.status;
@@ -831,7 +899,7 @@ After(async function(scenario) {
     // Maybe this should be in `Before`, but it would be so out of context there...
     scope.expected_in_report = [];
     let all_were_included = await scope.reportIncludesAllExpected( scope, { expected });
-    if ( all_were_included ) { scenario.result.status = `passed`; }
+    if ( all_were_included ) { scenario.result.status = `PASSED`; }
   }
 
   // If there is a page open, then sign out and close it
@@ -843,23 +911,26 @@ After(async function(scenario) {
     try {
       await scope.page.goto(`${ env_vars.BASE_URL }/user/sign-out`, {waitUntil: `domcontentloaded`});  
     } catch ( err ) {
-      await scope.addToReport(scope, {
-          type: `warning`,
-          value: `Trying to reset for the next test a second time because the first time failed.`
-        });
+      if ( env_vars.DEBUG ) {
+        console.log( `Trying to reset for the next test a second time because the first time failed.` );
+      }
+
       try {
         await scope.page.goto(`${ env_vars.BASE_URL }/user/sign-out`, {waitUntil: `domcontentloaded`});  
+
       } catch ( err ) {
         await scope.addToReport(scope, {
           type: `warning`,
-          value: `This test ${ original_scenario_status }, but also there was an error while trying to reset for the next test. Re-running until you don't get this error would probably be a good idea.`
+          value: `This test ${ original_scenario_status }, but also there `
+            + `was an error while trying to reset for the next test. You can try `
+            + `re-running the test or checking your server.`
         });
       }
     }
     
     await scope.page.close();
     scope.page = null;
-  }
+  }  // ends if scope.page
 });
 
 AfterAll(async function() {