fix: 修复测试基础设施和 biome 警告，确保 CI 通过

- 添加 afterEach(cleanup) 防止 DOM 泄漏
- 添加 window.matchMedia mock 支持 antd
- 修复验证测试使用组件内置错误渲染
- 移除未使用的导入 (fireEvent, within, React)
- 替换非空断言 ref.current! 为可选链 ref.current?
- 修复 biome 格式化问题

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: chenJJ-88

packages/editable-table/package.json

@@ -12,15 +12,23 @@
         },
         "./style.css": "./dist/EditableTable.css"
     },
-    "files": ["dist"],
+    "files": [
+        "dist"
+    ],
     "scripts": {
         "build": "tsup",
         "prepublishOnly": "pnpm run build",
         "type-check": "tsc --noEmit",
         "test": "vitest",
         "coverage": "vitest run --coverage"
     },
-    "keywords": ["react", "editable-table", "table", "virtual-scroll", "formily"],
+    "keywords": [
+        "react",
+        "editable-table",
+        "table",
+        "virtual-scroll",
+        "formily"
+    ],
     "author": "chenJJ-88",
     "license": "MIT",
     "repository": {

packages/editable-table/src/components/EditableTable/__tests__/EditableTable.test.tsx

@@ -5,9 +5,9 @@
  *  - Chinese labels (保存/取消/编辑/提交) match the component's own button text.
  *  - `userEvent` is used for realistic interactions; `fireEvent` only where needed.
  */
-import { act, fireEvent, render, screen, within } from '@testing-library/react';
+import { act, render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import React, { createRef } from 'react';
+import { createRef } from 'react';
 import { describe, expect, it, vi } from 'vitest';
 import EditableTable, { type EditableColumn, type EditableTableInstance } from '../index';
 
@@ -31,22 +31,14 @@ const BASE_COLUMNS: EditableColumn<Person>[] = [
         title: '姓名',
         dataIndex: 'name',
         editRender: ({ value, onChange }) => (
-            <input
-                aria-label="name-input"
-                value={String(value ?? '')}
-                onChange={(e) => onChange(e.target.value)}
-            />
+            <input aria-label="name-input" value={String(value ?? '')} onChange={(e) => onChange(e.target.value)} />
         ),
     },
     {
         title: '年龄',
         dataIndex: 'age',
         editRender: ({ value, onChange }) => (
-            <input
-                aria-label="age-input"
-                value={String(value ?? '')}
-                onChange={(e) => onChange(e.target.value)}
-            />
+            <input aria-label="age-input" value={String(value ?? '')} onChange={(e) => onChange(e.target.value)} />
         ),
     },
 ];
@@ -228,15 +220,8 @@ describe('validation', () => {
             title: '姓名',
             dataIndex: 'name',
             rules: [{ required: true, message: '姓名为必填项' }],
-            editRender: ({ value, onChange, error }) => (
-                <div>
-                    <input
-                        aria-label="name-input"
-                        value={String(value ?? '')}
-                        onChange={(e) => onChange(e.target.value)}
-                    />
-                    {error && <span role="alert">{error}</span>}
-                </div>
+            editRender: ({ value, onChange }) => (
+                <input aria-label="name-input" value={String(value ?? '')} onChange={(e) => onChange(e.target.value)} />
             ),
         },
     ];
@@ -254,8 +239,7 @@ describe('validation', () => {
 
         await userEvent.click(screen.getByRole('button', { name: '提交' }));
 
-        expect(await screen.findAllByRole('alert')).not.toHaveLength(0);
-        expect(screen.getByText('姓名为必填项')).toBeInTheDocument();
+        expect(await screen.findByText('姓名为必填项')).toBeInTheDocument();
     });
 
     it('shows validation error on change when validateTrigger is change', async () => {
@@ -302,17 +286,11 @@ describe('ref methods', () => {
     it('addRow appends a new row', async () => {
         const ref = createRef<EditableTableInstance<Person>>();
         render(
-            <EditableTable<Person>
-                ref={ref}
-                rowKey="id"
-                columns={BASE_COLUMNS}
-                dataSource={DATA}
-                editableMode="all"
-            />,
+            <EditableTable<Person> ref={ref} rowKey="id" columns={BASE_COLUMNS} dataSource={DATA} editableMode="all" />,
         );
 
         act(() => {
-            ref.current!.addRow({ id: '99', name: '新行', age: 0 });
+            ref.current?.addRow({ id: '99', name: '新行', age: 0 });
         });
 
         const nameInputs = screen.getAllByLabelText('name-input');
@@ -322,17 +300,11 @@ describe('ref methods', () => {
     it('removeRow removes the correct row', async () => {
         const ref = createRef<EditableTableInstance<Person>>();
         render(
-            <EditableTable<Person>
-                ref={ref}
-                rowKey="id"
-                columns={BASE_COLUMNS}
-                dataSource={DATA}
-                editableMode="all"
-            />,
+            <EditableTable<Person> ref={ref} rowKey="id" columns={BASE_COLUMNS} dataSource={DATA} editableMode="all" />,
         );
 
         act(() => {
-            ref.current!.removeRow(0);
+            ref.current?.removeRow(0);
         });
 
         const nameInputs = screen.getAllByLabelText('name-input') as HTMLInputElement[];
@@ -344,15 +316,9 @@ describe('ref methods', () => {
     it('getData returns the current data', () => {
         const ref = createRef<EditableTableInstance<Person>>();
         render(
-            <EditableTable<Person>
-                ref={ref}
-                rowKey="id"
-                columns={BASE_COLUMNS}
-                dataSource={DATA}
-                editableMode="all"
-            />,
+            <EditableTable<Person> ref={ref} rowKey="id" columns={BASE_COLUMNS} dataSource={DATA} editableMode="all" />,
         );
 
-        expect(ref.current!.getData()).toEqual(DATA);
+        expect(ref.current?.getData()).toEqual(DATA);
     });
 });

packages/editable-table/src/test/setup.ts

@@ -1 +1,5 @@
+import { cleanup } from '@testing-library/react';
 import '@testing-library/jest-dom/vitest';
+import { afterEach } from 'vitest';
+
+afterEach(cleanup);

packages/editable-table/src/utils/__tests__/validate.test.ts

@@ -173,9 +173,7 @@ describe('validateRow', () => {
     });
 
     it('ignores columns without rules', () => {
-        const noRuleColumns: EditableColumn<Row>[] = [
-            { title: '邮箱', dataIndex: 'email' },
-        ];
+        const noRuleColumns: EditableColumn<Row>[] = [{ title: '邮箱', dataIndex: 'email' }];
         const row: Row = { name: '', age: null, email: '' };
         expect(validateRow(row, noRuleColumns)).toEqual({});
     });

packages/fast-editable-table/package.json

@@ -11,15 +11,23 @@
             "types": "./dist/index.d.ts"
         }
     },
-    "files": ["dist"],
+    "files": [
+        "dist"
+    ],
     "scripts": {
         "build": "tsup",
         "prepublishOnly": "pnpm run build",
         "type-check": "tsc --noEmit",
         "test": "vitest",
         "coverage": "vitest run --coverage"
     },
-    "keywords": ["react", "editable-table", "formily", "antd", "table"],
+    "keywords": [
+        "react",
+        "editable-table",
+        "formily",
+        "antd",
+        "table"
+    ],
     "author": "chenJJ-88",
     "license": "MIT",
     "repository": {

packages/fast-editable-table/src/__tests__/FormilyEditableTable.test.tsx

@@ -6,9 +6,9 @@
  */
 import { createForm } from '@formily/core';
 import { FormProvider } from '@formily/react';
-import { act, render, screen, within } from '@testing-library/react';
+import { act, render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import * as React from 'react';
+import type * as React from 'react';
 import { describe, expect, it } from 'vitest';
 import { FormilyEditableTable } from '../FormilyEditableTable';
 import type { IColumn } from '../types';
@@ -29,7 +29,7 @@ function makeColumns(): IColumn[] {
     return [
         {
             title: '姓名',
-            render: ({ index, field }) => (
+            render: ({ index, field: _field }) => (
                 <FormilyEditableTable.Field name="name">
                     <input aria-label={`name-${index}`} />
                 </FormilyEditableTable.Field>
@@ -42,7 +42,10 @@ function makeColumns(): IColumn[] {
     ];
 }
 
-function renderTable(form: ReturnType<typeof createForm>, overrides: Partial<React.ComponentProps<typeof FormilyEditableTable>> = {}) {
+function renderTable(
+    form: ReturnType<typeof createForm>,
+    overrides: Partial<React.ComponentProps<typeof FormilyEditableTable>> = {},
+) {
     return render(
         <FormProvider form={form}>
             <FormilyEditableTable
@@ -169,7 +172,7 @@ describe('remove row', () => {
             arrayField.remove(0);
         });
 
-        const remaining = await screen.findByLabelText('name-0') as HTMLInputElement;
+        const remaining = (await screen.findByLabelText('name-0')) as HTMLInputElement;
         expect(remaining.value).toBe('李四');
         expect(screen.queryByLabelText('name-1')).not.toBeInTheDocument();
     });

packages/fast-editable-table/test/setup.ts

@@ -1 +1,20 @@
+import { cleanup } from '@testing-library/react';
 import '@testing-library/jest-dom/vitest';
+import { afterEach, vi } from 'vitest';
+
+// antd requires matchMedia
+Object.defineProperty(window, 'matchMedia', {
+    writable: true,
+    value: vi.fn().mockImplementation((query: string) => ({
+        matches: false,
+        media: query,
+        onchange: null,
+        addListener: vi.fn(),
+        removeListener: vi.fn(),
+        addEventListener: vi.fn(),
+        removeEventListener: vi.fn(),
+        dispatchEvent: vi.fn(),
+    })),
+});
+
+afterEach(cleanup);