You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by ff...@apache.org on 2019/03/21 03:51:37 UTC

[karaf] branch jdk12 updated: [KARAF-6203]ensure karaf can build and run with JDK12-fix XATest and JmsExampleTest

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

ffang pushed a commit to branch jdk12
in repository https://gitbox.apache.org/repos/asf/karaf.git


The following commit(s) were added to refs/heads/jdk12 by this push:
     new b9e2936  [KARAF-6203]ensure karaf can build and run with JDK12-fix XATest and JmsExampleTest
b9e2936 is described below

commit b9e2936a8fabcd00291f8a0557212c46d5320c45
Author: Freeman Fang <fr...@gmail.com>
AuthorDate: Thu Mar 21 11:51:23 2019 +0800

    [KARAF-6203]ensure karaf can build and run with JDK12-fix XATest and JmsExampleTest
---
 itests/test/src/test/java/org/apache/karaf/itests/XATest.java      | 7 -------
 .../test/java/org/apache/karaf/itests/examples/JmsExampleTest.java | 7 -------
 2 files changed, 14 deletions(-)

diff --git a/itests/test/src/test/java/org/apache/karaf/itests/XATest.java b/itests/test/src/test/java/org/apache/karaf/itests/XATest.java
index 1c14c55..026a473 100644
--- a/itests/test/src/test/java/org/apache/karaf/itests/XATest.java
+++ b/itests/test/src/test/java/org/apache/karaf/itests/XATest.java
@@ -22,7 +22,6 @@ import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.ops4j.pax.exam.Configuration;
-import org.ops4j.pax.exam.karaf.container.internal.JavaVersionUtil;
 import org.ops4j.pax.exam.MavenUtils;
 import org.ops4j.pax.exam.Option;
 import org.ops4j.pax.exam.junit.PaxExam;
@@ -64,12 +63,6 @@ public class XATest extends KarafTestSupport {
         result.add(replaceConfigurationFile("etc/org.ops4j.connectionfactory-artemis.cfg", getConfigFile("/org/apache/karaf/itests/features/org.ops4j.connectionfactory-artemis.cfg")));
         result.add(replaceConfigurationFile("etc/org.ops4j.datasource-derby.cfg", getConfigFile("/org/apache/karaf/itests/features/org.ops4j.datasource-derby.cfg")));
         result.add(replaceConfigurationFile("etc/xa-test-camel.xml", getConfigFile("/org/apache/karaf/itests/features/xa-test-camel.xml")));
-        if (JavaVersionUtil.getMajorVersion() >= 9) {
-            //need asm 6.x which support java9plus to run this test
-            result.add(replaceConfigurationFile("system/org/apache/karaf/features/standard/" 
-                + version + "/standard-" + version + "-features.xml", 
-                getConfigFile("/etc/feature.xml")));
-        }
         return result.toArray(new Option[result.size()]);
     }
 
diff --git a/itests/test/src/test/java/org/apache/karaf/itests/examples/JmsExampleTest.java b/itests/test/src/test/java/org/apache/karaf/itests/examples/JmsExampleTest.java
index 38214ec..c4ca654 100644
--- a/itests/test/src/test/java/org/apache/karaf/itests/examples/JmsExampleTest.java
+++ b/itests/test/src/test/java/org/apache/karaf/itests/examples/JmsExampleTest.java
@@ -21,7 +21,6 @@ import org.ops4j.pax.exam.Configuration;
 import org.ops4j.pax.exam.MavenUtils;
 import org.ops4j.pax.exam.Option;
 import org.ops4j.pax.exam.junit.PaxExam;
-import org.ops4j.pax.exam.karaf.container.internal.JavaVersionUtil;
 import org.ops4j.pax.exam.spi.reactors.ExamReactorStrategy;
 import org.ops4j.pax.exam.spi.reactors.PerClass;
 
@@ -53,12 +52,6 @@ public class JmsExampleTest extends KarafTestSupport {
         result.add(editConfigurationFilePut("etc/org.apache.karaf.features.cfg", "featuresBoot",
                 "instance,package,log,ssh,framework,system,eventadmin,feature,shell,management,service,jaas,deployer,diagnostic,wrap,bundle,config,kar,aries-blueprint,artemis,jms,pax-jms-artemis"));
         result.add(replaceConfigurationFile("etc/org.ops4j.connectionfactory-artemis.cfg", getConfigFile("/org/apache/karaf/itests/features/org.ops4j.connectionfactory-artemis.cfg")));
-        if (JavaVersionUtil.getMajorVersion() >= 9) {
-            //need asm 6.x which support java9plus to run this test
-            result.add(replaceConfigurationFile("system/org/apache/karaf/features/standard/"
-                            + version + "/standard-" + version + "-features.xml",
-                    getConfigFile("/etc/feature.xml")));
-        }
         return result.toArray(new Option[result.size()]);
     }