Skip to content

Align Gradle with upstream: Adopt test suites for smoke tests#4612

Closed
trask wants to merge 1 commit intomainfrom
gradle-test-suites
Closed

Align Gradle with upstream: Adopt test suites for smoke tests#4612
trask wants to merge 1 commit intomainfrom
gradle-test-suites

Conversation

@trask
Copy link
Member

@trask trask commented Feb 5, 2026

Replace manual source set and configuration setup with native Gradle JvmTestSuite. This aligns with upstream OpenTelemetry Java instrumentation patterns (adopted in v1.24.0) and provides better IDE integration.

@trask trask changed the title Adopt Gradle test suites for smoke tests Align Gradle with upstream: Adopt test suites for smoke tests Feb 5, 2026
@trask trask force-pushed the gradle-test-suites branch 6 times, most recently from ca9e4d2 to 239fcc5 Compare February 6, 2026 18:13
Replace manual source set and configuration setup with native Gradle
JvmTestSuite. This aligns with upstream OpenTelemetry Java instrumentation
patterns (adopted in v1.24.0) and provides better IDE integration.

Changes:
- Remove manual sourceSets.create("smokeTest") configuration
- Remove manual configuration extensions (smokeTestImplementation, etc.)
- Add testing.suites block with JvmTestSuite registration
- Move dependencies into test suite DSL
- Preserve all existing behavior (environment matrix, system properties,
  test logging, nested test classes with @Environment annotations)

Test suite automatically creates smokeTestImplementation and other
configurations, simplifying the build logic while maintaining full
backward compatibility.
@trask trask force-pushed the gradle-test-suites branch from 239fcc5 to c5eb325 Compare February 6, 2026 19:28
@trask trask marked this pull request as ready for review February 6, 2026 20:23
@trask trask closed this Feb 6, 2026
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