Skip to content

Increase test coverage - #268

Open
llrs-roche wants to merge 12 commits into
mainfrom
802_validation
Open

llrs-roche wants to merge 12 commits into
mainfrom
802_validation

Conversation

@llrs-roche

Copy link
Copy Markdown

Pull Request

Closes https://github.com/insightsengineering/coredev-tasks/issues/802

Adds tests to the package.

In addition I fixed two warnings related to tidyselect (select(all_of())) and ggplot2 (aes_() to aes() and an argument partial match on seq(lenght) to seq(length.out).

@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                           Stmts    Miss  Cover    Missing
-------------------------------  -------  ------  -------  ------------
R/g_boxplot.R                        116       0  100.00%
R/g_correlationplot.R                138       3  97.83%   407, 413-414
R/g_density_distribution_plot.R       86       0  100.00%
R/g_lineplot.R                       276       2  99.28%   382, 592
R/g_scatterplot.R                      5       0  100.00%
R/g_spaghettiplot.R                  101       0  100.00%
R/geom_axes_line.R                   167       0  100.00%
R/t_summarytable.R                   102       0  100.00%
R/utils.R                             70       0  100.00%
TOTAL                               1061       5  99.53%

Diff against main

Filename                           Stmts    Miss  Cover
-------------------------------  -------  ------  --------
R/g_boxplot.R                          0    -116  +100.00%
R/g_correlationplot.R                 +1    -134  +97.83%
R/g_density_distribution_plot.R        0     -86  +100.00%
R/g_lineplot.R                         0    -274  +99.28%
R/g_scatterplot.R                      0      -5  +100.00%
R/g_spaghettiplot.R                    0    -101  +100.00%
R/geom_axes_line.R                     0    -167  +100.00%
R/t_summarytable.R                     0    -102  +100.00%
R/utils.R                              0     -70  +100.00%
TOTAL                                 +1   -1055  +99.53%

Results for commit: b9c9eb4

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files    9 suites   11s ⏱️
 59 tests  59 ✅ 0 💤 0 ❌
187 runs  187 ✅ 0 💤 0 ❌

Results for commit b9c9eb4.

♻️ This comment has been updated with latest results.

@llrs-roche

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

This branch has not been deployed

No deployments
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.

1 participant