netliomax25-code commented on PR #2584: URL: https://github.com/apache/groovy/pull/2584#issuecomment-4659362224
Done. Both Copilot points are covered: 1. Added a `module: '..'` test to GrapeIvyTest so the check is pinned for all three coordinate keys, not just version and group. 2. Added matching dot-dot tests (version, group, module) to GrapeMavenTest for the Maven engine. That needed a `testImplementation projects.groovyTest` dependency there so `shouldFail` resolves. Also reworded the commit to reference GROOVY-12073. Ran both grape suites with `-Djunit.network=true` and the new tests pass. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
