chore: add test retry (#354)

add test retry
ChainSafe · Feb 1, 2024 · 02caf58 · 02caf58
1 parent eb95e0c
commit 02caf58
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -139,8 +139,14 @@ jobs:
           key: metamask-extension
       - name: Install
         run: yarn --immutable
-      - name: Tests
-        run: 'yarn run test:${{matrix.automation}}:${{ matrix.mm-version }} --timeout 50000'
+
+      - name: Run tests with retry
+        uses: nick-fields/retry@v2
+        with:
+          timeout_minutes: 5
+          max_attempts: 3
+          command: 'yarn run test:${{matrix.automation}}:${{ matrix.mm-version }} --timeout 50000'
+
       - uses: actions/upload-artifact@v3
         if: always()
         with:
@@ -181,8 +187,13 @@ jobs:
 
       - name: Install
         run: yarn --immutable
-      - name: Tests UserData
-        run: 'yarn run test:${{matrix.automation}}:userData --timeout 240000'
+
+      - name: Run Tests UserData with retry
+        uses: nick-fields/retry@v2
+        with:
+          timeout_minutes: 24
+          max_attempts: 3
+          command: 'yarn run test:${{matrix.automation}}:userData --timeout 240000'
 
   maybe-release:
     name: release

diff --git a/src/setup/setupActions.ts b/src/setup/setupActions.ts
@@ -45,7 +45,9 @@ export async function acceptTheRisks(
   metaMaskPage: DappeteerPage
 ): Promise<void> {
   await waitForOverlay(metaMaskPage);
-  await metaMaskPage.waitForTimeout(300);
+  await metaMaskPage.waitForSelector('[data-testid="experimental-area"]', {
+    visible: true,
+  });
   await clickOnElement(metaMaskPage, "I accept the risks");
 }