update to xbib netty http 4.1.39.2, xbib net 2.0.1, junit 5

build.gradle

@@ -21,7 +21,10 @@ subprojects {
     }
 
     dependencies {
-        testCompile "junit:junit:${project.property('junit.version')}"
+        testCompile "org.junit.jupiter:junit-jupiter-api:${project.property('junit.version')}"
+        testCompile "org.junit.jupiter:junit-jupiter-params:${project.property('junit.version')}"
+        testCompile "org.junit.jupiter:junit-jupiter-engine:${project.property('junit.version')}"
+        //testCompile "org.junit.vintage:junit-vintage-engine:${project.property('junit.version')}"
         testCompile "org.xbib:bibliographic-character-sets:${project.property('xbib-bibliographic-character-sets.version')}"
     }
 
@@ -45,10 +48,20 @@ subprojects {
     }
 
     test {
-        systemProperty 'java.util.logging.config.file', 'src/test/resources/logging.properties'
+        useJUnitPlatform()
+        failFast = false
         testLogging {
-            showStandardStreams = true
-            exceptionFormat = 'full'
+            events 'STARTED', 'PASSED', 'FAILED', 'SKIPPED'
+            showStandardStreams = false
+        }
+        afterSuite { desc, result ->
+            if (!desc.parent) {
+                println "\nTest result: ${result.resultType}"
+                println "Test summary: ${result.testCount} tests, " +
+                        "${result.successfulTestCount} succeeded, " +
+                        "${result.failedTestCount} failed, " +
+                        "${result.skippedTestCount} skipped"
+            }
         }
     }
 

gradle.properties

@@ -1,18 +1,16 @@
 group = org.xbib
 name = oai
-version = 2.0.0
+version = 2.0.1
 
-xbib-content.version = 2.0.0
-xbib-netty-http.version = 4.1.38.3
+xbib-content.version = 2.0.1
+xbib-netty-http.version = 4.1.39.2
 tcnative.version = 2.0.25.Final
 
 # test
-junit.version = 4.12
+junit.version = 5.5.1
+junit4.version = 4.12
 xbib-bibliographic-character-sets.version = 1.0.0
 xbib-marc.version = 2.0.0
 
 # doc
 asciidoclet.version = 1.5.4
-
-org.gradle.warning.mode = all
-org.gradle.daemon = false

gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,5 @@
-#Wed Aug 07 14:17:29 CEST 2019
-distributionUrl=https\://services.gradle.org/distributions/gradle-5.3.1-all.zip
+#Thu Aug 29 18:18:25 CEST 2019
+distributionUrl=https\://services.gradle.org/distributions/gradle-5.6-all.zip
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStorePath=wrapper/dists

gradlew

@@ -7,7 +7,7 @@
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
-#      http://www.apache.org/licenses/LICENSE-2.0
+#      https://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
@@ -125,8 +125,8 @@ if $darwin; then
     GRADLE_OPTS="$GRADLE_OPTS \"-Xdock:name=$APP_NAME\" \"-Xdock:icon=$APP_HOME/media/gradle.icns\""
 fi
 
-# For Cygwin, switch paths to Windows format before running java
-if $cygwin ; then
+# For Cygwin or MSYS, switch paths to Windows format before running java
+if [ "$cygwin" = "true" -o "$msys" = "true" ] ; then
     APP_HOME=`cygpath --path --mixed "$APP_HOME"`
     CLASSPATH=`cygpath --path --mixed "$CLASSPATH"`
     JAVACMD=`cygpath --unix "$JAVACMD"`

gradlew.bat

@@ -5,7 +5,7 @@
 @rem you may not use this file except in compliance with the License.
 @rem You may obtain a copy of the License at
 @rem
-@rem      http://www.apache.org/licenses/LICENSE-2.0
+@rem      https://www.apache.org/licenses/LICENSE-2.0
 @rem
 @rem Unless required by applicable law or agreed to in writing, software
 @rem distributed under the License is distributed on an "AS IS" BASIS,

oai-client/src/test/java/org/xbib/oai/client/ArxivClientTest.java

@@ -1,9 +1,8 @@
 package org.xbib.oai.client;
 
-import static org.junit.Assert.assertTrue;
-
 import io.netty.handler.codec.http.HttpHeaderNames;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import org.junit.jupiter.api.Test;
 import org.xbib.net.URL;
 import org.xbib.netty.http.client.Client;
 import org.xbib.netty.http.client.Request;
@@ -28,12 +27,12 @@ import java.util.logging.Logger;
 /**
  *
  */
-public class ArxivClientTest {
+class ArxivClientTest {
 
     private static final Logger logger = Logger.getLogger(ArxivClientTest.class.getName());
 
     @Test
-    public void testListRecordsArxiv() {
+    void testListRecordsArxiv() {
         final URL url = URL.create("http://export.arxiv.org/oai2/");
         try (OAIClient client = new OAIClient(url)) {
             IdentifyRequest identifyRequest = client.newIdentifyRequest();

oai-client/src/test/java/org/xbib/oai/client/BundeskunsthalleTest.java

@@ -1,8 +1,8 @@
 package org.xbib.oai.client;
 
 import io.netty.handler.codec.http.HttpHeaderNames;
-import org.junit.Ignore;
-import org.junit.Test;
+import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Test;
 import org.xbib.marc.Marc;
 import org.xbib.marc.json.MarcJsonWriter;
 import org.xbib.marc.xml.MarcContentHandler;
@@ -27,13 +27,13 @@ import java.util.logging.Logger;
 /**
  *
  */
-@Ignore
-public class BundeskunsthalleTest {
+@Disabled
+class BundeskunsthalleTest {
 
     private static final Logger logger = Logger.getLogger(BundeskunsthalleTest.class.getName());
 
     @Test
-    public void testListRecords() {
+    void testListRecords() {
         URL url = URL.create("https://www.bundeskunsthalle.de/cgi-bin/bib/oai-pmh");
         try (OAIClient oaiClient = new OAIClient(url)) {
             Client httpClient  = Client.builder()

oai-client/src/test/java/org/xbib/oai/client/DNBClientTest.java

@@ -1,8 +1,7 @@
 package org.xbib.oai.client;
 
 import io.netty.handler.codec.http.HttpHeaderNames;
-import org.junit.Ignore;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.xbib.net.URL;
 import org.xbib.netty.http.client.Client;
 import org.xbib.netty.http.client.Request;
@@ -28,12 +27,12 @@ import java.util.logging.Logger;
 /**
  *
  */
-public class DNBClientTest {
+class DNBClientTest {
 
     private static final Logger logger = Logger.getLogger(DNBClientTest.class.getName());
 
     @Test
-    public void testBibdat() {
+    void testBibdat() {
         URL url = URL.create("http://services.dnb.de/oai/repository");
         try (OAIClient oaiClient = new OAIClient(url)) {
             Client httpClient  = Client.builder()

oai-client/src/test/java/org/xbib/oai/client/DOAJClientTest.java

@@ -1,8 +1,8 @@
 package org.xbib.oai.client;
 
 import io.netty.handler.codec.http.HttpHeaderNames;
-import org.junit.Ignore;
-import org.junit.Test;
+import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Test;
 import org.xbib.net.URL;
 import org.xbib.netty.http.client.Client;
 import org.xbib.netty.http.client.Request;
@@ -25,13 +25,13 @@ import java.util.logging.Logger;
 /**
  *
  */
-public class DOAJClientTest {
+class DOAJClientTest {
 
     private static final Logger logger = Logger.getLogger(DOAJClientTest.class.getName());
 
     @Test
-    @Ignore // takes too long time
-    public void testListRecordsDOAJ() {
+    @Disabled // takes too long time
+    void testListRecordsDOAJ() {
         URL url = URL.create("https://doaj.org/oai");
         try (OAIClient oaiClient = new OAIClient(url)) {
             Client httpClient  = Client.builder()

oai-client/src/test/resources/logging.properties

@@ -1,5 +0,0 @@
-handlers = java.util.logging.ConsoleHandler
-.level = INFO
-java.util.logging.ConsoleHandler.level = INFO
-java.util.logging.ConsoleHandler.formatter = java.util.logging.SimpleFormatter
-java.util.logging.SimpleFormatter.format = %1$tFT%1$tT.%1$tL%1$tz [%4$-11s] [%3$s] %5$s %6$s%n

oai-server/build.gradle

@@ -1,4 +1,3 @@
 dependencies {
     compile project(':oai-common')
-
 }

oai-server/src/test/java/org/xbib/oai/server/SimpleServiceTest.java

@@ -1,6 +1,6 @@
 package org.xbib.oai.server;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.xbib.oai.server.identify.IdentifyServerRequest;
 import org.xbib.oai.server.identify.IdentifyServerResponse;
 
@@ -10,17 +10,18 @@ import javax.xml.stream.XMLOutputFactory;
 /**
  *
  */
-public class SimpleServiceTest {
+class SimpleServiceTest {
 
     @Test
-    public void testIdentifyService() throws Exception {
-        OAIServer service = OAIServiceFactory.getDefaultService();
+    void testIdentifyService() throws Exception {
         StringWriter sw = new StringWriter();
         XMLOutputFactory factory  = XMLOutputFactory.newInstance();
         IdentifyServerRequest request = new IdentifyServerRequest();
         IdentifyServerResponse response = new IdentifyServerResponse();
         response.setConsumer(factory.createXMLEventWriter(sw));
-        service.identify(request, response);
+        OAIServer service = OAIServiceFactory.getDefaultService();
+        if (service != null) {
+            service.identify(request, response);
+        }
     }
-
 }

oai-server/src/test/resources/log4j2.xml

@@ -1,13 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<configuration status="OFF">
-    <appenders>
-        <Console name="Console" target="SYSTEM_OUT">
-            <PatternLayout pattern="[%d{ABSOLUTE}][%-5p][%-25c][%t] %m%n"/>
-        </Console>
-    </appenders>
-    <Loggers>
-        <Root level="debug">
-            <AppenderRef ref="Console" />
-        </Root>
-    </Loggers>
-</configuration>