Merge pull request #1 from testradar/simplify-retries-implementation

Simplify test retries implementation
testradar · Feb 5, 2024 · 29eed31 · 29eed31
2 parents 0c015e8 + b037170
commit 29eed31
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -19,10 +19,7 @@ jobs:
           node-version: 16.x
           cache: 'npm'
       - run: npm ci
-      # `JEST_RETRIES` doesn't auto enable test retries.
-      # It needs to be used with `jest.retryTimes(numRetries)`.
-      # see: https://github.com/testradar/testradar-example-repo/blob/main/src/setupTests.ts#L7
-      - run: JEST_RETRIES=2 npm test
+      - run: npm test
 
   e2e-test:
     runs-on: ubuntu-latest
@@ -36,8 +33,6 @@ jobs:
           cache: 'npm'
       - name: Cypress Run
         uses: cypress-io/github-action@v5
-        env:
-          CYPRESS_RETRIES: 2
         with:
           command: npm run e2e
 

diff --git a/cypress.config.ts b/cypress.config.ts
@@ -7,4 +7,5 @@ export default defineConfig({
       setupTestRadar(on, { repositoryId: 674463592 });
     },
   },
+  retries: 2,
 });
diff --git a/src/setupTests.ts b/src/setupTests.ts
@@ -4,4 +4,4 @@
 // learn more: https://github.com/testing-library/jest-dom
 import '@testing-library/jest-dom';
 
-jest.retryTimes(Number(process.env.JEST_RETRIES ?? 0));
+jest.retryTimes(2);
-Original file line number
+Diff line change
@@ Expand Up / @@ -7,4 +7,5 @@ export default defineConfig({ @@
           setupTestRadar(on, { repositoryId: 674463592 });
         },
       },
+      retries: 2,
     });