Skip to content

remove testthat prefix from badge dropdown test#81

Merged
donyunardi merged 1 commit intomainfrom
80-remove-testhat-prefix@main
May 8, 2026
Merged

remove testthat prefix from badge dropdown test#81
donyunardi merged 1 commit intomainfrom
80-remove-testhat-prefix@main

Conversation

@donyunardi
Copy link
Copy Markdown
Contributor

@donyunardi donyunardi commented May 8, 2026

Fix #80

Remove testthat prefix so it can use the package version of expect_visible.

@donyunardi donyunardi linked an issue May 8, 2026 that may be closed by this pull request
@donyunardi donyunardi added the core label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

badge

Code Coverage Summary

Filename                  Stmts    Miss  Cover    Missing
----------------------  -------  ------  -------  -------------------------------------------------------------
R/as_picks.R                140      21  85.00%   63, 167-171, 182-186, 201-207, 229-231
R/assertion.R                 5       0  100.00%
R/call_utils.R              147      22  85.03%   23-28, 65, 132-138, 259, 279-280, 283-287, 292
R/helpers.R                   9       0  100.00%
R/interaction.R              42       1  97.62%   95
R/module_merge.R            257       2  99.22%   328, 607
R/module_picks.R            323      23  92.88%   47-53, 72, 109-110, 299-301, 303-307, 441, 490, 528, 540, 548
R/picks.R                   183       1  99.45%   335
R/print.R                    36       2  94.44%   50, 58
R/resolver.R                141      15  89.36%   110-118, 284-289
R/tidyselect-helpers.R       29       0  100.00%
R/tm_merge.R                 54       0  100.00%
R/ui_containers.R            55       0  100.00%
R/zzz.R                       5       5  0.00%    3-11
TOTAL                      1426      92  93.55%

Diff against main

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

Results for commit: d350e2d

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Unit Tests Summary

  1 files   11 suites   18s ⏱️
275 tests 261 ✅ 14 💤 0 ❌
409 runs  394 ✅ 15 💤 0 ❌

Results for commit d350e2d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
badge_dropdown 💀 $0.15$ $-0.15$ $-1$ $-1$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
assertion 👶 $+0.02$ unnamed
badge_dropdown 💀 $0.15$ $-0.15$ shinytest2_badge_dropdown_is_visible_when_clicking_on_it_multiple_times

Results for commit d315747

♻️ This comment has been updated with latest results.

@donyunardi donyunardi merged commit 1dff9d4 into main May 8, 2026
24 of 25 checks passed
@donyunardi donyunardi deleted the 80-remove-testhat-prefix@main branch May 8, 2026 18:12
@github-actions github-actions Bot locked and limited conversation to collaborators May 8, 2026
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.

Remove testhat:: prefix

2 participants