Skip to content

Increase timeout for test failing in CI#1715

Merged
donyunardi merged 1 commit into
mainfrom
increase-timeout@main
May 27, 2026
Merged

Increase timeout for test failing in CI#1715
donyunardi merged 1 commit into
mainfrom
increase-timeout@main

Conversation

@osenan

@osenan osenan commented May 25, 2026

Copy link
Copy Markdown
Contributor

The fixed test was failing in CI for not having enough timeout. This change increases the timeout only for this test, as the others are already passing and we do not want to increase the whole CI run for the tests.

To review it, see if the CI is passing. Additionally, we can check for errors locally if required.

@github-actions

Copy link
Copy Markdown
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/decorators-helpers.R               59       1  98.31%   97
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      54  45.45%   78-133
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           210       7  96.67%   119-120, 316, 343, 355, 362-363
R/module_nested_tabs.R              411      42  89.78%   120, 178-183, 211, 315-330, 350-354, 409, 527-530, 534-537, 541-544
R/module_session_info.R              18       0  100.00%
R/module_snapshot_manager.R         276       9  96.74%   315-319, 386, 389-391
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                     304      26  91.45%   130, 179, 235, 265-285, 315-316
R/module_transform_data.R           164      25  84.76%   77-81, 85-89, 106, 112, 119-120, 134-138, 188, 273-277
R/module_validate_error.R            73       0  100.00%
R/modules.R                         345      50  85.51%   170-174, 229-232, 332, 339, 353, 472, 570-576, 706-712, 725-733, 748-763, 796, 808-816
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                 23       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                      48       2  95.83%   153-154
R/teal_transform_module.R            45       0  100.00%
R/TealAppDriver.R                   363     255  29.75%   68-73, 125, 133-162, 193-202, 212, 227-275, 300-575, 603-628, 683, 685-688, 707-721, 738-739, 750-767
R/utils.R                           291      49  83.16%   38, 408-457, 545-554
R/validate_inputs.R                  57       0  100.00%
R/validations.R                      58       0  100.00%
R/zzz.R                              23       1  95.65%   25
TOTAL                              3927     626  84.06%

Diff against main

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

Results for commit: 8a3820c

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   36 suites   2m 35s ⏱️
485 tests 417 ✅ 68 💤 0 ❌
752 runs  684 ✅ 68 💤 0 ❌

Results for commit 8a3820c.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
module_session_info 💚 $207.96$ $-185.66$ $0$ $0$ $0$ $0$
module_teal 💔 $114.19$ $+2.99$ $0$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
module_session_info 💚 $207.95$ $-185.66$ 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

Results for commit b56baee

♻️ This comment has been updated with latest results.

@osenan osenan requested a review from a team May 25, 2026 15:23
@donyunardi donyunardi merged commit 016cf8c into main May 27, 2026
25 checks passed
@donyunardi donyunardi deleted the increase-timeout@main branch May 27, 2026 03:24
@github-actions github-actions Bot locked and limited conversation to collaborators May 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants