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

Issue #3316: use pitest testing of checks.sizes package in shippable CI #3631

Merged
merged 1 commit into from Dec 9, 2016
Merged

Issue #3316: use pitest testing of checks.sizes package in shippable CI #3631

merged 1 commit into from Dec 9, 2016

Conversation

Efremenkov
Copy link
Contributor

Issue #3316.

Due to the fact that pitest works for a long time, and CI will perceive it as an timeout error, we need to split pitest testing packages on maven profiles. At first I added the sizes package of checks to separate maven profile and added pitest testing of this package in shippable CI.

@codecov-io
Copy link

codecov-io commented Dec 9, 2016

Current coverage is 100% (diff: 100%)

Merging #3631 into master will not change coverage

@@           master   #3631   diff @@
=====================================
  Files         275     275          
  Lines       13325   13325          
  Methods         0       0          
  Messages        0       0          
  Branches     3037    3037          
=====================================
  Hits        13325   13325          
  Misses          0       0          
  Partials        0       0          

Powered by Codecov. Last update ec2834e...b1619c8

@romani romani merged commit ba178a2 into checkstyle:master Dec 9, 2016
@romani
Copy link
Member

romani commented Dec 9, 2016

pitest validation on "size" package took - 5m 55s

@Efremenkov Efremenkov deleted the pitest branch December 10, 2016 18:57
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

Successfully merging this pull request may close these issues.

None yet

3 participants