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.

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

Test failed. Because each processing's coverage is not fine. But total coverage was fine. #593

Closed
kouyaf77 opened this issue Jun 26, 2017 · 3 comments

Comments

@kouyaf77
Copy link

kouyaf77 commented Jun 26, 2017

Is it possible to check only total coverage?
I'm using simplecov (0.14.1) and parallel_tests (2.14.1)

[...]

Coverage report generated for RSpec:248423 to /vagrant/Source/Rails/xxx/coverage. 3741 / 10680 LOC (35.03%) covered.
Coverage (35.03%) is below the expected minimum coverage (80.00%).

[...]

Coverage report generated for RSpec:248423, RSpec:248434 to /vagrant/Source/Rails/xxx/coverage. 4929 / 8455 LOC (58.3%) covered.
Coverage (58.30%) is below the expected minimum coverage (80.00%).

[...]

Coverage report generated for RSpec:24838, RSpec:248423, RSpec:248434 to /vagrant/Source/Rails/xxx/coverage. 5572 / 7620 LOC (73.12%) covered.
Coverage (73.12%) is below the expected minimum coverage (80.00%).

[...]

Coverage report generated for RSpec:24838, RSpec:248423, RSpec:248434, RSpec:248472 to /vagrant/Source/Rails/xxx/coverage. 6176 / 7127 LOC (86.66%) covered.
Tests Failed

3340 examples, 0 failures, 22 pendings
@PragTob
Copy link
Collaborator

PragTob commented Jun 27, 2017

Ugh. If it doesn't do it with this setting then no it probably isn't possible atm (to the best of my knowledge) :|

That said, I wanted to enable & use that setting on our own CI and we are also running with parallel_tests. So I might have to fix that either way... still if you have a PR/idea I'd be glad to have it :D

@PragTob
Copy link
Collaborator

PragTob commented Jun 27, 2017

It seems the problem was brought up before as part of #559 and there seems to be a workaround as per this comment

@kouyaf77
Copy link
Author

kouyaf77 commented Jul 3, 2017

@PragTob
I see. Thanks a lot.
I'm going to refer this comment

@kouyaf77 kouyaf77 closed this as completed Jul 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants