-
Notifications
You must be signed in to change notification settings - Fork 787
_ #817
Comments
I have the same problem, my repo is: To reproduce just run |
I have the same issue. I did a bit of investigation and it seems to be at this point: At the point it errors on my project, However the coverage report is generating correctly! I put in a local hack to stop the error from showing: Not sure what's causing this issue though as I'm not too familiar with the Istanbul source, so would be good to get some suggestions on a fix to the underlying problem? FYI: This issue started appearing when I updated our project to |
it is solved with: #819 |
Hello. When it's going to be fixed? I cannot get proper html report |
No news ? |
Dev's on holiday? How long is this going to take? |
Reverting |
I am using gulp-istanbul 1.1.3 which depends on istanbul 0.4.x and I encountered this issue (when it was using istanbul 0.4.5). Manually forcing gulp-istanbul to use istanbul 0.4.4 resulted in the same error. @heronblade Is it possible another change helped you fix this or was it just the downgrade? Also, this issue seems to be stuck without any updates or progress. Is there anything users can provide to help the project maintainers root cause this? It seems related to source maps but my mocha test setup doesn't really do anything with source maps so I'm not sure where to try to fiddle with things to get around this error. |
Reverting to istanbul 0.4.4 did not solve the issue for me, either. |
Bump. So this issue is an unsolved blocker? Any other workaround suggestions. Istanbul seems like the only game in time for modern js coverage so I'm a bit lost where to go from here. |
FWIW, I gave up using gulp to drive my mocha test run. When I instead invoked nyc via an npm script, it worked fine. I think that means using this package by itself is no longer supported. |
Check out #602 for a resolution |
I am using simple istanbul and still recieveing this error. Does anyone know how to solve it? It is generating lcov.info correctly but not index.html |
FWIW I switched to using nyc via its CLI and it's working. I think this means this package is unmaintained/dead for other use cases. |
In my case, i am using protractor + istanbul to get coverage for integration testing. Checking code, |
I've started seeing this issue a couple days ago. A week ago everything worked smoothly. Looks like our js source code was changed and somehow uncovered this bug.
We are combining coverage from a set of files. Looks like calculation is fine, but the report generation is failed
Another possible reason can be that we switched from angulajs-1.7.0 to angularjs-1.7.3 a month ago fixed: the issue was that our source code and coverage information was not synchronized, we are using karma/istanbul/jasmine for e2e tests coverage calculation |
Fixed: I remove karma-coverage and used |
For |
Hi , Solved it for me . Please can this issue be resolved so that we don't have to patch Istanbul ourselves. |
The real problem seems not to be in "html" reporter module. I ran into the same problem and after some investigations, I make a patch myself on the "html" reporter module to skip the error and I realize that the highlighted uncovered lines seems to be shifted with the real code. Hope this trick can help another desperate code coverage lover, and sorry for my bad english |
_
The text was updated successfully, but these errors were encountered: