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.

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

Reports "NaN total" #64

Closed
tunnckoCore opened this issue Feb 1, 2019 · 5 comments · Fixed by #65 or jestjs/jest#7779
Closed

Reports "NaN total" #64

tunnckoCore opened this issue Feb 1, 2019 · 5 comments · Fixed by #65 or jestjs/jest#7779

Comments

@tunnckoCore
Copy link

Heya!

I just notice that

Test Suites: 32 passed, 32 total
Tests:       32 passed, NaN total
Snapshots:   0 total
Time:        3.635s

I'm not sure if it was a bug before upgrading to Jest 24.

I may look later on the source code.
@SimenB
Copy link
Member

SimenB commented Feb 2, 2019

This is a bug in Jest following jestjs/jest#6996 (will be fixed by jest-community/create-jest-runner#14).

However, Jest itself shouldn't bug out, so I'll fix it there

@SimenB SimenB mentioned this issue Feb 2, 2019
Merged
@SimenB
Copy link
Member

SimenB commented Feb 2, 2019

PR: jestjs/jest#7779

@SimenB SimenB reopened this Feb 2, 2019
@SimenB
Copy link
Member

SimenB commented Feb 2, 2019

Will be fixed by either upgrading to [email protected] (PR welcome!) or wait for the next release of Jest (which hopefully is really soon)

@tunnckoCore
Copy link
Author

I thought that too, but noticed it only when run with eslint runner.

Okey, thanks :)

@SimenB SimenB mentioned this issue Jun 8, 2019
Merged
@SimenB SimenB closed this as completed in #65 Jun 8, 2019
@SimenB
Copy link
Member

SimenB commented Jun 8, 2019 •
edited
Loading

Belatedly, 0.7.4 had been published with a fix (although it was also fixed in Jest some time ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@SimenB @tunnckoCore