ci: update new job to match latest patterns and rebuild

porcellus · porcellus · commit 79f3ca2b7286 · 2025-11-25T01:11:31.000+01:00
diff --git a/.github/workflows/unit-test.yml b/.github/workflows/unit-test.yml
@@ -1,60 +1,67 @@
 name: "Unit Tests"
 
 on:
-  pull_request:
-    types:
-      - opened
-      - reopened
-      - synchronize
-  push:
-    branches:
-      - master
-      - "v[0-9]+.[0-9]+"
-    tags:
-      - "(dev-)?v[0-9]+.[0-9]+.[0-9]+"
+    pull_request:
+        types:
+            - opened
+            - reopened
+            - synchronize
+            - labeled
+            - unlabeled
+    push:
+        branches:
+            - "[0-9]+.[0-9]+"
+
+# Only one instance of this workflow will run on the same ref (PR/Branch/Tag)
+# Previous runs will be cancelled.
+concurrency:
+    group: ${{ github.workflow }}-${{ github.ref }}
+    cancel-in-progress: true
 
 jobs:
-  define-versions:
-    runs-on: ubuntu-latest
-    outputs:
-      fdiVersions: ${{ steps.versions.outputs.fdiVersions }}
-      cdiVersions: ${{ steps.versions.outputs.cdiVersions }}
-      nodeVersions: '["18.20", "20.19", "22.14"]'
-    steps:
-      - uses: actions/checkout@v4
-      - uses: supertokens/get-supported-versions-action@main
-        id: versions
-        with:
-          has-fdi: true
-          has-cdi: true
+    define-versions:
+        if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'run-tests')
+        runs-on: ubuntu-latest
+        outputs:
+            fdiVersions: ${{ steps.versions.outputs.fdiVersions }}
+            cdiVersions: ${{ steps.versions.outputs.cdiVersions }}
+            nodeVersions: '["18.20", "20.19", "22.14"]'
+        steps:
+            - uses: actions/checkout@v4
+            - uses: supertokens/get-supported-versions-action@main
+              id: versions
+              with:
+                  has-fdi: true
+                  has-cdi: true
 
-  test:
-    runs-on: ubuntu-latest
-    needs: define-versions
-    strategy:
-      fail-fast: false
-      matrix:
-        node-version: ${{ fromJSON(needs.define-versions.outputs.nodeVersions) }}
-        cdi-version: ${{ fromJSON(needs.define-versions.outputs.cdiVersions) }}
-    steps:
-      - uses: actions/checkout@v4
-      - uses: supertokens/get-versions-action@main
-        id: versions
-        with:
-          driver-name: node
-          cdi-version: ${{ matrix.cdi-version }}
-        env:
-          SUPERTOKENS_API_KEY: ${{ secrets.SUPERTOKENS_API_KEY }}
-      - uses: actions/setup-node@v4
-        with:
-          node-version: ${{ matrix.node-version }}
-      - name: Install dependencies
-        run: |
-          npm install
-      - name: Run unit tests
-        run: |
-          docker compose up --wait
-          npm run test
-          docker compose down
-        env:
-          SUPERTOKENS_CORE_VERSION: ${{ steps.versions.outputs.coreVersionXy }}
+    test:
+        if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'run-tests')
+        runs-on: ubuntu-latest
+        needs: define-versions
+        strategy:
+            fail-fast: false
+            matrix:
+                node-version: ${{ fromJSON(needs.define-versions.outputs.nodeVersions) }}
+                cdi-version: ${{ fromJSON(needs.define-versions.outputs.cdiVersions) }}
+        steps:
+            - uses: actions/checkout@v4
+            - uses: supertokens/get-versions-action@main
+              id: versions
+              with:
+                  driver-name: node
+                  cdi-version: ${{ matrix.cdi-version }}
+              env:
+                  SUPERTOKENS_API_KEY: ${{ secrets.SUPERTOKENS_API_KEY }}
+            - uses: actions/setup-node@v4
+              with:
+                  node-version: ${{ matrix.node-version }}
+            - name: Install dependencies
+              run: |
+                  npm install
+            - name: Run unit tests
+              run: |
+                  docker compose up --wait
+                  npm run test
+                  docker compose down
+              env:
+                  SUPERTOKENS_CORE_VERSION: ${{ steps.versions.outputs.coreVersionXy }}
diff --git a/.mocharc.yml b/.mocharc.yml
@@ -6,4 +6,4 @@ timeout: 40000
 reporter: mocha-multi-reporters
 require: test/hooks.js
 reporter-option:
-  - configFile=mocha-multi-reporters.json
+    - configFile=mocha-multi-reporters.json
diff --git a/lib/build/recipe/oauth2provider/constants.js b/lib/build/recipe/oauth2provider/constants.js
diff --git a/lib/build/recipe/session/constants.js b/lib/build/recipe/session/constants.js
diff --git a/mocha-multi-reporters.json b/mocha-multi-reporters.json
@@ -6,4 +6,4 @@
     "mochaJsonOutputReporterReporterOptions": {
         "output": "test-results.json"
     }
-}
+}
diff --git a/package.json b/package.json
@@ -196,4 +196,4 @@
     "browser": {
         "fs": false
     }
-}
+}
diff --git a/test/emailpassword/override.test.js b/test/emailpassword/override.test.js
@@ -97,7 +97,8 @@ describe(`overrideTest: ${printPath("[test/emailpassword/override.test.js]")}`,
                                             loginMethods: [
                                                 {
                                                     ...response.user.loginMethods[0],
-                                                    recipeUserId: response.user.loginMethods[0].recipeUserId.getAsString(),
+                                                    recipeUserId:
+                                                        response.user.loginMethods[0].recipeUserId.getAsString(),
                                                 },
                                             ],
                                         };
@@ -116,7 +117,8 @@ describe(`overrideTest: ${printPath("[test/emailpassword/override.test.js]")}`,
                                             loginMethods: [
                                                 {
                                                     ...response.user.loginMethods[0],
-                                                    recipeUserId: response.user.loginMethods[0].recipeUserId.getAsString(),
+                                                    recipeUserId:
+                                                        response.user.loginMethods[0].recipeUserId.getAsString(),
                                                 },
                                             ],
                                         };
@@ -234,7 +236,8 @@ describe(`overrideTest: ${printPath("[test/emailpassword/override.test.js]")}`,
                                             loginMethods: [
                                                 {
                                                     ...response.user.loginMethods[0],
-                                                    recipeUserId: response.user.loginMethods[0].recipeUserId.getAsString(),
+                                                    recipeUserId:
+                                                        response.user.loginMethods[0].recipeUserId.getAsString(),
                                                 },
                                             ],
                                         };
@@ -253,7 +256,8 @@ describe(`overrideTest: ${printPath("[test/emailpassword/override.test.js]")}`,
                                             loginMethods: [
                                                 {
                                                     ...response.user.loginMethods[0],
-                                                    recipeUserId: response.user.loginMethods[0].recipeUserId.getAsString(),
+                                                    recipeUserId:
+                                                        response.user.loginMethods[0].recipeUserId.getAsString(),
                                                 },
                                             ],
                                         };
diff --git a/test/session/sessionHandlingFuncsWithoutReq.test.js b/test/session/sessionHandlingFuncsWithoutReq.test.js
@@ -434,7 +434,8 @@ describe(`Session handling functions without modifying response: ${printPath(
                 tokensAfterRefresh.refreshToken,
                 true
             );
-            const tokensAfterRefreshWithDisable = sessionAfterRefreshWithDisabledAntiCsrf.getAllSessionTokensDangerously();
+            const tokensAfterRefreshWithDisable =
+                sessionAfterRefreshWithDisabledAntiCsrf.getAllSessionTokensDangerously();
             assert.strictEqual(tokensAfterRefreshWithDisable.accessAndFrontTokenUpdated, true);
         });
 
diff --git a/test/thirdparty/oidc.test.js b/test/thirdparty/oidc.test.js
@@ -26,7 +26,6 @@ describe(`oidcTest: ${printPath("[test/thirdparty/oidc.test.js]")}`, function ()
         ProcessState.getInstance().reset();
     });
 
-
     it("should use code challenge method S256 if supported by the provider", async function () {
         const connectionURI = await createCoreApplicationWithMultitenancy();
 
diff --git a/test/thirdparty/override.test.js b/test/thirdparty/override.test.js
@@ -269,7 +269,8 @@ describe(`overrideTest: ${printPath("[test/thirdparty/override.test.js]")}`, fun
                                             loginMethods: [
                                                 {
                                                     ...response.user.loginMethods[0],
-                                                    recipeUserId: response.user.loginMethods[0].recipeUserId.getAsString(),
+                                                    recipeUserId:
+                                                        response.user.loginMethods[0].recipeUserId.getAsString(),
                                                 },
                                             ],
                                         };
diff --git a/test/thirdparty/provider.config.test.js b/test/thirdparty/provider.config.test.js
@@ -73,7 +73,9 @@ describe(`providerConfigTest: ${printPath("[test/thirdparty/provider.config.test
         });
 
         for (let i = 0; i < 10; i++) {
-            await (await ThirdParty.getProvider("public", "google")).getAuthorisationRedirectURL({
+            await (
+                await ThirdParty.getProvider("public", "google")
+            ).getAuthorisationRedirectURL({
                 redirectURIOnProviderDashboard: "http://localhost:8080",
                 userContext: {},
             });

Original file line number	Diff line number	Diff line change
`@@ -6,4 +6,4 @@`
`6`	`6`	`"mochaJsonOutputReporterReporterOptions": {`
`7`	`7`	`"output": "test-results.json"`
`8`	`8`	`}`
`9`		`-}`
	`9`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -196,4 +196,4 @@`
`196`	`196`	`"browser": {`
`197`	`197`	`"fs": false`
`198`	`198`	`}`
`199`		`-}`
	`199`	`+}`