Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Update UI Tests to use Galata #273

Merged
merged 3 commits into from
Nov 8, 2021
Merged

Update UI Tests to use Galata #273

merged 3 commits into from
Nov 8, 2021

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Nov 8, 2021

This will help with #228.

Update to @jupyterlab/galata which is built on top of @playwright/test, and brings a couple of helper functions we should be able to use in RetroLab too.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2021

Binder 👈 Launch RetroLab on Binder

@jtpio
Copy link
Member Author

jtpio commented Nov 8, 2021

We can also add visual regression tests and reference snapshots to help detect issues like #263, for example when updating to newer @jupyterlab packages.

@jtpio
Copy link
Member Author

jtpio commented Nov 8, 2021

This produces HTML reports:

image

New reference snapshots are generated on failure.

@jtpio jtpio marked this pull request as ready for review November 8, 2021 13:45
@jtpio
Copy link
Member Author

jtpio commented Nov 8, 2021

cc @fcollonval if you want to have a look. Things seems to be working fine also when opening multiple browser tabs.

I opened jupyterlab/jupyterlab#11414 to discuss how we can maybe improve the waitForApplication logic upstream.

Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great thanks @jtpio

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants