From cbca43c34f46c8c883b3e402425057634aa752aa Mon Sep 17 00:00:00 2001 From: Hyukjin Kwon Date: Fri, 26 Dec 2025 11:04:38 +0900 Subject: [PATCH] Remove obsolate todo in r/tests/testthat/test-compute-arith.R --- r/tests/testthat/test-compute-arith.R | 4 ---- 1 file changed, 4 deletions(-) diff --git a/r/tests/testthat/test-compute-arith.R b/r/tests/testthat/test-compute-arith.R index 7d77b33f262..0da42b0307a 100644 --- a/r/tests/testthat/test-compute-arith.R +++ b/r/tests/testthat/test-compute-arith.R @@ -71,8 +71,6 @@ test_that("Division", { expect_equal(a / 2 / 2, Array$create(c(1:4 / 2 / 2, NA_real_))) expect_equal(a %/% 2L %/% 2L, Array$create(c(0L, 0L, 0L, 1L, NA_integer_))) expect_equal(a / 0, Array$create(c(Inf, Inf, Inf, Inf, NA_real_))) - # TODO add tests for integer division %/% by 0 - # see https://issues.apache.org/jira/browse/ARROW-14297 b <- a$cast(float64()) expect_equal(b / 2, Array$create(c(1:4 / 2, NA_real_))) @@ -81,8 +79,6 @@ test_that("Division", { expect_equal(b %/% 2, Array$create(c(0, 1, 1, 2, NA_real_))) expect_equal(b %/% 2L, Array$create(c(0, 1, 1, 2, NA_real_))) expect_equal(b / 0, Array$create(c(Inf, Inf, Inf, Inf, NA_real_))) - # TODO add tests for integer division %/% by 0 - # see https://issues.apache.org/jira/browse/ARROW-14297 # the behavior of %/% matches R's (i.e. the integer of the quotient, not # simply dividing two integers)