Skip to content

Isolate Gradle TestKit fixtures - #970

Merged
iuliiasobolevska merged 1 commit into
masterfrom
test/isolate-testkit-fixtures
Sep 26, 2026
Merged

iuliiasobolevska merged 1 commit into
masterfrom
test/isolate-testkit-fixtures

Conversation

@iuliiasobolevska

@iuliiasobolevska iuliiasobolevska commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • copy Gradle TestKit fixtures into a fresh JUnit temporary directory for every test
  • give each test a separate Gradle TestKit home to prevent concurrent cache lock contention
  • run nested builds out of process and keep their output isolated from parallel tests
  • keep configuration-cache round trips within the same isolated project
  • assert after each test that checked-in fixture directories contain no .gradle or build state

Performance

An isolated CodegenGradlePluginTest run with --rerun-tasks --profile completed in 8.951 seconds, compared with 10.382 seconds on master—a 13.8% improvement across all 19 tests.

Verification

  • forced four-way parallel execution of all 19 CodegenGradlePluginTest tests
  • ./gradlew --info check publishToMavenLocal --warning-mode all
  • checked-in src/test/resources contains no generated .gradle or build directories after the suite

@iuliiasobolevska iuliiasobolevska added the enhancement New feature or request label Sep 24, 2026
@iuliiasobolevska
iuliiasobolevska force-pushed the test/isolate-testkit-fixtures branch from 22997b5 to 3549c0d Compare September 24, 2026 20:34
@iuliiasobolevska
iuliiasobolevska marked this pull request as ready for review September 24, 2026 21:09
Correct the no-schema output assertions for both configured and default paths, and verify Jackson override output. Parameterize equivalent fixture cases to keep the tests focused.
@iuliiasobolevska
iuliiasobolevska force-pushed the test/isolate-testkit-fixtures branch from 3549c0d to fd31881 Compare September 26, 2026 16:23
@iuliiasobolevska
iuliiasobolevska merged commit 3d48989 into master Sep 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants