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 Jest for tests #1391

Closed
wants to merge 1 commit into from
Closed

Add Jest for tests #1391

wants to merge 1 commit into from

Conversation

damassi
Copy link
Member

@damassi damassi commented Aug 3, 2018

Noticed in #1390 that some tests were starting to get written so wanted to suggest Jest before things got too far along. This lib is really futuristic as far as test runners go and is basically zero config. It looks for files with a file.test.js extension (or files placed in a __tests__ directory) so you can very easily co-locate tests alongside their sources which tends to dramatically increase test-coverage over time since everything is side-by-side.

jest

(Also, not sure if you've used Enzyme before but it's great for writing component tests and works nicely with Jest.)

@@ -141,5 +143,8 @@
},
"optionalDependencies": {
"fsevents": "*"
},
"jest": {
"testURL": "http://localhost"
Copy link
Member Author

@damassi damassi Aug 3, 2018

Choose a reason for hiding this comment

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

Can remove this once this patch is published. JSDom (used underneath Jest) made a change upstream and broke libs all across the internet.

Copy link
Member

Choose a reason for hiding this comment

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

thx much

@dzucconi
Copy link
Member

dzucconi commented Aug 3, 2018

Cool I'm with it. Will just add swap out Mocha in my branch and co-locate my spec for that other fn instead of dealing with this merge. 👍

@dzucconi dzucconi closed this Aug 3, 2018
@damassi damassi deleted the add-jest branch August 3, 2018 18:12
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