Skip to content

Conversation

@cpsievert
Copy link
Collaborator

test(filter = "shinytest") currently fails since shinytest no longer supports the latest shiny

This PR migrates the tests to shinytest2 and gets test(filter = "shinytest") passing again

@cpsievert cpsievert requested a review from Copilot June 12, 2025 15:45

This comment was marked as spam.

@cpsievert cpsievert marked this pull request as ready for review June 12, 2025 16:15
@cpsievert cpsievert merged commit d1f897a into main Jun 12, 2025
12 checks passed
@cpsievert cpsievert deleted the shinytest2 branch June 12, 2025 16:28
cpsievert added a commit to teunbrand/thematic that referenced this pull request Jun 12, 2025
cpsievert added a commit that referenced this pull request Jun 12, 2025
* stabilise default getter

* try to fix ggplot_build shenannigans

* be more strict about restoring blank properties

* skip test

* namespace `is_theme_element()`

* Code review

* Use dev evaluate to fix README rendering bug (#159)

* Use dev evaluate to fix README rendering bug

* Avoid Remotes field

* `devtools::build_readme()` (GitHub Actions)

---------

Co-authored-by: cpsievert <[email protected]>

* Migrate from shinytest to shinytest2 (#158)

* Migrate to testthat edition 3 (#160)

* Avoid using deprecated context()

* Try setting testthat edition 3

* Skip on CI and add a note about why

* Update visual baselines (using ggplot2 v3.5.2 and R 4.5.0)

* Update visual baselines (using ggplot2-dev v4.0.0 and R 4.5.0)

* Update shiny visual baselines (using ggplot2-dev v4.0.0 and R 4.5.0)

* Revert "Code review"

This reverts commit fd3735b.

* Undo ggplot2 requirement

---------

Co-authored-by: Carson Sievert <[email protected]>
Co-authored-by: cpsievert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants