You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicecomb.apache.org by li...@apache.org on 2022/07/17 09:06:13 UTC
[servicecomb-java-chassis] branch master updated: [SCB-2475] migrate common-rest module to junit5 part 3 (#3193)
This is an automated email from the ASF dual-hosted git repository.
liubao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git
The following commit(s) were added to refs/heads/master by this push:
new 37df78fec [SCB-2475] migrate common-rest module to junit5 part 3 (#3193)
37df78fec is described below
commit 37df78fec71f58442cc05051c7cbdc0c684fb2b3
Author: TingTing Wang <19...@qq.com>
AuthorDate: Sun Jul 17 17:06:08 2022 +0800
[SCB-2475] migrate common-rest module to junit5 part 3 (#3193)
---
.../rest/RestProducerInvocationCreatorTest.java | 36 +++++++++++-----------
.../common/rest/TestAbstractRestInvocation.java | 14 ++++-----
.../common/rest/TestRestProducerInvocation.java | 16 +++++-----
.../common/rest/VertxRestInvocationTest.java | 2 +-
.../common/rest/codec/fix/TestDoSFix.java | 10 +++---
.../common/rest/codec/param/TestBodyProcessor.java | 14 ++++-----
.../rest/codec/param/TestCookieProcessor.java | 13 ++++----
.../common/rest/codec/param/TestFormProcessor.java | 9 +++---
.../rest/codec/param/TestHeaderProcessor.java | 17 +++-------
.../common/rest/codec/param/TestPathProcessor.java | 6 ++--
.../rest/codec/param/TestQueryProcessor.java | 5 ++-
.../codec/param/TestQueryProcessorCreator.java | 3 +-
.../codec/param/TestRestClientRequestImpl.java | 2 +-
.../codec/produce/TestProduceJsonProcessor.java | 8 ++---
.../produce/TestProduceTextPlainProcessor.java | 4 +--
.../rest/codec/query/QueryCodecTestBase.java | 2 +-
.../common/rest/definition/TestPath.java | 10 +++---
.../rest/definition/TestRestOperationMeta.java | 10 +++---
.../definition/path/PathVarParamWriterTest.java | 2 +-
...HttpServerFilterBeforeSendResponseExecutor.java | 4 +--
.../filter/inner/RestServerCodecFilterTest.java | 36 +++++++++++-----------
.../filter/inner/TestServerRestArgsFilter.java | 12 ++++----
.../common/rest/locator/TestMicroservicePaths.java | 10 +++---
.../TestClassPathStaticResourceHandler.java | 6 ++--
24 files changed, 120 insertions(+), 131 deletions(-)
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/RestProducerInvocationCreatorTest.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/RestProducerInvocationCreatorTest.java
index ed6d807ec..94604e256 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/RestProducerInvocationCreatorTest.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/RestProducerInvocationCreatorTest.java
@@ -43,13 +43,13 @@ import org.apache.servicecomb.foundation.vertx.http.HttpServletRequestEx;
import org.apache.servicecomb.foundation.vertx.http.HttpServletResponseEx;
import org.apache.servicecomb.swagger.invocation.exception.CommonExceptionData;
import org.apache.servicecomb.swagger.invocation.exception.InvocationException;
-import org.junit.AfterClass;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.Test;
import io.vertx.core.json.Json;
import io.vertx.ext.web.RoutingContext;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.mockito.MockedStatic;
import org.mockito.Mockito;
@@ -57,33 +57,33 @@ import java.util.ArrayList;
public class RestProducerInvocationCreatorTest {
- RoutingContext routingContext = Mockito.mock(RoutingContext.class);
+ final RoutingContext routingContext = Mockito.mock(RoutingContext.class);
- MicroserviceMeta microserviceMeta = Mockito.mock(MicroserviceMeta.class);
+ final MicroserviceMeta microserviceMeta = Mockito.mock(MicroserviceMeta.class);
- ServicePathManager servicePathManager = Mockito.mock(ServicePathManager.class);
+ final ServicePathManager servicePathManager = Mockito.mock(ServicePathManager.class);
- RestOperationMeta restOperationMeta = Mockito.mock(RestOperationMeta.class);
+ final RestOperationMeta restOperationMeta = Mockito.mock(RestOperationMeta.class);
- Endpoint endpoint = Mockito.mock(Endpoint.class);
+ final Endpoint endpoint = Mockito.mock(Endpoint.class);
- HttpServletRequestEx requestEx = Mockito.mock(HttpServletRequestEx.class);
+ final HttpServletRequestEx requestEx = Mockito.mock(HttpServletRequestEx.class);
- HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
+ final HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
- OperationLocator locator = Mockito.mock(OperationLocator.class);
+ final OperationLocator locator = Mockito.mock(OperationLocator.class);
- InvocationRuntimeType invocationRuntimeType = Mockito.mock(InvocationRuntimeType.class);
+ final InvocationRuntimeType invocationRuntimeType = Mockito.mock(InvocationRuntimeType.class);
- OperationMeta operationMeta = Mockito.mock(OperationMeta.class);
+ final OperationMeta operationMeta = Mockito.mock(OperationMeta.class);
- SchemaMeta schemaMeta = Mockito.mock(SchemaMeta.class);
+ final SchemaMeta schemaMeta = Mockito.mock(SchemaMeta.class);
RestProducerInvocationCreator creator;
static SCBEngine engine;
- @BeforeClass
+ @BeforeAll
public static void beforeClass() {
ArchaiusUtils.resetConfig();
ConfigUtil.installDynamicConfig();
@@ -92,13 +92,13 @@ public class RestProducerInvocationCreatorTest {
engine.setStatus(SCBStatus.UP);
}
- @AfterClass
+ @AfterAll
public static void afterClass() {
engine.destroy();
ArchaiusUtils.resetConfig();
}
- @Before
+ @BeforeEach
public void setUp() {
creator = new RestVertxProducerInvocationCreator(routingContext, microserviceMeta, endpoint,
requestEx, responseEx);
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestAbstractRestInvocation.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestAbstractRestInvocation.java
index 56625c135..0b7734dd5 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestAbstractRestInvocation.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestAbstractRestInvocation.java
@@ -73,15 +73,15 @@ import org.apache.servicecomb.swagger.invocation.exception.CommonExceptionData;
import org.apache.servicecomb.swagger.invocation.exception.InvocationException;
import org.hamcrest.MatcherAssert;
import org.hamcrest.Matchers;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Assertions;
import com.google.common.eventbus.Subscribe;
import io.vertx.core.MultiMap;
import io.vertx.core.buffer.Buffer;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.mockito.MockedStatic;
import org.mockito.Mockito;
@@ -89,11 +89,11 @@ public class TestAbstractRestInvocation {
HttpServletRequestEx requestEx = Mockito.mock(HttpServletRequestEx.class);
- HttpServletResponse servletResponse = Mockito.mock(HttpServletResponse.class);
+ final HttpServletResponse servletResponse = Mockito.mock(HttpServletResponse.class);
HttpServletResponseEx responseEx;
- ReferenceConfig endpoint = Mockito.mock(ReferenceConfig.class);
+ final ReferenceConfig endpoint = Mockito.mock(ReferenceConfig.class);
Map<String, Object> arguments = new HashMap<>();
@@ -126,7 +126,7 @@ public class TestAbstractRestInvocation {
AbstractRestInvocation restInvocation = new AbstractRestInvocationForTest();
- @Before
+ @BeforeEach
public void setup() {
ConfigUtil.installDynamicConfig();
scbEngine = SCBBootstrap.createSCBEngineForTest()
@@ -147,7 +147,7 @@ public class TestAbstractRestInvocation {
restInvocation.setHttpServerFilters(httpServerFilters);
}
- @After
+ @AfterEach
public void teardown() {
ArchaiusUtils.resetConfig();
scbEngine.destroy();
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestRestProducerInvocation.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestRestProducerInvocation.java
index 75f0db77a..fee6a0e1f 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestRestProducerInvocation.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/TestRestProducerInvocation.java
@@ -40,20 +40,20 @@ import org.apache.servicecomb.foundation.vertx.http.HttpServletRequestEx;
import org.apache.servicecomb.foundation.vertx.http.HttpServletResponseEx;
import org.apache.servicecomb.registry.api.registry.Microservice;
import org.apache.servicecomb.swagger.invocation.exception.InvocationException;
-import org.junit.AfterClass;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
import org.mockito.MockedStatic;
import org.mockito.Mockito;
public class TestRestProducerInvocation {
- Transport transport = Mockito.mock(Transport.class);
+ final Transport transport = Mockito.mock(Transport.class);
HttpServletRequestEx requestEx = Mockito.mock(HttpServletRequestEx.class);
- HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
+ final HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
RestProducerInvocation restProducerInvocation;
@@ -61,7 +61,7 @@ public class TestRestProducerInvocation {
boolean scheduleInvocation;
- static List<HttpServerFilter> httpServerFilters = SPIServiceUtils.getSortedService(HttpServerFilter.class);
+ static final List<HttpServerFilter> httpServerFilters = SPIServiceUtils.getSortedService(HttpServerFilter.class);
static SCBEngine scbEngine;
@@ -71,7 +71,7 @@ public class TestRestProducerInvocation {
static MicroserviceMeta microserviceMeta;
- @BeforeClass
+ @BeforeAll
public static void classSetup() {
ConfigUtil.installDynamicConfig();
scbEngine = SCBBootstrap.createSCBEngineForTest()
@@ -82,7 +82,7 @@ public class TestRestProducerInvocation {
microserviceMeta = operationMeta.getMicroserviceMeta();
}
- @AfterClass
+ @AfterAll
public static void classTeardown() {
scbEngine.destroy();
ArchaiusUtils.resetConfig();
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/VertxRestInvocationTest.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/VertxRestInvocationTest.java
index 421624128..a004e746a 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/VertxRestInvocationTest.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/VertxRestInvocationTest.java
@@ -19,7 +19,7 @@ package org.apache.servicecomb.common.rest;
import org.apache.servicecomb.core.Invocation;
import org.apache.servicecomb.foundation.vertx.http.VertxServerRequestToHttpServletRequest;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
import io.vertx.ext.web.RoutingContext;
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/fix/TestDoSFix.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/fix/TestDoSFix.java
index 338b5f7f2..ae8abc8a6 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/fix/TestDoSFix.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/fix/TestDoSFix.java
@@ -33,15 +33,15 @@ import com.google.common.base.Strings;
import org.junit.jupiter.api.Test;
public class TestDoSFix {
- static ObjectMapper mapper = new RestObjectMapper();
+ static final ObjectMapper mapper = new RestObjectMapper();
- static String invalidNum = Strings.repeat("9", 100_0000);
+ static final String invalidNum = Strings.repeat("9", 100_0000);
- static String invalidStr = "\"" + invalidNum + "\"";
+ static final String invalidStr = "\"" + invalidNum + "\"";
- static String invalidArrNum = "[" + invalidNum + "]";
+ static final String invalidArrNum = "[" + invalidNum + "]";
- static String invalidArrStr = "[\"" + invalidNum + "\"]";
+ static final String invalidArrStr = "[\"" + invalidNum + "\"]";
public static class Model {
public Color color;
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestBodyProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestBodyProcessor.java
index 04a8b8697..0fecdf86e 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestBodyProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestBodyProcessor.java
@@ -34,8 +34,6 @@ import org.apache.servicecomb.common.rest.codec.param.BodyProcessorCreator.BodyP
import org.apache.servicecomb.common.rest.codec.param.BodyProcessorCreator.RawJsonBodyProcessor;
import org.apache.servicecomb.foundation.vertx.stream.BufferInputStream;
import org.junit.jupiter.api.Assertions;
-import org.junit.Before;
-import org.junit.Test;
import com.fasterxml.jackson.databind.type.TypeFactory;
@@ -44,22 +42,24 @@ import io.netty.buffer.Unpooled;
import io.vertx.core.MultiMap;
import io.vertx.core.buffer.Buffer;
import io.vertx.core.http.impl.headers.HeadersMultiMap;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestBodyProcessor {
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
+ final HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
MultiMap headers;
- RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
+ final RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
ParamValueProcessor processor;
- ByteBuf inputBodyByteBuf = Unpooled.buffer();
+ final ByteBuf inputBodyByteBuf = Unpooled.buffer();
- BufferInputStream inputStream = new BufferInputStream(inputBodyByteBuf);
+ final BufferInputStream inputStream = new BufferInputStream(inputBodyByteBuf);
Buffer outputBodyBuffer;
@@ -92,7 +92,7 @@ public class TestBodyProcessor {
initInputStream();
}
- @Before
+ @BeforeEach
public void before() {
headers = new HeadersMultiMap();
value = "value";
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestCookieProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestCookieProcessor.java
index 87c613858..ffe1e36f0 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestCookieProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestCookieProcessor.java
@@ -28,23 +28,22 @@ import org.apache.servicecomb.common.rest.codec.RestClientRequest;
import org.apache.servicecomb.common.rest.codec.RestObjectMapperFactory;
import org.apache.servicecomb.common.rest.codec.param.CookieProcessorCreator.CookieProcessor;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.type.TypeFactory;
import com.fasterxml.jackson.databind.util.StdDateFormat;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
-
public class TestCookieProcessor {
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
+ final HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
- Map<String, String> cookies = new HashMap<>();
+ final Map<String, String> cookies = new HashMap<>();
- RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
+ final RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
- String cookieName = "v1";
+ final String cookieName = "v1";
- String cookieValue = "c1v";
+ final String cookieValue = "c1v";
private CookieProcessor createProcessor(String name, Class<?> type) {
return new CookieProcessor(name, TypeFactory.defaultInstance().constructType(type), null, true);
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestFormProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestFormProcessor.java
index 626391f74..2b7e233e7 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestFormProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestFormProcessor.java
@@ -32,21 +32,20 @@ import org.apache.servicecomb.common.rest.codec.param.FormProcessorCreator.FormP
import org.hamcrest.MatcherAssert;
import org.hamcrest.Matchers;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.JavaType;
import com.fasterxml.jackson.databind.type.TypeFactory;
import io.swagger.models.parameters.FormParameter;
import io.swagger.models.properties.ArrayProperty;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
-
public class TestFormProcessor {
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
+ final HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
- Map<String, Object> forms = new HashMap<>();
+ final Map<String, Object> forms = new HashMap<>();
- RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
+ final RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
private FormProcessor createProcessor(String name, Type type) {
return createProcessor(name, type, null, true);
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestHeaderProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestHeaderProcessor.java
index 55ebf588c..c3f07c582 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestHeaderProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestHeaderProcessor.java
@@ -34,7 +34,6 @@ import org.apache.servicecomb.common.rest.codec.param.HeaderProcessorCreator.Hea
import org.hamcrest.MatcherAssert;
import org.hamcrest.Matchers;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.JavaType;
import com.fasterxml.jackson.databind.type.TypeFactory;
@@ -42,14 +41,15 @@ import com.fasterxml.jackson.databind.util.StdDateFormat;
import io.swagger.models.parameters.HeaderParameter;
import io.swagger.models.properties.ArrayProperty;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestHeaderProcessor {
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
+ final HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
- Map<String, String> headers = new HashMap<>();
+ final Map<String, String> headers = new HashMap<>();
- RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
+ final RestClientRequest clientRequest = Mockito.mock(RestClientRequest.class);
private HeaderProcessor createProcessor(String name, Type type) {
return createProcessor(name, type, null, true);
@@ -69,15 +69,6 @@ public class TestHeaderProcessor {
return new HeaderProcessor(headerParameter, javaType);
}
- /*private void createClientRequest() {
- clientRequest = new MockUp<RestClientRequest>() {
- @Mock
- void putHeader(String name, String value) {
- headers.put(name, value);
- }
- }.getMockInstance();
- }*/
-
@Test
public void testGetValueNormal() throws Exception {
Mockito.when(request.getHeader("h1")).thenReturn("h1v");
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestPathProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestPathProcessor.java
index 95a2fc17f..39bbb2177 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestPathProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestPathProcessor.java
@@ -25,15 +25,15 @@ import javax.servlet.http.HttpServletRequest;
import org.apache.servicecomb.common.rest.RestConst;
import org.apache.servicecomb.common.rest.codec.param.PathProcessorCreator.PathProcessor;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.type.TypeFactory;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestPathProcessor {
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
+ final HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
- Map<String, String> pathVars = new HashMap<>();
+ final Map<String, String> pathVars = new HashMap<>();
ParamValueProcessor processor;
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessor.java
index 2d275ace1..8683ef108 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessor.java
@@ -23,18 +23,17 @@ import org.apache.servicecomb.common.rest.codec.param.QueryProcessorCreator.Quer
import org.hamcrest.MatcherAssert;
import org.hamcrest.Matchers;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.JavaType;
import com.fasterxml.jackson.databind.type.TypeFactory;
import io.swagger.models.parameters.QueryParameter;
import io.swagger.models.properties.ArrayProperty;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
-
public class TestQueryProcessor {
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
+ final HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
private ParamValueProcessor createProcessor(String name, Class<?> type, String collectionFormat) {
return createProcessor(name, type, null, true, collectionFormat);
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessorCreator.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessorCreator.java
index 10fb73d5b..f7387db4c 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessorCreator.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestQueryProcessorCreator.java
@@ -22,7 +22,7 @@ import javax.servlet.http.HttpServletRequest;
import org.apache.servicecomb.common.rest.codec.param.QueryProcessorCreator.QueryProcessor;
import org.apache.servicecomb.foundation.test.scaffolding.config.ArchaiusUtils;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
import com.fasterxml.jackson.databind.type.TypeFactory;
@@ -78,5 +78,6 @@ public class TestQueryProcessorCreator {
result = (String) processor.convertValue(null, TypeFactory.defaultInstance().constructType(String.class));
result = (String) processor.getValue(request);
Assertions.assertNull(result);
+ ArchaiusUtils.resetConfig();
}
}
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestRestClientRequestImpl.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestRestClientRequestImpl.java
index 10e197b35..7a6ccdf67 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestRestClientRequestImpl.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/param/TestRestClientRequestImpl.java
@@ -47,7 +47,7 @@ import io.vertx.core.http.HttpClientRequest;
public class TestRestClientRequestImpl {
private HttpClientRequest request;
- private Context context = Mockito.mock(Context.class);
+ private final Context context = Mockito.mock(Context.class);
@BeforeEach
public void before() {
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceJsonProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceJsonProcessor.java
index 7dcda4e64..db8e71cd5 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceJsonProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceJsonProcessor.java
@@ -24,18 +24,18 @@ import java.io.ByteArrayOutputStream;
import java.nio.charset.StandardCharsets;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.JavaType;
import com.fasterxml.jackson.databind.exc.MismatchedInputException;
import com.fasterxml.jackson.databind.type.TypeFactory;
import io.vertx.core.buffer.Buffer;
+import org.junit.jupiter.api.Test;
public class TestProduceJsonProcessor {
- ProduceProcessor pp = ProduceProcessorManager.INSTANCE.findDefaultJsonProcessor();
+ final ProduceProcessor pp = ProduceProcessorManager.INSTANCE.findDefaultJsonProcessor();
- JavaType stringType = TypeFactory.defaultInstance().constructType(String.class);
+ final JavaType stringType = TypeFactory.defaultInstance().constructType(String.class);
@Test
public void testEncodeResponseNull() throws Exception {
@@ -48,7 +48,7 @@ public class TestProduceJsonProcessor {
}
@Test
- public void testdecodeResponseNull() throws Exception {
+ public void testDecodeResponseNull() throws Exception {
JavaType resultType = TypeFactory.unknownType();
Object result = pp.decodeResponse(Buffer.buffer(), resultType);
Assertions.assertNull(result);
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceTextPlainProcessor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceTextPlainProcessor.java
index 36d97808f..d7413d938 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceTextPlainProcessor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/produce/TestProduceTextPlainProcessor.java
@@ -32,9 +32,9 @@ import io.vertx.core.buffer.Buffer;
import org.junit.jupiter.api.Test;
public class TestProduceTextPlainProcessor {
- ProduceProcessor pp = ProduceProcessorManager.INSTANCE.findDefaultPlainProcessor();
+ final ProduceProcessor pp = ProduceProcessorManager.INSTANCE.findDefaultPlainProcessor();
- JavaType stringType = TypeFactory.defaultInstance().constructType(String.class);
+ final JavaType stringType = TypeFactory.defaultInstance().constructType(String.class);
@Test
public void testEncodeResponseNull() throws Exception {
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/query/QueryCodecTestBase.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/query/QueryCodecTestBase.java
index f3a4a6841..5865a2b4c 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/query/QueryCodecTestBase.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/codec/query/QueryCodecTestBase.java
@@ -32,7 +32,7 @@ import io.swagger.models.properties.ArrayProperty;
public class QueryCodecTestBase {
QueryCodec codec;
- String queryName = "q";
+ final String queryName = "q";
void should_encode(String encodedValue, Object... values) throws Exception {
URLPathStringBuilder builder = new URLPathStringBuilder();
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestPath.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestPath.java
index eb3bceb1f..8c1abca87 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestPath.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestPath.java
@@ -25,23 +25,23 @@ import org.apache.servicecomb.common.rest.definition.path.PathRegExp;
import org.apache.servicecomb.common.rest.definition.path.QueryVarParamWriter;
import org.apache.servicecomb.common.rest.definition.path.URLPathBuilder;
import org.apache.servicecomb.common.rest.definition.path.URLPathBuilder.URLPathStringBuilder;
-import org.junit.After;
+import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Assertions;
-import org.junit.Before;
-import org.junit.Test;
import io.swagger.models.parameters.Parameter;
import io.swagger.models.parameters.PathParameter;
import io.swagger.models.parameters.QueryParameter;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestPath {
- @Before
+ @BeforeEach
public void setUp() {
}
- @After
+ @AfterEach
public void tearDown() {
}
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
index afefdafb1..30e627909 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/TestRestOperationMeta.java
@@ -38,14 +38,14 @@ import org.apache.servicecomb.core.definition.OperationMeta;
import org.apache.servicecomb.core.definition.SchemaMeta;
import org.apache.servicecomb.foundation.test.scaffolding.config.ArchaiusUtils;
import org.hamcrest.MatcherAssert;
-import org.junit.AfterClass;
+import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.Assertions;
-import org.junit.BeforeClass;
-import org.junit.Test;
import com.fasterxml.jackson.annotation.JsonView;
import io.swagger.models.Swagger;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestRestOperationMeta {
@@ -160,7 +160,7 @@ public class TestRestOperationMeta {
RestOperationMeta operationMeta;
- @BeforeClass
+ @BeforeAll
public static void classSetup() {
ConfigUtil.installDynamicConfig();
scbEngine = SCBBootstrap.createSCBEngineForTest()
@@ -169,7 +169,7 @@ public class TestRestOperationMeta {
swagger = Mockito.spy(scbEngine.getProducerMicroserviceMeta().ensureFindSchemaMeta("sid1").getSwagger());
}
- @AfterClass
+ @AfterAll
public static void classTeardown() {
scbEngine.destroy();
ArchaiusUtils.resetConfig();
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/path/PathVarParamWriterTest.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/path/PathVarParamWriterTest.java
index 5aa4db137..99ef928b4 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/path/PathVarParamWriterTest.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/definition/path/PathVarParamWriterTest.java
@@ -23,7 +23,7 @@ import java.util.Map;
import org.apache.servicecomb.common.rest.definition.RestParam;
import org.apache.servicecomb.common.rest.definition.path.URLPathBuilder.URLPathStringBuilder;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class PathVarParamWriterTest {
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/TestHttpServerFilterBeforeSendResponseExecutor.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/TestHttpServerFilterBeforeSendResponseExecutor.java
index 4bb0c20a4..169c2fda5 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/TestHttpServerFilterBeforeSendResponseExecutor.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/TestHttpServerFilterBeforeSendResponseExecutor.java
@@ -37,9 +37,9 @@ public class TestHttpServerFilterBeforeSendResponseExecutor {
@Mock
HttpServletResponseEx responseEx;
- List<HttpServerFilter> httpServerFilters = new ArrayList<>();
+ final List<HttpServerFilter> httpServerFilters = new ArrayList<>();
- HttpServerFilterBeforeSendResponseExecutor executor =
+ final HttpServerFilterBeforeSendResponseExecutor executor =
new HttpServerFilterBeforeSendResponseExecutor(httpServerFilters, invocation, responseEx);
@BeforeEach
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/RestServerCodecFilterTest.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/RestServerCodecFilterTest.java
index 14e9b8f34..cd1279f7d 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/RestServerCodecFilterTest.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/RestServerCodecFilterTest.java
@@ -47,43 +47,43 @@ import org.apache.servicecomb.foundation.test.scaffolding.config.ArchaiusUtils;
import org.apache.servicecomb.foundation.test.scaffolding.exception.RuntimeExceptionWithoutStackTrace;
import org.apache.servicecomb.foundation.vertx.http.HttpServletResponseEx;
import org.apache.servicecomb.swagger.invocation.Response;
-import org.junit.AfterClass;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.Test;
import com.fasterxml.jackson.databind.type.TypeFactory;
import io.vertx.core.MultiMap;
import io.vertx.core.json.Json;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
import javax.servlet.http.Part;
public class RestServerCodecFilterTest {
- RestServerCodecFilter codecFilter = new RestServerCodecFilter();
+ final RestServerCodecFilter codecFilter = new RestServerCodecFilter();
Invocation invocation;
- Endpoint endpoint = Mockito.mock(Endpoint.class);
+ final Endpoint endpoint = Mockito.mock(Endpoint.class);
- OperationMeta operationMeta = Mockito.mock(OperationMeta.class);
+ final OperationMeta operationMeta = Mockito.mock(OperationMeta.class);
- SchemaMeta schemaMeta = Mockito.mock(SchemaMeta.class);
+ final SchemaMeta schemaMeta = Mockito.mock(SchemaMeta.class);
- MicroserviceMeta microserviceMeta = Mockito.mock(MicroserviceMeta.class);
+ final MicroserviceMeta microserviceMeta = Mockito.mock(MicroserviceMeta.class);
- InvocationRuntimeType invocationRuntimeType = Mockito.mock(InvocationRuntimeType.class);
+ final InvocationRuntimeType invocationRuntimeType = Mockito.mock(InvocationRuntimeType.class);
- RestOperationMeta restOperationMeta = Mockito.mock(RestOperationMeta.class);
+ final RestOperationMeta restOperationMeta = Mockito.mock(RestOperationMeta.class);
- HttpTransportContext transportContext = Mockito.mock(HttpTransportContext.class);
+ final HttpTransportContext transportContext = Mockito.mock(HttpTransportContext.class);
- HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
+ final HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
- MultiMap headers = MultiMap.caseInsensitiveMultiMap();
+ final MultiMap headers = MultiMap.caseInsensitiveMultiMap();
- FilterNode nextNode = new FilterNode((invocation, next) -> {
+ final FilterNode nextNode = new FilterNode((invocation, next) -> {
Response response = Response.ok("ok");
response.setHeaders(headers);
return CompletableFuture.completedFuture(response);
@@ -91,7 +91,7 @@ public class RestServerCodecFilterTest {
static SCBEngine engine;
- @BeforeClass
+ @BeforeAll
public static void beforeClass() {
ArchaiusUtils.resetConfig();
ConfigUtil.installDynamicConfig();
@@ -100,13 +100,13 @@ public class RestServerCodecFilterTest {
engine.setStatus(SCBStatus.UP);
}
- @AfterClass
+ @AfterAll
public static void afterClass() {
engine.destroy();
ArchaiusUtils.resetConfig();
}
- @Before
+ @BeforeEach
public void setUp() {
Mockito.when(operationMeta.getSchemaMeta()).thenReturn(schemaMeta);
Mockito.when(schemaMeta.getMicroserviceMeta()).thenReturn(microserviceMeta);
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/TestServerRestArgsFilter.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/TestServerRestArgsFilter.java
index 2a190a09e..b1cef1a61 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/TestServerRestArgsFilter.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/filter/inner/TestServerRestArgsFilter.java
@@ -25,25 +25,25 @@ import org.apache.servicecomb.foundation.vertx.http.HttpServletResponseEx;
import org.apache.servicecomb.swagger.invocation.Response;
import org.apache.servicecomb.swagger.invocation.response.ResponsesMeta;
import org.junit.jupiter.api.Assertions;
-import org.junit.Test;
import com.fasterxml.jackson.databind.type.TypeFactory;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestServerRestArgsFilter {
- Invocation invocation = Mockito.mock(Invocation.class);
+ final Invocation invocation = Mockito.mock(Invocation.class);
- HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
+ final HttpServletResponseEx responseEx = Mockito.mock(HttpServletResponseEx.class);
- Response response = Mockito.mock(Response.class);
+ final Response response = Mockito.mock(Response.class);
- Part part = Mockito.mock(Part.class);
+ final Part part = Mockito.mock(Part.class);
boolean invokedSendPart;
- ServerRestArgsFilter filter = new ServerRestArgsFilter();
+ final ServerRestArgsFilter filter = new ServerRestArgsFilter();
@Test
public void asyncBeforeSendResponse_part() {
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/locator/TestMicroservicePaths.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/locator/TestMicroservicePaths.java
index 2a0b6641f..6192bf723 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/locator/TestMicroservicePaths.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/locator/TestMicroservicePaths.java
@@ -24,10 +24,10 @@ import org.apache.servicecomb.core.bootstrap.SCBBootstrap;
import org.apache.servicecomb.foundation.common.exceptions.ServiceCombException;
import org.apache.servicecomb.foundation.test.scaffolding.config.ArchaiusUtils;
import org.apache.servicecomb.foundation.test.scaffolding.log.LogCollector;
-import org.junit.AfterClass;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestMicroservicePaths {
@@ -35,7 +35,7 @@ public class TestMicroservicePaths {
static MicroservicePaths paths;
- @BeforeClass
+ @BeforeAll
public static void setup() {
ConfigUtil.installDynamicConfig();
scbEngine = SCBBootstrap.createSCBEngineForTest()
@@ -46,7 +46,7 @@ public class TestMicroservicePaths {
paths = spm.producerPaths;
}
- @AfterClass
+ @AfterAll
public static void teardown() {
scbEngine.destroy();
ArchaiusUtils.resetConfig();
diff --git a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/resource/TestClassPathStaticResourceHandler.java b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/resource/TestClassPathStaticResourceHandler.java
index 144b67ca8..a48aa291a 100644
--- a/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/resource/TestClassPathStaticResourceHandler.java
+++ b/common/common-rest/src/test/java/org/apache/servicecomb/common/rest/resource/TestClassPathStaticResourceHandler.java
@@ -31,14 +31,14 @@ import org.apache.servicecomb.swagger.invocation.Response;
import org.apache.servicecomb.swagger.invocation.exception.CommonExceptionData;
import org.apache.servicecomb.swagger.invocation.exception.InvocationException;
import org.junit.jupiter.api.Assertions;
-import org.junit.BeforeClass;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
public class TestClassPathStaticResourceHandler {
static ClassPathStaticResourceHandler handler = new ClassPathStaticResourceHandler();
- @BeforeClass
+ @BeforeAll
public static void setup() {
handler.setWebRoot("web-root/");
}