Fix testWithJava21 task running zero tests by andimarek · Pull Request #4227 · graphql-java/graphql-java · GitHub
Skip to content

Fix testWithJava21 task running zero tests#4227

Merged
andimarek merged 1 commit intomasterfrom
fix-testWithJava21-no-tests
Jan 26, 2026
Merged

Fix testWithJava21 task running zero tests#4227
andimarek merged 1 commit intomasterfrom
fix-testWithJava21-no-tests

Conversation

@andimarek
Copy link
Copy Markdown
Member

Summary

  • The testWithJava21 Gradle task was missing testClassesDirs, classpath, and dependsOn configuration, causing it to discover zero test classes and silently succeed with 0 tests executed
  • CI logs confirm: testWithJava21 completed in ~2 minutes with "0 tests run in 0 ms" while testWithJava17 and testWithJava11 each ran 5336 tests in ~7 minutes
  • Added the same configuration that testWithJava17 and testWithJava11 already have

Test plan

  • Verify the testWithJava21 CI job now reports 5336 tests (matching Java 11 and Java 17)
  • Verify the testWithJava21 CI job duration is ~7 minutes (matching Java 11 and Java 17), no longer ~2 minutes
  • Verify all other CI matrix jobs still pass

🤖 Generated with Claude Code

The testWithJava21 task was missing testClassesDirs, classpath, and
dependsOn configuration that testWithJava17 and testWithJava11 have.
Without these, Gradle finds no test classes and silently succeeds
with 0 tests executed.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 26, 2026

@andimarek andimarek merged commit f1c521c into master Jan 26, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant