ci: remove self-host in build.yml (#1363)

Co-authored-by: DarkSky <darksky2048@gmail.com>

.github/workflows/build.yml

@@ -18,7 +18,7 @@ jobs:
   build:
     name: Build on Pull Request
     if: github.ref != 'refs/heads/master'
-    runs-on: self-hosted
+    runs-on: ubuntu-latest
     environment: development
 
     steps:
@@ -40,6 +40,9 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nextjs-${{ hashFiles('**/pnpm-lock.yaml') }}-
 
+      - run: pnpm i
+      - run: pnpm lint --max-warnings=0
+
       - name: Build
         run: pnpm i && pnpm build
         env:
@@ -60,9 +63,13 @@ jobs:
           name: artifact
           path: ./apps/web/.next
 
-  lint:
+  e2e-test:
-    name: Lint and E2E Test
+    name: E2E Test
-    runs-on: self-hosted
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        shard: [1, 2, 3, 4]
     environment: development
     needs: build
 
@@ -81,20 +88,12 @@ jobs:
           name: artifact
           path: ./apps/web/.next
 
-      - name: Lint & E2E Test & Unit Test
+      - name: Install playwright browsers
-        run: |
+        run: npx playwright install chromium
-          pnpm lint --max-warnings=0
+      - name: Run playwright tests
-          npx playwright install chromium
+        run: pnpm test -- --forbid-only --shard=${{ matrix.shard }}/${{ strategy.job-total }}
-          pnpm run test:coverage
+        env:
-          pnpm run test:unit
+          COVERAGE: true
-        # env:
-        #   NEXT_PUBLIC_FIREBASE_API_KEY: ${{ secrets.NEXT_PUBLIC_FIREBASE_API_KEY }}
-        #   NEXT_PUBLIC_FIREBASE_AUTH_DOMAIN: ${{ secrets.NEXT_PUBLIC_FIREBASE_AUTH_DOMAIN }}
-        #   NEXT_PUBLIC_FIREBASE_PROJECT_ID: ${{ secrets.NEXT_PUBLIC_FIREBASE_PROJECT_ID }}
-        #   NEXT_PUBLIC_FIREBASE_STORAGE_BUCKET: ${{ secrets.NEXT_PUBLIC_FIREBASE_STORAGE_BUCKET }}
-        #   NEXT_PUBLIC_FIREBASE_MESSAGING_SENDER_ID: ${{ secrets.NEXT_PUBLIC_FIREBASE_MESSAGING_SENDER_ID }}
-        #   NEXT_PUBLIC_FIREBASE_APP_ID: ${{ secrets.NEXT_PUBLIC_FIREBASE_APP_ID }}
-        #   NEXT_PUBLIC_FIREBASE_MEASUREMENT_ID: ${{ secrets.NEXT_PUBLIC_FIREBASE_MEASUREMENT_ID }}
 
       - name: Collect code coverage report
         run: pnpm exec nyc report -t .nyc_output --report-dir .coverage --reporter=lcov
@@ -112,14 +111,55 @@ jobs:
         if: ${{ failure() }}
         uses: actions/upload-artifact@v2
         with:
-          name: test-results-main
+          name: test-results-e2e
+          path: ./test-results
+          if-no-files-found: ignore
+
+  unit-test:
+    name: Unit Test
+    runs-on: ubuntu-latest
+    environment: development
+    needs: build
+
+    steps:
+      - uses: actions/checkout@v3
+      - uses: pnpm/action-setup@v2
+      - uses: actions/setup-node@v3
+        with:
+          node-version-file: '.nvmrc'
+          cache: 'pnpm'
+      - run: pnpm i
+
+      - name: Download artifact
+        uses: actions/download-artifact@v3
+        with:
+          name: artifact
+          path: ./apps/web/.next
+
+      - name: Unit Test
+        run: pnpm run test:unit:coverage
+
+      - name: Upload unit test coverage results
+        uses: codecov/codecov-action@v3
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: ./.coverage/store/lcov.info
+          flags: unittest
+          name: affine
+          fail_ci_if_error: true
+
+      - name: Upload test results
+        if: ${{ failure() }}
+        uses: actions/upload-artifact@v2
+        with:
+          name: test-results-unit
           path: ./test-results
           if-no-files-found: ignore
 
   build-frontend:
     name: Build frontend
     if: github.ref == 'refs/heads/master'
-    runs-on: self-hosted
+    runs-on: ubuntu-latest
     environment: production
 
     steps:

playwright.config.ts

@@ -1,4 +1,7 @@
-import type { PlaywrightTestConfig } from '@playwright/test';
+import type {
+  PlaywrightTestConfig,
+  PlaywrightWorkerOptions,
+} from '@playwright/test';
 // import { devices } from '@playwright/test';
 
 /**
@@ -12,9 +15,12 @@ import type { PlaywrightTestConfig } from '@playwright/test';
  */
 const config: PlaywrightTestConfig = {
   testDir: './tests',
-  timeout: 30 * 1000,
+  fullyParallel: true,
+  timeout: process.env.CI ? 50_000 : 30_000,
   use: {
-    browserName: 'chromium',
+    browserName:
+      (process.env.BROWSER as PlaywrightWorkerOptions['browserName']) ??
+      'chromium',
     viewport: { width: 1440, height: 800 },
     actionTimeout: 5 * 1000,
     locale: 'en-US',
@@ -25,11 +31,13 @@ const config: PlaywrightTestConfig = {
     // Record video only when retrying a test for the first time.
     video: 'on-first-retry',
   },
-
-  fullyParallel: true,
   forbidOnly: !!process.env.CI,
-  retries: process.env.CI ? 3 : 0,
+  workers: 4,
-  workers: process.env.CI ? '100%' : undefined,
+  retries: 1,
+  // 'github' for GitHub Actions CI to generate annotations, plus a concise 'dot'
+  // default 'list' when running locally
+  // See https://playwright.dev/docs/test-reporters#github-actions-annotations
+  reporter: process.env.CI ? 'github' : 'list',
 
   webServer: {
     command: 'pnpm build && pnpm start -p 8080',
@@ -41,11 +49,11 @@ const config: PlaywrightTestConfig = {
       ENABLE_DEBUG_PAGE: '1',
     },
   },
-
-  // 'github' for GitHub Actions CI to generate annotations, plus a concise 'dot'
-  // default 'list' when running locally
-  // See https://playwright.dev/docs/test-reporters#github-actions-annotations
-  reporter: process.env.CI ? 'github' : 'list',
 };
 
+if (process.env.CI) {
+  config.retries = 3;
+  config.workers = '50%';
+}
+
 export default config;