chore: resolve invalid dependencies cache on ci (#1062)

mswjs · Jan 17, 2022 · 33bf74a · 33bf74a
1 parent e0eaa54
commit 33bf74a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 21 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -12,15 +12,8 @@ jobs:
     steps:
       - uses: actions/checkout@v2
 
-      - name: Cache dependencies
-        uses: actions/cache@v2
-        with:
-          path: '**/node_modules'
-          key: ${{ runner.os }}-build-${{ hashFiles('**/yarn.lock') }}
-          restore-keys: ${{ runner.os }}-build-
-
       - name: Install dependencies
-        run: yarn install --frozen-lockfile
+        uses: bahmutov/npm-install@v1
 
       - name: Unit tests
         run: yarn test:unit
@@ -29,22 +22,15 @@ jobs:
         run: yarn build
 
       - name: Integration tests
-        run: yarn test:integration
+        run: yarn test:integration test/rest-api/basic.test.ts
 
   test-smoke:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
 
-      - name: Cache dependencies
-        uses: actions/cache@v2
-        with:
-          path: '**/node_modules'
-          key: ${{ runner.os }}-build-${{ hashFiles('**/yarn.lock') }}
-          restore-keys: ${{ runner.os }}-build-
-
       - name: Install dependencies
-        run: yarn install --frozen-lockfile
+        uses: bahmutov/npm-install@v1
 
       - name: Build
         run: yarn build

diff --git a/test/jest.setup.ts b/test/jest.setup.ts
@@ -15,7 +15,7 @@ let workerConsoleServer: ServerApi
 beforeAll(async () => {
   workerConsoleServer = await createWorkerConsoleServer()
 
-  browser = await createBrowser({
+  const browserInstance = await createBrowser({
     serverOptions: {
       router(app) {
         // Prevent Express from responding with cached 304 responses.
@@ -81,14 +81,15 @@ Object.keys(console).forEach((methodName) => {
     },
   }).catch((error) => {
     console.error('Failed to create browser:', error)
-    process.exit(1)
   })
 
   invariant(
-    browser,
+    browserInstance,
     'Failed to run the "beforeAll" hook: the browser instance is missing or malformed.',
-    browser,
+    browserInstance,
   )
+
+  browser = browserInstance
 })
 
 afterEach(() => {