Skip to content

Conversation

@zhangfengcdt
Copy link
Member

@zhangfengcdt zhangfengcdt commented Nov 25, 2025

Did you read the Contributor Guide?

Is this PR related to a ticket?

  • No:
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

Fixes R build failures caused by breaking changes in testthat 3.3.0 (released Nov 13, 2025). The failures started appearing on Nov 24, 2025 when GitHub Actions updated to the latest testthat version.

How was this patch tested?

All R tests should pass on ci

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@zhangfengcdt zhangfengcdt changed the title TEMP - test and fix the r ci build bugfix(R): Fix testthat compatibility issues with version 3.3.0 Nov 25, 2025
@zhangfengcdt zhangfengcdt changed the title bugfix(R): Fix testthat compatibility issues with version 3.3.0 [CI] bugfix(R): Fix testthat compatibility issues with version 3.3.0 Nov 25, 2025
@zhangfengcdt zhangfengcdt marked this pull request as ready for review November 25, 2025 01:25
@zhangfengcdt zhangfengcdt requested a review from Copilot November 25, 2025 01:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses breaking changes in testthat 3.3.0 that caused R build failures, specifically by removing deprecated context() calls and simplifying the test reporter configuration.

Key Changes:

  • Removed all context() function calls from test files (deprecated in testthat 3.0.0, now causing failures in 3.3.0)
  • Simplified test reporter configuration to use testthat's default reporter instead of custom MultiReporter setup

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

File Description
R/tests/testthat.R Replaced custom MultiReporter configuration with default reporter to ensure compatibility with testthat 3.3.0
R/tests/testthat/test-*.R (13 files) Removed deprecated context() calls from all test files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jiayuasu jiayuasu merged commit 3796f0e into apache:master Nov 25, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants