@suffolklitlab/alkiln 4.2.1-custom-setup-timeout.1 → 4.2.1-custom-setup-timeout.2

package/.github/workflows/test_tests.yml

@@ -78,12 +78,12 @@ jobs:
         with:
           name: report
           path: ./**/*_report_*
-      - name: log artifact
+      - name: upload debug_log artifact
         uses: actions/upload-artifact@v2
         if: ${{ always() }}
         with:
-          name: log file
-          path: ./log.txt
+          name: debug_log file
+          path: ./debug_log.txt
       - name: Cleanup
         if: ${{ always() }}
         run: npm run takedown

package/CHANGELOG.md

@@ -42,6 +42,12 @@ Format:
 ### Added
 - GitHub environment variable MAX_SECONDS_FOR_SETUP to set a custom maximum time for setup and takedown for packages that take a long time to load.
 
+## [4.2.0] - 2022-04-05
+### Changed
+- Pin versions of dependencies in action.yml and package.json
+- Change log.txt file to debug_log.txt and upload as github artifact.
+- Update cheerio version to remove vulnerabilities.
+
 ## [4.1.1] - 2022-04-01
 ### Added
 - Step to set vars to secret variables. That is, using environment variables while hiding the names of those variables in the report, errors, and console logs.

package/action.yml

@@ -82,7 +82,7 @@ runs:
           },
           "dependencies": {
             "colors": "1.4.0",
-            "@cucumber/cucumber": "^7.3.2",
+            "@cucumber/cucumber": "7.3.2",
             "@suffolklitlab/alkiln": "$ALKILN_VERSION"
           }
         }

package/lib/docassemble/docassemble_api_interface.js

@@ -356,11 +356,14 @@ da_i.wait_for_server_to_restart = async ( task_id, options ) => {
   let default_timeout = DEFAULT_MAX_REQUEST_MS;
   let timeout = options.timeout || default_timeout;
   // Other defaults
-  let default_wait_per_loop = 5 * 1000;
-  let default_max_loops = timeout/default_wait_per_loop;
+  let default_wait_between_loops = 4 * 1000;
+  let default_wait_for_response = 1 * 1000;
+  let default_max_loops = timeout/( default_wait_between_loops + default_wait_for_response );
   // Actual values
-  let wait_ms_per_loop = options.wait_ms_per_loop || default_wait_per_loop;
+  let wait_between_loops = options.wait_between_loops || default_wait_between_loops;
+  let wait_for_response = options.wait_for_response || default_wait_for_response;
   let max_loops = options.max_loops || default_max_loops;
+  let total_wait = wait_between_loops + wait_for_response;
 
   // Flags for stopping the `while` loop
   let num_tries = 0;
@@ -372,7 +375,7 @@ da_i.wait_for_server_to_restart = async ( task_id, options ) => {
   while ( !done_restarting && num_tries <= max_loops ) {
     num_tries++;
     // Wait only a short time for the task to finish, not the full default timeout
-    done_restarting = await da_i.has_task_finished( task_id, { timeout: 1 * 1000 } );
+    done_restarting = await da_i.has_task_finished( task_id, { timeout: wait_for_response } );
     if ( !done_restarting ) {
       if ( num_tries < max_loops ) {
         // Show a slightly different message after a number of tries to reassure
@@ -381,16 +384,17 @@ da_i.wait_for_server_to_restart = async ( task_id, options ) => {
         if ( num_tries > 3 ) { msg_start = `Still not complete`; }
 
         // When the server is busy, the task timeout causes this loop to actually
-        // take 6 seconds to complete, not 5, but I'm not sure how to make that clear
+        // take 5 seconds to complete, but this only waits for 4. I'm not sure how
+        // to make that clear
         log.info({ pre: `${ msg_start }. Check ${ num_tries } of ${ max_loops }. `
-          + `Will check again in ${ wait_ms_per_loop/1000 } seconds.`
+          + `Will check again in ${( wait_between_loops + wait_for_response )/1000 } seconds.`
         });
-        await time.waitForTimeout( wait_ms_per_loop );
+        await time.waitForTimeout( wait_between_loops );
 
       } else {
         // This is our last round, let's wrap it up
         let server_msg = `ALKiln setup: It seems like the docassemble server `
-          + `didn't finish uploading the code after ${ max_loops * (wait_ms_per_loop/1000) } `
+          + `didn't finish uploading the code after ${ timeout/1000 } `
           + `seconds. Some things you can try: check your server is up, try `
           + `re-running the test, and try pulling the package manually `
           + `into the playground.`;

package/lib/scope.js

@@ -60,7 +60,7 @@ module.exports = {
     let scenario = await scope.makeSureReportPartsExist( scope );
     scenario.lines.push({ type, value });
 
-    fs.appendFileSync('log.txt', `Report:\ntype: ${ type }, value: ${ value }`);
+    fs.appendFileSync('debug_log.txt', `Report:\ntype: ${ type }, value: ${ value }`);
     return scenario;
     
   },  // Ends scope.addToReport()

package/lib/steps.js

@@ -68,7 +68,7 @@ BeforeAll(async () => {
   scope.timeout = 30 * 1000;
   scope.showif_timeout = 600;
   scope.report = new Map();
-  fs.writeFileSync('log.txt', "");
+  fs.writeFileSync('debug_log.txt', "");
 });
 
 Before(async (scenario) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suffolklitlab/alkiln",
-  "version": "4.2.1-custom-setup-timeout.1",
+  "version": "4.2.1-custom-setup-timeout.2",
   "description": "Integrated automated end-to-end testing with docassemble, puppeteer, and cucumber.",
   "main": "lib/index.js",
   "scripts": {
@@ -34,16 +34,16 @@
   },
   "homepage": "https://suffolklitlab.org/docassemble-AssemblyLine-documentation/docs/automated_integrated_testing/",
   "dependencies": {
-    "@cucumber/cucumber": "^7.3.2",
-    "axios": "^0.24.0",
-    "chai": "^4.2.0",
-    "cheerio": "^1.0.0-rc.5",
-    "dotenv": "^8.2.0",
-    "mocha": "^9.2.0",
-    "puppeteer": "^13.1.3",
-    "qs": "^6.10.2",
-    "sanitize-filename": "^1.6.3",
-    "uuid": "^8.3.2"
+    "@cucumber/cucumber": "7.3.2",
+    "axios": "0.24.0",
+    "chai": "4.2.0",
+    "cheerio": "1.0.0-rc.10",
+    "dotenv": "8.2.0",
+    "mocha": "9.2.0",
+    "puppeteer": "13.1.3",
+    "qs": "6.10.2",
+    "sanitize-filename": "1.6.3",
+    "uuid": "8.3.2"
   },
   "bin": {
     "alkiln-setup": "./lib/docassemble/setup.js",