Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport PR #15524: Fix visual tests #15578

Merged
merged 7 commits into from Jan 30, 2024

Conversation

krassowski
Copy link
Member

Backport PR #15524 on branch 4.0.x (Fix visual tests, allow to disable termianal cursor blinking)

* Try to make the test terminal snapshot more robust

by mock `tree . -L 2` command

* Workaround `test-` dirs issue by capturing `/data` instead of `/`

* Update Playwright Snapshots

* Run customization docs snapshots in serial

* Ignore all test files in galata tests

* Update Playwright Snapshots

* Disable concurrency in `build:examples` to fix flaky test

This was already use in `build:src`

* Ensure no flaky focus indicator on snapshot

(cherry picked from commit c26a0e7)
@krassowski krassowski added this to the 4.0.x milestone Dec 29, 2023
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@krassowski
Copy link
Member Author

bot please update documentation snapshots

Copy link
Contributor

Documentation snapshots updated.

@krassowski
Copy link
Member Author

bot please update galata snapshots

Copy link
Contributor

Galata snapshots updated.

@krassowski krassowski closed this Dec 29, 2023
@krassowski krassowski reopened this Dec 29, 2023
@krassowski krassowski marked this pull request as ready for review January 30, 2024 10:57
@krassowski
Copy link
Member Author

Some documentation snapshots are still flaky but the main visual regression tests job passes - merging.

@krassowski krassowski merged commit 353707e into jupyterlab:4.0.x Jan 30, 2024
75 of 76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant