test: Remove redundant R version checks from tests

[Copilot]

Removes the custom skip_if_no_R4() helper and all R version skip checks from tests, as they are redundant given that the package already requires R >= 4.1.0 in DESCRIPTION.

Changes

• Removed custom skip_if_no_R4() function definition (6 lines)
• Removed 12 redundant R version skip checks

# Before
skip_if_no_R4 <- function() {
  if (R.Version()$major < 4) {
    skip("R 4.0.0 or newer not available for testing")
  }
}

test_that("dbplyr generic scalars translated correctly", {
  skip_if_no_R4()
  skip_if_not_installed("dbplyr")
  # ...
})

# After
test_that("dbplyr generic scalars translated correctly", {
  skip_if_not_installed("dbplyr")
  # ...
})

All changes are in tests/testthat/test-backend-dbplyr__duckdb_connection.R.

Since the package DESCRIPTION specifies R (>= 4.1.0) as a dependency, these version checks are unnecessary and have been removed to simplify the test code.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Uses skip_unless_r_version?</issue_title>
<issue_description>

duckdb-r/tests/testthat/test-backend-dbplyr__duckdb_connection.R

Line 1 in d35b0d0

skip_if_no_R4 <- function() {

{testthat} 3.3.0 ships testthat::skip_unless_r() as a shorthand that would obviate this helper. Would you like a PR doing so?</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Uses skip_unless_r_version? #1815

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[krlmlr]

@copilot: We require R >= 4.1.0 in DESCRIPTION, remove all skips.

[Copilot]

@copilot: We require R >= 4.1.0 in DESCRIPTION, remove all skips.

Done. Removed all R version skips in commit fe0e570.

[Copilot]

Pull request overview

This PR removes redundant R version checking logic from the test suite. Since the package DESCRIPTION already requires R (>= 4.1.0), the custom skip_if_no_R4() helper function and all its calls are unnecessary - tests will never run on R versions below 4.1.0.

• Removed the skip_if_no_R4() helper function definition (6 lines)
• Removed 12 calls to skip_if_no_R4() from test cases