search

caitp / karma-coveralls

A Karma plugin to upload coverage reports to coveralls.io
65 stars 13 forks source link

multiple browser coverage #36

Open julkue opened 8 years ago

julkue commented 8 years ago

I'm asking myself what happens when karma-coverage generates coverage reports for multiple browsers!? Coveralls.io always shows the coverage for the last tested browser. So I'm assuming that this reporter either sends all reports or just the last one. But it would make more sense to just send the average value of all tests. @caitp what do you think?

caitp commented 8 years ago

Sending reports for multiple browsers has been part of the roadmap, but I haven't found a way to do it. If you have suggestions, PRs welcome

julkue commented 8 years ago

@caitp What is your idea how to implement this? However, unfortunately I don't use coveralls.io anymore. Currently I'm experiencing with codacy coverage.

Btw: In my opinion you should point this out in the readme. This is essential for cross-browser tests.

caitp commented 8 years ago

Looking at the coveralls API, it may be possible to have multiple coveralls jobs for the same source digest.

If I get some time this week, I'll see if I can take advantage of this in a useful way (or, you're welcome to give it a try too).

Short of that, I think we'd have to talk to Coveralls directly about supporting this use-case.

julkue commented 8 years ago

@caitp I've already reported an issue but didn't get a reply (which is one of the reasons why I will not use it in future).

caitp commented 6 years ago

I hear you man :( (sorry for the late response)