From 24d44398abc48a9967391a67f8edf4071dd32a0a Mon Sep 17 00:00:00 2001 From: DenisGabriela Date: Mon, 8 Oct 2018 14:38:27 +0300 Subject: [PATCH 1/5] ATS-35 : T-Engines: update Docker Transformers to Spring Boot 2.0.x GA - upgraded to Spring boot 2.0.5 - added dependency to spring-boot-starter-web (before it was brought in by thymeleaf 1.5.15) - removed try-catch since UnsupportedEncodingException is not thrown anymore by HierarchicalUriComponents.encodeUriComponent(..) - updated WebApplicationConfig to implement WebMvcConfigurer (WebMvcConfigurerAdapter is deprecated) - updated import of LocalServerPort - updated tests to use new method from InvocationOnMock --- .../LibreOfficeControllerTest.java | 6 ++--- alfresco-transformer-base/pom.xml | 4 +++ .../AbstractTransformerController.java | 25 +++++++------------ .../transformer/WebApplicationConfig.java | 7 +++--- .../transformer/AbstractHttpRequestTest.java | 2 +- .../AbstractTransformerControllerTest.java | 4 +-- pom.xml | 2 +- 7 files changed, 24 insertions(+), 26 deletions(-) diff --git a/alfresco-docker-libreoffice/src/test/java/org/alfresco/transformer/LibreOfficeControllerTest.java b/alfresco-docker-libreoffice/src/test/java/org/alfresco/transformer/LibreOfficeControllerTest.java index cab9ebea..b88bb8f5 100644 --- a/alfresco-docker-libreoffice/src/test/java/org/alfresco/transformer/LibreOfficeControllerTest.java +++ b/alfresco-docker-libreoffice/src/test/java/org/alfresco/transformer/LibreOfficeControllerTest.java @@ -84,14 +84,14 @@ public class LibreOfficeControllerTest extends AbstractTransformerControllerTest doAnswer((Answer) invocation -> { - File sourceFile = invocation.getArgumentAt(0, File.class); - File targetFile = invocation.getArgumentAt(1, File.class); + File sourceFile = invocation.getArgument(0); + File targetFile = invocation.getArgument(1); String actualTargetExtension = StringUtils.getFilenameExtension(targetFile.getAbsolutePath()); assertNotNull(sourceFile); assertNotNull(targetFile); - Long actualTimeout = invocation.getArgumentAt(2, Long.class); + Long actualTimeout = invocation.getArgument(2); assertNotNull(actualTimeout); if (expectedTimeout != null) { diff --git a/alfresco-transformer-base/pom.xml b/alfresco-transformer-base/pom.xml index 815a7f19..c6d750ed 100644 --- a/alfresco-transformer-base/pom.xml +++ b/alfresco-transformer-base/pom.xml @@ -17,6 +17,10 @@ org.springframework.boot spring-boot-starter-thymeleaf + + org.springframework.boot + spring-boot-starter-web + org.springframework.boot spring-boot-starter-test diff --git a/alfresco-transformer-base/src/main/java/org/alfresco/transformer/AbstractTransformerController.java b/alfresco-transformer-base/src/main/java/org/alfresco/transformer/AbstractTransformerController.java index 64e1222f..7813f98d 100644 --- a/alfresco-transformer-base/src/main/java/org/alfresco/transformer/AbstractTransformerController.java +++ b/alfresco-transformer-base/src/main/java/org/alfresco/transformer/AbstractTransformerController.java @@ -720,22 +720,15 @@ public abstract class AbstractTransformerController protected ResponseEntity createAttachment(String targetFilename, File targetFile, Long testDelay) { - try - { - Resource targetResource = load(targetFile); - targetFilename = UriUtils.encodePath(StringUtils.getFilename(targetFilename), "UTF-8"); - ResponseEntity body = ResponseEntity.ok().header(HttpHeaders.CONTENT_DISPOSITION, - "attachment; filename*= UTF-8''" + targetFilename).body(targetResource); - LogEntry.setTargetSize(targetFile.length()); - long time = LogEntry.setStatusCodeAndMessage(200, "Success"); - time += LogEntry.addDelay(testDelay); - getProbeTestTransformInternal().recordTransformTime(time); - return body; - } - catch (UnsupportedEncodingException e) - { - throw new TransformException(500, "Filename encoding error", e); - } + Resource targetResource = load(targetFile); + targetFilename = UriUtils.encodePath(StringUtils.getFilename(targetFilename), "UTF-8"); + ResponseEntity body = ResponseEntity.ok().header(HttpHeaders.CONTENT_DISPOSITION, + "attachment; filename*= UTF-8''" + targetFilename).body(targetResource); + LogEntry.setTargetSize(targetFile.length()); + long time = LogEntry.setStatusCodeAndMessage(200, "Success"); + time += LogEntry.addDelay(testDelay); + getProbeTestTransformInternal().recordTransformTime(time); + return body; } /** diff --git a/alfresco-transformer-base/src/main/java/org/alfresco/transformer/WebApplicationConfig.java b/alfresco-transformer-base/src/main/java/org/alfresco/transformer/WebApplicationConfig.java index 1e9ca806..df599f28 100644 --- a/alfresco-transformer-base/src/main/java/org/alfresco/transformer/WebApplicationConfig.java +++ b/alfresco-transformer-base/src/main/java/org/alfresco/transformer/WebApplicationConfig.java @@ -28,12 +28,13 @@ package org.alfresco.transformer; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; import org.springframework.web.client.RestTemplate; -import org.springframework.web.servlet.config.annotation.InterceptorRegistry; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; import org.alfresco.transform.client.model.TransformRequestValidator; +import org.springframework.web.servlet.config.annotation.InterceptorRegistry; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; @Configuration -public class WebApplicationConfig extends WebMvcConfigurerAdapter { +public class WebApplicationConfig implements WebMvcConfigurer +{ @Override public void addInterceptors(InterceptorRegistry registry) { diff --git a/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractHttpRequestTest.java b/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractHttpRequestTest.java index f45fcbe1..47e5343a 100644 --- a/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractHttpRequestTest.java +++ b/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractHttpRequestTest.java @@ -27,7 +27,7 @@ package org.alfresco.transformer; import org.junit.Test; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.context.embedded.LocalServerPort; +import org.springframework.boot.web.server.LocalServerPort; import org.springframework.boot.test.web.client.TestRestTemplate; import org.springframework.http.*; import org.springframework.util.LinkedMultiValueMap; diff --git a/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractTransformerControllerTest.java b/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractTransformerControllerTest.java index c58f893f..bef4562a 100644 --- a/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractTransformerControllerTest.java +++ b/alfresco-transformer-base/src/test/java/org/alfresco/transformer/AbstractTransformerControllerTest.java @@ -138,7 +138,7 @@ public abstract class AbstractTransformerControllerTest { public RuntimeExec.ExecutionResult answer(InvocationOnMock invocation) throws Throwable { - Map actualProperties = invocation.getArgumentAt(0, Map.class); + Map actualProperties = invocation.getArgument(0); assertEquals("There should be 3 properties", 3, actualProperties.size()); String actualOptions = actualProperties.get("options"); @@ -160,7 +160,7 @@ public abstract class AbstractTransformerControllerTest assertEquals("expectedOptions", expectedOptions, actualOptions); } - Long actualTimeout = invocation.getArgumentAt(1, Long.class); + Long actualTimeout = invocation.getArgument(1); assertNotNull(actualTimeout); if (expectedTimeout != null) { diff --git a/pom.xml b/pom.xml index 91375056..3d5fdab5 100644 --- a/pom.xml +++ b/pom.xml @@ -18,7 +18,7 @@ latest 2.0.8 3.5.37 - 1.5.15.RELEASE + 2.0.5.RELEASE 7.3 8.8 3.0.1.1 From ca0867c3b821e73f7c22a3e8d1ee8f93a25bc3e4 Mon Sep 17 00:00:00 2001 From: DenisGabriela Date: Mon, 8 Oct 2018 16:26:27 +0300 Subject: [PATCH 2/5] ATS-35 : T-Engines: update Docker Transformers to Spring Boot 2.0.x GA - override mockito & byte-buddy transitive dependencies (from Spring Boot 2.0.5 - starter parent) for Java 11 - mockito 2.15.0 to 2.22.0 - byte-buddy from 1.17.11 to 1.9.0 --- alfresco-transformer-base/pom.xml | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/alfresco-transformer-base/pom.xml b/alfresco-transformer-base/pom.xml index c6d750ed..a53d7a85 100644 --- a/alfresco-transformer-base/pom.xml +++ b/alfresco-transformer-base/pom.xml @@ -34,6 +34,32 @@ org.alfresco alfresco-transform-data-model + + + + + org.mockito + mockito-core + 2.22.0 + test + + + org.mockito + mockito-inline + 2.22.0 + test + + + net.bytebuddy + byte-buddy + 1.9.0 + + + net.bytebuddy + byte-buddy-agent + 1.9.0 + + From 27aa38209b5d8c8c18b17b8ac9f64a8ed2d0e392 Mon Sep 17 00:00:00 2001 From: DenisGabriela Date: Mon, 8 Oct 2018 17:00:54 +0300 Subject: [PATCH 3/5] ATS-35 : T-Engines: update Docker Transformers to Spring Boot 2.0.x GA - updated objenesis to 2.6 --- alfresco-transformer-base/pom.xml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/alfresco-transformer-base/pom.xml b/alfresco-transformer-base/pom.xml index a53d7a85..265fef49 100644 --- a/alfresco-transformer-base/pom.xml +++ b/alfresco-transformer-base/pom.xml @@ -59,6 +59,11 @@ byte-buddy-agent 1.9.0 + + org.objenesis + objenesis + 2.6 + From 824c07573633f7938f8bdf826e4bebe0f04bb229 Mon Sep 17 00:00:00 2001 From: DenisGabriela Date: Mon, 8 Oct 2018 17:20:32 +0300 Subject: [PATCH 4/5] ATS-35 : T-Engines: update Docker Transformers to Spring Boot 2.0.x GA - added mockito-core 2.22 to dependency-management --- alfresco-transformer-base/pom.xml | 6 ------ pom.xml | 9 +++++++++ 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/alfresco-transformer-base/pom.xml b/alfresco-transformer-base/pom.xml index 265fef49..f9ed2ee9 100644 --- a/alfresco-transformer-base/pom.xml +++ b/alfresco-transformer-base/pom.xml @@ -40,7 +40,6 @@ org.mockito mockito-core - 2.22.0 test @@ -59,11 +58,6 @@ byte-buddy-agent 1.9.0 - - org.objenesis - objenesis - 2.6 - diff --git a/pom.xml b/pom.xml index 3d5fdab5..d9a6e262 100644 --- a/pom.xml +++ b/pom.xml @@ -98,6 +98,15 @@ 3.2.6 + + + + + org.mockito + mockito-core + 2.22.0 + test + From 9704964a241b5ccb7d190c15f0e5500848522b08 Mon Sep 17 00:00:00 2001 From: DenisGabriela Date: Mon, 8 Oct 2018 17:38:53 +0300 Subject: [PATCH 5/5] ATS-35 : T-Engines: update Docker Transformers to Spring Boot 2.0.x GA - added test scope to buddy dependencies --- alfresco-transformer-base/pom.xml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/alfresco-transformer-base/pom.xml b/alfresco-transformer-base/pom.xml index f9ed2ee9..cd21190e 100644 --- a/alfresco-transformer-base/pom.xml +++ b/alfresco-transformer-base/pom.xml @@ -52,11 +52,13 @@ net.bytebuddy byte-buddy 1.9.0 + test net.bytebuddy byte-buddy-agent 1.9.0 + test