Skip to content

Remove decommissioned test code - #16096

Merged
jdaugherty merged 2 commits into
8.0.xfrom
remove-decommissioned-test-code
Aug 10, 2026
Merged

Remove decommissioned test code#16096
jdaugherty merged 2 commits into
8.0.xfrom
remove-decommissioned-test-code

Conversation

@matrei

@matrei matrei commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This pull request removes legacy Ant-based JUnit reporting support from the grails-test-core module. The main changes involve deleting custom JUnit report formatter classes and their factory, as well as cleaning up related dependencies from the build configuration.

Dependency cleanup:

  • Removed Ant and JUnit-related dependencies from grails-test-core/build.gradle, including org.apache.ant:ant, org.apache.ant:ant-junit, and various JUnit and Spock dependencies that were only used by the deleted reporting classes. [1] [2]

Removal of legacy JUnit reporting implementation:

  • Deleted the JUnitReports, JUnitReportsFactory, PlainFormatter, and XMLFormatter classes from grails-test-core/src/main/groovy/org/grails/test/report/junit/, which previously provided Ant-based JUnit report formatting and output. [1] [2] [3] [4]

matrei added 2 commits August 5, 2026 10:13
This code was used for generating test reports prior to the switch to Gradle.
@testlens-app

testlens-app Bot commented Aug 5, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: 8ac14e9
▶️ Tests: 53567 executed
⚪️ Checks: 60/60 completed


Learn more about TestLens at testlens.app.

@jdaugherty

Copy link
Copy Markdown
Contributor

@matrei we removed support, but it's been requested we add junit support back via #15588 - aren't these needed for that?

@matrei

matrei commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

@matrei we removed support, but it's been requested we add junit support back via #15588 - aren't these needed for that?

@jdaugherty As far as I understand, these are support classes for pre-Gradle test reporting.

@matrei matrei added this to the grails:8.0.0-RC1 milestone Aug 5, 2026
@matrei matrei moved this to In Progress in Apache Grails Aug 5, 2026
@jdaugherty
jdaugherty merged commit 8f1fa16 into 8.0.x Aug 10, 2026
61 of 62 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Apache Grails Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants