Move handling of the file browser settings to a separate plugin, enable file browser single click navigation #2140
Annotations
7 errors and 2 notices
Update snapshots:
ui-tests/test/links.spec.ts#L1
1) [firefox] › test/links.spec.ts:21:7 › Local Links › Open the current directory ────────────────
Test timeout of 60000ms exceeded.
|
Update snapshots:
ui-tests/test/links.spec.ts#L25
1) [firefox] › test/links.spec.ts:21:7 › Local Links › Open the current directory ────────────────
Error: proxy.waitForEvent: Test timeout of 60000ms exceeded.
=========================== logs ===========================
waiting for event "popup"
============================================================
23 |
24 | const [current] = await Promise.all([
> 25 | page.waitForEvent('popup'),
| ^
26 | page.getByText('Current Directory').last().click(),
27 | ]);
28 |
at /home/runner/work/notebook/notebook/ui-tests/test/links.spec.ts:25:12
|
Test:
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:
1626 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
|
Test:
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:
1625 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
|
Test:
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:
3027 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
|
Test:
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:
2976 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
|
Test
Process completed with exit code 1.
|
Update snapshots
1 flaky
[firefox] › test/links.spec.ts:21:7 › Local Links › Open the current directory ─────────────────
34 passed (2.2m)
|
Test
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
33 passed (2.1m)
|
Loading