Skip to content

fix: markdown should be consistent with the inline editor #7695

fix: markdown should be consistent with the inline editor

fix: markdown should be consistent with the inline editor #7695

Triggered via pull request November 18, 2024 13:16
Status Failure
Total duration 4m 18s
Artifacts

test.yml

on: pull_request
Install Dependencies
1m 7s
Install Dependencies
Matrix: E2E Test
Matrix: Unit Test
Build & Test
0s
Build & Test
Fit to window
Zoom out
Zoom in

Annotations

11 errors, 20 warnings, and 20 notices
attachment.spec.ts:406:1 › should attachment can be deleted: tests/attachment.spec.ts#L69
1) attachment.spec.ts:406: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/blocksuite/blocksuite/tests/attachment.spec.ts:69:30) at /home/runner/work/blocksuite/blocksuite/tests/attachment.spec.ts:412:3
edgeless/paste-block.spec.ts:39:3 › pasting blocks › pasting a note block: tests/edgeless/paste-block.spec.ts#L62
1) edgeless/paste-block.spec.ts:39:3 › pasting blocks › pasting a note block ───────────────────── Error: Timed out 5000ms waiting for expect(locator).toContainText(expected) Locator: locator('affine-edgeless-note[data-block-id="10"]').locator('[data-block-id]').first() - Expected string - 1 + Received string + 21 - hello + + + + + 50 + + + Type '/' for commands + + + + + + + + + + + + + Call log: - expect.toContainText with timeout 5000ms - waiting for locator('affine-edgeless-note[data-block-id="10"]').locator('[data-block-id]').first() - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-block-id="11">…</affine-paragraph> - unexpected value " 50 Type '/' for commands " - locator resolved to <affine-paragraph data-
hotkey/hotkey.spec.ts:274:1 › should cut work single line: tests/hotkey/hotkey.spec.ts#L283
1) hotkey/hotkey.spec.ts:274:1 › should cut work single line ───────────────────────────────────── Error: Snapshot comparison failed: { "type": "block", "id": "0", "flavour": "affine:page", "version": 2, "props": { "title": { "$blocksuite:internal:text$": true, "delta": [] } }, "children": [ { "type": "block", "id": "1", "flavour": "affine:note", "version": 1, "props": { "xywh": "[0,0,498,92]", "background": "--affine-note-background-white", "index": "a0", "hidden": false, "displayMode": "both", "edgeless": { "style": { "borderRadius": 8, "borderSize": 4, "borderStyle": "none", "shadowType": "--affine-note-shadow-box" } } }, "children": [ { "type": "block", "id": "2", "flavour": "affine:paragraph", "version": 1, "props": { "type": "text", "text": { "$blocksuite:internal:text$": true, "delta": [ { "insert": "hello" } ] } }, "children": [] } ] } ] } Expected: /home/runner/work/blocksuite/blocksuite/tests/snapshots/hotkey/hotkey.spec.ts/should-cut-work-single-line-init.json Received: /home/runner/work/blocksuite/blocksuite/tests/test-results/hotkey-hotkey-should-cut-work-single-line/should-cut-work-single-line-init-actual.json 281 | // cut 282 | await page.keyboard.press(`${SHORT_KEY}+x`); > 283 | expect(await getPageSnapshot(page, true)).toMatchSnapshot( | ^ 284 | `${testInfo.title}_init.json` 285 | ); 286 | await undoByKeyboard(page); at /home/runner/work/blocksuite/blocksuite/tests/hotkey/hotkey.spec.ts:283:45
hotkey/multiline.spec.ts:100:1 › should cut work multiple line: tests/hotkey/multiline.spec.ts#L110
2) hotkey/multiline.spec.ts:100:1 › should cut work multiple line ──────────────────────────────── Error: Snapshot comparison failed: { "type": "block", "id": "0", "flavour": "affine:page", "version": 2, "props": { "title": { "$blocksuite:internal:text$": true, "delta": [] } }, "children": [ { "type": "block", "id": "1", "flavour": "affine:note", "version": 1, "props": { "xywh": "[0,0,498,92]", "background": "--affine-note-background-white", "index": "a0", "hidden": false, "displayMode": "both", "edgeless": { "style": { "borderRadius": 8, "borderSize": 4, "borderStyle": "none", "shadowType": "--affine-note-shadow-box" } } }, "children": [ { "type": "block", "id": "2", "flavour": "affine:paragraph", "version": 1, "props": { "type": "text", "text": { "$blocksuite:internal:text$": true, "delta": [ { "insert": "123" } ] } }, "children": [] }, { "type": "block", "id": "3", "flavour": "affine:paragraph", "version": 1, "props": { "type": "text", "text": { "$blocksuite:internal:text$": true, "delta": [ { "insert": "456" } ] } }, "children": [] }, { "type": "block", "id": "4", "flavour": "affine:paragraph", "version": 1, "props": { "type": "text", "text": { "$blocksuite:internal:text$": true, "delta": [ { "insert": "789" } ] } }, "children": [] } ] } ] } Expected: /home/runner/work/blocksuite/blocksuite/tests/snapshots/hotkey/multiline.spec.ts/should-cut-work-multiple-line-init.json Received: /home/runner/work/blocksuite/blocksuite/tests/test-results/hotkey-multiline-should-cut-work-multiple-line/should-cut-work-multiple-line-init-actual.json 108 | // cut 109 | await page.keyboard.press(`${SHORT_KEY}+x`); > 110 | expect(await getPageSnapshot(page, true)).toMatchSnapshot( | ^ 111 | `${testInfo.title}_init.json` 112 | ); 113 | await undoByKeyboard(page); at /home/runner/work/blocksuite/blocksuite/tests/hotkey/multiline.spec.ts:110:45
src/__tests__/adapters/markdown.unit.spec.ts > snapshot to markdown > inline latex: packages/blocks/src/__tests__/adapters/markdown.unit.spec.ts#L1109
AssertionError: expected 'inline $$E=mc^2$$ latex\n' to be 'inline $E=mc^2$ latex\n' // Object.is equality - Expected + Received - inline $E=mc^2$ latex + inline $$E=mc^2$$ latex ❯ src/__tests__/adapters/markdown.unit.spec.ts:1109:25
src/__tests__/adapters/markdown.unit.spec.ts > markdown to snapshot > inline latex: packages/blocks/src/__tests__/adapters/markdown.unit.spec.ts#L3657
AssertionError: expected { type: 'block', …(4) } to deeply equal { type: 'block', …(4) } - Expected + Received Object { "children": Array [ Object { "children": Array [], "flavour": "affine:paragraph", "id": "matchesReplaceMap[1]", "props": Object { "text": Object { "$blocksuite:internal:text$": true, "delta": Array [ Object { - "insert": "inline ", - }, - Object { - "attributes": Object { - "latex": "E=mc^2", - }, - "insert": " ", - }, - Object { - "insert": " latex", + "insert": "inline $E=mc^2$ latex", }, ], }, "type": "text", }, "type": "block", }, ], "flavour": "affine:note", "id": "matchesReplaceMap[0]", "props": Object { "background": "--affine-note-background-white", "displayMode": "both", "hidden": false, "index": "a0", "xywh": "[0,0,800,95]", }, "type": "block", } ❯ src/__tests__/adapters/markdown.unit.spec.ts:3657:49
Unit Test (1)
Process completed with exit code 1.
edgeless/shortcut.spec.ts:119:1 › should not switch shapes in editing: tests/utils/actions/edgeless.ts#L357
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 - waiting for element to be visible, enabled and stable - element is not stable - retrying click action, attempt #1 - waiting for element to be visible, enabled and stable - element is not stable - retrying click action, attempt #2 - waiting 20ms - waiting for element to be visible, enabled and stable - element is not stable - retrying click action, attempt #3 - waiting 100ms - waiting for element to be visible, enabled and stable - element was detached from the DOM, retrying at utils/actions/edgeless.ts:357 355 | .locator('edgeless-slide-menu edgeless-tool-icon-button') 356 | .filter({ hasText: shape }); > 357 | await squareShapeButton.click(); | ^ 358 | break; 359 | } 360 | } at setEdgelessTool (/home/runner/work/blocksuite/blocksuite/tests/utils/actions/edgeless.ts:357:31) at /home/runner/work/blocksuite/blocksuite/tests/edgeless/shortcut.spec.ts:137:3
edgeless/note/drag-handle.spec.ts:122:1 › drag handle should work across multiple notes: tests/edgeless/note/drag-handle.spec.ts#L141
1) edgeless/note/drag-handle.spec.ts:122:1 › drag handle should work across multiple notes ─────── Error: Timed out 5000ms waiting for expect(locator).toBeHidden() Locator: locator('.affine-drag-handle-container') Expected: hidden Received: visible Call log: - expect.toBeHidden with timeout 5000ms - waiting for locator('.affine-drag-handle-container') - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" - locator resolved to <div draggable="true" class="affine-drag-handle-container">…</div> - unexpected value "visible" 139 | await page.mouse.dblclick(CENTER_X, CENTER_Y - 20); 140 | await dragHandleFromBlockToBlockBottomById(page, '3', '7'); > 141 | await expect(page.locator('.affine-drag-handle-container')).toBeHidden(); | ^ 142 | await waitNextFrame(page); 143 | await assertRichTexts(page, ['456', '789', '000', '123']); 144 | at /home/runner/work/blocksuite/blocksuite/tests/edgeless/note/drag-handle.spec.ts:141:63
edgeless/edgeless-text.spec.ts:248:3 › edgeless text block › edgeless text should maintain selection when deleting across multiple lines: tests/utils/asserts.ts#L598
1) edgeless/edgeless-text.spec.ts:248:3 › edgeless text block › edgeless text should maintain selection when deleting across multiple lines Error: expect(received).toBe(expected) // Object.is equality Expected: "ab" Received: "b" at utils/asserts.ts:598 596 | { id } 597 | ); > 598 | expect(actual).toBe(str); | ^ 599 | } 600 | 601 | export async function assertBlockProps( at assertBlockTextContent (/home/runner/work/blocksuite/blocksuite/tests/utils/asserts.ts:598:18) at /home/runner/work/blocksuite/blocksuite/tests/edgeless/edgeless-text.spec.ts:269:5
Build & Test
Process completed with exit code 1.
E2E Test (16)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (2)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (19)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (17)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (3)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (20)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (4)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (18)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (1)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (10)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (15)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (14)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (13)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (12)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (5)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (6)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (11)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (9)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (7)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
E2E Test (8)
Codecov: Failed to properly upload report: The process '/home/runner/work/_actions/codecov/codecov-action/v4/dist/codecov' failed with exit code 1
🎭 Playwright Run Summary
1 skipped 46 passed (44.2s)
🎭 Playwright Run Summary
1 skipped 47 passed (45.5s)
🎭 Playwright Run Summary
47 passed (55.2s)
🎭 Playwright Run Summary
47 passed (48.7s)
🎭 Playwright Run Summary
4 skipped 44 passed (50.1s)
🎭 Playwright Run Summary
3 skipped 44 passed (44.2s)
🎭 Playwright Run Summary
47 passed (58.8s)
🎭 Playwright Run Summary
47 passed (54.2s)
🎭 Playwright Run Summary
1 flaky attachment.spec.ts:406:1 › should attachment can be deleted ──────────────────────────────────── 2 skipped 45 passed (56.7s)
🎭 Playwright Run Summary
1 flaky edgeless/paste-block.spec.ts:39:3 › pasting blocks › pasting a note block ────────────────────── 1 skipped 45 passed (1.1m)
🎭 Playwright Run Summary
1 skipped 46 passed (58.2s)
🎭 Playwright Run Summary
2 flaky hotkey/hotkey.spec.ts:274:1 › should cut work single line ────────────────────────────────────── hotkey/multiline.spec.ts:100:1 › should cut work multiple line ───────────────────────────────── 45 passed (1.1m)
🎭 Playwright Run Summary
47 passed (1.1m)
🎭 Playwright Run Summary
1 flaky edgeless/shortcut.spec.ts:119:1 › should not switch shapes in editing ────────────────────────── 2 skipped 44 passed (1.1m)
🎭 Playwright Run Summary
6 skipped 41 passed (1.4m)
🎭 Playwright Run Summary
47 passed (1.3m)
🎭 Playwright Run Summary
1 skipped 46 passed (1.6m)
🎭 Playwright Run Summary
1 flaky edgeless/note/drag-handle.spec.ts:122:1 › drag handle should work across multiple notes ──────── 10 skipped 36 passed (1.6m)
🎭 Playwright Run Summary
1 flaky edgeless/edgeless-text.spec.ts:248:3 › edgeless text block › edgeless text should maintain selection when deleting across multiple lines 1 skipped 45 passed (1.6m)
🎭 Playwright Run Summary
47 passed (1.7m)