Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce shinytest2 #1127

Merged
merged 71 commits into from
Mar 18, 2024
Merged

Introduce shinytest2 #1127

Show file tree
Hide file tree
Changes from 50 commits
Commits
Show all changes
71 commits
Select commit Hold shift + click to select a range
86bc726
feat: add some shinytest2 tests
vedhav Mar 5, 2024
d3340da
feat: add test for hamburger show/hide
vedhav Mar 5, 2024
b239ba6
chore: apply changes requested from @gogonzo
vedhav Mar 6, 2024
fe3d833
feat: add some more tests
vedhav Mar 6, 2024
041e910
fix: push local changes
vedhav Mar 7, 2024
70758f1
chore: add some helper comments
vedhav Mar 7, 2024
fcf5602
chore: rename to `var_name`
vedhav Mar 7, 2024
647bc51
feat: add a function to set input
vedhav Mar 7, 2024
bf1f1da
e2e: adds snapshot manager specs
averissimo Mar 7, 2024
091d180
e2e: reuse open_filter_manager
averissimo Mar 7, 2024
2dea57b
style: minor formatting
averissimo Mar 7, 2024
3021787
fix: missing occurence on param rename
averissimo Mar 7, 2024
83b48f1
feat: add `TealAppDriver` class and migrate existing tests to use it.
vedhav Mar 8, 2024
2850256
chore: use sprintf and build docs
vedhav Mar 8, 2024
b700f55
Merge branch 'main' into 503-introduce-shinytest2@main
vedhav Mar 8, 2024
f947a8c
Merge branch 'snapshot@503-introduce-shinytest2@main' into 503-introd…
vedhav Mar 8, 2024
2b5013c
chore: migrate the snapshot tests to use the `TealAppDriver`
vedhav Mar 8, 2024
5329b2d
chore: remove function used for testing
vedhav Mar 8, 2024
b907714
Update tests/testthat/test-shinytest2-teal.R
vedhav Mar 8, 2024
2d13df0
feat: add `set_input` method to easily set inputs with input ids as s…
vedhav Mar 8, 2024
d546b02
Removes redundant `app` from R6 class (#1139)
averissimo Mar 8, 2024
c5ccfe7
chore: superseded functions and adds missing double colon
averissimo Mar 8, 2024
49b3293
Update R/TealAppDriver.R
averissimo Mar 8, 2024
287fda8
Merge branch 'main' into 503-introduce-shinytest2@main
cicdguy Mar 8, 2024
c1addbe
enable CI/CD for 503-introduce-shinytest2@main branch (#1142)
m7pr Mar 8, 2024
9107790
typos
m7pr Mar 11, 2024
6458cf9
[skip actions] Bump version to 0.15.2.9003
m7pr Mar 11, 2024
f5fa038
feat: extend `remove_filter_var` and address some comments
vedhav Mar 11, 2024
685e7e1
chore: split the tests into individual test files
vedhav Mar 11, 2024
a773004
chore: append testhat name space
vedhav Mar 11, 2024
687158a
chore: remove duplicate tests
vedhav Mar 11, 2024
9a90065
chore: rename some methods of the driver
vedhav Mar 11, 2024
30a84bf
chore: use `set_input` to set input using do.call
vedhav Mar 11, 2024
8ec02d5
feat: extend `get_active_data_filters` to get filter vars for all the…
vedhav Mar 11, 2024
f1c2180
docs: update docs
vedhav Mar 11, 2024
455825f
chore: remove duplicate code
vedhav Mar 11, 2024
18801d4
chore: add class comments similar to teal.slice
vedhav Mar 11, 2024
a8e934f
chore: simplify boolean logic as suggested by @chlebowa
vedhav Mar 11, 2024
5ee714c
fix: use the proper root to fetch the module label
vedhav Mar 11, 2024
867e7ff
chore: USE.NAMES = FALSE
vedhav Mar 11, 2024
1fbae58
Merge branch 'main' into 503-introduce-shinytest2@main
vedhav Mar 11, 2024
22df0a2
chore: rename the namespace getters
vedhav Mar 12, 2024
c58a589
[skip actions] Bump version to 0.15.2.9003
vedhav Mar 12, 2024
60963fc
chore: remove unused functions
vedhav Mar 12, 2024
fbd1d7b
test: add a check to see if filter panel changes the module content
vedhav Mar 12, 2024
c2f0ed6
test: check that module server gets executed only when module becomes…
vedhav Mar 12, 2024
11c24b5
[skip roxygen] [skip vbump] Roxygen Man Pages Auto Update
dependabot-preview[bot] Mar 12, 2024
29cbca7
fix: set the namespaces if they are not set
vedhav Mar 12, 2024
4329d3c
chore: apply suggestions from @averissimo - simplify calls
vedhav Mar 12, 2024
9c9d903
test: check connection of teal reporter module
vedhav Mar 12, 2024
289f983
503 `$open_url` method for `TealAppDriver` (#1155)
m7pr Mar 15, 2024
5227686
[skip actions] Bump version to 0.15.2.9003
m7pr Mar 15, 2024
1478763
503 `$active_module_element` method for `TealAppDriver` (#1156)
m7pr Mar 15, 2024
3254be8
chore: remove snapshot manager tests after #1011
averissimo Mar 15, 2024
a00f9ea
chore: remove snapshot manager function until #1011
averissimo Mar 15, 2024
1eb40a2
503 add `{checkmate}` assertions to `TealAppDriver` (#1157)
m7pr Mar 15, 2024
d74b99d
chore: split the test into two parts
vedhav Mar 15, 2024
f970bd8
feat: add a method for fetching all the active filters
vedhav Mar 15, 2024
f283807
Merge branch 'main' into 503-introduce-shinytest2@main
vedhav Mar 15, 2024
a21fc3d
[skip roxygen] [skip vbump] Roxygen Man Pages Auto Update
dependabot-preview[bot] Mar 15, 2024
afba505
Revert "feat: add a method for fetching all the active filters"
vedhav Mar 15, 2024
7e7e405
docs: improve class title
vedhav Mar 15, 2024
66dfc49
test: add test suggestions from @gogonzo
vedhav Mar 18, 2024
3b510fb
[skip actions] Bump version to 0.15.2.9004
vedhav Mar 18, 2024
7bacc4a
feat: get only displayed elements using js as suggested by @averissimo
vedhav Mar 18, 2024
0f9a2ed
revert: ci triggers on this branch
vedhav Mar 18, 2024
7c5a0d3
chore: fix typo
vedhav Mar 18, 2024
690cd28
503 shinytest2 removes `helper_NS` method (#1147)
averissimo Mar 18, 2024
53e3aca
[skip roxygen] [skip vbump] Roxygen Man Pages Auto Update
dependabot-preview[bot] Mar 18, 2024
ac2b020
docs: update man
averissimo Mar 18, 2024
66dd0b9
[skip roxygen] [skip vbump] Roxygen Man Pages Auto Update
dependabot-preview[bot] Mar 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .github/workflows/check.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,11 @@ on:
- ready_for_review
branches:
- main
- 503-introduce-shinytest2@main
push:
branches:
- main
- 503-introduce-shinytest2@main
workflow_dispatch:

jobs:
Expand Down
5 changes: 4 additions & 1 deletion DESCRIPTION
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ Type: Package
Package: teal
Title: Exploratory Web Apps for Analyzing Clinical Trials Data
Version: 0.15.2.9003
Date: 2024-03-08
Date: 2024-03-12
Authors@R: c(
person("Dawid", "Kaledkowski", , "[email protected]", role = c("aut", "cre"),
comment = c(ORCID = "0000-0001-9533-457X")),
Expand Down Expand Up @@ -60,6 +60,8 @@ Suggests:
MultiAssayExperiment,
R6,
rmarkdown (>= 2.19),
rvest,
shinytest2,
shinyvalidate,
testthat (>= 3.1.5),
withr (>= 2.1.0),
Expand All @@ -84,6 +86,7 @@ LazyData: true
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.3.1
Collate:
'TealAppDriver.R'
'dummy_functions.R'
'get_rcode_utils.R'
'include_css_js.R'
Expand Down
Loading
Loading