run mock auth tests without azure creds

taskcluster · Sep 16, 2020 · 286d7ed · 286d7ed
1 parent 0919569
commit 286d7ed
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/changelog/issue-3503.md b/changelog/issue-3503.md
@@ -0,0 +1,4 @@
+audience: developers
+level: silent
+reference: issue 3503
+---
diff --git a/services/auth/test/helper.js b/services/auth/test/helper.js
@@ -55,6 +55,10 @@ exports.withCfg = (mock, skipping) => {
     return;
   }
   suiteSetup(async function() {
+    if (skipping()) {
+      return;
+    }
+
     exports.cfg = await exports.load('cfg');
 
     exports.load.save();
@@ -66,12 +70,20 @@ exports.withCfg = (mock, skipping) => {
       description: 'testing',
     })));
 
-    // override cfg.app.azureAccounts based on the azure secret
-    const sec = exports.secrets.get('azure');
-    exports.load.cfg('app.azureAccounts', { [sec.accountId]: sec.accessKey });
+    // override cfg.app.azureAccounts based on the azure secret, or mock it
+    if (mock) {
+      exports.load.cfg('app.azureAccounts', {});
+    } else {
+      const sec = exports.secrets.get('azure');
+      exports.load.cfg('app.azureAccounts', { [sec.accountId]: sec.accessKey });
+    }
   });
 
   suiteTeardown(async function() {
+    if (skipping()) {
+      return;
+    }
+
     exports.load.restore();
   });
 };