Skip to content

chore: fix eslint in blocksuite #20463

chore: fix eslint in blocksuite

chore: fix eslint in blocksuite #20463

Triggered via pull request December 20, 2024 13:51
Status Success
Total duration 16m 1s
Artifacts 8

build-test.yml

on: pull_request
Matrix: build-native
Build Server native
1m 5s
Build Server native
Build @affine/electron renderer
2m 22s
Build @affine/electron renderer
test-build-mobile-app  /  output-env
3s
test-build-mobile-app / output-env
Matrix: Analyze
Matrix: Legacy Blocksuite E2E Test
Matrix: E2E Mobile Test
Matrix: E2E Test
Matrix: server-e2e-test
Matrix: Server Copilot E2E Test
Server Test
7m 28s
Server Test
Server Copilot Api Test
24s
Server Copilot Api Test
Matrix: desktop-test
test-build-mobile-app  /  build-android-web
2m 42s
test-build-mobile-app / build-android-web
test-build-mobile-app  /  build-ios-web
2m 32s
test-build-mobile-app / build-ios-web
test-build-mobile-app  /  android
2m 31s
test-build-mobile-app / android
test-build-mobile-app  /  ios
2m 32s
test-build-mobile-app / ios
3, 2, 1 Launch
0s
3, 2, 1 Launch
Fit to window
Zoom out
Zoom in

Annotations

12 errors, 60 warnings, and 24 notices
selection/native.spec.ts:266:1 › cursor move to up and down with children block: blocksuite/tests-legacy/selection/native.spec.ts#L295
1) selection/native.spec.ts:266:1 › cursor move to up and down with children block ─────────────── Error: expect(received).toBeGreaterThanOrEqual(expected) Expected: >= 12 Received: -1 293 | const textTwo = await getInlineSelectionText(page); 294 | expect(textTwo).toBe('arrow down test 1'); > 295 | expect(indexTwo).toBeGreaterThanOrEqual(12); | ^ 296 | expect(indexTwo).toBeLessThanOrEqual(17); 297 | await page.keyboard.press('ArrowDown'); 298 | const textThree = await getInlineSelectionText(page); at /home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/selection/native.spec.ts:295:20
attachment.spec.ts:418:1 › should attachment can be deleted: blocksuite/tests-legacy/attachment.spec.ts#L69
1) attachment.spec.ts:418:1 › should attachment can be deleted ─────────────────────────────────── TimeoutError: page.waitForEvent: Timeout 5000ms exceeded while waiting for event "filechooser" =========================== logs =========================== waiting for event "filechooser" ============================================================ 67 | await expect(slashMenu).toBeVisible(); 68 | > 69 | const fileChooser = page.waitForEvent('filechooser'); | ^ 70 | await pressEnter(page); 71 | await sleep(100); 72 | await (await fileChooser).setFiles(FILE_PATH); at insertAttachment (/home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/attachment.spec.ts:69:30) at /home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/attachment.spec.ts:424:3
clipboard/list.spec.ts:307:1 › paste non-nested lists to a nested list: blocksuite/tests-legacy/utils/asserts.ts#L209
2) clipboard/list.spec.ts:307:1 › paste non-nested lists to a nested list ──────────────────────── Error: expect(received).toEqual(expected) // deep equality - Expected - 2 + Received + 2 Array [ - "123", - "456aaa", + "aaa123", + "456", "bbb", ] at utils/asserts.ts:209 207 | }); 208 | }, currentEditorIndex); > 209 | expect(actualTexts).toEqual(texts); | ^ 210 | } 211 | 212 | export async function assertEdgelessCanvasText(page: Page, text: string) { at assertRichTexts (/home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/utils/asserts.ts:209:23) at /home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/clipboard/list.spec.ts:342:3
[Mobile Safari] › explorer-folder.spec.ts:49:5 › create a folder: tests/affine-mobile/e2e/explorer-folder.spec.ts#L19
1) [Mobile Safari] › explorer-folder.spec.ts:49:5 › create a folder ────────────────────────────── Error: expect(received).toEqual(expected) // deep equality Expected: "rename-input" Received: null 17 | const isRenameInputFocused = async (page: Page) => { 18 | const focusElTestid = await getAttrOfActiveElement(page); > 19 | expect(focusElTestid).toEqual('rename-input'); | ^ 20 | }; 21 | 22 | const createRootFolder = async (page: Page, name: string) => { at isRenameInputFocused (/home/runner/work/AFFiNE/AFFiNE/tests/affine-mobile/e2e/explorer-folder.spec.ts:19:25) at createRootFolder (/home/runner/work/AFFiNE/AFFiNE/tests/affine-mobile/e2e/explorer-folder.spec.ts:27:3) at /home/runner/work/AFFiNE/AFFiNE/tests/affine-mobile/e2e/explorer-folder.spec.ts:50:16
[Mobile Safari] › explorer-tag.spec.ts:59:5 › rename a tag from explorer: tests/affine-mobile/e2e/explorer-tag.spec.ts#L67
1) [Mobile Safari] › explorer-tag.spec.ts:59:5 › rename a tag from explorer ────────────────────── Error: expect(received).toEqual(expected) // deep equality Expected: "rename-input" Received: null 65 | await menu.getByTestId('rename-tag').tap(); 66 | const focusedTestid = await getAttrOfActiveElement(page); > 67 | expect(focusedTestid).toEqual('rename-input'); | ^ 68 | await page.keyboard.type(appendedName); 69 | await menu.getByTestId('rename-confirm').tap(); 70 | await expect(tag).not.toBeVisible(); at /home/runner/work/AFFiNE/AFFiNE/tests/affine-mobile/e2e/explorer-tag.spec.ts:67:25
edgeless/note/resize.spec.ts:138:1 › resize note then auto size and custom size: blocksuite/tests-legacy/utils/asserts.ts#L888
1) edgeless/note/resize.spec.ts:138:1 › resize note then auto size and custom size ─────────────── Error: expect(received).toBeCloseTo(expected, precision) Expected: 192 Received: 92 Expected precision: 0 Expected difference: < 0.5 Received difference: 100 at utils/asserts.ts:888 886 | expect(a.y).toBeCloseTo(b.y, 0); 887 | expect(a.w).toBeCloseTo(b.w, 0); > 888 | expect(a.h).toBeCloseTo(b.h, 0); | ^ 889 | } 890 | 891 | export function assertDOMRectEqual(a: DOMRect, b: DOMRect) { at assertRectEqual (/home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/utils/asserts.ts:888:15) at assertNoteRectEqual (/home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/utils/asserts.ts:881:3) at /home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/edgeless/note/resize.spec.ts:188:3
edgeless/shortcut.spec.ts:119:1 › should not switch shapes in editing: blocksuite/tests-legacy/utils/actions/edgeless.ts#L368
1) edgeless/shortcut.spec.ts:119:1 › should not switch shapes in editing ───────────────────────── TimeoutError: locator.click: Timeout 5000ms exceeded. Call log: - waiting for locator('edgeless-slide-menu edgeless-tool-icon-button').filter({ hasText: 'Square' }) - locator resolved to <edgeless-tool-icon-button role="button">…</edgeless-tool-icon-button> - attempting click action 2 × waiting for element to be visible, enabled and stable - element is not stable - retrying click action - waiting 20ms - waiting for element to be visible, enabled and stable - element was detached from the DOM, retrying at utils/actions/edgeless.ts:368 366 | .locator('edgeless-slide-menu edgeless-tool-icon-button') 367 | .filter({ hasText: shape }); > 368 | await squareShapeButton.click(); | ^ 369 | break; 370 | } 371 | } at setEdgelessTool (/home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/utils/actions/edgeless.ts:368:31) at /home/runner/work/AFFiNE/AFFiNE/blocksuite/tests-legacy/edgeless/shortcut.spec.ts:137:3
peek-view.spec.ts:112:5 › can open peek view for embedded frames: tests/affine-local/e2e/peek-view.spec.ts#L1
1) peek-view.spec.ts:112:5 › can open peek view for embedded frames ────────────────────────────── Test timeout of 50000ms exceeded.
peek-view.spec.ts:112:5 › can open peek view for embedded frames: tests/affine-local/e2e/peek-view.spec.ts#L184
1) peek-view.spec.ts:112:5 › can open peek view for embedded frames ────────────────────────────── Error: expect(locator).not.toBeVisible() Locator: getByTestId('peek-view-modal') Expected: not visible Received: visible Call log: - expect.not.toBeVisible with timeout 5000ms - waiting for getByTestId('peek-view-modal') 182 | // can close modal when navigate 183 | await openHomePage(page); > 184 | await expect(page.getByTestId('peek-view-modal')).not.toBeVisible(); | ^ 185 | }); 186 | 187 | test.skip('can open peek view for fav link', async ({ page }) => { at /home/runner/work/AFFiNE/AFFiNE/tests/affine-local/e2e/peek-view.spec.ts:184:57
doc-info-modal.spec.ts:123:5 › add custom property: tests/affine-local/e2e/doc-info-modal.spec.ts#L1
1) doc-info-modal.spec.ts:123:5 › add custom property ──────────────────────────────────────────── Test timeout of 50000ms exceeded.
doc-info-modal.spec.ts:123:5 › add custom property: tests/kit/utils/properties.ts#L136
1) doc-info-modal.spec.ts:123:5 › add custom property ──────────────────────────────────────────── Error: page.waitForTimeout: Test timeout of 50000ms exceeded. at ../../kit/utils/properties.ts:136 134 | await page.keyboard.press('Escape'); 135 | } > 136 | await page.waitForTimeout(500); | ^ 137 | }; 138 | 139 | export const expectPropertyOrdering = async ( at addCustomProperty (/home/runner/work/AFFiNE/AFFiNE/tests/kit/utils/properties.ts:136:14) at /home/runner/work/AFFiNE/AFFiNE/tests/affine-local/e2e/doc-info-modal.spec.ts:132:3
links.spec.ts:249:5 › allow switching to embed view when linking to the other document with mode: tests/affine-local/e2e/links.spec.ts#L286
1) links.spec.ts:249:5 › allow switching to embed view when linking to the other document with mode TimeoutError: locator.click: Timeout 5000ms exceeded. Call log: - waiting for locator('affine-embed-linked-doc-block') - locator resolved to <affine-embed-linked-doc-block contenteditable="false" data-block-id="xANEsNCEvchfTvhYzeM4i">…</affine-embed-linked-doc-block> - attempting click action 2 × waiting for element to be visible, enabled and stable - element is visible, enabled and stable - scrolling into view if needed - done scrolling - <iframe src="about:blank" id="webpack-dev-server-client-overlay"></iframe> intercepts pointer events - retrying click action - waiting 20ms 2 × waiting for element to be visible, enabled and stable - element is visible, enabled and stable - scrolling into view if needed - done scrolling - <iframe src="about:blank" id="webpack-dev-server-client-overlay"></iframe> intercepts pointer events - retrying click action - waiting 100ms 9 × waiting for element to be visible, enabled and stable - element is visible, enabled and stable - scrolling into view if needed - done scrolling - <iframe src="about:blank" id="webpack-dev-server-client-overlay"></iframe> intercepts pointer events - retrying click action - waiting 500ms 284 | 285 | await expect(cardLink).toBeVisible(); > 286 | await cardLink.click(); | ^ 287 | 288 | // refocus 289 | await cardLink.click(); at /home/runner/work/AFFiNE/AFFiNE/tests/affine-local/e2e/links.spec.ts:286:18
Check yarn binary
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
test-build-mobile-app / output-env
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build Server native
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Server Copilot Api Test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Server Copilot E2E Test (2, 3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Server Copilot E2E Test (1, 3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Server Copilot E2E Test (3, 3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Run native tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build @affine/electron renderer
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build AFFiNE native (x86_64-unknown-linux-gnu)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
test-build-mobile-app / build-ios-web
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
test-build-mobile-app / build-android-web
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Mobile Test (5)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Mobile Test (4)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-mobile/[Mobile Chrome] › explorer-favorite.spec.ts#L1
tests/affine-mobile/[Mobile Chrome] › explorer-favorite.spec.ts took 45.7s
Legacy Blocksuite E2E Test (10)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (1)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Mobile Test (1)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-mobile/[Mobile Safari] › back-button-visibility.spec.ts#L1
tests/affine-mobile/[Mobile Safari] › back-button-visibility.spec.ts took 47.7s
Legacy Blocksuite E2E Test (8)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (2)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (9)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Mobile Test (3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (7)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Mobile Test (2)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-mobile/[Mobile Safari] › explorer-folder.spec.ts#L1
tests/affine-mobile/[Mobile Safari] › explorer-folder.spec.ts took 53.0s
Legacy Blocksuite E2E Test (5)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (6)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Legacy Blocksuite E2E Test (4)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Test (4)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-local/local-first-favorites-items.spec.ts#L1
tests/affine-local/local-first-favorites-items.spec.ts took 42.7s
E2E Test (5)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
test-build-mobile-app / android
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Test (1)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Test (2)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E Test (3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-local/local-first-avatar.spec.ts#L1
tests/affine-local/local-first-avatar.spec.ts took 15.3s
Lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (javascript)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (javascript)
1 issue was detected with this workflow: CodeQL language 'javascript' is referenced by more than one entry in the 'language' matrix parameter for job 'analyze'. This may result in duplicate alerts. Please edit the 'language' matrix parameter to keep only one of the following: 'javascript', 'typescript'.
Analyze (javascript)
Unable to validate code scanning workflow: DuplicateLanguageInMatrix
Analyze (typescript)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Analyze (typescript)
1 issue was detected with this workflow: CodeQL language 'javascript' is referenced by more than one entry in the 'language' matrix parameter for job 'analyze'. This may result in duplicate alerts. Please edit the 'language' matrix parameter to keep only one of the following: 'javascript', 'typescript'.
Analyze (typescript)
Unable to validate code scanning workflow: DuplicateLanguageInMatrix
Server Test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Server Desktop E2E Test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-desktop-cloud/basic.spec.ts#L1
tests/affine-desktop-cloud/basic.spec.ts took 58.6s
Server E2E Test 3/3
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-cloud/workspace.spec.ts#L1
tests/affine-cloud/workspace.spec.ts took 1.4m
Server E2E Test 1/3
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-cloud/collaboration.spec.ts#L1
tests/affine-cloud/collaboration.spec.ts took 2.6m
Slow Test: tests/affine-cloud/login.spec.ts#L1
tests/affine-cloud/login.spec.ts took 1.3m
Unit Test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Server E2E Test 2/3
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Slow Test: tests/affine-cloud/share-page.spec.ts#L1
tests/affine-cloud/share-page.spec.ts took 3.2m
Slow Test: tests/affine-cloud/open-in-app.spec.ts#L1
tests/affine-cloud/open-in-app.spec.ts took 52.5s
Slow Test: tests/affine-cloud/page-history.spec.ts#L1
tests/affine-cloud/page-history.spec.ts took 47.8s
Desktop Test (ubuntu-latest, linux, x64, x86_64-unknown-linux-gnu, true)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
3, 2, 1 Launch
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
🎭 Playwright Run Summary
7 passed (1.3m)
🎭 Playwright Run Summary
7 passed (1.4m)
🎭 Playwright Run Summary
1 flaky selection/native.spec.ts:266:1 › cursor move to up and down with children block ──────────────── 3 skipped 94 passed (1.3m)
🎭 Playwright Run Summary
2 flaky attachment.spec.ts:418:1 › should attachment can be deleted ──────────────────────────────────── clipboard/list.spec.ts:307:1 › paste non-nested lists to a nested list ───────────────────────── 3 skipped 93 passed (1.4m)
🎭 Playwright Run Summary
1 flaky [Mobile Safari] › explorer-folder.spec.ts:49:5 › create a folder ─────────────────────────────── 7 passed (2.0m)
🎭 Playwright Run Summary
3 skipped 95 passed (1.4m)
🎭 Playwright Run Summary
4 skipped 94 passed (1.5m)
🎭 Playwright Run Summary
98 passed (1.4m)
🎭 Playwright Run Summary
8 passed (1.6m)
🎭 Playwright Run Summary
1 skipped 97 passed (1.7m)
🎭 Playwright Run Summary
1 flaky [Mobile Safari] › explorer-tag.spec.ts:59:5 › rename a tag from explorer ─────────────────────── 7 passed (2.0m)
🎭 Playwright Run Summary
1 flaky edgeless/note/resize.spec.ts:138:1 › resize note then auto size and custom size ──────────────── 12 skipped 85 passed (2.2m)
🎭 Playwright Run Summary
6 skipped 92 passed (2.4m)
🎭 Playwright Run Summary
1 flaky edgeless/shortcut.spec.ts:119:1 › should not switch shapes in editing ────────────────────────── 4 skipped 93 passed (2.5m)
🎭 Playwright Run Summary
1 skipped 97 passed (2.8m)
🎭 Playwright Run Summary
1 skipped 35 passed (3.5m)
🎭 Playwright Run Summary
1 flaky peek-view.spec.ts:112:5 › can open peek view for embedded frames ─────────────────────────────── 1 skipped 34 passed (3.8m)
🎭 Playwright Run Summary
1 skipped 35 passed (3.4m)
🎭 Playwright Run Summary
1 flaky doc-info-modal.spec.ts:123:5 › add custom property ───────────────────────────────────────────── 35 passed (4.0m)
🎭 Playwright Run Summary
1 flaky links.spec.ts:249:5 › allow switching to embed view when linking to the other document with mode 2 skipped 33 passed (4.1m)
🎭 Playwright Run Summary
1 passed (1.2m)
🎭 Playwright Run Summary
2 passed (1.7m)
🎭 Playwright Run Summary
1 skipped 7 passed (4.3m)
🎭 Playwright Run Summary
1 skipped 10 passed (5.4m)

Artifacts

Produced during runtime
Name Size
affine.darwin-arm64.node
1.91 MB
affine.darwin-x64.node
2.04 MB
affine.linux-x64-gnu.node
2.08 MB
affine.win32-x64-msvc.node
2.43 MB
android
60.7 MB
ios
60.7 MB
server-native.node
4.32 MB
web
62.7 MB