Skip to content

Commit c936387

Browse files
committed
remove duplicate tests and suppress warnings when testing for errors
1 parent 460fe63 commit c936387

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

tests/testthat/test-utils.R

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -168,20 +168,16 @@ test_that("assert_sufficient_f_args alerts if the provided f has defaults for th
168168
expect_no_error(assert_sufficient_f_args(f_xsgt, setting = "b"))
169169
expect_no_error(assert_sufficient_f_args(f_xsgt_dots, setting = "b"))
170170
expect_error(suppressWarnings(assert_sufficient_f_args(f_xs_dots, setting = "b")),
171-
regexp = "window data to `f`'s x argument",
171+
regexp = "pass the window data to `f`'s x argument",
172172
class = "epiprocess__assert_sufficient_f_args__required_args_contain_defaults")
173173

174174
# forwarding unnamed dots should not:
175175
expect_error(assert_sufficient_f_args(f_xsgt, "b"),
176176
class = "epiprocess__assert_sufficient_f_args__required_args_contain_defaults")
177177
expect_error(assert_sufficient_f_args(f_xsgt_dots, "b"),
178178
class = "epiprocess__assert_sufficient_f_args__required_args_contain_defaults")
179-
expect_error(assert_sufficient_f_args(f_xs_dots, "b"),
180-
class = "epiprocess__assert_sufficient_f_args__required_args_contain_defaults")
181-
182-
# forwarding no dots should produce a different error message in some cases:
183-
expect_error(assert_sufficient_f_args(f_xs_dots),
184-
regexp = "window data and group key to `f`'s x and setting argument",
179+
expect_error(suppressWarnings(assert_sufficient_f_args(f_xs_dots, "b")),
180+
regexp = "pass the window data and group key to `f`'s x and setting argument",
185181
class = "epiprocess__assert_sufficient_f_args__required_args_contain_defaults")
186182
})
187183

0 commit comments

Comments
 (0)