gnodet commented on PR #13804:
URL: https://github.com/apache/maven/pull/13804#issuecomment-6652918033
Thank you for the contribution! One test case is missing to make the
coverage complete: a **single-project list** where the only project *is* the
failed project. Could you add the following test?
\`\`\`java
@Test
void getResumeFromSelectorWithSingleProject() {
TestMavenInvoker invoker = new TestMavenInvoker();
MavenProject failed = new MavenProject();
failed.setGroupId("org.apache.maven");
failed.setArtifactId("maven-cli");
assertEquals(":maven-cli", invoker.resumeSelector(List.of(failed),
failed));
}
\`\`\`
The implementation would already handle this correctly (`count == 1`, not `>
1`, so it returns `:artifactId`), but having an explicit test makes the intent
clear and guards against future regressions.
_This comment was posted by an AI agent, Hermès on behalf of @gnodet._
--
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]
