Skip to content

Commit 8792d2f

Browse files
committed
Amend tests to detect new cli messages
1 parent c46951e commit 8792d2f

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/testthat/test-supp.R

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ test_that("supp data that does not match the main data will raise a warning but
226226

227227
expect_warning(
228228
out <- combine_supp(safetyData::sdtm_ae, sdtm_suppae_extra),
229-
"Not all rows of SUPP were merged"
229+
"Some SUPP records were not merged into the main dataset"
230230
)
231231
expect_s3_class(out, "data.frame")
232232
})
@@ -329,7 +329,7 @@ test_that("combine_supp errors when QNAM already exists in dataset", {
329329

330330
expect_error(
331331
combine_supp(simple_ae, simple_suppae),
332-
"already in the original dataset"
332+
"Column name conflict detected when combining SUPP data"
333333
)
334334
})
335335

@@ -472,7 +472,7 @@ test_that("combine_supp: extra SUPP rows that do not match core raise a warning
472472

473473
expect_warning(
474474
out <- combine_supp(pc, supppc_extra),
475-
"Not all rows of SUPP were merged"
475+
"Some SUPP records were not merged into the main dataset"
476476
)
477477
expect_s3_class(out, "data.frame")
478478
expect_equal(nrow(out), nrow(pc))

0 commit comments

Comments
 (0)