Skip to content

Migrate from Mocha to Jest #2550

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.

Sign up for GitHub

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

Jump to bottom
Closed
4 tasks done
jsjoeio opened this issue Jan 8, 2021 · 1 comment
Closed
4 tasks done

Migrate from Mocha to Jest #2550

jsjoeio opened this issue Jan 8, 2021 · 1 comment
Assignees
@jsjoeio
Labels
enhancement Some improvement that isn't a feature

Comments

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 8, 2021 •
edited
Loading

As discussed with @code-asher and @nhooyr, we've decided to migrate from Mocha to Jest as our testing framework of choice for the following reasons:

  • Jest has a larger community, more active and more popular in the JS ecosystem
  • more members of the team are familiar with it
  • built-in support for code coverage

This issue is for transparency and to track the migration.

TODOS

  • Add jest and make sure it works (ts-jest Is one of the recommended approaches, easy setup)
  • Use codemods to migrate mocha tests over (helpful resource)
  • Remove mocha fully
  • Add code coverage (including shield to README)
@jsjoeio jsjoeio self-assigned this Jan 8, 2021
@jsjoeio jsjoeio added the enhancement Some improvement that isn't a feature label Jan 8, 2021
@jsjoeio jsjoeio mentioned this issue Jan 11, 2021
Merged
3 tasks
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Jan 26, 2021

Closing via #2564

@jsjoeio jsjoeio closed this as completed Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jsjoeio jsjoeio

Labels
enhancement Some improvement that isn't a feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jsjoeio