@suffolklitlab/alkiln 5.12.0-delete5 → 5.12.0-delete6

package/.github/workflows/github_server.yml

@@ -75,12 +75,14 @@ jobs:
         shell: bash
 
       #### Developer note: You can probably leave this out
-      - name: "Create a new normal user with an API key env var"
+      - name: "Add a low-level user and their API key (in an env var)"
         env:
           SERVER_URL: "${{ steps.github_server.outputs.SERVER_URL }}"
           KEY: ${{ steps.github_server.outputs.DOCASSEMBLE_DEVELOPER_API_KEY }}
         run: |
           # Create user and their API key
+
+          # Install json processor
           sudo apt-get install jq
 
           # https://docassemble.org/docs/api.html#user_new
@@ -98,7 +100,7 @@ jobs:
           user_id=$(echo "$user_data" | jq '.user_id' -r)
 
           # https://docassemble.org/docs/api.html#api_user_user_id_api_post
-          new_key=$(curl -X POST \
+          new_key_with_quotes=$(curl -X POST \
             ${{ env.SERVER_URL }}/api/user/"$user_id"/api \
             -H 'Content-Type: application/json' \
             -d '{
@@ -106,16 +108,18 @@ jobs:
               "name": "alkiln_key",
               "method": "none"
             }')
-          echo "User new key:"
-          echo "$new_key"
+          echo "new key (watch out, this contains actual quotation marks): $new_key_with_quotes"
+          no_end_quote="${new_key_with_quotes%\"}"
+          unquoted="${no_end_quote#\"}"
+          echo "unquoted: $unquoted"
 
-          echo "USER_NO_PERMISSIONS_API_KEY=$new_key" >> "$GITHUB_ENV"
+          echo "USER_NO_PERMISSIONS_API_KEY=$unquoted" >> "$GITHUB_ENV"
 
-      ### Developer note: You can probably leave this out
-      # Optional debugging to explore things like docker issues
-      - name: Docker debug tmate session
-        #if: ${{ failure() && github.event.inputs.enable_tmate == 'true' }}
-        uses: mxschmitt/action-tmate@v3
+      #### Developer note: You can probably leave this out
+      ## Optional debugging to explore things like docker issues
+      #- name: Docker debug tmate session
+      #  #if: ${{ failure() && github.event.inputs.enable_tmate == 'true' }}
+      #  uses: mxschmitt/action-tmate@v3
 
       #### Developer note: You can probably leave this out
       - name: "Create other env vars for our own tests from server output"

package/.github/workflows/playground.yml

@@ -77,6 +77,6 @@ jobs:
           # Internal: docassemble changes sometimes break @a11y tests. That is
           # a da problem, not ours. We will trigger these tests manually when we
           # want to check up on this.
-          ALKILN_TAG_EXPRESSION: "${{ env.ALKILN_TAG_EXPRESSION }}"
+          ALKILN_TAG_EXPRESSION: "@del3"
           #### Developer note: You can probably leave this out
           ALKILN_VERSION: delete

package/CHANGELOG.md

@@ -58,6 +58,7 @@ Format:
 - Test deleting interviews upon interview completion for signed in users.
 - Switch to mostly using a lowest-access-level user account for actually running interviews.
 - Added @temp_error to neutralize tests that are getting undesired errors right now, but which we anticipate being fixed soon. We want to ignore them for a short while. For example, an upstream change temporarily breaking some tests.
+- Excluded docassemble/ALKiln files from log code checks.
 
 ## [5.12.0] - 2024-06-06
 

package/lib/scope.js

@@ -3327,6 +3327,4 @@ module.exports = {
     return none_included;
   },
 
-  // Test that all interviews have been deleted
-
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suffolklitlab/alkiln",
-  "version": "5.12.0-delete5",
+  "version": "5.12.0-delete6",
   "description": "Integrated automated end-to-end testing with docassemble, puppeteer, and cucumber.",
   "main": "lib/index.js",
   "scripts": {

package/tests/log_codes/check_codes.sh

@@ -16,7 +16,7 @@ fi
 # The grepped files will include the list of all the codes that have been
 # removed and are no longer used. Syntax used works with GitHub cli ([[:digit:]])
 # https://stackoverflow.com/a/6901221
-lines=$(grep -roh --exclude="tests/log_codes/check_codes.sh" --exclude="debug_log.txt" --exclude="cucumber-report.txt" --exclude-dir="./docassemble" --exclude-dir="node_modules" --exclude-dir='alkiln-*' --exclude-dir='_alkiln*' 'ALK[[:digit:]][[:digit:]][[:digit:]][[:digit:]]' "$directory")
+lines=$(grep -roh --exclude="tests/log_codes/check_codes.sh" --exclude="debug_log.txt" --exclude="cucumber-report.txt" --exclude-dir="ALKilnTests" --exclude-dir="node_modules" --exclude-dir='alkiln-*' --exclude-dir='_alkiln*' 'ALK[[:digit:]][[:digit:]][[:digit:]][[:digit:]]' "$directory")
 sorted=$(echo "$lines" | sort -n)