Skip to content

Switch UI test runner from karma to jest - #5163

Merged
matt-boris merged 10 commits into
mainfrom
ui/introduce-unit-tests
Feb 14, 2022
Merged

Switch UI test runner from karma to jest#5163
matt-boris merged 10 commits into
mainfrom
ui/introduce-unit-tests

Conversation

@lotas

@lotas lotas commented Feb 10, 2022

Copy link
Copy Markdown
Contributor

Jest will allow to write better integration and snapshot tests for react components

Related to: https://mozilla-hub.atlassian.net/browse/FCP-25

@lotas
lotas requested a review from a team as a code owner February 10, 2022 11:34
@lotas
lotas requested review from matt-boris and petemoore and removed request for a team February 10, 2022 11:34
@lotas
lotas force-pushed the ui/introduce-unit-tests branch from 239afe6 to 1dee2b4 Compare February 11, 2022 13:47
@lotas
lotas force-pushed the ui/introduce-unit-tests branch from 555cc5b to 5c66fc8 Compare February 11, 2022 14:41
@lotas lotas changed the title Draft: Switch UI test runner from karma to jest Switch UI test runner from karma to jest Feb 14, 2022

@matt-boris matt-boris left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I feel we should also remove the cypress examples before merging to main.

Comment thread ui/test/e2e/package.json Outdated
Comment thread ui/test/e2e/package.json Outdated

@matt-boris matt-boris left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, thanks Yarik!

@matt-boris
matt-boris enabled auto-merge February 14, 2022 16:55
@matt-boris
matt-boris merged commit d733a51 into main Feb 14, 2022
@matt-boris
matt-boris deleted the ui/introduce-unit-tests branch February 14, 2022 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants