Merge pull request #352 from cougil/feature/migrate-tests-to-junit-5

Author: Brutus5000

build.gradle

@@ -148,8 +148,8 @@ sourceSets {
   inttest {
     java.srcDir 'src/inttest/java'
     resources.srcDir 'src/inttest/resources'
-    compileClasspath += main.output + test.output
-    runtimeClasspath += main.output + test.output
+    compileClasspath += sourceSets.test.compileClasspath
+    runtimeClasspath += sourceSets.test.runtimeClasspath
   }
 }
 
@@ -165,6 +165,9 @@ task inttest(type: Test) {
 
   setTestClassesDirs(files(sourceSets.inttest.output))
   classpath = sourceSets.inttest.runtimeClasspath
+
+  useJUnitPlatform()
+
   mustRunAfter test
 }
 
@@ -205,8 +208,7 @@ dependencyManagement {
 }
 
 test {
-  useJUnitPlatform {
-  }
+  useJUnitPlatform()
   testLogging {
     events("passed", "skipped", "failed")
   }
@@ -272,10 +274,10 @@ dependencies {
   testCompile("org.springframework.boot:spring-boot-starter-test")
   testCompile("org.springframework.restdocs:spring-restdocs-mockmvc")
   testCompile("org.springframework.security:spring-security-test")
-  testImplementation("junit:junit")
   testImplementation("org.junit.jupiter:junit-jupiter:${jUnit5Version}")
   testImplementation("org.junit.jupiter:junit-jupiter-params:${jUnit5Version}")
   testImplementation("org.mockito:mockito-junit-jupiter:${mockitoVersion}")
+  testImplementation("org.junit.jupiter:junit-jupiter-migrationsupport:${jUnit5Version}")
   testRuntimeOnly("org.junit.vintage:junit-vintage-engine:${jUnit5Version}")
   testCompile("com.h2database:h2:${h2Version}")
   testCompile("com.jayway.jsonpath:json-path:${jsonPath}")

src/inttest/java/com/faforever/api/AbstractIntegrationTest.java

@@ -19,8 +19,8 @@
 import com.github.jasminb.jsonapi.ResourceConverter;
 import com.github.jasminb.jsonapi.exceptions.DocumentSerializationException;
 import org.json.JSONObject;
-import org.junit.Before;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.internal.util.collections.Sets;
 import org.skyscreamer.jsonassert.JSONAssert;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -30,7 +30,7 @@
 import org.springframework.test.context.ActiveProfiles;
 import org.springframework.test.context.jdbc.Sql;
 import org.springframework.test.context.jdbc.Sql.ExecutionPhase;
-import org.springframework.test.context.junit4.SpringRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.MvcResult;
 import org.springframework.test.web.servlet.request.RequestPostProcessor;
@@ -44,7 +44,7 @@
 
 import static org.springframework.security.test.web.servlet.setup.SecurityMockMvcConfigurers.springSecurity;
 
-@RunWith(SpringRunner.class)
+@ExtendWith(SpringExtension.class)
 @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
 @ActiveProfiles(ApplicationProfile.INTEGRATION_TEST)
 @Import(OAuthHelper.class)
@@ -67,7 +67,7 @@ public abstract class AbstractIntegrationTest {
   protected ObjectMapper objectMapper;
   protected ResourceConverter resourceConverter;
 
-  @Before
+  @BeforeEach
   public void setUp() {
     this.mockMvc = MockMvcBuilders
       .webAppContextSetup(this.context)

src/inttest/java/com/faforever/api/avatar/AvatarControllerTest.java

@@ -6,8 +6,8 @@
 import com.faforever.api.security.AuditService;
 import com.faforever.api.security.OAuthScope;
 import com.faforever.api.utils.FileHandlingHelper;
-import org.junit.After;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.mock.mockito.SpyBean;
 import org.springframework.http.MediaType;
@@ -136,7 +136,7 @@ public void cannotDeleteWithoutScope() throws Exception {
     verify(auditServiceSpy, times(0)).logMessage(any());
   }
 
-  @After
+  @AfterEach
   public void tearDown() throws Exception {
     Files.deleteIfExists(Paths.get("build/cache/avatars/avatar1.png"));
     Files.deleteIfExists(Paths.get("build/cache/avatars/avatar3.png"));

src/inttest/java/com/faforever/api/clan/ClanControllerTest.java

@@ -6,7 +6,7 @@
 import com.faforever.api.error.ErrorCode;
 import com.faforever.api.player.PlayerRepository;
 import com.faforever.api.security.FafUserDetails;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpHeaders;
 import org.springframework.security.core.context.SecurityContextHolder;

src/inttest/java/com/faforever/api/data/AvatarAssignmentElideTest.java

@@ -6,7 +6,7 @@
 import com.faforever.commons.api.dto.Avatar;
 import com.faforever.commons.api.dto.AvatarAssignment;
 import com.faforever.commons.api.dto.Player;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.http.HttpHeaders;
 import org.springframework.security.test.context.support.WithUserDetails;
 import org.springframework.test.context.jdbc.Sql;

src/inttest/java/com/faforever/api/data/BanTest.java

@@ -7,7 +7,8 @@
 import com.faforever.commons.api.dto.BanLevel;
 import com.faforever.commons.api.dto.ModerationReport;
 import com.faforever.commons.api.dto.Player;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpHeaders;
 import org.springframework.test.context.jdbc.Sql;
 import org.springframework.test.context.jdbc.Sql.ExecutionPhase;

src/inttest/java/com/faforever/api/data/ClanElideTest.java

@@ -8,7 +8,7 @@
 import com.faforever.api.player.PlayerRepository;
 import lombok.SneakyThrows;
 import org.json.JSONObject;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpHeaders;
 import org.springframework.security.test.context.support.WithUserDetails;

src/inttest/java/com/faforever/api/data/DomainBlacklistTest.java

@@ -3,7 +3,7 @@
 import com.faforever.api.AbstractIntegrationTest;
 import com.faforever.api.data.domain.GroupPermission;
 import com.faforever.api.security.OAuthScope;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.http.HttpHeaders;
 import org.springframework.security.test.context.support.WithUserDetails;
 import org.springframework.test.context.jdbc.Sql;

src/inttest/java/com/faforever/api/data/Ladder1v1MapTest.java

@@ -3,7 +3,7 @@
 import com.faforever.api.AbstractIntegrationTest;
 import com.faforever.api.data.domain.GroupPermission;
 import com.faforever.api.security.OAuthScope;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.http.HttpHeaders;
 import org.springframework.test.context.jdbc.Sql;
 import org.springframework.test.context.jdbc.Sql.ExecutionPhase;

src/inttest/java/com/faforever/api/data/MapVersionElideTest.java

@@ -3,7 +3,7 @@
 import com.faforever.api.AbstractIntegrationTest;
 import com.faforever.api.data.domain.GroupPermission;
 import com.faforever.api.security.OAuthScope;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.http.HttpHeaders;
 import org.springframework.security.test.context.support.WithUserDetails;
 import org.springframework.test.context.jdbc.Sql;