Skip to content

Commit

Permalink
Use CdmExecutionSettings in all helper functions
Browse files Browse the repository at this point in the history
  • Loading branch information
anthonysena committed Nov 13, 2024
1 parent b4772a8 commit a54cb57
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions R/StrategusModule.R
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ StrategusModule <- R6::R6Class(
.replaceCovariateSettingsCohortTableNames <- function(covariateSettings, executionSettings) {
errorMessages <- checkmate::makeAssertCollection()
checkmate::assertList(covariateSettings, min.len = 1, add = errorMessages)
checkmate::assertClass(executionSettings, "ExecutionSettings", add = errorMessages)
checkmate::assertClass(executionSettings, "CdmExecutionSettings", add = errorMessages)
checkmate::reportAssertions(collection = errorMessages)

.replaceProperties <- function(s) {
Expand All @@ -343,7 +343,7 @@ StrategusModule <- R6::R6Class(
.replaceCovariateSettings <- function(moduleSettings, executionSettings) {
errorMessages <- checkmate::makeAssertCollection()
checkmate::assertList(moduleSettings, min.len = 1, add = errorMessages)
checkmate::assertClass(executionSettings, "ExecutionSettings", add = errorMessages)
checkmate::assertClass(executionSettings, "CdmExecutionSettings", add = errorMessages)
checkmate::reportAssertions(collection = errorMessages)

# A helper function to perform the replacement
Expand Down

0 comments on commit a54cb57

Please sign in to comment.