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

Pylbo: extra unit tests + code coverage #74

Merged
merged 10 commits into from
Jul 30, 2021
Merged

Conversation

n-claes
Copy link
Owner

@n-claes n-claes commented Jul 16, 2021

PR description

This PR is a follow-up on #70 and adds a lot of unit tests for the pylbo framework. Code coverage for pylbo is enabled as well, partially closing #58.

@n-claes n-claes added pylbo Affects the post-processing framework Pylbo tests-unit Modified or new unit tests labels Jul 16, 2021
@n-claes n-claes added this to the v1.1 milestone Jul 16, 2021
@n-claes n-claes changed the title Tests/pylbo testing Pylbo: extra unit tests + code coverage Jul 16, 2021
@n-claes n-claes linked an issue Jul 16, 2021 that may be closed by this pull request
Repository owner deleted a comment from codecov bot Jul 16, 2021
Repository owner deleted a comment from codecov bot Jul 16, 2021
@n-claes n-claes force-pushed the tests/pylbo_testing branch 2 times, most recently from 0dae102 to a093f7c Compare July 16, 2021 14:45
Repository owner deleted a comment from codecov bot Jul 16, 2021
Repository owner deleted a comment from codecov bot Jul 29, 2021
@codecov
Copy link

codecov bot commented Jul 29, 2021

Codecov Report

Merging #74 (bee1504) into develop (35ec9ea) will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #74      +/-   ##
===========================================
+ Coverage    81.32%   81.46%   +0.14%     
===========================================
  Files           78       99      +21     
  Lines         3983     5876    +1893     
===========================================
+ Hits          3239     4787    +1548     
- Misses         744     1089     +345     
Flag Coverage Δ
legolas 81.32% <ø> (?)
pylbo 81.77% <100.00%> (?)

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

@n-claes n-claes merged commit d10a3a3 into develop Jul 30, 2021
@n-claes n-claes deleted the tests/pylbo_testing branch July 30, 2021 07:42
@n-claes n-claes mentioned this pull request Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pylbo Affects the post-processing framework Pylbo tests-unit Modified or new unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI code coverage
1 participant