Skip to content

Updating and improving unit testing#42

Merged
michaeldenes merged 8 commits intomainfrom
improving_unit_tests
Jun 7, 2024
Merged

Updating and improving unit testing#42
michaeldenes merged 8 commits intomainfrom
improving_unit_tests

Conversation

@michaeldenes
Copy link
Member

A PR that improves the unit testing, and adds new testing for new particleset creation functionality.

michaeldenes and others added 4 commits June 7, 2024 13:25
…he kernels unit testing. This allows us to remove superfluous code (like the `download_plasticparcels_dataset()` in tests that don't use the unbeaching kernel.
Merging main into improved unit testing branch for a new pull request
@codecov-commenter
Copy link

codecov-commenter commented Jun 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.71%. Comparing base (4dc18c0) to head (8326777).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   69.87%   66.71%   -3.16%     
==========================================
  Files           8        8              
  Lines         790      712      -78     
==========================================
- Hits          552      475      -77     
+ Misses        238      237       -1     
Flag Coverage Δ
unit-tests 66.71% <100.00%> (-3.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@michaeldenes michaeldenes merged commit 920fd8e into main Jun 7, 2024
@michaeldenes michaeldenes deleted the improving_unit_tests branch June 7, 2024 14:02
fanqi203 pushed a commit to fanqi203/plasticparcels that referenced this pull request Jul 8, 2025
…ests

Updating and improving unit testing
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.

3 participants