From 2637f65b90cf90bb45093e2e902e4ce716abae67 Mon Sep 17 00:00:00 2001 From: Antoine Rey <antoine.rey@gmail.com> Date: Wed, 17 Aug 2016 18:31:38 +0200 Subject: [PATCH] Upgrade to Spring Boot 1.4.0 --- pom.xml | 26 +++---------------- .../petclinic/PetClinicApplication.java | 2 +- .../petclinic/config/DandelionConfig.java | 5 ++-- .../ClinicServiceSpringDataJpaTests.java | 6 ++--- .../petclinic/web/CrashControllerTests.java | 5 ++-- .../petclinic/web/OwnerControllerTests.java | 4 +-- .../petclinic/web/PetControllerTests.java | 16 +++++------- .../petclinic/web/VetControllerTests.java | 5 ++-- .../petclinic/web/VisitControllerTests.java | 4 +-- 9 files changed, 22 insertions(+), 51 deletions(-) diff --git a/pom.xml b/pom.xml index de84dae..06741b4 100644 --- a/pom.xml +++ b/pom.xml @@ -4,11 +4,11 @@ <modelVersion>4.0.0</modelVersion> <groupId>org.springframework.samples</groupId> <artifactId>springboot-petclinic</artifactId> - <version>1.3.5-SNAPSHOT</version> + <version>1.4.0-SNAPSHOT</version> <parent> <groupId>org.springframework.boot</groupId> <artifactId>spring-boot-starter-parent</artifactId> - <version>1.3.5.RELEASE</version> + <version>1.4.0.RELEASE</version> </parent> <name>petclinic</name> <packaging>war</packaging> @@ -20,15 +20,10 @@ <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding> - <!-- Test --> - <assertj.version>2.2.0</assertj.version> - <hamcrest.version>1.3</hamcrest.version> - <!-- Dates --> <jodatime-hibernate.version>1.3</jodatime-hibernate.version> <jodatime-jsptags.version>1.1.1</jodatime-jsptags.version> - <jadira-usertype-core.version>3.2.0.GA</jadira-usertype-core.version> - + <jadira-usertype-core.version>5.0.0.GA</jadira-usertype-core.version> <!-- Web dependencies --> <webjars-bootstrap.version>3.3.6</webjars-bootstrap.version> @@ -80,13 +75,6 @@ </dependency> - <!-- JSon --> - <dependency> - <groupId>com.jayway.jsonpath</groupId> - <artifactId>json-path</artifactId> - <scope>test</scope> - </dependency> - <!-- used for EhCacheCacheManager --> <dependency> <groupId>org.aspectj</groupId> @@ -145,14 +133,6 @@ </exclusions> </dependency> - <!-- Test Artifacts --> - <dependency> - <groupId>org.assertj</groupId> - <artifactId>assertj-core</artifactId> - <version>${assertj.version}</version> - <scope>test</scope> - </dependency> - <!-- Dandelion --> <dependency> <groupId>com.github.dandelion</groupId> diff --git a/src/main/java/org/springframework/samples/petclinic/PetClinicApplication.java b/src/main/java/org/springframework/samples/petclinic/PetClinicApplication.java index d1b31fe..456b061 100644 --- a/src/main/java/org/springframework/samples/petclinic/PetClinicApplication.java +++ b/src/main/java/org/springframework/samples/petclinic/PetClinicApplication.java @@ -19,7 +19,7 @@ package org.springframework.samples.petclinic; import org.springframework.boot.SpringApplication; import org.springframework.boot.autoconfigure.SpringBootApplication; import org.springframework.boot.builder.SpringApplicationBuilder; -import org.springframework.boot.context.web.SpringBootServletInitializer; +import org.springframework.boot.web.support.SpringBootServletInitializer; /** * PetClinic Spring Boot Application. diff --git a/src/main/java/org/springframework/samples/petclinic/config/DandelionConfig.java b/src/main/java/org/springframework/samples/petclinic/config/DandelionConfig.java index d6ec44d..b13737f 100644 --- a/src/main/java/org/springframework/samples/petclinic/config/DandelionConfig.java +++ b/src/main/java/org/springframework/samples/petclinic/config/DandelionConfig.java @@ -20,12 +20,11 @@ import com.github.dandelion.core.web.DandelionFilter; import com.github.dandelion.core.web.DandelionServlet; import com.github.dandelion.datatables.core.web.filter.DatatablesFilter; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.context.embedded.FilterRegistrationBean; -import org.springframework.boot.context.embedded.ServletRegistrationBean; +import org.springframework.boot.web.servlet.FilterRegistrationBean; +import org.springframework.boot.web.servlet.ServletRegistrationBean; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; import org.springframework.context.annotation.Description; -import org.springframework.context.annotation.Profile; import org.springframework.core.env.Environment; import javax.annotation.PostConstruct; diff --git a/src/test/java/org/springframework/samples/petclinic/service/ClinicServiceSpringDataJpaTests.java b/src/test/java/org/springframework/samples/petclinic/service/ClinicServiceSpringDataJpaTests.java index d0781b8..9dc4011 100644 --- a/src/test/java/org/springframework/samples/petclinic/service/ClinicServiceSpringDataJpaTests.java +++ b/src/test/java/org/springframework/samples/petclinic/service/ClinicServiceSpringDataJpaTests.java @@ -1,10 +1,8 @@ package org.springframework.samples.petclinic.service; import org.junit.runner.RunWith; -import org.springframework.boot.test.SpringApplicationConfiguration; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.samples.petclinic.PetClinicApplication; -import org.springframework.test.context.ActiveProfiles; -import org.springframework.test.context.ContextConfiguration; import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; /** @@ -15,7 +13,7 @@ import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; */ @RunWith(SpringJUnit4ClassRunner.class) -@SpringApplicationConfiguration(classes = PetClinicApplication.class) +@SpringBootTest(classes = PetClinicApplication.class) public class ClinicServiceSpringDataJpaTests extends AbstractClinicServiceTests { } diff --git a/src/test/java/org/springframework/samples/petclinic/web/CrashControllerTests.java b/src/test/java/org/springframework/samples/petclinic/web/CrashControllerTests.java index 41d233e..30a0a64 100644 --- a/src/test/java/org/springframework/samples/petclinic/web/CrashControllerTests.java +++ b/src/test/java/org/springframework/samples/petclinic/web/CrashControllerTests.java @@ -5,13 +5,12 @@ import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.SpringApplicationConfiguration; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.samples.petclinic.PetClinicApplication; import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import org.springframework.test.context.web.WebAppConfiguration; import org.springframework.test.web.servlet.MockMvc; import org.springframework.test.web.servlet.setup.MockMvcBuilders; -import org.springframework.web.servlet.handler.SimpleMappingExceptionResolver; import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; @@ -22,7 +21,7 @@ import static org.springframework.test.web.servlet.result.MockMvcResultMatchers. * @author Colin But */ @RunWith(SpringJUnit4ClassRunner.class) -@SpringApplicationConfiguration(classes = PetClinicApplication.class) +@SpringBootTest(classes = PetClinicApplication.class) @WebAppConfiguration // Waiting https://github.com/spring-projects/spring-boot/issues/5574 @Ignore diff --git a/src/test/java/org/springframework/samples/petclinic/web/OwnerControllerTests.java b/src/test/java/org/springframework/samples/petclinic/web/OwnerControllerTests.java index c6e0aed..36f7d80 100644 --- a/src/test/java/org/springframework/samples/petclinic/web/OwnerControllerTests.java +++ b/src/test/java/org/springframework/samples/petclinic/web/OwnerControllerTests.java @@ -4,7 +4,7 @@ import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.SpringApplicationConfiguration; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.samples.petclinic.PetClinicApplication; import org.springframework.test.context.ActiveProfiles; import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; @@ -24,7 +24,7 @@ import static org.springframework.test.web.servlet.result.MockMvcResultMatchers. * @author Colin But */ @RunWith(SpringJUnit4ClassRunner.class) -@SpringApplicationConfiguration(classes = PetClinicApplication.class) +@SpringBootTest(classes = PetClinicApplication.class) @WebAppConfiguration @ActiveProfiles("test") public class OwnerControllerTests { diff --git a/src/test/java/org/springframework/samples/petclinic/web/PetControllerTests.java b/src/test/java/org/springframework/samples/petclinic/web/PetControllerTests.java index fb327fe..060ef0e 100644 --- a/src/test/java/org/springframework/samples/petclinic/web/PetControllerTests.java +++ b/src/test/java/org/springframework/samples/petclinic/web/PetControllerTests.java @@ -1,33 +1,29 @@ package org.springframework.samples.petclinic.web; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.model; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.view; - import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.SpringApplicationConfiguration; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.format.support.DefaultFormattingConversionService; -import org.springframework.format.support.FormattingConversionServiceFactoryBean; import org.springframework.samples.petclinic.PetClinicApplication; import org.springframework.test.context.ActiveProfiles; -import org.springframework.test.context.ContextConfiguration; import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import org.springframework.test.context.web.WebAppConfiguration; import org.springframework.test.web.servlet.MockMvc; import org.springframework.test.web.servlet.setup.MockMvcBuilders; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; + /** * Test class for the {@link PetController} * * @author Colin But */ @RunWith(SpringJUnit4ClassRunner.class) -@SpringApplicationConfiguration(classes = PetClinicApplication.class) +@SpringBootTest(classes = PetClinicApplication.class) @WebAppConfiguration @ActiveProfiles("test") public class PetControllerTests { diff --git a/src/test/java/org/springframework/samples/petclinic/web/VetControllerTests.java b/src/test/java/org/springframework/samples/petclinic/web/VetControllerTests.java index 6f77223..e03c948 100644 --- a/src/test/java/org/springframework/samples/petclinic/web/VetControllerTests.java +++ b/src/test/java/org/springframework/samples/petclinic/web/VetControllerTests.java @@ -1,11 +1,10 @@ package org.springframework.samples.petclinic.web; -import org.hamcrest.xml.HasXPath; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.SpringApplicationConfiguration; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.http.MediaType; import org.springframework.samples.petclinic.PetClinicApplication; import org.springframework.test.context.ActiveProfiles; @@ -23,7 +22,7 @@ import static org.springframework.test.web.servlet.result.MockMvcResultMatchers. * Test class for the {@link VetController} */ @RunWith(SpringJUnit4ClassRunner.class) -@SpringApplicationConfiguration(classes = PetClinicApplication.class) +@SpringBootTest(classes = PetClinicApplication.class) @WebAppConfiguration @ActiveProfiles("test") public class VetControllerTests { diff --git a/src/test/java/org/springframework/samples/petclinic/web/VisitControllerTests.java b/src/test/java/org/springframework/samples/petclinic/web/VisitControllerTests.java index b5e28cd..dbb5069 100644 --- a/src/test/java/org/springframework/samples/petclinic/web/VisitControllerTests.java +++ b/src/test/java/org/springframework/samples/petclinic/web/VisitControllerTests.java @@ -4,7 +4,7 @@ import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.SpringApplicationConfiguration; +import org.springframework.boot.test.context.SpringBootTest; import org.springframework.samples.petclinic.PetClinicApplication; import org.springframework.test.context.ActiveProfiles; import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; @@ -22,7 +22,7 @@ import static org.springframework.test.web.servlet.result.MockMvcResultMatchers. * @author Colin But */ @RunWith(SpringJUnit4ClassRunner.class) -@SpringApplicationConfiguration(classes = PetClinicApplication.class) +@SpringBootTest(classes = PetClinicApplication.class) @WebAppConfiguration @ActiveProfiles("test") public class VisitControllerTests { -- GitLab