Remove need for message index in YChat API methods #436
build.yml
on: pull_request
Build jupyter_chat
2m 41s
Typing test
2m 20s
Check Links
1m 53s
Python test on extensions
2m 13s
Integration tests
8m 11s
Integration tests notebook
1m 27s
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 |
|