Skip to content

Remove need for message index in YChat API methods #436

Remove need for message index in YChat API methods

Remove need for message index in YChat API methods #436

Re-run triggered December 17, 2024 08:39
Status Failure
Total duration 8m 24s
Artifacts 3

build.yml

on: pull_request
Build chat extension
2m 33s
Build chat extension
Python test on extensions
2m 13s
Python test on extensions
Integration tests
8m 11s
Integration tests
Integration tests notebook
1m 27s
Integration tests notebook
Fit to window
Zoom out
Zoom in

Annotations

10 errors, 8 warnings, and 2 notices
tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section: ui-tests/tests/commands.spec.ts#L122
1) tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section ───────────── Error: Screenshot comparison failed: 6 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/commands.spec.ts-snapshots/launcher-tile-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-commands--launcher-s-cfe04--a-launcher-item-in-section/launcher-tile-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-commands--launcher-s-cfe04--a-launcher-item-in-section/launcher-tile-diff.png 120 | // Chat tile 121 | const tile = page.locator('.jp-LauncherCard[data-category="Chat"]'); > 122 | expect(await tile.screenshot()).toMatchSnapshot('launcher-tile.png'); | ^ 123 | }); 124 | 125 | test('should open modal create from the launcher', async ({ page }) => { at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/commands.spec.ts:122:37
tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section: ui-tests/tests/commands.spec.ts#L122
1) tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section ───────────── Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: Screenshot comparison failed: 6 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/commands.spec.ts-snapshots/launcher-tile-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-commands--launcher-s-cfe04--a-launcher-item-in-section-retry1/launcher-tile-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-commands--launcher-s-cfe04--a-launcher-item-in-section-retry1/launcher-tile-diff.png 120 | // Chat tile 121 | const tile = page.locator('.jp-LauncherCard[data-category="Chat"]'); > 122 | expect(await tile.screenshot()).toMatchSnapshot('launcher-tile.png'); | ^ 123 | }); 124 | 125 | test('should open modal create from the launcher', async ({ page }) => { at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/commands.spec.ts:122:37
tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section: ui-tests/tests/commands.spec.ts#L122
1) tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section ───────────── Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: Screenshot comparison failed: 6 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/commands.spec.ts-snapshots/launcher-tile-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-commands--launcher-s-cfe04--a-launcher-item-in-section-retry2/launcher-tile-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-commands--launcher-s-cfe04--a-launcher-item-in-section-retry2/launcher-tile-diff.png 120 | // Chat tile 121 | const tile = page.locator('.jp-LauncherCard[data-category="Chat"]'); > 122 | expect(await tile.screenshot()).toMatchSnapshot('launcher-tile.png'); | ^ 123 | }); 124 | 125 | test('should open modal create from the launcher', async ({ page }) => { at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/commands.spec.ts:122:37
tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read: ui-tests/tests/notifications.spec.ts#L1
2) tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read Test timeout of 60000ms exceeded.
tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read: ui-tests/tests/notifications.spec.ts#L1
2) tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Test timeout of 60000ms exceeded.
tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read: ui-tests/tests/notifications.spec.ts#L120
2) tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(received).toHaveLength(expected) Expected length: 1 Received length: 0 Received array: [] 118 | ); 119 | let notifications = await page.notifications; > 120 | expect(notifications).toHaveLength(1); | ^ 121 | 122 | await messages.last().scrollIntoViewIfNeeded(); 123 | await page.waitForCondition( at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts:120:27
tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label: ui-tests/tests/notifications.spec.ts#L209
3) tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label ──── Error: Screenshot comparison failed: 15 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts-snapshots/tab-with-unread-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-notifications--notif-8e6a5--unread-symbol-in-tab-label/tab-with-unread-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-notifications--notif-8e6a5--unread-symbol-in-tab-label/tab-with-unread-diff.png 207 | }); 208 | expect(await beforePseudo).toHaveProperty('content', '"* "'); > 209 | expect(await tab.screenshot()).toMatchSnapshot('tab-with-unread.png'); | ^ 210 | 211 | await messages.last().scrollIntoViewIfNeeded(); 212 | expect(await tab.screenshot()).toMatchSnapshot('tab-without-unread.png'); at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts:209:36
tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label: ui-tests/tests/notifications.spec.ts#L209
3) tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label ──── Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: Screenshot comparison failed: 15 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts-snapshots/tab-with-unread-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-notifications--notif-8e6a5--unread-symbol-in-tab-label-retry1/tab-with-unread-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-notifications--notif-8e6a5--unread-symbol-in-tab-label-retry1/tab-with-unread-diff.png 207 | }); 208 | expect(await beforePseudo).toHaveProperty('content', '"* "'); > 209 | expect(await tab.screenshot()).toMatchSnapshot('tab-with-unread.png'); | ^ 210 | 211 | await messages.last().scrollIntoViewIfNeeded(); 212 | expect(await tab.screenshot()).toMatchSnapshot('tab-without-unread.png'); at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts:209:36
tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label: ui-tests/tests/notifications.spec.ts#L209
3) tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label ──── Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: Screenshot comparison failed: 15 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts-snapshots/tab-with-unread-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-notifications--notif-8e6a5--unread-symbol-in-tab-label-retry2/tab-with-unread-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-notifications--notif-8e6a5--unread-symbol-in-tab-label-retry2/tab-with-unread-diff.png 207 | }); 208 | expect(await beforePseudo).toHaveProperty('content', '"* "'); > 209 | expect(await tab.screenshot()).toMatchSnapshot('tab-with-unread.png'); | ^ 210 | 211 | await messages.last().scrollIntoViewIfNeeded(); 212 | expect(await tab.screenshot()).toMatchSnapshot('tab-without-unread.png'); at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/notifications.spec.ts:209:36
tests/ui-config.spec.ts:97:7 › #stackedMessages › messages should be stacked by default: ui-tests/tests/ui-config.spec.ts#L110
4) tests/ui-config.spec.ts:97:7 › #stackedMessages › messages should be stacked by default ─────── Error: Screenshot comparison failed: 59 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/ui-config.spec.ts-snapshots/stacked-messages-linux.png Received: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-ui-config--stackedMe-dc9c3-hould-be-stacked-by-default/stacked-messages-actual.png Diff: /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/test-results/tests-ui-config--stackedMe-dc9c3-hould-be-stacked-by-default/stacked-messages-diff.png 108 | ); 109 | > 110 | expect(await messagesContainer.screenshot()).toMatchSnapshot( | ^ 111 | 'stacked-messages.png' 112 | ); 113 | }); at /home/runner/work/jupyter-chat/jupyter-chat/ui-tests/tests/ui-config.spec.ts:110:50
Build jupyter_chat
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Typing test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Check Links
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Python test on extensions
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build chat extension
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Integration tests notebook
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "jupyterlab_chat-artifacts", "jupyterlab_chat-notebook-playwright-tests", "jupyterlab_chat-playwright-tests". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
Integration tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
🎭 Playwright Run Summary
2 passed (12.6s)
🎭 Playwright Run Summary
5 failed tests/commands.spec.ts:114:7 › #launcher › should have a launcher item in section ────────────── tests/notifications.spec.ts:108:7 › #notifications › should remove notification when the message is read tests/notifications.spec.ts:195:7 › #notifications › should add unread symbol in tab label ───── tests/ui-config.spec.ts:97:7 › #stackedMessages › messages should be stacked by default ──────── tests/ui-config.spec.ts:115:7 › #stackedMessages › should update settings value stackMessages on existing chat 6 flaky tests/chat-file.spec.ts:166:7 › #ychat › should add an id to the chat metadata ───────────────── tests/chat-file.spec.ts:236:7 › #outofband › should add a message from file ──────────────────── tests/notifications.spec.ts:128:7 › #notifications › should update existing notification on new message tests/side-panel.spec.ts:174:9 › #sidepanel › #openingClosing › should list existing chat ────── tests/unread.spec.ts:56:9 › #messagesNavigation › navigation without unread message › should navigate to last message tests/unread.spec.ts:197:9 › #messagesNavigation › navigation with new unread message › should navigate to new unread message 2 skipped 75 passed (7.0m)

Artifacts

Produced during runtime
Name Size
jupyterlab_chat-artifacts
1.68 MB
jupyterlab_chat-notebook-playwright-tests
436 KB
jupyterlab_chat-playwright-tests
1.47 MB