Move handling of the file browser settings to a separate plugin, enable file browser single click navigation #2138
Annotations
13 errors and 3 notices
[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:
1601 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:
1574 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
|
[chromium] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page:
ui-tests/test/settings.spec.ts#L36
2) [chromium] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page
Error: Screenshot comparison failed:
2889 pixels (ratio 0.01 of all image pixels) are different.
Expected: /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts-snapshots/top-hidden-chromium-linux.png
Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-chromium/top-hidden-actual.png
Diff: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-chromium/top-hidden-diff.png
34 | await page.reload({ waitUntil: 'networkidle' });
35 | await page.menu.getMenuItem(showHeaderPath);
> 36 | expect(await page.screenshot()).toMatchSnapshot('top-hidden.png', {
| ^
37 | maxDiffPixels: 300,
38 | });
39 |
at /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts:36:37
|
[chromium] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page:
ui-tests/test/settings.spec.ts#L36
2) [chromium] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: Screenshot comparison failed:
2889 pixels (ratio 0.01 of all image pixels) are different.
Expected: /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts-snapshots/top-hidden-chromium-linux.png
Received: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-chromium-retry1/top-hidden-actual.png
Diff: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-chromium-retry1/top-hidden-diff.png
34 | await page.reload({ waitUntil: 'networkidle' });
35 | await page.menu.getMenuItem(showHeaderPath);
> 36 | expect(await page.screenshot()).toMatchSnapshot('top-hidden.png', {
| ^
37 | maxDiffPixels: 300,
38 | });
39 |
at /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts:36:37
|
ui-tests (chromium)
Process completed with exit code 1.
|
ui-tests (firefox)
Canceling since a higher priority waiting request for 'UI Tests-7481' exists
|
ui-tests (firefox)
The operation was canceled.
|
[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
1) [firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page
Error: Screenshot comparison failed:
1617 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
|
[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
1) [firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: Screenshot comparison failed:
1617 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-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-firefox-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
|
[firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page:
ui-tests/test/settings.spec.ts#L36
2) [firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page
Error: Screenshot comparison failed:
2967 pixels (ratio 0.01 of all image pixels) are different.
Expected: /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts-snapshots/top-hidden-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-hidden-actual.png
Diff: /home/runner/work/notebook/notebook/ui-tests/test-results/test-settings-Settings-Sho-cda2b-ed-after-reloading-the-page-firefox/top-hidden-diff.png
34 | await page.reload({ waitUntil: 'networkidle' });
35 | await page.menu.getMenuItem(showHeaderPath);
> 36 | expect(await page.screenshot()).toMatchSnapshot('top-hidden.png', {
| ^
37 | maxDiffPixels: 300,
38 | });
39 |
at /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts:36:37
|
[firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page:
ui-tests/test/settings.spec.ts#L36
2) [firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: Screenshot comparison failed:
2967 pixels (ratio 0.01 of all image pixels) are different.
Expected: /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts-snapshots/top-hidden-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-hidden-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-hidden-diff.png
34 | await page.reload({ waitUntil: 'networkidle' });
35 | await page.menu.getMenuItem(showHeaderPath);
> 36 | expect(await page.screenshot()).toMatchSnapshot('top-hidden.png', {
| ^
37 | maxDiffPixels: 300,
38 | });
39 |
at /home/runner/work/notebook/notebook/ui-tests/test/settings.spec.ts:36:37
|
[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('Open')
Expected: visible
Received: hidden
Call log:
- expect.toBeVisible with timeout 5000ms
- waiting for getByRole('toolbar').getByText('Open')
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
|
ui-tests (firefox)
Process completed with exit code 1.
|
🎭 Playwright Run Summary
35 passed (1.7m)
|
🎭 Playwright Run Summary
2 failed
[chromium] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page
[chromium] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page
33 passed (1.9m)
|
🎭 Playwright Run Summary
2 failed
[firefox] › test/mobile.spec.ts:30:7 › Mobile › The layout should be more compact on the file browser page
[firefox] › test/settings.spec.ts:23:7 › Settings › Should be persisted after reloading the page
1 flaky
[firefox] › test/filebrowser.spec.ts:31:7 › File Browser › Select one file ─────────────────────
32 passed (2.2m)
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
notebook-chromium-test-assets
|
1.4 MB |
|
notebook-chromium-test-report
|
2.02 MB |
|
notebook-chromium-updated-snapshots
|
310 KB |
|
notebook-dist-2138
|
9.5 MB |
|
notebook-firefox-test-assets
|
2.02 MB |
|
notebook-firefox-test-report
|
2.62 MB |
|
notebook-pkgs-2138
|
57.1 KB |
|