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

Add coverage checking to the Java build #685

Closed
mcpierce opened this issue Apr 2, 2021 · 1 comment
Closed

Add coverage checking to the Java build #685

mcpierce opened this issue Apr 2, 2021 · 1 comment
Assignees
Labels
build Issues related to the build environment feature A new feature or enhancement to an exisiting feature
Milestone

Comments

@mcpierce
Copy link
Contributor

mcpierce commented Apr 2, 2021

The build should begin enforcing code coverage requirements, and fail when test coverage drops below an allowed threshold.

@mcpierce mcpierce added build Issues related to the build environment feature A new feature or enhancement to an exisiting feature labels Apr 2, 2021
@mcpierce mcpierce self-assigned this Apr 2, 2021
@mcpierce mcpierce added this to the 0.9 milestone Apr 2, 2021
mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 2, 2021
@mcpierce
Copy link
Contributor Author

mcpierce commented Apr 2, 2021

This initial commit adds an overall rule of 50% code coverage.

However, I added rules (commented out) to change that to require code coverage at 90% with no classes excluded. When that's achieved we can close out this ticket.

mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 2, 2021
mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 3, 2021
Avoided changing the functional code to accomodate testing.
mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 12, 2021
 * Added the models package to the list of exclusions.
 * Added the ComiXedApp class to the list of exclusions.
 * Changed the runtime tasks to remove "model" from their packaging.
mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 12, 2021
 * Added the models package to the list of exclusions.
 * Added the ComiXedApp class to the list of exclusions.
 * Changed the runtime tasks to remove "model" from their packaging.
mcpierce added a commit that referenced this issue Apr 12, 2021
 * Added the models package to the list of exclusions.
 * Added the ComiXedApp class to the list of exclusions.
 * Changed the runtime tasks to remove "model" from their packaging.
mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 13, 2021
 * Renamed Task => PersistedTask
 * Renamed WorkerTask => Task

Also updated instance variables and added Lombok annotations.
mcpierce added a commit that referenced this issue Apr 14, 2021
 * Renamed Task => PersistedTask
 * Renamed WorkerTask => Task

Also updated instance variables and added Lombok annotations.
mcpierce added a commit that referenced this issue Apr 14, 2021
 * Renamed Task => PersistedTask
 * Renamed WorkerTask => Task

Also updated instance variables and added Lombok annotations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues related to the build environment feature A new feature or enhancement to an exisiting feature
Projects
None yet
Development

No branches or pull requests

1 participant