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

12: add custom assertion to DRY #23

Merged
merged 1 commit into from
Feb 12, 2024
Merged

12: add custom assertion to DRY #23

merged 1 commit into from
Feb 12, 2024

Conversation

danielinteractive
Copy link
Collaborator

@danielinteractive danielinteractive commented Feb 9, 2024

closes #12

(DRY: Don't Repeat Yourself :-))

Copy link
Contributor

github-actions bot commented Feb 9, 2024

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  ---------
R/aalen_johansen.R               62       0  100.00%
R/assert.R                        5       0  100.00%
R/generate_data.R                27       0  100.00%
R/inc_prop.R                      7       0  100.00%
R/one_minus_kaplan_meier.R       21       0  100.00%
R/prop_trans_inc_dens_ce.R       23       0  100.00%
R/prop_trans_inc_dens.R          10       0  100.00%
TOTAL                           155       0  100.00%

Diff against main

Filename                      Stmts    Miss  Cover
--------------------------  -------  ------  --------
R/aalen_johansen.R               -3       0  +100.00%
R/assert.R                       +5       0  +100.00%
R/inc_prop.R                     -3       0  +100.00%
R/one_minus_kaplan_meier.R       -3       0  +100.00%
R/prop_trans_inc_dens_ce.R       -3       0  +100.00%
R/prop_trans_inc_dens.R          -3       0  +100.00%
TOTAL                           -10       0  +100.00%

Results for commit: e5dbebc

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Feb 9, 2024

Unit Tests Summary

 1 files   7 suites   1s ⏱️
12 tests 12 ✅ 0 💤 0 ❌
19 runs  19 ✅ 0 💤 0 ❌

Results for commit e5dbebc.

Copy link
Contributor

github-actions bot commented Feb 9, 2024

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
assert 👶 $+0.04$ $+6$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
assert 👶 $+0.02$ assert_ae_data_fails_as_expected
assert 👶 $+0.02$ assert_ae_data_passes_as_expected

Results for commit 05d1692

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@kuenzelt kuenzelt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks very much!

@danielinteractive danielinteractive merged commit b990b8c into main Feb 12, 2024
23 checks passed
@danielinteractive danielinteractive deleted the 12_assertion branch February 12, 2024 14:36
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.

Add custom assertion for data to avoid repetition
2 participants