From a468619032039f594b38e8614a099c9cfaaf3cf8 Mon Sep 17 00:00:00 2001 From: Nic Crane Date: Thu, 18 Jun 2026 14:26:20 +0000 Subject: [PATCH 1/2] update class path name --- r/tests/testthat/test-duckdb.R | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/r/tests/testthat/test-duckdb.R b/r/tests/testthat/test-duckdb.R index 8b572268e35f..f1719a68f661 100644 --- a/r/tests/testthat/test-duckdb.R +++ b/r/tests/testthat/test-duckdb.R @@ -129,7 +129,7 @@ test_that("to_duckdb then to_arrow", { select(-fct) # alter the class of ds_rt's connection to simulate some other database - class(ds_rt$src$con) <- "some_other_connection" + class(ds_rt$con) <- "some_other_connection" expect_error( to_arrow(ds_rt), From 68643233b240eaa34df2c60fcea58368d1b528d9 Mon Sep 17 00:00:00 2001 From: Nic Crane Date: Thu, 18 Jun 2026 17:40:15 +0100 Subject: [PATCH 2/2] skip on older dbplyr --- r/tests/testthat/test-duckdb.R | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/r/tests/testthat/test-duckdb.R b/r/tests/testthat/test-duckdb.R index f1719a68f661..c0e69026488d 100644 --- a/r/tests/testthat/test-duckdb.R +++ b/r/tests/testthat/test-duckdb.R @@ -123,6 +123,10 @@ test_that("to_duckdb then to_arrow", { ) # Now check errors + # dbplyr 2.6.0 added "con" to the allowed $ fields on tbl_lazy; + # older versions only allow "src" and "lazy_query" + skip_if(packageVersion("dbplyr") < "2.6.0") + ds_rt <- ds |> to_duckdb() |> # factors don't roundtrip https://github.com/duckdb/duckdb/issues/1879