insightsengineering / teal.modules.general

General Purpose Teal Modules
https://insightsengineering.github.io/teal.modules.general/
Other
9 stars 13 forks source link

vbump testthat #763

Closed pawelru closed 4 months ago

pawelru commented 4 months ago

fix min_isolated strategy https://github.com/insightsengineering/teal.modules.general/actions/runs/9630557319/job/26561246365

══ Failed tests ════════════════════════════════════════════════════════════════
── Error (test-examples.R:143:1): (code run outside of `test_that()`) ──────────
Error: 'is_checking' is not an exported object from 'namespace:testthat'
Backtrace:
    ▆
 1. └─teal.modules.general (local) rd_files() at test-examples.R:143:1
── Error (test-tm_g_bivariate.R:243:3): tm_g_bivariate determines `color`, `size` and `fill` when `color_setting` is TRUE ──
Error: 'expect_contains' is not an exported object from 'namespace:testthat'

[ FAIL 2 | WARN 0 | SKIP 78 | PASS 66 ]

is_checking is available since 3.1.9 (and it's missing in 3.1.8)

test runs available here: https://github.com/insightsengineering/teal.modules.general/actions/workflows/scheduled.yaml?query=branch%3Apawelru-patch-1

github-actions[bot] commented 4 months ago

CLA Assistant Lite bot ✅ All contributors have signed the CLA

pawelru commented 4 months ago

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

github-actions[bot] commented 4 months ago

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  ------------------------------------
R/tm_a_pca.R                    827     827  0.00%    108-1068
R/tm_a_regression.R             773     773  0.00%    153-1031
R/tm_data_table.R               185     185  0.00%    93-332
R/tm_file_viewer.R              173     173  0.00%    44-252
R/tm_front_page.R               133     122  8.27%    70-228
R/tm_g_association.R            330     330  0.00%    135-537
R/tm_g_bivariate.R              672     410  38.99%   303-769, 810, 921, 938, 956, 967-989
R/tm_g_distribution.R          1050    1050  0.00%    122-1311
R/tm_g_response.R               351     351  0.00%    154-578
R/tm_g_scatterplot.R            722     722  0.00%    230-1053
R/tm_g_scatterplotmatrix.R      278     259  6.83%    165-472, 533, 547
R/tm_missing_data.R            1069    1069  0.00%    92-1317
R/tm_outliers.R                 985     985  0.00%    134-1263
R/tm_t_crosstable.R             251     251  0.00%    141-440
R/tm_variable_browser.R         830     825  0.60%    79-1071, 1109-1293
R/utils.R                        99      96  3.03%    82-267
R/zzz.R                           2       2  0.00%    2-3
TOTAL                          8730    8430  3.44%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 47cb0c5319ddf9273bc3870aa449ceb9cb5fb5ba

Minimum allowed coverage is 80%

:recycle: This comment has been updated with latest results

github-actions[bot] commented 4 months ago

Unit Tests Summary

  1 files   22 suites   13m 4s :stopwatch: 147 tests 147 :white_check_mark: 0 :zzz: 0 :x: 478 runs  478 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 47cb0c53.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 4 months ago

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
shinytest2-tm_a_pca 💚 $115.51$ $-1.70$ $0$ $0$ $0$ $0$
shinytest2-tm_g_scatterplot 💚 $73.74$ $-1.29$ $0$ $0$ $0$ $0$
shinytest2-tm_misssing_data 💔 $47.11$ $+1.24$ $0$ $0$ $0$ $0$
shinytest2-tm_outliers 💚 $105.32$ $-3.67$ $0$ $0$ $0$ $0$
shinytest2-tm_variable_browser 💚 $70.86$ $-9.37$ $0$ $0$ $0$ $0$
Additional test case details | Test Suite | $Status$ | Time on `main` | $±Time$ | Test Case | |:-----|:----:|:----:|:----:|:-----| | shinytest2-tm_misssing_data | 💔 | $12.39$ | $+2.27$ | e2e_tm_missing_data_Default_settings_and_visibility_of_the_summary_graph | | shinytest2-tm_variable_browser | 💚 | $13.25$ | $-2.74$ | e2e_tm_variable_browser_changing_display_density_encoding_doesn_t_show_errors. | | shinytest2-tm_variable_browser | 💚 | $15.33$ | $-1.60$ | e2e_tm_variable_browser_changing_outlier_definition_encoding_doesn_t_show_errors. | | shinytest2-tm_variable_browser | 💚 | $15.41$ | $-1.43$ | e2e_tm_variable_browser_changing_plot_setting_encodings_doesn_t_show_errors. | | shinytest2-tm_variable_browser | 💚 | $10.72$ | $-2.97$ | e2e_tm_variable_browser_content_is_displayed_correctly. |

Results for commit 9d13b7b5ed6b86bbfd16b8a6965a316339a212fb

♻️ This comment has been updated with latest results.