NEWS bullet about condition

Author: DavisVaughan

NEWS.md

@@ -1,5 +1,7 @@
 # dplyr (development version)
 
+* `if_else()` no longer allows `condition` to be a logical array. It must be a logical vector with no `dim` attribute (#7723).
+
 * `if_else()` has gotten significantly faster and uses much less memory due to a rewrite in C via `vctrs::vec_if_else()` (#7723).
 
 * Passing `size` to `if_else()` is now deprecated. The output size is always taken from the `condition` (#7722).

tests/testthat/_snaps/if-else.md

@@ -22,6 +22,14 @@
       Error in `if_else()`:
       ! `condition` must be a logical vector, not an integer vector.
 
+# `condition` can't be an array (#7723)
+
+    Code
+      if_else(array(TRUE), 1, 2)
+    Condition
+      Error in `if_else()`:
+      ! `condition` must be a logical vector, not a logical vector.
+
 # `true`, `false`, and `missing` must recycle to the size of `condition`
 
     Code

tests/testthat/test-if-else.R

@@ -74,6 +74,14 @@ test_that("`condition` must be logical (and isn't cast to logical!)", {
   })
 })
 
+test_that("`condition` can't be an array (#7723)", {
+  expect_snapshot(error = TRUE, {
+    # TODO: Error message will improve with
+    # https://github.com/r-lib/rlang/pull/1832
+    if_else(array(TRUE), 1, 2)
+  })
+})
+
 test_that("`true`, `false`, and `missing` must recycle to the size of `condition`", {
   x <- 1:3
   bad <- 1:2