Migrate Java 25 concurrent smoke tests#11913
Conversation
There was a problem hiding this comment.
More details
Migration from Groovy/Spock to JUnit 5 Java passes validation. The Java smoke tests are functionally equivalent to the Groovy originals with improved null safety (Optional return types, Objects.equals) and stricter span count assertions that match the expected @WithSpan annotations in test applications. Removed Java 8-only Groovy compiler workaround cleanly.
📊 Validated against 5 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit 4738d4a · What is Autotest? · Any feedback? Reach out in #autotest
|
🎯 Code Coverage (details) 🔗 Commit SHA: 4738d4a | Docs | Datadog PR Page | Give us feedback! |
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
typotter
left a comment
There was a problem hiding this comment.
lgtm for flagging. thanks!
What Does This Do
Migrates the Java 25 concurrent smoke tests from Groovy/Spock to JUnit 5 Java and adds shared base classes to support Java smoke-tests migration .
Motivation
Removes the module's Java 8-only Groovy compiler workaround while keeping the broader Groovy migration incremental.
Additional Notes
Awaitilityfor trace polling and a Java 25 toolchain.