search

UMM-CSci-3601 / intro-to-git

A group exercise practicing with various git ideas and tools
MIT License
2 stars 7 forks source link

Add test coverage #116

Closed NicMcPhee closed 8 months ago

NicMcPhee commented 8 months ago

This adds the relevant bits from the iteration template that tell Gradle to compute and check the Jacoco test coverage stats.

Jacoco won't let us exclude just a method from the test coverage stats; we have to exclude whole classes. It then seemed to make sense to pull the main() method out into a new Main class, and then exclude that class from the code coverage stats.

We then exclude Main.class since it's just the really simple main() method.

I updated the Markdown files to make sure that the they were up-to-date with the refactoring I did here. While I was there I got excited and made several other edits that really had nothing to do with this PR – sorry. :cry: If anyone wants me to separate them out, just say so.

Closes #114

NicMcPhee commented 8 months ago

Good catch! I had "Merge when ready" turned on, so it already committed, but I'm happy about the change.