search

gotwarlost / istanbul

Yet another JS code coverage tool that computes statement, line, function and branch coverage with module loader hooks to transparently add coverage when running tests. Supports all JS coverage use cases including unit tests, server side functional tests and browser tests. Built for scale.
Other
8.7k stars 786 forks source link

Cobertura reporter makes non valid method when using v8 coverage #927

Closed jehy closed 4 years ago

jehy commented 4 years ago

If you use Node 12 or 13 and v8 coverage provider, istanbul generates wrong method names, including non valid < and > characters, like this:

<method name="map.<computed>.f" hits="1" signature="()V">

Bug can be reproduced with jest and with c8: https://github.com/facebook/jest/issues/9561 https://github.com/bcoe/c8/issues/149

I can try to provide a minimal example, but it seems like a simple fix would be to replace <> with &lt; and &gt;.

jehy commented 4 years ago

Sorry, wrong repo, actual issue here: https://github.com/istanbuljs/istanbuljs/issues/527 Why is issues section open on a deprecated repo? oO