The Methods' body on the TealAppDriver class should be defined as internal function calls to trigger all checks in R CMD check.
This would avoid problems such as #1197 and follow the same pattern as shinytest2::AppDriver source code.
TealAppDriver <- R6::R6Class( # nolint: object_name.
"TealAppDriver",
inherit = shinytest2::AppDriver,
# public methods ----
public = list(
initialize = function(a, b) {
private$a <- a
private$b <- b
},
foo = function(d, e) {
private$a + self$bar() + d + e
},
bar = function() {
private$b * 2
}
),
private = list(a = 0, b = 0)
)
To:
TealAppDriver <- R6::R6Class( # nolint: object_name.
"TealAppDriver",
inherit = shinytest2::AppDriver,
# public methods ----
public = list(
initialize = function(a, b) app_driver_initialize(self, private, a, b),
foo = function(d, e) app_driver_foo(self, private, d ,e),
bar = function() app_driver_bar(self, private)
),
private = list(a = 0, b = 0)
)
# Functions below defined on a different file (either 1 file per function or grouping related functions in a file `TealAppDriver-methods.R`)
app_driver_initialize <- function(self, private, a, b) {
private$a <- a
private$b <- b
}
app_driver_foo <- function(self, private, d, e) {
private$a + self$bar() + d + e
}
app_driver_bar <- function(self, private) {
private$b * 2
}
Code of Conduct
[X] I agree to follow this project's Code of Conduct.
Contribution Guidelines
[X] I agree to follow this project's Contribution Guidelines.
Security Policy
[X] I agree to follow this project's Security Policy.
Feature description
The Methods' body on the
TealAppDriver
class should be defined as internal function calls to trigger all checks in R CMD check.This would avoid problems such as #1197 and follow the same pattern as
shinytest2::AppDriver
source code.To:
Code of Conduct
Contribution Guidelines
Security Policy