Skip to content

fix(edgeless): set default shape style as general type and text font-family as sans-serif #13392

fix(edgeless): set default shape style as general type and text font-family as sans-serif

fix(edgeless): set default shape style as general type and text font-family as sans-serif #13392

Triggered via pull request November 13, 2023 10:31
Status Success
Total duration 11m 48s
Artifacts

test.yaml

on: pull_request
Node.js Environment Compatibility
49s
Node.js Environment Compatibility
Unit test
1m 6s
Unit test
Virgo E2E test
1m 9s
Virgo E2E test
Matrix: Playground E2E test
Fit to window
Zoom out
Zoom in

Annotations

21 errors and 11 notices
selection/native.spec.ts:887:1 › Delete the blank line between two dividers: tests/utils/asserts.ts#L187
1) selection/native.spec.ts:887:1 › Delete the blank line between two dividers ─────────────────── Error: expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 0 Array [ "", - "", ] at utils/asserts.ts:187 185 | }); 186 | }, currentEditorIndex); > 187 | expect(actualTexts).toEqual(texts); | ^ 188 | } 189 | 190 | export async function assertSelectionPath(page: Page, expected: string[]) { at assertRichTexts (/home/runner/work/blocksuite/blocksuite/tests/utils/asserts.ts:187:23) at /home/runner/work/blocksuite/blocksuite/tests/selection/native.spec.ts:899:3
edgeless/shape.spec.ts:399:1 › auto wrap text in shape: tests/edgeless/shape.spec.ts#L1
1) edgeless/shape.spec.ts:399:1 › auto wrap text in shape ──────────────────────────────────────── Test finished within timeout of 30000ms, but tearing down "context" ran out of time. Please allow more time for the test, since teardown is attributed towards the test timeout budget.
edgeless/auto-complete.spec.ts:108:5 › auto-complete › drag on auto-complete button › drag on right auto-complete button to add note: tests/utils/asserts.ts#L187
1) edgeless/auto-complete.spec.ts:108:5 › auto-complete › drag on auto-complete button › drag on right auto-complete button to add note Error: expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 1 Array [ - "hello", + "llo", ] at utils/asserts.ts:187 185 | }); 186 | }, currentEditorIndex); > 187 | expect(actualTexts).toEqual(texts); | ^ 188 | } 189 | 190 | export async function assertSelectionPath(page: Page, expected: string[]) { at assertRichTexts (/home/runner/work/blocksuite/blocksuite/tests/utils/asserts.ts:187:23) at /home/runner/work/blocksuite/blocksuite/tests/edgeless/auto-complete.spec.ts:124:7
edgeless/basic.spec.ts:201:1 › the tooltip of more button should be hidden when the action menu is shown: tests/edgeless/basic.spec.ts#L224
2) edgeless/basic.spec.ts:201:1 › the tooltip of more button should be hidden when the action menu is shown Error: expect.toBeVisible: Error: strict mode violation: locator('.affine-tooltip') resolved to 2 elements: 1) <div role="tooltip" class="affine-tooltip">…</div> aka getByRole('tooltip', { name: 'Select V' }) 2) <div role="tooltip" class="affine-tooltip">…</div> aka getByRole('tooltip', { name: 'More' }) =========================== logs =========================== expect.toBeVisible with timeout 5000ms waiting for locator('.affine-tooltip') ============================================================ 222 | 223 | await page.mouse.move(moreButtonBox.x + 10, moreButtonBox.y + 10); > 224 | await expect(tooltip).toBeVisible(); | ^ 225 | 226 | await page.mouse.click(moreButtonBox.x + 10, moreButtonBox.y + 10); 227 | await expect(tooltip).toBeHidden(); at /home/runner/work/blocksuite/blocksuite/tests/edgeless/basic.spec.ts:224:25
hotkey.spec.ts:567:1 › should multiple line format hotkey work: tests/hotkey.spec.ts#L1
1) hotkey.spec.ts:567:1 › should multiple line format hotkey work ──────────────────────────────── Test timeout of 30000ms exceeded.
hotkey.spec.ts:567:1 › should multiple line format hotkey work: tests/utils/actions/drag.ts#L21
1) hotkey.spec.ts:567:1 › should multiple line format hotkey work ──────────────────────────────── Error: mouse.move: Target closed at utils/actions/drag.ts:21 19 | await page.mouse.move(x1, y1); 20 | await page.mouse.down(); > 21 | await page.mouse.move(x2, y2, { steps }); | ^ 22 | await options?.beforeMouseUp?.(); 23 | await page.mouse.up(); 24 | } at dragBetweenCoords (/home/runner/work/blocksuite/blocksuite/tests/utils/actions/drag.ts:21:20) at dragBetweenIndices (/home/runner/work/blocksuite/blocksuite/tests/utils/actions/drag.ts:53:3) at /home/runner/work/blocksuite/blocksuite/tests/hotkey.spec.ts:573:3
hotkey.spec.ts:913:1 › should ctrl+enter create new block: tests/utils/asserts.ts#L187
2) hotkey.spec.ts:913:1 › should ctrl+enter create new block ───────────────────────────────────── Error: expect(received).toEqual(expected) // deep equality - Expected - 2 + Received + 2 Array [ - "1", - "23", + "12", + "3", ] at utils/asserts.ts:187 185 | }); 186 | }, currentEditorIndex); > 187 | expect(actualTexts).toEqual(texts); | ^ 188 | } 189 | 190 | export async function assertSelectionPath(page: Page, expected: string[]) { at assertRichTexts (/home/runner/work/blocksuite/blocksuite/tests/utils/asserts.ts:187:23) at /home/runner/work/blocksuite/blocksuite/tests/hotkey.spec.ts:923:3
linked-page.spec.ts:464:3 › reference node › should create reference node works: tests/linked-page.spec.ts#L1
1) linked-page.spec.ts:464:3 › reference node › should create reference node works ─────────────── Test timeout of 30000ms exceeded.
linked-page.spec.ts:464:3 › reference node › should create reference node works: tests/utils/actions/click.ts#L68
1) linked-page.spec.ts:464:3 › reference node › should create reference node works ─────────────── Error: locator.click: Target closed =========================== logs =========================== waiting for locator('debug-menu').locator('sl-button[content="Add New Page"]') locator resolved to <sl-button size="small" data-valid="" data-optional="" vari…>…</sl-button> attempting click action waiting for element to be visible, enabled and stable element is visible, enabled and stable scrolling into view if needed done scrolling ============================================================ at utils/actions/click.ts:68 66 | export async function addNewPage(page: Page) { 67 | const { addNewPageBtn } = getDebugMenu(page); > 68 | await addNewPageBtn.click(); | ^ 69 | const pageMetas = await page.evaluate(() => { 70 | const { workspace } = window; 71 | return workspace.meta.pageMetas; at addNewPage (/home/runner/work/blocksuite/blocksuite/tests/utils/actions/click.ts:68:23) at /home/runner/work/blocksuite/blocksuite/tests/linked-page.spec.ts:468:33
paragraph.spec.ts:1118:1 › press arrow down should move caret to the start of line: tests/paragraph.spec.ts#L1
1) paragraph.spec.ts:1118:1 › press arrow down should move caret to the start of line ──────────── Test timeout of 30000ms exceeded.
paragraph.spec.ts:1118:1 › press arrow down should move caret to the start of line: tests/paragraph.spec.ts#L1122
1) paragraph.spec.ts:1118:1 › press arrow down should move caret to the start of line ──────────── Error: page.evaluate: Target closed 1120 | }) => { 1121 | await enterPlaygroundRoom(page); > 1122 | await page.evaluate(() => { | ^ 1123 | const { page } = window; 1124 | const pageId = page.addBlock('affine:page', { 1125 | title: new page.Text(), at /home/runner/work/blocksuite/blocksuite/tests/paragraph.spec.ts:1122:14
paragraph.spec.ts:1579:1 › delete at the start of paragraph (multiple notes): tests/utils/asserts.ts#L187
2) paragraph.spec.ts:1579:1 › delete at the start of paragraph (multiple notes) ────────────────── Error: expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 2 Array [ - "123456", + "123", + "456", ] at utils/asserts.ts:187 185 | }); 186 | }, currentEditorIndex); > 187 | expect(actualTexts).toEqual(texts); | ^ 188 | } 189 | 190 | export async function assertSelectionPath(page: Page, expected: string[]) { at assertRichTexts (/home/runner/work/blocksuite/blocksuite/tests/utils/asserts.ts:187:23) at /home/runner/work/blocksuite/blocksuite/tests/paragraph.spec.ts:1608:3
edgeless/basic.spec.ts:255:1 › Before and after switching to Edgeless: tests/edgeless/basic.spec.ts#L1
1) edgeless/basic.spec.ts:255:1 › Before and after switching to Edgeless, the previous zoom ratio and position when Edgeless was opened should be remembered Test timeout of 30000ms exceeded.
edgeless/block-hub.spec.ts:21:1 › block hub should drag and drop a card into existing note: tests/utils/asserts.ts#L187
2) edgeless/block-hub.spec.ts:21:1 › block hub should drag and drop a card into existing note ──── Error: expect(received).toEqual(expected) // deep equality - Expected - 1 + Received + 1 Array [ "123", - "000", + "00", "456", "789", ] at utils/asserts.ts:187 185 | }); 186 | }, currentEditorIndex); > 187 | expect(actualTexts).toEqual(texts); | ^ 188 | } 189 | 190 | export async function assertSelectionPath(page: Page, expected: string[]) { at assertRichTexts (/home/runner/work/blocksuite/blocksuite/tests/utils/asserts.ts:187:23) at /home/runner/work/blocksuite/blocksuite/tests/edgeless/block-hub.spec.ts:49:3
edgeless/note.spec.ts:87:1 › resize note in edgeless mode: tests/edgeless/note.spec.ts#L1
1) edgeless/note.spec.ts:87:1 › resize note in edgeless mode ───────────────────────────────────── Test timeout of 30000ms exceeded.
edgeless/note.spec.ts:87:1 › resize note in edgeless mode: tests/utils/actions/misc.ts#L237
1) edgeless/note.spec.ts:87:1 › resize note in edgeless mode ───────────────────────────────────── Error: page.goto: Page closed =========================== logs =========================== navigating to "http://localhost:5173/starter/?room=playwright-26377318&blobStorage=idb", waiting until "load" ============================================================ at utils/actions/misc.ts:237 235 | url.searchParams.set('room', room); 236 | url.searchParams.set('blobStorage', blobStorage?.join(',') || 'idb'); > 237 | await page.goto(url.toString()); | ^ 238 | const readyPromise = waitForPageReady(page); 239 | 240 | // See https://github.com/microsoft/playwright/issues/5546 at enterPlaygroundRoom (/home/runner/work/blocksuite/blocksuite/tests/utils/actions/misc.ts:237:14) at /home/runner/work/blocksuite/blocksuite/tests/edgeless/note.spec.ts:88:9
attachment.spec.ts:112:1 › can insert attachment from slash menu: tests/attachment.spec.ts#L1
1) attachment.spec.ts:112:1 › can insert attachment from slash menu ────────────────────────────── Test timeout of 30000ms exceeded.
attachment.spec.ts:112:1 › can insert attachment from slash menu: tests/utils/actions/misc.ts#L237
1) attachment.spec.ts:112:1 › can insert attachment from slash menu ────────────────────────────── Error: page.goto: Page closed =========================== logs =========================== navigating to "http://localhost:5173/starter/?room=playwright-75162146&blobStorage=idb", waiting until "load" ============================================================ at utils/actions/misc.ts:237 235 | url.searchParams.set('room', room); 236 | url.searchParams.set('blobStorage', blobStorage?.join(',') || 'idb'); > 237 | await page.goto(url.toString()); | ^ 238 | const readyPromise = waitForPageReady(page); 239 | 240 | // See https://github.com/microsoft/playwright/issues/5546 at enterPlaygroundRoom (/home/runner/work/blocksuite/blocksuite/tests/utils/actions/misc.ts:237:14) at /home/runner/work/blocksuite/blocksuite/tests/attachment.spec.ts:113:9
block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks: tests/block-hub.spec.ts#L124
2) block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks ───── Error: Timed out 5000ms waiting for expect(locator).toBeInViewport() Locator: locator('.affine-rich-text').filter({ hasText: 'paragraph 0' }) Expected: in viewport Received: outside viewport Call log: - expect.toBeInViewport with timeout 5000ms - waiting for locator('.affine-rich-text').filter({ hasText: 'paragraph 0' }) - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" 122 | await waitNextFrame(page, 2000); 123 | // now paragraph0 is in viewport > 124 | await expect(paragraph0).toBeInViewport(); | ^ 125 | 126 | await assertStoreMatchJSX( 127 | page, at /home/runner/work/blocksuite/blocksuite/tests/block-hub.spec.ts:124:28
block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks: tests/block-hub.spec.ts#L124
2) block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks ───── Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: Timed out 5000ms waiting for expect(locator).toBeInViewport() Locator: locator('.affine-rich-text').filter({ hasText: 'paragraph 0' }) Expected: in viewport Received: outside viewport Call log: - expect.toBeInViewport with timeout 5000ms - waiting for locator('.affine-rich-text').filter({ hasText: 'paragraph 0' }) - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" 122 | await waitNextFrame(page, 2000); 123 | // now paragraph0 is in viewport > 124 | await expect(paragraph0).toBeInViewport(); | ^ 125 | 126 | await assertStoreMatchJSX( 127 | page, at /home/runner/work/blocksuite/blocksuite/tests/block-hub.spec.ts:124:28
block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks: tests/block-hub.spec.ts#L124
2) block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks ───── Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: Timed out 5000ms waiting for expect(locator).toBeInViewport() Locator: locator('.affine-rich-text').filter({ hasText: 'paragraph 0' }) Expected: in viewport Received: outside viewport Call log: - expect.toBeInViewport with timeout 5000ms - waiting for locator('.affine-rich-text').filter({ hasText: 'paragraph 0' }) - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" - locator resolved to <div contenteditable="true" data-virgo-root="true" cl…>…</div> - unexpected value "viewport ratio 0" 122 | await waitNextFrame(page, 2000); 123 | // now paragraph0 is in viewport > 124 | await expect(paragraph0).toBeInViewport(); | ^ 125 | 126 | await assertStoreMatchJSX( 127 | page, at /home/runner/work/blocksuite/blocksuite/tests/block-hub.spec.ts:124:28
🎭 Playwright Run Summary
12 passed (40.4s)
🎭 Playwright Run Summary
1 skipped 65 passed (2.8m)
🎭 Playwright Run Summary
1 flaky selection/native.spec.ts:887:1 › Delete the blank line between two dividers ──────────────────── 1 skipped 64 passed (2.9m)
🎭 Playwright Run Summary
1 flaky edgeless/shape.spec.ts:399:1 › auto wrap text in shape ───────────────────────────────────────── 2 skipped 63 passed (3.2m)
🎭 Playwright Run Summary
2 flaky edgeless/auto-complete.spec.ts:108:5 › auto-complete › drag on auto-complete button › drag on right auto-complete button to add note edgeless/basic.spec.ts:201:1 › the tooltip of more button should be hidden when the action menu is shown 1 skipped 63 passed (3.2m)
🎭 Playwright Run Summary
2 flaky hotkey.spec.ts:567:1 › should multiple line format hotkey work ───────────────────────────────── hotkey.spec.ts:913:1 › should ctrl+enter create new block ────────────────────────────────────── 2 skipped 62 passed (6.2m)
🎭 Playwright Run Summary
1 flaky linked-page.spec.ts:464:3 › reference node › should create reference node works ──────────────── 2 skipped 63 passed (6.3m)
🎭 Playwright Run Summary
2 flaky paragraph.spec.ts:1118:1 › press arrow down should move caret to the start of line ───────────── paragraph.spec.ts:1579:1 › delete at the start of paragraph (multiple notes) ─────────────────── 64 passed (7.5m)
🎭 Playwright Run Summary
2 flaky edgeless/basic.spec.ts:255:1 › Before and after switching to Edgeless, the previous zoom ratio and position when Edgeless was opened should be remembered edgeless/block-hub.spec.ts:21:1 › block hub should drag and drop a card into existing note ───── 64 passed (9.2m)
🎭 Playwright Run Summary
1 flaky edgeless/note.spec.ts:87:1 › resize note in edgeless mode ────────────────────────────────────── 1 skipped 64 passed (9.8m)
🎭 Playwright Run Summary
2 flaky attachment.spec.ts:112:1 › can insert attachment from slash menu ─────────────────────────────── block-hub.spec.ts:18:1 › auto-scroll should be activate when adding blank lines or blocks ────── 64 passed (9.9m)