build: junit 5

- migrate tests to use junit 5 platform

Author: edgar-espina-wpp

.travis.yml

@@ -50,8 +50,8 @@
     </dependency>
 
     <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-engine</artifactId>
       <scope>test</scope>
     </dependency>
 

handlebars-caffeine/pom.xml

@@ -5,8 +5,8 @@
  */
 package com.github.jknack.handlebars.cache;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.fail;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.fail;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
@@ -15,7 +15,7 @@
 import java.io.IOException;
 import java.util.function.Function;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.mockito.ArgumentCaptor;
 
 import com.github.benmanes.caffeine.cache.Cache;

handlebars-caffeine/src/test/java/com/github/jknack/handlebars/cache/CaffeineTemplateCacheTest.java

@@ -51,8 +51,8 @@
     </dependency>
 
     <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-engine</artifactId>
       <scope>test</scope>
     </dependency>
 

handlebars-guava-cache/pom.xml

@@ -5,7 +5,7 @@
  */
 package com.github.jknack.handlebars.cache;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.ArgumentMatchers.same;
@@ -17,7 +17,8 @@
 import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutionException;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
 
 import com.github.jknack.handlebars.HandlebarsException;
 import com.github.jknack.handlebars.Parser;
@@ -108,50 +109,63 @@ public void clear() throws IOException {
   }
 
   @SuppressWarnings("unchecked")
-  @Test(expected = IllegalStateException.class)
+  @Test
   public void executionExceptionWithRuntimeException() throws IOException, ExecutionException {
-    TemplateSource source = mock(TemplateSource.class);
+    Assertions.assertThrows(
+        IllegalStateException.class,
+        () -> {
+          TemplateSource source = mock(TemplateSource.class);
 
-    Parser parser = mock(Parser.class);
+          Parser parser = mock(Parser.class);
 
-    Cache<TemplateSource, Template> cache = mock(Cache.class);
-    when(cache.get(eq(source), any(Callable.class)))
-        .thenThrow(new ExecutionException(new IllegalStateException()));
+          Cache<TemplateSource, Template> cache = mock(Cache.class);
+          when(cache.get(eq(source), any(Callable.class)))
+              .thenThrow(new ExecutionException(new IllegalStateException()));
 
-    new GuavaTemplateCache(cache).get(source, parser);
+          new GuavaTemplateCache(cache).get(source, parser);
 
-    verify(cache).get(eq(source), any(Callable.class));
+          verify(cache).get(eq(source), any(Callable.class));
+        });
   }
 
   @SuppressWarnings("unchecked")
-  @Test(expected = Error.class)
+  @Test
   public void executionExceptionWithError() throws IOException, ExecutionException {
-    TemplateSource source = mock(TemplateSource.class);
+    Assertions.assertThrows(
+        Error.class,
+        () -> {
+          TemplateSource source = mock(TemplateSource.class);
 
-    Parser parser = mock(Parser.class);
+          Parser parser = mock(Parser.class);
 
-    Cache<TemplateSource, Template> cache = mock(Cache.class);
-    when(cache.get(eq(source), any(Callable.class))).thenThrow(new ExecutionException(new Error()));
+          Cache<TemplateSource, Template> cache = mock(Cache.class);
+          when(cache.get(eq(source), any(Callable.class)))
+              .thenThrow(new ExecutionException(new Error()));
 
-    new GuavaTemplateCache(cache).get(source, parser);
+          new GuavaTemplateCache(cache).get(source, parser);
 
-    verify(cache).get(eq(source), any(Callable.class));
+          verify(cache).get(eq(source), any(Callable.class));
+        });
   }
 
   @SuppressWarnings("unchecked")
-  @Test(expected = HandlebarsException.class)
+  @Test
   public void executionExceptionWithCheckedException() throws IOException, ExecutionException {
-    TemplateSource source = mock(TemplateSource.class);
+    Assertions.assertThrows(
+        HandlebarsException.class,
+        () -> {
+          TemplateSource source = mock(TemplateSource.class);
 
-    Parser parser = mock(Parser.class);
+          Parser parser = mock(Parser.class);
 
-    Cache<TemplateSource, Template> cache = mock(Cache.class);
-    when(cache.get(eq(source), any(Callable.class)))
-        .thenThrow(new ExecutionException(new IOException()));
+          Cache<TemplateSource, Template> cache = mock(Cache.class);
+          when(cache.get(eq(source), any(Callable.class)))
+              .thenThrow(new ExecutionException(new IOException()));
 
-    new GuavaTemplateCache(cache).get(source, parser);
+          new GuavaTemplateCache(cache).get(source, parser);
 
-    verify(cache).get(eq(source), any(Callable.class));
+          verify(cache).get(eq(source), any(Callable.class));
+        });
   }
 
   private TemplateSource source(final String filename) throws IOException {

handlebars-guava-cache/src/test/java/com/github/jknack/handlebars/cache/GuavaTemplateCacheTest.java

@@ -7,7 +7,7 @@
 
 import java.io.IOException;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import com.github.jknack.handlebars.AbstractTest;
 import com.google.common.collect.ImmutableList;

handlebars-guava-cache/src/test/java/com/github/jknack/handlebars/i371/Issue371.java

@@ -6,12 +6,12 @@
 package com.github.jknack.handlebars.io;
 
 import static java.lang.System.out;
-import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 import java.io.File;
 import java.util.concurrent.TimeUnit;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import com.google.common.base.Stopwatch;
 

handlebars-guava-cache/src/test/java/com/github/jknack/handlebars/io/GuavaCachedTemplateLoaderTest.java

@@ -57,8 +57,8 @@
     </dependency>
 
     <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-engine</artifactId>
       <scope>test</scope>
     </dependency>
 

handlebars-helpers/pom.xml

@@ -5,11 +5,11 @@
  */
 package com.github.jknack.handlebars.helper;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.io.IOException;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import com.github.jknack.handlebars.AbstractTest;
 import com.github.jknack.handlebars.Context;

handlebars-helpers/src/test/java/com/github/jknack/handlebars/helper/AssignHelperTest.java

@@ -7,7 +7,7 @@
 
 import java.io.IOException;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import com.github.jknack.handlebars.AbstractTest;
 import com.github.jknack.handlebars.Handlebars;