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

Add unit tests #53

Merged
merged 24 commits into from
Mar 11, 2021
Merged

Add unit tests #53

merged 24 commits into from
Mar 11, 2021

Conversation

pverscha
Copy link
Member

This PR adds Jest unit tests and a GitHub action that will automatically be executed after each push. Visual snapshots are used in this package to make sure that the appearance of visualizations didn't change due to an update. Note that the Treeview visualization cannot be tested using this approach at this point in time, due to lacking support for advanced SVG manipulations in JSDom (jsdom/jsdom#2531).

The GitHub action also creates an artefact that can be downloaded, which contains all screenshots produced and all screenshot diffs if a test failed. Annotations that point to which line in the source code caused a failure are also automatically added.

.github/workflows/test.yml Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
pverscha and others added 3 commits March 11, 2021 16:23
Co-authored-by: Bart Mesuere <Bart.Mesuere@UGent.be>
Co-authored-by: Bart Mesuere <Bart.Mesuere@UGent.be>
Co-authored-by: Bart Mesuere <Bart.Mesuere@UGent.be>
@pverscha pverscha requested a review from bmesuere March 11, 2021 15:26
@pverscha pverscha merged commit 1e38e0d into develop Mar 11, 2021
@pverscha pverscha deleted the feature/tests branch March 11, 2021 15:28
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

2 participants