Skip to content

Update pre-commit hooks #2266

Update pre-commit hooks

Update pre-commit hooks #2266

Triggered via pull request January 7, 2025 09:07
Status Success
Total duration 8m 40s
Artifacts 5

ui-tests.yml

on: pull_request
Matrix: ui-tests
Fit to window
Zoom out
Zoom in

Annotations

3 errors, 2 warnings, and 2 notices
[firefox] › test/filebrowser.spec.ts:32:7 › File Browser › Select one file: ui-tests/test/filebrowser.spec.ts#L41
1) [firefox] › test/filebrowser.spec.ts:32:7 › File Browser › Select one file ──────────────────── Error: expect(locator).toBeVisible() Locator: getByRole('toolbar').getByText('Open') Expected: visible Received: hidden Call log: - expect.toBeVisible with timeout 5000ms - waiting for getByRole('toolbar').getByText('Open') 39 | 40 | ['Rename', 'Delete', 'Open', 'Download', 'Delete'].forEach(async (text) => { > 41 | expect(toolbar.getByText(text)).toBeVisible(); | ^ 42 | }); 43 | }); 44 | at forEach (/home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:41:39) at /home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:40:56
[firefox] › test/filebrowser.spec.ts:45:7 › File Browser › Select files and folders: ui-tests/test/filebrowser.spec.ts#L56
2) [firefox] › test/filebrowser.spec.ts:45:7 › File Browser › Select files and folders ─────────── Error: expect(locator).toBeHidden() Locator: getByRole('toolbar').getByText('Open') Expected: hidden Received: visible Call log: - expect.toBeHidden with timeout 5000ms - waiting for getByRole('toolbar').getByText('Open') 54 | 55 | expect(toolbar.getByText('Rename')).toBeHidden(); > 56 | expect(toolbar.getByText('Open')).toBeHidden(); | ^ 57 | expect(toolbar.getByText('Delete')).toBeVisible(); 58 | }); 59 | at /home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:56:39
[firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page: ui-tests/test/mobile.spec.ts#L38
3) [firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page Error: Screenshot comparison failed: 361 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/notebook/notebook/ui-tests/test/mobile.spec.ts-snapshots/tree-firefox-linux.png Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-mobile-Mobile-The-lay-60036-ct-on-the-file-browser-page-firefox/tree-actual.png Diff: /home/runner/work/notebook/notebook/ui-tests/test-results/test-mobile-Mobile-The-lay-60036-ct-on-the-file-browser-page-firefox/tree-diff.png 36 | await page.waitForSelector('#top-panel-wrapper', { state: 'hidden' }); 37 | > 38 | expect(await page.screenshot()).toMatchSnapshot('tree.png', { | ^ 39 | maxDiffPixels: 300, 40 | }); 41 | }); at /home/runner/work/notebook/notebook/ui-tests/test/mobile.spec.ts:38:37
Build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
ui-tests (chromium)
No files were found with the provided path: ui-tests/test-results. No artifacts will be uploaded.
🎭 Playwright Run Summary
36 passed (1.7m)
🎭 Playwright Run Summary
3 flaky [firefox] › test/filebrowser.spec.ts:32:7 › File Browser › Select one file ───────────────────── [firefox] › test/filebrowser.spec.ts:45:7 › File Browser › Select files and folders ──────────── [firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page 33 passed (2.2m)

Artifacts

Produced during runtime
Name Size
notebook-chromium-test-report
328 KB
notebook-dist-2266
26.5 MB
notebook-firefox-test-assets
1.55 MB
notebook-firefox-test-report
2.21 MB
notebook-pkgs-2266
57.8 KB