fix: skip cli verification when publishPub is false (#119)

zeshuaro · Dec 3, 2023 · fc46428 · fc46428
1 parent cea527b
commit fc46428
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/verifyConditions.ts b/src/verifyConditions.ts
@@ -19,13 +19,12 @@ export const verifyConditions = async (
     }
 
     await getGoogleIdentityToken(GOOGLE_SERVICE_ACCOUNT_KEY);
+    await verifyCommand(cli);
   } else {
     logger.log(
-      `Skipping Google service account key verification as publishPub is ${publishPub}`
+      `Skipping Google service account key and ${cli} CLI verification as publishPub is ${publishPub}`
     );
   }
-
-  await verifyCommand(cli);
 };
 
 const verifyCommand = async (command: string) => {

diff --git a/tests/verifyConditions.test.ts b/tests/verifyConditions.test.ts
@@ -48,8 +48,8 @@ describe('verifyConditions', () => {
 
     await verifyConditions(config, context);
 
-    expect(execa).toBeCalledWith(cli);
     expect(getGoogleIdentityToken).toBeCalledTimes(0);
+    expect(execa).toBeCalledTimes(0);
   });
 
   test('error due to missing environment variable', async () => {