From 5da139100c98ae257352c64c1ef74f7df06fed43 Mon Sep 17 00:00:00 2001 From: Gabriel Belingueres Date: Fri, 30 Nov 2018 15:57:30 -0300 Subject: [PATCH 1/2] Solved issue #48. Changed text to assert by "OS name:", which appears in both 3.5.x Maven versions and Maven 3.6.0. --- src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java b/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java index 8cd96f6c..41e1af3f 100644 --- a/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java +++ b/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java @@ -103,7 +103,7 @@ public void testExecuteBinaryOnPath() assertTrue( out.contains( "Apache Maven" ) ); assertTrue( out.contains( "Maven home:" ) ); assertTrue( out.contains( "Java version:" ) ); - assertTrue( out.contains( "Java home:" ) ); + assertTrue( out.contains( "OS name:" ) ); } catch ( Exception e ) { From 9ed0f487579f24531c0061e92b25ec7435ed5e48 Mon Sep 17 00:00:00 2001 From: Gabriel Belingueres Date: Sat, 1 Dec 2018 11:06:36 -0300 Subject: [PATCH 2/2] Deleted "OS name:" assertion, as suggested by Robert Scholte. --- src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java | 1 - 1 file changed, 1 deletion(-) diff --git a/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java b/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java index 41e1af3f..841219af 100644 --- a/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java +++ b/src/test/java/org/codehaus/plexus/util/cli/CommandlineTest.java @@ -103,7 +103,6 @@ public void testExecuteBinaryOnPath() assertTrue( out.contains( "Apache Maven" ) ); assertTrue( out.contains( "Maven home:" ) ); assertTrue( out.contains( "Java version:" ) ); - assertTrue( out.contains( "OS name:" ) ); } catch ( Exception e ) {