ci(root): remove redundant API unit test (#6678)

novuhq · Oct 11, 2024 · 3671e95 · 3671e95
1 parent db33f89
commit 3671e95
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/.github/workflows/on-pr.yml b/.github/workflows/on-pr.yml
@@ -244,6 +244,7 @@ jobs:
       test-e2e-affected: ${{ contains(fromJson(needs.get-affected.outputs.test-e2e), '@novu/api') || contains(fromJson(needs.get-affected.outputs.test-e2e), '@novu/worker') }}
       test-e2e-ee-affected: ${{ contains(fromJson(needs.get-affected.outputs.test-e2e-ee), '@novu/api') || contains(fromJson(needs.get-affected.outputs.test-e2e-ee), '@novu/worker') }}
       job-name: ${{ matrix.name }}
+      test-unit: false
     secrets: inherit    
 
   test_e2e_web:

diff --git a/.github/workflows/reusable-api-e2e.yml b/.github/workflows/reusable-api-e2e.yml
@@ -14,6 +14,11 @@ on:
         required: false
         default: true
         type: boolean
+      test-unit:
+        description: 'detect if we should run unit tests'
+        required: false
+        default: true
+        type: boolean
       ee:
         description: 'use the ee version of api'
         required: false
@@ -117,6 +122,7 @@ jobs:
         run: sudo kill -9 $(sudo lsof -t -i:1342)
 
       - name: Run unit tests
+        if: inputs.test-unit
         run: |
           cd apps/api && pnpm test