Skip to content

Fix bug when loading geojson layers in notebook and update notebook test #1043

Fix bug when loading geojson layers in notebook and update notebook test

Fix bug when loading geojson layers in notebook and update notebook test #1043

Triggered via pull request January 10, 2025 15:28
Status Failure
Total duration 10m 55s
Artifacts 4

build.yml

on: pull_request
Build docs with JupyterLite deployment
1m 8s
Build docs with JupyterLite deployment
test_isolated
27s
test_isolated
Integration tests
6m 17s
Integration tests
Fit to window
Zoom out
Zoom in

Annotations

7 errors, 8 warnings, and 1 notice
tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct: ui-tests/tests/notebook.spec.ts#L51
1) tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct Error: expect(Buffer).toMatchSnapshot(expected) Expected an image 656px by 550px, received 656px by 601px. 27928 pixels (ratio 0.08 of all image pixels) are different. Expected: /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts-snapshots/light-Notebook-ipynb-cell-2-linux.png Received: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-da5b3-l-outputs-should-be-correct/light-Notebook-ipynb-cell-2-actual.png Diff: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-da5b3-l-outputs-should-be-correct/light-Notebook-ipynb-cell-2-diff.png 49 | 50 | for (let c = 0; c < numCellImages; ++c) { > 51 | expect(results[c]).toMatchSnapshot( | ^ 52 | getCaptureImageName(contextPrefix, notebook, c) 53 | ); 54 | } at testCellOutputs (/home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:51:26) at /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:81:5
tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct: ui-tests/tests/notebook.spec.ts#L51
1) tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(Buffer).toMatchSnapshot(expected) Expected an image 656px by 550px, received 656px by 601px. 27928 pixels (ratio 0.08 of all image pixels) are different. Expected: /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts-snapshots/light-Notebook-ipynb-cell-2-linux.png Received: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-da5b3-l-outputs-should-be-correct-retry1/light-Notebook-ipynb-cell-2-actual.png Diff: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-da5b3-l-outputs-should-be-correct-retry1/light-Notebook-ipynb-cell-2-diff.png 49 | 50 | for (let c = 0; c < numCellImages; ++c) { > 51 | expect(results[c]).toMatchSnapshot( | ^ 52 | getCaptureImageName(contextPrefix, notebook, c) 53 | ); 54 | } at testCellOutputs (/home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:51:26) at /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:81:5
tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct: ui-tests/tests/notebook.spec.ts#L51
1) tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(Buffer).toMatchSnapshot(expected) Expected an image 656px by 550px, received 656px by 601px. 27928 pixels (ratio 0.08 of all image pixels) are different. Expected: /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts-snapshots/light-Notebook-ipynb-cell-2-linux.png Received: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-da5b3-l-outputs-should-be-correct-retry2/light-Notebook-ipynb-cell-2-actual.png Diff: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-da5b3-l-outputs-should-be-correct-retry2/light-Notebook-ipynb-cell-2-diff.png 49 | 50 | for (let c = 0; c < numCellImages; ++c) { > 51 | expect(results[c]).toMatchSnapshot( | ^ 52 | getCaptureImageName(contextPrefix, notebook, c) 53 | ); 54 | } at testCellOutputs (/home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:51:26) at /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:81:5
tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct: ui-tests/tests/notebook.spec.ts#L51
2) tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct Error: expect(Buffer).toMatchSnapshot(expected) Expected an image 656px by 550px, received 656px by 601px. 310899 pixels (ratio 0.79 of all image pixels) are different. Expected: /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts-snapshots/dark-Notebook-ipynb-cell-2-linux.png Received: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-aebed-l-outputs-should-be-correct/dark-Notebook-ipynb-cell-2-actual.png Diff: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-aebed-l-outputs-should-be-correct/dark-Notebook-ipynb-cell-2-diff.png 49 | 50 | for (let c = 0; c < numCellImages; ++c) { > 51 | expect(results[c]).toMatchSnapshot( | ^ 52 | getCaptureImageName(contextPrefix, notebook, c) 53 | ); 54 | } at testCellOutputs (/home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:51:26) at /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:88:5
tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct: ui-tests/tests/notebook.spec.ts#L51
2) tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(Buffer).toMatchSnapshot(expected) Expected an image 656px by 550px, received 656px by 601px. 310899 pixels (ratio 0.79 of all image pixels) are different. Expected: /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts-snapshots/dark-Notebook-ipynb-cell-2-linux.png Received: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-aebed-l-outputs-should-be-correct-retry1/dark-Notebook-ipynb-cell-2-actual.png Diff: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-aebed-l-outputs-should-be-correct-retry1/dark-Notebook-ipynb-cell-2-diff.png 49 | 50 | for (let c = 0; c < numCellImages; ++c) { > 51 | expect(results[c]).toMatchSnapshot( | ^ 52 | getCaptureImageName(contextPrefix, notebook, c) 53 | ); 54 | } at testCellOutputs (/home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:51:26) at /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:88:5
tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct: ui-tests/tests/notebook.spec.ts#L51
2) tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(Buffer).toMatchSnapshot(expected) Expected an image 656px by 550px, received 656px by 601px. 310899 pixels (ratio 0.79 of all image pixels) are different. Expected: /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts-snapshots/dark-Notebook-ipynb-cell-2-linux.png Received: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-aebed-l-outputs-should-be-correct-retry2/dark-Notebook-ipynb-cell-2-actual.png Diff: /home/runner/work/jupytergis/jupytergis/ui-tests/test-results/tests-notebook-Notebook-AP-aebed-l-outputs-should-be-correct-retry2/dark-Notebook-ipynb-cell-2-diff.png 49 | 50 | for (let c = 0; c < numCellImages; ++c) { > 51 | expect(results[c]).toMatchSnapshot( | ^ 52 | getCaptureImageName(contextPrefix, notebook, c) 53 | ); 54 | } at testCellOutputs (/home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:51:26) at /home/runner/work/jupytergis/jupytergis/ui-tests/tests/notebook.spec.ts:88:5
Integration tests
Process completed with exit code 1.
Check Links
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint check
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
python_tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build jupytergis
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build jupytergis
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
test_isolated
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build docs with JupyterLite deployment
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
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 failed tests/notebook.spec.ts:77:7 › Notebook API Visual Regression › Light theme: Cell outputs should be correct tests/notebook.spec.ts:84:7 › Notebook API Visual Regression › Dark theme: Cell outputs should be correct 47 passed (4.6m)

Artifacts

Produced during runtime
Name Size
extension-artifacts
46.8 MB
galata-apss
350 Bytes
github-pages
51.6 MB
jupytergis-playwright-tests
49.5 MB