Skip to content

Commit

Permalink
[185839488]: API no longer guards against dup var names
Browse files Browse the repository at this point in the history
  • Loading branch information
gergness committed Oct 26, 2023
1 parent 507bef6 commit e5264a7
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions tests/testthat/test-add-variable.R
Original file line number Diff line number Diff line change
Expand Up @@ -410,14 +410,6 @@ with_test_authentication({
expect_true(is.Numeric(ds$newvariable))
expect_identical(mean(ds$newvariable), 10.5)
})
test_that("adding variable with duplicate name fails", {
expect_error(
addVariables(ds, VariableDefinition(df$v5,
name = "New var4", alias = "newVar4"
)),
"Variable with name: New var4 already exists"
)
})
test_that("Variable lengths must match, in an R way", {
expect_error(
ds[["not valid"]] <- 1:7,
Expand Down

0 comments on commit e5264a7

Please sign in to comment.