refactor(tests): 优化测试代码格式，移除多余分号并调整换行

Author: people-sea

.github/workflows/frontend-test.yml

@@ -77,6 +77,7 @@ jobs:
         working-directory: web
         run: ./.github/ci/requirement.install.sh
       - name: Start Backend Server
+        id: start_backend_server
         working-directory: web
         run: ./.github/ci/run.start-backend-server.sh
         env:
@@ -87,12 +88,13 @@ jobs:
         working-directory: web
         run: ./.github/ci/run.test.sh
       - name: Upload E2E Test Report
+        if: always()
         uses: actions/upload-artifact@v4
         with:
           path: web/playwright-report/
           retention-days: 1
       - name: Stop Backend Server
-        if: always()
+        if: steps.start_backend_server.outcome == 'success'
         working-directory: web
         run: ./.github/ci/run.stop-backend-server.sh
 

web/tests/e2e/pages/loginPage.ts

@@ -1,15 +1,15 @@
-import { Page } from '@playwright/test';
+import type { Page } from '@playwright/test'
 
 export class LoginPage {
   constructor(public page: Page) { }
 
   async goto() {
-    await this.page.goto('/login');
+    await this.page.goto('/login')
   }
 
   async login(username: string, password: string) {
-    await this.page.fill('input[name="username"]', username);
-    await this.page.fill('input[name="password"]', password);
-    await this.page.click('button[name="登录"]');
+    await this.page.fill('input[name="username"]', username)
+    await this.page.fill('input[name="password"]', password)
+    await this.page.click('button[name="登录"]')
   }
-}
+}

web/tests/e2e/specs/base.spec.ts

@@ -1,23 +1,21 @@
-import { test, expect } from '@playwright/test';
+import { expect, test } from '@playwright/test'
 
 test.describe.serial('Base Tests', () => {
-
   test('Api Test', async ({ request }) => {
-    const response = await request.get('http://localhost:9501');
+    const response = await request.get('http://localhost:9501')
 
-    expect(response.status()).toBe(200);
+    expect(response.status()).toBe(200)
 
-    expect(await response.text()).toEqual('welcome use mineAdmin');
-  });
+    expect(await response.text()).toEqual('welcome use mineAdmin')
+  })
 
   test('Page Check', async ({ page }) => {
-    await page.goto('/');
-
-    await page.waitForLoadState('networkidle');
+    await page.goto('/')
 
-    await expect(page.getByRole('button', { name: '登录' })).toBeVisible();
+    await page.waitForLoadState('networkidle')
 
-    expect(await page.title()).toContain('MineAdmin');
-  });
+    await expect(page.getByRole('button', { name: '登录' })).toBeVisible()
 
-});
+    expect(await page.title()).toContain('MineAdmin')
+  })
+})

web/tests/e2e/specs/login.spec.ts

@@ -1,10 +1,10 @@
-import { test, expect } from '@playwright/test';
-import { LoginPage } from '../pages/loginPage.ts';
+import { test } from '@playwright/test'
+import { LoginPage } from '../pages/loginPage.ts'
 
 test.describe('登录模块', () => {
   test('正确账号可以登录', async ({ page }) => {
-    const loginPage = new LoginPage(page);
-    await loginPage.goto();
-    await loginPage.login('admin', '123456');
-  });
-});
+    const loginPage = new LoginPage(page)
+    await loginPage.goto()
+    await loginPage.login('admin', '123456')
+  })
+})