This is an automated email from the ASF dual-hosted git repository. elharo pushed a commit to branch deps in repository https://gitbox.apache.org/repos/asf/maven-invoker.git
commit 7ff3ab18465cfcd1ca4c28b14b8d738d1255f3ca Author: Elliotte Rusty Harold <elh...@ibiblio.org> AuthorDate: Wed Sep 27 08:11:43 2023 -0400 update maven-shared-utils to 3.4.2 --- pom.xml | 2 +- .../java/org/apache/maven/shared/invoker/DefaultInvokerTest.java | 7 +++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/pom.xml b/pom.xml index 587dad2..751dfdf 100644 --- a/pom.xml +++ b/pom.xml @@ -85,7 +85,7 @@ under the License. <dependency> <groupId>org.apache.maven.shared</groupId> <artifactId>maven-shared-utils</artifactId> - <version>3.3.4</version> + <version>3.4.2</version> </dependency> <dependency> <groupId>javax.inject</groupId> diff --git a/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java b/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java index cccb51a..b0d9f28 100644 --- a/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java +++ b/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java @@ -31,6 +31,7 @@ import org.apache.maven.shared.utils.StringUtils; import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertTrue; public class DefaultInvokerTest { @@ -84,15 +85,13 @@ public class DefaultInvokerTest { InvocationResult result = invoker.execute(request); // We check the exception to be sure the failure is based on timeout. - assertEquals( - "Error while executing external command, process killed.", - result.getExecutionException().getMessage()); + assertTrue(result.getExecutionException().getMessage().contains("timed out")); // WARN - Windows issue MSHARED-867 - Maven and child surefire test process stays alive on Windows // workaround implemented in this test to timeout test after 15 sec // please also check timeout logic in maven-shared-utils - // exitCode can't be used cause in case of an timeout it's not correctly + // exitCode can't be used because in case of a timeout it's not correctly // set in DefaultInvoker. Need to think about this. // assertEquals( 1, result.getExitCode() ); }