Conversation
we migrate 8 tests: - DependencyPeriodActionTest - DependencyResolverQueueTest - DependencyResolverTest - DependencyResolverTest - JarReaderTest - LocationsCollectingTransformerTest - EndpointPeriodicActionTest - LogPeriodicActionTest
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
There was a problem hiding this comment.
More details
The Java tests keep the behavior of the removed Groovy tests. The production code change only makes convertToURI visible to package tests.
🤖 Datadog Autotest · Commit ad46be5 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
🟢 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. |
What Does This Do
we migrate 8 tests:
Motivation
this is part of the effort to migrate groovy tests to Java/JUnit
part1: #12430
part2: #12461
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]