test: added retries for ci

.config/vite.browser.mts

@@ -18,6 +18,7 @@ export default mergeConfig(baseConfig, defineConfig({
         fakeTimers: {
             toFake: ['setTimeout', 'clearTimeout', 'setInterval', 'clearInterval', 'Date']
         },
+        retry: process.env.CI ? 3 : 0,
         // for whatever reason using exclude-s makes the vite never start the browser, so we use skip-s instead.
         // exclude: [
         //     './packages/crypto-node/**',

.github/workflows/test.yaml

@@ -57,7 +57,13 @@ jobs:
       - name: 'Initialize browser'
         run: pnpm exec playwright install --with-deps ${{ matrix.browser }}
       - name: 'Run tests'
-        run: pnpm run test:browser --browser.name=${{ matrix.browser }}
+        # i wish we didn't have to do this, but vitest in browser is very flaky
+        # see: https://github.com/vitest-dev/vitest/issues/4173
+        uses: nick-fields/retry@v2
+        with:
+          max_attempts: 3
+          timeout_minutes: 30
+          command: pnpm run test:browser --browser.name=${{ matrix.browser }}
   
   e2e:
     runs-on: ubuntu-latest