Skip to content

Conversation

@llrs-roche
Copy link
Contributor

Pull Request

Follow r-hub/rhub#655 (comment)

This is trying to fix the issue blindly as we don't know what generates those files or directories, but this should clean up whatever they are.

@github-actions
Copy link
Contributor

badge

Code Coverage Summary

Filename                          Stmts    Miss  Cover    Missing
------------------------------  -------  ------  -------  ---------------------------------------------------------------------------------------
R/after.R                            59      21  64.41%   42-52, 64, 69, 77-79, 81-89, 100, 104-105
R/checkmate.R                        24       0  100.00%
R/dummy_functions.R                  61       2  96.72%   54, 56
R/include_css_js.R                   11       0  100.00%
R/init.R                            136       0  100.00%
R/module_bookmark_manager.R          99      62  37.37%   54-58, 78-133, 138-139, 151
R/module_data_summary.R             177       8  95.48%   40, 50, 205, 236-240
R/module_filter_data.R               64       0  100.00%
R/module_filter_manager.R           207       7  96.62%   116-117, 313, 340, 352, 359-360
R/module_init_data.R                 84       0  100.00%
R/module_nested_tabs.R              371      36  90.30%   163, 267-282, 302-306, 324, 361, 479-482, 486-489, 493-496
R/module_session_info.R              18       0  100.00%
R/module_snapshot_manager.R         272       9  96.69%   302-306, 373, 376-378
R/module_source_code.R               69       0  100.00%
R/module_teal_lockfile.R            131      53  59.54%   45-57, 60-62, 76, 86-88, 100-102, 110-119, 122, 124, 126-127, 142-146, 161-162, 177-186
R/module_teal_reporter.R            122       9  92.62%   60, 77-78, 81, 98, 128, 142, 144, 158
R/module_teal.R                     211       7  96.68%   127, 142-143, 183, 217, 260-261
R/module_transform_data.R           116       6  94.83%   48, 132-136
R/module_validate_error.R            73       0  100.00%
R/modules.R                         291      61  79.04%   170-174, 229-232, 357-377, 385, 391, 568-574, 587-595, 610-625, 658, 670-678
R/reporter_previewer_module.R        41      12  70.73%   41, 45, 68-85
R/teal_data_module-eval_code.R       23       0  100.00%
R/teal_data_module-within.R           7       0  100.00%
R/teal_data_module.R                 20       0  100.00%
R/teal_data_utils.R                  49       0  100.00%
R/teal_modifiers.R                   57       0  100.00%
R/teal_slices-store.R                29       0  100.00%
R/teal_slices.R                      63       0  100.00%
R/teal_transform_module.R            45       0  100.00%
R/TealAppDriver.R                   298     298  0.00%    50-621
R/utils.R                           291      48  83.51%   403-452, 540-549
R/validate_inputs.R                  32       0  100.00%
R/validations.R                      58       0  100.00%
R/zzz.R                              19       0  100.00%
TOTAL                              3628     639  82.39%

Diff against main

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

Results for commit: d7759ca

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

Unit Tests Summary

  1 files   32 suites   2m 34s ⏱️
404 tests 343 ✅ 61 💤 0 ❌
620 runs  559 ✅ 61 💤 0 ❌

Results for commit d7759ca.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
module_session_info 💚 $18.61$ $-1.30$ $0$ $0$ $0$ $0$
module_teal 💚 $131.07$ $-6.21$ $0$ $0$ $0$ $0$
shinytest2-disable_report 👶 $+0.34$ $+4$ $+4$ $0$ $0$
shinytest2-disable_src 👶 $+0.19$ $+2$ $+2$ $0$ $0$
shinytest2-show-rcode 👶 $+0.27$ $+3$ $+3$ $0$ $0$
shinytest2-teal_modifiers 👶 $+0.47$ $+7$ $+7$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
module_session_info 💚 $18.60$ $-1.30$ srv_session_info_lockfile_creation_process_is_invoked_for_teal.lockfile.mode_enabled_and_snapshot_is_copied_to_teal_app.lock_and_removed_after_session_ended
shinytest2-decorators 💀 $0.03$ $-0.03$ unnamed
shinytest2-disable_report 👶 $+0.09$ Add_to_report_button_is_not_disabled_by_default.
shinytest2-disable_report 👶 $+0.09$ Report_button_is_active_on_a_nested_module_by_default
shinytest2-disable_report 👶 $+0.08$ Report_button_is_disabled_on_a_module_changed_by_disable_report_
shinytest2-disable_report 👶 $+0.09$ Report_button_is_disabled_on_nested_modules_changed_by_disable_report_
shinytest2-disable_src 👶 $+0.10$ Show_R_Code_button_is_disabled_on_a_module
shinytest2-disable_src 👶 $+0.09$ Show_R_Code_is_disabled_on_nested_modules_changed_with_disable_src
shinytest2-reporter 💀 $0.02$ $-0.02$ unnamed
shinytest2-show-rcode 👶 $+0.09$ e2e_Module_with_Show_R_Code_has_code
shinytest2-show-rcode 👶 $+0.10$ e2e_Module_with_Show_R_Code_has_modal_with_two_dismiss_and_two_copy_to_clipboard_buttons
shinytest2-show-rcode 👶 $+0.09$ e2e_Module_with_Show_R_Code_initializes_with_visible_button
shinytest2-teal_data_module 💀 $0.02$ $-0.02$ unnamed
shinytest2-teal_modifiers 👶 $+0.07$ e2e_add_landing_modal_displays_landing_modal_on_app_startup
shinytest2-teal_modifiers 👶 $+0.07$ e2e_add_landing_modal_modal_can_be_dismissed
shinytest2-teal_modifiers 👶 $+0.07$ e2e_combined_modifiers_displays_all_customizations_when_chained_together
shinytest2-teal_modifiers 👶 $+0.07$ e2e_modify_footer_displays_custom_footer_in_the_app
shinytest2-teal_modifiers 👶 $+0.07$ e2e_modify_header_displays_custom_header_in_the_app
shinytest2-teal_modifiers 👶 $+0.07$ e2e_modify_title_sets_custom_title_in_the_page_title_head_title_displays_custom_favicon_in_the_app
shinytest2-teal_modifiers 👶 $+0.07$ e2e_modify_title_sets_custom_title_in_the_page_title_head_title_displays_custom_title_in_the_app

Results for commit e991288

♻️ This comment has been updated with latest results.

@llrs-roche llrs-roche requested a review from a team November 19, 2025 10:59
@llrs-roche llrs-roche mentioned this pull request Nov 19, 2025
33 tasks
@osenan osenan self-assigned this Nov 19, 2025
Copy link
Contributor

@osenan osenan left a comment

Choose a reason for hiding this comment

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

Thanks for the link. I think it should work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants