test: fix test interceptor

Author: mutoe

.github/workflows/test.yml

@@ -127,7 +127,7 @@ jobs:
         run: pnpm test:playwright
 
       - uses: actions/upload-artifact@v4
-        if: always()
+        if: ${{ !cancelled() }}
         with:
           name: playwright-report
           path: playwright-report/

playwright.config.ts

@@ -33,8 +33,8 @@ export default defineConfig({
     /* Base URL to use in actions like `await page.goto('/')`. */
     baseURL,
 
-    navigationTimeout: isCI ? 10_000 : 4000,
-    actionTimeout: isCI ? 10_000 : 4000,
+    navigationTimeout: isCI ? 10_000 : 10_000,
+    actionTimeout: isCI ? 10_000 : 10_000,
 
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     screenshot: 'only-on-failure',
@@ -49,14 +49,14 @@ export default defineConfig({
           name: 'chromium',
           use: { ...devices['Desktop Chrome'] },
         },
-        {
-          name: 'firefox',
-          use: { ...devices['Desktop Firefox'] },
-        },
-        {
-          name: 'webkit',
-          use: { ...devices['Desktop Safari'] },
-        },
+        // {
+        //   name: 'firefox',
+        //   use: { ...devices['Desktop Firefox'] },
+        // },
+        // {
+        //   name: 'webkit',
+        //   use: { ...devices['Desktop Safari'] },
+        // },
       ]
     : [
         {
@@ -69,7 +69,7 @@ export default defineConfig({
 
   /* Run your local dev server before starting the tests */
   webServer: {
-    command: isCI ? 'pnpm serve' : 'npm run dev',
+    command: isCI ? 'pnpm serve' : 'pnpm dev',
     url: baseURL,
     reuseExistingServer: !isCI,
     ignoreHTTPSErrors: true,

playwright/page-objects/conduit.page-object.ts

@@ -2,10 +2,10 @@ import fs from 'node:fs/promises'
 import path from 'node:path'
 import url from 'node:url'
 import type { Page, Response } from '@playwright/test'
-import type { User } from 'src/services/api.ts'
+import type { User } from 'src/services/api'
 import { Route } from '../constant'
-import { expect } from '../extends.ts'
-import { boxedStep } from '../utils/test-decorators.ts'
+import { expect } from '../extends'
+import { boxedStep } from '../utils/test-decorators'
 
 const __dirname = url.fileURLToPath(new URL('.', import.meta.url))
 const fixtureDir = path.join(__dirname, '../../cypress/fixtures')
@@ -23,6 +23,8 @@ export class ConduitPageObject {
     timeout?: number
   } = {}): Promise<() => Promise<Response>> {
     await this.page.route(url, async route => {
+      if (route.request().url().endsWith('.ts'))
+        return await route.continue()
       if (route.request().method() !== method)
         return await route.continue()