Skip to content

Commit 2a5e4ab

Browse files
Use linter variable in outer_negation_linter() tests (#2212)
1 parent 3c48940 commit 2a5e4ab

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

tests/testthat/test-outer_negation_linter.R

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,43 +18,45 @@ test_that("outer_negation_linter skips allowed usages", {
1818
})
1919

2020
test_that("outer_negation_linter blocks simple disallowed usages", {
21+
linter <- outer_negation_linter()
22+
2123
expect_lint(
2224
"any(!x)",
2325
rex::rex("!all(x) is better than any(!x)"),
24-
outer_negation_linter()
26+
linter
2527
)
2628

2729
expect_lint(
2830
"all(!foo(x))",
2931
rex::rex("!any(x) is better than all(!x)"),
30-
outer_negation_linter()
32+
linter
3133
)
3234

3335
# na.rm doesn't change the recommendation
3436
expect_lint(
3537
"any(!x, na.rm = TRUE)",
3638
rex::rex("!all(x) is better than any(!x)"),
37-
outer_negation_linter()
39+
linter
3840
)
3941

4042
# also catch nested usage
4143
expect_lint(
4244
"all(!(x + y))",
4345
rex::rex("!any(x) is better than all(!x)"),
44-
outer_negation_linter()
46+
linter
4547
)
4648

4749
# catch when all inputs are negated
4850
expect_lint(
4951
"any(!x, !y)",
5052
rex::rex("!all(x) is better than any(!x)"),
51-
outer_negation_linter()
53+
linter
5254
)
5355

5456
expect_lint(
5557
"all(!x, !y, na.rm = TRUE)",
5658
rex::rex("!any(x) is better than all(!x)"),
57-
outer_negation_linter()
59+
linter
5860
)
5961
})
6062

0 commit comments

Comments
 (0)