nasa / Common-Metadata-Repository

Apache License 2.0
337 stars 92 forks source link

CMR 9792: Int test cleanup part 1 #2113

Closed jmaeng72 closed 3 months ago

jmaeng72 commented 4 months ago

Overview

What is the feature/fix?

Linter errors existed in system-int-test project

What is the Solution?

Since I was already looking at integration tests for research, I decided to do some linter fixes. These are only partial of the many linter errors in the repo.

What areas of the application does this impact?

integration tests

Checklist

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 35.89744% with 25 lines in your changes are missing coverage. Please review.

Project coverage is 24.75%. Comparing base (289c472) to head (869ac84). Report is 6 commits behind head on master.

Files Patch % Lines
...test/src/cmr/system_int_test/utils/ingest_util.clj 38.46% 8 Missing :warning:
...nt-test/src/cmr/system_int_test/utils/tag_util.clj 0.00% 4 Missing :warning:
...tem-int-test/src/cmr/system_int_test/data2/kml.clj 25.00% 3 Missing :warning:
...em-int-test/src/cmr/system_int_test/data2/core.clj 33.33% 2 Missing :warning:
...nt-test/src/cmr/system_int_test/data2/opendata.clj 0.00% 2 Missing :warning:
...rc/cmr/system_int_test/utils/subscription_util.clj 0.00% 2 Missing :warning:
...st/src/cmr/system_int_test/utils/variable_util.clj 0.00% 2 Missing :warning:
...t-test/src/cmr/system_int_test/data2/atom_json.clj 50.00% 1 Missing :warning:
...st/src/cmr/system_int_test/utils/provider_util.clj 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2113 +/- ## =========================================== - Coverage 57.81% 24.75% -33.06% =========================================== Files 1044 1044 Lines 70318 70296 -22 Branches 1966 1031 -935 =========================================== - Hits 40653 17404 -23249 - Misses 27808 51861 +24053 + Partials 1857 1031 -826 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.