-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Simple ES5 test file fails at coverage #1214
Comments
Simple ping here, just wondering if my issue is an actual issue. I have somehow trouble getting good coverage support with jest + babel or even jest + ES5 code like in this example. If coverage is not completely working today, maybe we should say so in the README? |
We are working on improving code coverage over the next few months. cc @DmitriiAbramov. |
Yep that's why I thought we could update the documentation to reflect the coverage status of today, to avoid people spending a lot of time on it. |
I'm assuming this is fixed now. @DmitriiAbramov rewrote Jest's code coverage support. Please try with |
@cpojer config:
output:
|
The current pre-release might have an issue on windows that we'll need to resolve before the final then. Sorry about that. |
Looks like the issue on Windows blames to #1379 (b56e368). I created a separate issue to look into it: #1462. Thanks for reporting, @ManuelDeLeon! |
* feat: implement spyOnProperty method, fixes #5106 * style: fix indentation of mock, runtime, jasmine2 * test: fix failing tests for spyOnProperty * test: implement tests for #1214 * style: fix eslint errors * refactor: proxy spyOnProperty call behind spyOn * refactor: remove useless console.log * style: fix eslint errors * types: remove declaration of spyOnProperty * docs: add documentation for accessType argument of spyOn * docs: fix typo in spyOn docs * test(spyOn): fix typo in should throw on invalid input * test(spyOn): add tests for setters * docs(spyOn): add example for spying on setters * style: fix eslint errors * refactor: format error messages with getErrorMsg() * style: fix eslint errors * revert: restore snapshotState.getUncheckedKeys()
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I am tryiing to user
jest --coverage
without luck, I somehow endup with weird coverage report.I have setup a reproducible repository at https://github.com/vvo/test-coverage-jest.
index.js:
test.js
result
Same with mocha:
I have setup a reproducible repository at https://github.com/vvo/test-coverage-jest.
Debug data:
The text was updated successfully, but these errors were encountered: