From 1893a2e47eb1078be461c323b99ab8ee240df7db Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vicen=C3=ADkov=C3=A1=20Jitka?= <Jitka.Vicenikova@partners.cz> Date: Fri, 5 May 2023 17:48:59 +0200 Subject: [PATCH] fix: tests for car service & visualization it --- core/src/main/java/cz/muni/pa165/service/CarService.java | 2 +- .../src/test/java/cz/muni/pa165/rest/VisualizationIT.java | 7 +++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/core/src/main/java/cz/muni/pa165/service/CarService.java b/core/src/main/java/cz/muni/pa165/service/CarService.java index 60052f1..085b116 100644 --- a/core/src/main/java/cz/muni/pa165/service/CarService.java +++ b/core/src/main/java/cz/muni/pa165/service/CarService.java @@ -113,7 +113,7 @@ public class CarService extends DomainService<Car> { try { sendPostRequest(savedCar); - } catch (RestClientException | IllegalArgumentException e) { + } catch (Exception e) { log.debug(String.format("The visualization module is not reachable on the URL: %s, exception %s", VISUALIZATION_MODULE_URL + VISUALIZATION_MODULE_URI, e)); } diff --git a/visualization/src/test/java/cz/muni/pa165/rest/VisualizationIT.java b/visualization/src/test/java/cz/muni/pa165/rest/VisualizationIT.java index c18b872..2d73c00 100644 --- a/visualization/src/test/java/cz/muni/pa165/rest/VisualizationIT.java +++ b/visualization/src/test/java/cz/muni/pa165/rest/VisualizationIT.java @@ -1,6 +1,5 @@ package cz.muni.pa165.rest; -import com.fasterxml.jackson.annotation.JsonTypeInfo; import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule; import cz.muni.pa165.generated.model.CarDto; @@ -14,6 +13,8 @@ import org.springframework.boot.test.context.SpringBootTest; import org.springframework.http.MediaType; import org.springframework.test.web.servlet.MockMvc; +import java.util.Map; + import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; @@ -36,7 +37,7 @@ class VisualizationIT { log.info("testGenerateCarPdf() running"); var carDto = new CarDto().id(1L); - var requestBody = mapper.writeValueAsString(carDto); + var requestBody = mapper.writeValueAsString(Map.of("car", carDto)); String response = mockMvc.perform( post("/visualization") @@ -50,6 +51,4 @@ class VisualizationIT { Assertions.assertNotNull(response); Assertions.assertEquals("application/pdf", response); } - - } -- GitLab