This is an automated email from the ASF dual-hosted git repository.

sor pushed a commit to branch 1564
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git


The following commit(s) were added to refs/heads/1564 by this push:
     new 1d84b39  Upgrade surefire-junit-platform to use JUnit Platform 1.3.1
1d84b39 is described below

commit 1d84b3927285a257af39cb99042722a9f275f395
Author: Christian Stein <sormu...@gmail.com>
AuthorDate: Wed Sep 26 13:33:32 2018 +0200

    Upgrade surefire-junit-platform to use JUnit Platform 1.3.1
    
    Also upgraded internal unit test to use JUnit Jupiter 5.3.1 which
    introduce a new parameter `ConfigurationParameters` in the constructor
    of `ClassTestDescriptor`.
---
 .../java/org/apache/maven/surefire/its/JUnitPlatformIT.java    | 10 +++++-----
 surefire-providers/surefire-junit-platform/pom.xml             |  4 ++--
 .../maven/surefire/junitplatform/RunListenerAdapterTest.java   |  5 ++++-
 .../maven/surefire/junitplatform/TestMethodFilterTest.java     |  6 +++++-
 4 files changed, 16 insertions(+), 9 deletions(-)

diff --git 
a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java 
b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java
index 87f338a..fbe527d 100644
--- 
a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java
+++ 
b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformIT.java
@@ -67,7 +67,7 @@ public class JUnitPlatformIT
                 .executeTest()
                 .verifyErrorFree( 1 );
 
-        validator.verifyTextInLog( "test(compact) classpath" ) // is fine, all 
versions are coming from the project
+        validator.verifyTextInLog( "test(compact) classpath" )
                 .verifyTextInLog( "apiguardian-api-1.0.0.jar" )
                 .verifyTextInLog( "junit-jupiter-api-5.0.0.jar" )
                 .verifyTextInLog( "junit-jupiter-engine-5.0.0.jar" )
@@ -75,14 +75,14 @@ public class JUnitPlatformIT
                 .verifyTextInLog( "junit-platform-engine-1.0.0.jar" )
                 .verifyTextInLog( "opentest4j-1.0.0.jar" );
 
-        validator.verifyTextInLog( "provider(compact) classpath" ) // is 
broken, some versions remain on "1.2.0"
+        validator.verifyTextInLog( "provider(compact) classpath" )
                 .verifyTextInLog( "surefire-api-" ) // and current version, 
like "2.22.1-SNAPSHOT.jar"
                 .verifyTextInLog( "surefire-junit-platform-" ) // and current 
version, like "2.22.1-SNAPSHOT.jar"
                 .verifyTextInLog( "apiguardian-api-1.0.0.jar" )
                 .verifyTextInLog( "junit-jupiter-engine-5.0.0.jar" )
-                .verifyTextInLog( "junit-platform-commons-1.0.0.jar" ) // is 
always "1.2.0"
-                .verifyTextInLog( "junit-platform-engine-1.0.0.jar" ) // is 
always "1.2.0"
-                .verifyTextInLog( "junit-platform-launcher-1.0.0.jar" ) // is 
always "1.2.0"
+                .verifyTextInLog( "junit-platform-commons-1.0.0.jar" )
+                .verifyTextInLog( "junit-platform-engine-1.0.0.jar" )
+                .verifyTextInLog( "junit-platform-launcher-1.3.1.jar" ) // is 
always "1.3.1" for now
                 .verifyTextInLog( "opentest4j-1.0.0.jar" );
     }
 
diff --git a/surefire-providers/surefire-junit-platform/pom.xml 
b/surefire-providers/surefire-junit-platform/pom.xml
index 18be313..4a57646 100644
--- a/surefire-providers/surefire-junit-platform/pom.xml
+++ b/surefire-providers/surefire-junit-platform/pom.xml
@@ -86,12 +86,12 @@
         <dependency>
             <groupId>org.junit.platform</groupId>
             <artifactId>junit-platform-launcher</artifactId>
-            <version>1.2.0</version>
+            <version>1.3.1</version>
         </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter-engine</artifactId>
-            <version>5.2.0</version>
+            <version>5.3.1</version>
             <scope>test</scope>
         </dependency>
         <dependency>
diff --git 
a/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/RunListenerAdapterTest.java
 
b/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/RunListenerAdapterTest.java
index 0cf526c..5fa9bad 100644
--- 
a/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/RunListenerAdapterTest.java
+++ 
b/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/RunListenerAdapterTest.java
@@ -48,6 +48,7 @@ import org.junit.Test;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.engine.descriptor.ClassTestDescriptor;
 import org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor;
+import org.junit.platform.engine.ConfigurationParameters;
 import org.junit.platform.engine.TestDescriptor;
 import org.junit.platform.engine.TestDescriptor.Type;
 import org.junit.platform.engine.TestExecutionResult;
@@ -68,6 +69,8 @@ import org.mockito.InOrder;
  */
 public class RunListenerAdapterTest
 {
+    private static final ConfigurationParameters configParams = 
mock(ConfigurationParameters.class);
+
     private RunListener listener;
 
     private RunListenerAdapter adapter;
@@ -444,7 +447,7 @@ public class RunListenerAdapterTest
 
     private static TestDescriptor newClassDescriptor()
     {
-        return new ClassTestDescriptor( UniqueId.root( "class", 
MyTestClass.class.getName() ), MyTestClass.class );
+        return new ClassTestDescriptor( UniqueId.root( "class", 
MyTestClass.class.getName() ), MyTestClass.class, configParams );
     }
 
     private static TestIdentifier newSourcelessChildIdentifierWithParent(
diff --git 
a/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/TestMethodFilterTest.java
 
b/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/TestMethodFilterTest.java
index efb0589..aeb86a7 100644
--- 
a/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/TestMethodFilterTest.java
+++ 
b/surefire-providers/surefire-junit-platform/src/test/java/org/apache/maven/surefire/junitplatform/TestMethodFilterTest.java
@@ -31,6 +31,7 @@ import org.apache.maven.surefire.testset.TestListResolver;
 import org.junit.Test;
 import org.junit.jupiter.engine.descriptor.ClassTestDescriptor;
 import org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor;
+import org.junit.platform.engine.ConfigurationParameters;
 import org.junit.platform.engine.FilterResult;
 import org.junit.platform.engine.UniqueId;
 
@@ -41,6 +42,9 @@ import org.junit.platform.engine.UniqueId;
  */
 public class TestMethodFilterTest
 {
+
+    private static final ConfigurationParameters configParams = 
mock(ConfigurationParameters.class);
+
     private final TestListResolver resolver = mock( TestListResolver.class );
 
     private final TestMethodFilter filter = new TestMethodFilter( 
this.resolver );
@@ -90,7 +94,7 @@ public class TestMethodFilterTest
     private static ClassTestDescriptor newClassTestDescriptor()
     {
         UniqueId uniqueId = UniqueId.forEngine( "class" );
-        return new ClassTestDescriptor( uniqueId, TestClass.class );
+        return new ClassTestDescriptor( uniqueId, TestClass.class, 
configParams );
     }
 
     public static class TestClass

Reply via email to