Skip to content

Add the file filter button to the file browser toolbar #2130

Add the file filter button to the file browser toolbar

Add the file filter button to the file browser toolbar #2130

Triggered via pull request October 15, 2024 18:08
Status Failure
Total duration 10m 47s
Artifacts 8

ui-tests.yml

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

Annotations

13 errors and 4 notices
[chromium] › test/notebook.spec.ts:32:7 › Notebook › Renaming the notebook should be possible: ui-tests/test/notebook.spec.ts#L1
1) [chromium] › test/notebook.spec.ts:32:7 › Notebook › Renaming the notebook should be possible ─ Test timeout of 60000ms exceeded.
[chromium] › test/notebook.spec.ts:32:7 › Notebook › Renaming the notebook should be possible: ui-tests/test/notebook.spec.ts#L52
1) [chromium] › test/notebook.spec.ts:32:7 › Notebook › Renaming the notebook should be possible ─ Error: proxy.waitForNavigation: Test timeout of 60000ms exceeded. =========================== logs =========================== waiting for navigation until "load" ============================================================ 50 | await page.click('text="Rename"'), 51 | // wait until the URL is updated > 52 | await page.waitForNavigation(), | ^ 53 | ]); 54 | 55 | // Check the URL contains the new name at /home/runner/work/notebook/notebook/ui-tests/test/notebook.spec.ts:52:18
[chromium] › 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
1) [chromium] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page Error: Screenshot comparison failed: 446 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/notebook/notebook/ui-tests/test/mobile.spec.ts-snapshots/tree-chromium-linux.png Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-mobile-Mobile-The-lay-60036-ct-on-the-file-browser-page-chromium/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-chromium/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
[chromium] › 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
1) [chromium] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: Screenshot comparison failed: 446 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/notebook/notebook/ui-tests/test/mobile.spec.ts-snapshots/tree-chromium-linux.png Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-mobile-Mobile-The-lay-60036-ct-on-the-file-browser-page-chromium-retry1/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-chromium-retry1/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
ui-tests (chromium)
Process completed with exit code 1.
[firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file: ui-tests/test/filebrowser.spec.ts#L39
1) [firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file ──────────────────── Error: expect(locator).toBeVisible() Locator: getByRole('toolbar').getByText('Download') Expected: visible Received: hidden Call log: - expect.toBeVisible with timeout 5000ms - waiting for getByRole('toolbar').getByText('Download') 37 | 38 | ['Rename', 'Delete', 'Open', 'Download', 'Delete'].forEach(async (text) => { > 39 | expect(toolbar.getByText(text)).toBeVisible(); | ^ 40 | }); 41 | }); 42 | at forEach (/home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:39:39) at /home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:38:56
[firefox] › test/filebrowser.spec.ts:43:7 › File Browser › Select files and folders: ui-tests/test/filebrowser.spec.ts#L55
2) [firefox] › test/filebrowser.spec.ts:43:7 › File Browser › Select files and folders ─────────── Error: expect(locator).toBeVisible() Locator: getByRole('toolbar').getByText('Delete') Expected: visible Received: hidden Call log: - expect.toBeVisible with timeout 5000ms - waiting for getByRole('toolbar').getByText('Delete') 53 | expect(toolbar.getByText('Rename')).toBeHidden(); 54 | expect(toolbar.getByText('Open')).toBeHidden(); > 55 | expect(toolbar.getByText('Delete')).toBeVisible(); | ^ 56 | }); 57 | 58 | test('Select files and open', async ({ page, tmpPath }) => { at /home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:55:41
[firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page: ui-tests/test/settings.spec.ts#L45
1) [firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page Error: Screenshot comparison failed: 350 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts-snapshots/top-visible-firefox-linux.png Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-firefox/top-visible-actual.png Diff: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-firefox/top-visible-diff.png 43 | await page.reload({ waitUntil: 'networkidle' }); 44 | await page.menu.getMenuItem(showHeaderPath); > 45 | expect(await page.screenshot()).toMatchSnapshot('top-visible.png', { | ^ 46 | maxDiffPixels: 300, 47 | }); 48 | }); at /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts:45:37
[firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page: ui-tests/test/settings.spec.ts#L45
1) [firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: Screenshot comparison failed: 550 pixels (ratio 0.01 of all image pixels) are different. Expected: /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts-snapshots/top-visible-firefox-linux.png Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-firefox-retry1/top-visible-actual.png Diff: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-firefox-retry1/top-visible-diff.png 43 | await page.reload({ waitUntil: 'networkidle' }); 44 | await page.menu.getMenuItem(showHeaderPath); > 45 | expect(await page.screenshot()).toMatchSnapshot('top-visible.png', { | ^ 46 | maxDiffPixels: 300, 47 | }); 48 | }); at /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts:45:37
[firefox] › test/filebrowser.spec.ts:20:7 › File Browser › Select one folder: ui-tests/test/filebrowser.spec.ts#L28
2) [firefox] › test/filebrowser.spec.ts:20:7 › File Browser › Select one folder ────────────────── Error: expect(locator).toBeVisible() Locator: getByRole('toolbar').getByText('Delete') Expected: visible Received: hidden Call log: - expect.toBeVisible with timeout 5000ms - waiting for getByRole('toolbar').getByText('Delete') 26 | 27 | expect(toolbar.getByText('Rename')).toBeVisible(); > 28 | expect(toolbar.getByText('Delete')).toBeVisible(); | ^ 29 | }); 30 | 31 | test('Select one file', async ({ page, tmpPath }) => { at /home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:28:41
[firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file: ui-tests/test/filebrowser.spec.ts#L39
3) [firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file ──────────────────── Error: expect(locator).toBeVisible() Locator: getByRole('toolbar').getByText('Delete') Expected: visible Received: hidden Call log: - expect.toBeVisible with timeout 5000ms - waiting for getByRole('toolbar').getByText('Delete') 37 | 38 | ['Rename', 'Delete', 'Open', 'Download', 'Delete'].forEach(async (text) => { > 39 | expect(toolbar.getByText(text)).toBeVisible(); | ^ 40 | }); 41 | }); 42 | at forEach (/home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:39:39) at /home/runner/work/notebook/notebook/ui-tests/test/filebrowser.spec.ts:38:56
[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
4) [firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page Error: Screenshot comparison failed: 487 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
ui-tests (firefox)
Process completed with exit code 1.
🎭 Playwright Run Summary
1 flaky [chromium] › test/notebook.spec.ts:32:7 › Notebook › Renaming the notebook should be possible ── 34 passed (2.1m)
🎭 Playwright Run Summary
1 failed [chromium] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page 34 passed (1.8m)
🎭 Playwright Run Summary
2 flaky [firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file ───────────────────── [firefox] › test/filebrowser.spec.ts:43:7 › File Browser › Select files and folders ──────────── 33 passed (1.9m)
🎭 Playwright Run Summary
1 failed [firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page 3 flaky [firefox] › test/filebrowser.spec.ts:20:7 › File Browser › Select one folder ─────────────────── [firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file ───────────────────── [firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page 31 passed (2.3m)

Artifacts

Produced during runtime
Name Size
notebook-chromium-test-assets
571 KB
notebook-chromium-test-report
1.18 MB
notebook-chromium-updated-snapshots
314 KB
notebook-dist-2130
9.5 MB
notebook-firefox-test-assets
2.98 MB
notebook-firefox-test-report
3.63 MB
notebook-firefox-updated-snapshots
367 KB
notebook-pkgs-2130
56.9 KB