Skip to content

Commit

Permalink
Merge georust#700
Browse files Browse the repository at this point in the history
700: Minor: Remove test_all_general() from code coverage report. r=michaelkirk a=martinfrances107

- [ x] I agree to follow the project's [code of conduct](https://github.com/georust/geo/blob/master/CODE_OF_CONDUCT.md).
---

I was using `cargo tarpaulin -o Html` to generate a code coverage webpage.

All looks good this is a well tested crate.
 
But I noticed the report painted the test function test_all_general() red  --- as having no coverage

This is a minor oversight .. test harnesses and test infrastructure should be excluded for the consideration.



Co-authored-by: Martin <martinfrances107@hotmail.com>
  • Loading branch information
bors[bot] and martinfrances107 authored Jan 31, 2022
2 parents 010662f + 7e83c0e commit 471d15a
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions geo/tests/jts_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ fn init_logging() {
}

#[test]
#[cfg(not(tarpaulin_include))]
fn test_all_general() {
init_logging();

Expand Down

0 comments on commit 471d15a

Please sign in to comment.