Skip to content

Conversation

@averissimo
Copy link
Contributor

Pull Request

Attemps to correct errors in integration test using identical defaults to teal::TealAppDriver()

  • Shiny app did not become stable in 15000ms.
  • Element not found as it may not have been rendered yet

@averissimo averissimo enabled auto-merge (squash) November 28, 2025 12:33
@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  ------------------------------------------------------
R/basic_table_args.R             23       0  100.00%
R/draggable_buckets.R            87      12  86.21%   116-121, 151-156
R/get_dt_rows.R                  13       0  100.00%
R/ggplot2_args.R                 49       0  100.00%
R/nested_closeable_modal.R       20      20  0.00%    83-103
R/optionalInput.R               255      79  69.02%   190, 315-389, 402-409, 411-416, 434, 436, 501, 583-596
R/panel_group.R                  39      39  0.00%    50-136
R/plot_with_settings.R          309      16  94.82%   299-305, 327, 364, 373-374, 390, 578-579, 581, 583
R/standard_layout.R              52       4  92.31%   90-93
R/table_with_settings.R         158       1  99.37%   100
R/utils.R                         7       0  100.00%
R/verbatim_popup.R              105      35  66.67%   113-114, 116, 124-155
R/white_small_well.R              7       0  100.00%
TOTAL                          1124     206  81.67%

Diff against main

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

Results for commit: c4d0ee3

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

Unit Tests Summary

  1 files   16 suites   2m 1s ⏱️
170 tests 170 ✅ 0 💤 0 ❌
386 runs  386 ✅ 0 💤 0 ❌

Results for commit c4d0ee3.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
plot_with_settings_ui 💔 $53.10$ $+1.23$ $0$ $0$ $0$ $0$
table_with_settings_ui 💔 $32.55$ $+2.95$ $0$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
plot_with_settings_ui 💔 $3.49$ $+1.11$ e2e_teal.widgets_plot_with_settings_it_is_possible_to_set_height_and_width_for_the_plot_on_the_third_button_dropdown_menu_without_errors
table_with_settings_ui 💔 $4.15$ $+1.51$ e2e_teal.widgets_table_with_settings_the_click_on_expand_button_opens_a_modal_with_a_table

Results for commit 2b090f0

♻️ This comment has been updated with latest results.

@averissimo averissimo disabled auto-merge November 28, 2025 12:41
@averissimo averissimo enabled auto-merge (squash) November 28, 2025 13:13
@averissimo averissimo merged commit f8bd6a3 into main Nov 28, 2025
26 checks passed
@averissimo averissimo deleted the integration-increase_timeout branch November 28, 2025 14:18
@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants