-
-
Notifications
You must be signed in to change notification settings - Fork 149
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
Jest tests on UI #6751
Jest tests on UI #6751
Conversation
b99e642
to
f2d241d
Compare
60dc68c
to
2458d4a
Compare
5623b55
to
ce9920f
Compare
c1031fc
to
5b55e17
Compare
84cce3f
to
aab075e
Compare
@ylavoie is this really ready for merging? Or is this branch still waiting for Coveralls? |
I was waiting. They answered 5 days ago, see lemurheavy/coveralls-public#1662. I'll fix that. |
004a9f2
to
4351b39
Compare
Seems ok now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
@@ -125,16 +125,19 @@ jobs: | |||
postgres: "13" | |||
BROWSER: "chrome" | |||
COVERAGE: 1 | |||
JOB_COUNT: 2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changes the build matrix, which means adjustment is required for the required jobs on branch protection...
* Add jest * Simplify GitHub Actions * Coverage with local perl * Do not collect coverage from tests themselves * Ignore coverage in project sources * Fix devel option * Move UI tests to UI * Freshen package-lock
* Add jest * Simplify GitHub Actions * Coverage with local perl * Do not collect coverage from tests themselves * Ignore coverage in project sources * Fix devel option * Move UI tests to UI * Freshen package-lock
This PR adds:
Jest testing isolates the UI and mock all server interaction, allowing robust, repetitive and quick tests