set environment var

matomo-org · Feb 29, 2024 · 1d2ea4c · 1d2ea4c
1 parent a721fe0
commit 1d2ea4c
Show file tree

Hide file tree

Showing 2 changed files with 43 additions and 38 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -66,6 +66,8 @@ jobs:
 
       - name: Test
         run: npm test
+        env:
+          USE_LOCALTUNNEL: ${{ secrets.USE_LOCALTUNNEL }}
 
       - name: Archive test artifacts (actual)
         uses: actions/upload-artifact@v3

diff --git a/tests/appscript/config.spec.ts b/tests/appscript/config.spec.ts
@@ -70,52 +70,55 @@ describe('config', () => {
       expect(result).toEqual(getExpectedResponse(result, 'config', 'step2'));
     });
 
-    if (process.env.USE_LOCALTUNNEL) {
-      describe('#localtunnel tests', () => {
-        let server: ReturnType<typeof makeMatomo4MockServer>;
-        let tunnel;
-
-        if (process.env.USE_LOCALTUNNEL) {
-          beforeAll(async () => {
-            // create localtunnel
-            tunnel = await localtunnel({
-              port: 3000,
-              host: process.env.USE_LOCALTUNNEL,
-            });
+    describe('#localtunnel tests', () => {
+      let server: ReturnType<typeof makeMatomo4MockServer>;
+      let tunnel;
+
+      if (process.env.USE_LOCALTUNNEL) {
+        beforeAll(async () => {
+          // create localtunnel
+          tunnel = await localtunnel({
+            port: 3000,
+            host: process.env.USE_LOCALTUNNEL,
           });
+        });
 
-          afterEach(async () => {
-            if (server) {
-              await new Promise((r) => {
-                server.close(r);
-              });
-              server = null;
-            }
-          });
+        afterEach(async () => {
+          if (server) {
+            await new Promise((r) => {
+              server.close(r);
+            });
+            server = null;
+          }
+        });
 
-          afterAll(async () => {
-            await tunnel.close();
-          });
+        afterAll(async () => {
+          await tunnel.close();
+        });
+      }
+
+      it('should return the expected response when the Matomo version is < 4.14', async () => {
+        if (!process.env.USE_LOCALTUNNEL) {
+          console.log('*** SKIPPING TEST ***');
+          return;
         }
 
-        it('should return the expected response when the Matomo version is < 4.14', async () => {
-          server = makeMatomo4MockServer(3000);
+        server = makeMatomo4MockServer(3000);
 
-          // use the mock server's path that forces a non-random error
-          const setCredentialsResult = await Clasp.run('setCredentials', {
-            userToken: {
-              username: tunnel.url,
-              token: 'ignored',
-            },
-          });
+        // use the mock server's path that forces a non-random error
+        const setCredentialsResult = await Clasp.run('setCredentials', {
+          userToken: {
+            username: tunnel.url,
+            token: 'ignored',
+          },
+        });
 
-          expect(setCredentialsResult).toEqual({ errorCode: 'NONE' });
+        expect(setCredentialsResult).toEqual({ errorCode: 'NONE' });
 
-          let result = await Clasp.run('getConfig', {});
-          result = cleanUpSelects(result);
-          expect(result).toEqual(getExpectedResponse(result, 'config', 'oldMatomoVersion'));
-        });
+        let result = await Clasp.run('getConfig', {});
+        result = cleanUpSelects(result);
+        expect(result).toEqual(getExpectedResponse(result, 'config', 'oldMatomoVersion'));
       });
-    }
+    });
   });
 });