This is an automated email from the ASF dual-hosted git repository.
albumenj pushed a commit to branch 3.1
in repository https://gitbox.apache.org/repos/asf/dubbo.git
The following commit(s) were added to refs/heads/3.1 by this push:
new 1fbeed8e55 Remove public modifier in metadata and registry (#10980)
1fbeed8e55 is described below
commit 1fbeed8e552d221822b06740cf31312aa7aa29c5
Author: Albumen Kevin <[email protected]>
AuthorDate: Tue Nov 22 10:47:54 2022 +0800
Remove public modifier in metadata and registry (#10980)
---
.../metadata/AbstractServiceNameMappingTest.java | 2 +-
.../apache/dubbo/metadata/MetadataInfoTest.java | 18 ++++----
.../dubbo/metadata/ServiceNameMappingTest.java | 10 ++---
.../report/MetadataReportInstanceTest.java | 3 +-
.../report/identifier/KeyTypeEnumTest.java | 4 +-
.../report/identifier/MetadataIdentifierTest.java | 4 +-
.../support/AbstractMetadataReportFactoryTest.java | 10 ++---
.../report/support/AbstractMetadataReportTest.java | 18 ++++----
.../JAXRSServiceRestMetadataResolverTest.java | 6 +--
.../SpringMvcServiceRestMetadataResolverTest.java | 6 +--
.../metadata/test/JTestMetadataReport4Test.java | 2 +-
.../protobuf/ProtobufTypeBuilderTest.java | 4 +-
.../builder/ArrayTypeDefinitionBuilderTest.java | 6 +--
.../CollectionTypeDefinitionBuilderTest.java | 6 +--
.../builder/EnumTypeDefinitionBuilderTest.java | 6 +--
.../builder/GeneralTypeDefinitionBuilderTest.java | 6 +--
.../builder/MapTypeDefinitionBuilderTest.java | 6 +--
.../PrimitiveTypeDefinitionBuilderTest.java | 6 +--
.../builder/ServiceDefinitionBuilderTest.java | 4 +-
.../builder/SimpleTypeDefinitionBuilderTest.java | 6 +--
.../AnnotatedMethodParameterProcessorTest.java | 2 +-
.../processing/util/AnnotationUtilsTest.java | 18 ++++----
.../annotation/processing/util/FieldUtilsTest.java | 20 ++++-----
.../processing/util/LoggerUtilsTest.java | 8 ++--
.../processing/util/MemberUtilsTest.java | 10 ++---
.../processing/util/MethodUtilsTest.java | 22 +++++-----
.../util/ServiceAnnotationUtilsTest.java | 14 +++----
.../annotation/processing/util/TypeUtilsTest.java | 46 ++++++++++-----------
.../apache/dubbo/metadata/tools/CompilerTest.java | 4 +-
.../metadata/tools/DefaultRestServiceTest.java | 4 +-
.../dubbo/metadata/tools/RestServiceTest.java | 4 +-
.../metadata/tools/SpringRestServiceTest.java | 4 +-
.../metadata/tools/StandardRestServiceTest.java | 4 +-
.../store/redis/RedisMetadataReportTest.java | 14 +++----
.../zookeeper/ZookeeperMetadataReportTest.java | 20 ++++-----
.../registry/ListenerRegistryWrapperTest.java | 4 +-
.../dubbo/registry/PerformanceRegistryTest.java | 4 +-
.../dubbo/registry/RegistryFactoryWrapperTest.java | 4 +-
.../AbstractServiceDiscoveryFactoryTest.java | 4 +-
.../client/DefaultServiceInstanceTest.java | 6 +--
.../registry/client/InstanceAddressURLTest.java | 6 +--
.../client/ServiceDiscoveryRegistryTest.java | 10 ++---
.../ServiceInstancesChangedListenerTest.java | 2 +-
.../metadata/MetadataServiceNameMappingTest.java | 8 ++--
.../metadata/MetadataServiceURLBuilderTest.java | 2 +-
.../ProtocolPortsMetadataCustomizerTest.java | 4 +-
.../ServiceInstanceMetadataCustomizerTest.java | 6 +--
.../SpringCloudMetadataServiceURLBuilderTest.java | 4 +-
.../StandardMetadataServiceURLBuilderTest.java | 4 +-
.../metadata/store/MetaCacheManagerTest.java | 6 +--
.../DefaultMigrationAddressComparatorTest.java | 4 +-
.../client/migration/MigrationInvokerTest.java | 8 ++--
.../client/migration/MigrationRuleHandlerTest.java | 4 +-
.../migration/MigrationRuleListenerTest.java | 8 ++--
.../client/migration/model/MigrationRuleTest.java | 4 +-
.../registry/integration/DynamicDirectoryTest.java | 6 +--
.../registry/integration/RegistryProtocolTest.java | 16 ++++----
.../support/AbstractRegistryFactoryTest.java | 8 ++--
.../registry/support/AbstractRegistryTest.java | 48 +++++++++++-----------
.../registry/support/FailbackRegistryTest.java | 10 ++---
.../multicast/MulticastRegistryFactoryTest.java | 4 +-
.../multiple/MultipleRegistry2S2RTest.java | 10 ++---
.../registry/nacos/NacosRegistryFactoryTest.java | 6 +--
.../dubbo/registry/nacos/NacosRegistryTest.java | 12 +++---
.../nacos/NacosServiceDiscoveryFactoryTest.java | 4 +-
.../registry/nacos/NacosServiceDiscoveryTest.java | 10 ++---
.../nacos/util/NacosNamingServiceUtilsTest.java | 8 ++--
.../registry/zookeeper/ZookeeperRegistryTest.java | 26 ++++++------
.../zookeeper/ZookeeperServiceDiscoveryTest.java | 4 +-
69 files changed, 300 insertions(+), 301 deletions(-)
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/AbstractServiceNameMappingTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/AbstractServiceNameMappingTest.java
index 7b346754b8..61d73d8d2d 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/AbstractServiceNameMappingTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/AbstractServiceNameMappingTest.java
@@ -71,7 +71,7 @@ class AbstractServiceNameMappingTest {
}
@Test
- public void testGetAndListener() {
+ void testGetAndListener() {
URL registryURL = URL.valueOf("registry://127.0.0.1:7777/test");
registryURL = registryURL.addParameter(SUBSCRIBED_SERVICE_NAMES_KEY,
"registry-app1");
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/MetadataInfoTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/MetadataInfoTest.java
index ccd91deb05..544aaa5721 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/MetadataInfoTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/MetadataInfoTest.java
@@ -47,7 +47,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
/**
* Some construction and filter cases are covered in
InMemoryMetadataServiceTest
*/
-public class MetadataInfoTest {
+class MetadataInfoTest {
private static URL url =
URL.valueOf("dubbo://30.225.21.30:20880/org.apache.dubbo.registry.service.DemoService2?"
+
"REGISTRY_CLUSTER=registry1&anyhost=true&application=demo-provider2&delay=5000&deprecated=false&dubbo=2.0.2"
+
"&dynamic=true&generic=false&group=greeting&interface=org.apache.dubbo.registry.service.DemoService2"
+
@@ -67,7 +67,7 @@ public class MetadataInfoTest {
"&side=provider&timeout=5000×tamp=1629970068002&version=1.0.0¶ms-filter=-customized,excluded");
@Test
- public void testEmptyRevision() {
+ void testEmptyRevision() {
MetadataInfo metadataInfo = new MetadataInfo("demo");
metadataInfo.setApp("demo");
@@ -75,7 +75,7 @@ public class MetadataInfoTest {
}
@Test
- public void testParamsFilterIncluded() {
+ void testParamsFilterIncluded() {
MetadataInfo metadataInfo = new MetadataInfo("demo");
// export normal url again
@@ -93,7 +93,7 @@ public class MetadataInfoTest {
}
@Test
- public void testParamsFilterExcluded() {
+ void testParamsFilterExcluded() {
MetadataInfo metadataInfo = new MetadataInfo("demo");
// export normal url again
@@ -111,7 +111,7 @@ public class MetadataInfoTest {
}
@Test
- public void testEqualsAndRevision() {
+ void testEqualsAndRevision() {
// same metadata
MetadataInfo metadataInfo = new MetadataInfo("demo");
metadataInfo.addService(url);
@@ -144,7 +144,7 @@ public class MetadataInfoTest {
}
@Test
- public void testChanged() {
+ void testChanged() {
MetadataInfo metadataInfo = new MetadataInfo("demo");
metadataInfo.addService(url);
metadataInfo.addService(url2);
@@ -156,7 +156,7 @@ public class MetadataInfoTest {
}
@Test
- public void testJsonFormat() {
+ void testJsonFormat() {
MetadataInfo metadataInfo = new MetadataInfo("demo");
// export normal url again
@@ -172,7 +172,7 @@ public class MetadataInfoTest {
}
@Test
- public void testJdkSerialize() throws IOException, ClassNotFoundException,
NoSuchFieldException, IllegalAccessException {
+ void testJdkSerialize() throws IOException, ClassNotFoundException,
NoSuchFieldException, IllegalAccessException {
ByteArrayOutputStream byteArrayOutputStream = new
ByteArrayOutputStream();
ObjectOutputStream objectOutputStream = new
ObjectOutputStream(byteArrayOutputStream);
MetadataInfo metadataInfo = new MetadataInfo("demo");
@@ -195,7 +195,7 @@ public class MetadataInfoTest {
}
@Test
- public void testCal() {
+ void testCal() {
MetadataInfo metadataInfo = new MetadataInfo("demo");
// export normal url again
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/ServiceNameMappingTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/ServiceNameMappingTest.java
index 7a03cd49e0..322359fa00 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/ServiceNameMappingTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/ServiceNameMappingTest.java
@@ -42,12 +42,12 @@ import static org.junit.jupiter.api.Assertions.assertThrows;
*/
/**
- * {@link ServiceNameMapping} Test
+ * {@link org.apache.dubbo.metadata.ServiceNameMapping} Test
*
* @since 2.7.8
*//*
-public class ServiceNameMappingTest {
+class ServiceNameMappingTest {
private static final URL BASE_URL = URL.valueOf("dubbo://127.0.0.1:20880");
@@ -83,7 +83,7 @@ public class ServiceNameMappingTest {
}
@Test
- public void testDeprecatedMethods() {
+ void testDeprecatedMethods() {
assertThrows(UnsupportedOperationException.class, () -> {
serviceNameMapping.map(null, null, null, null);
});
@@ -94,7 +94,7 @@ public class ServiceNameMappingTest {
}
@Test
- public void testMap() {
+ void testMap() {
String serviceInterface = ServiceNameMapping.class.getName();
String key = applicationName;
String group = buildGroup(serviceInterface, null, null, null);
@@ -104,7 +104,7 @@ public class ServiceNameMappingTest {
}
@Test
- public void testGet() {
+ void testGet() {
String serviceInterface = ServiceNameMapping.class.getName();
URL url = BASE_URL.setServiceInterface(serviceInterface);
serviceNameMapping.map(url);
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/MetadataReportInstanceTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/MetadataReportInstanceTest.java
index 1e8f7e6c4a..0f2d3cb4b6 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/MetadataReportInstanceTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/MetadataReportInstanceTest.java
@@ -27,7 +27,6 @@ import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
-import java.util.Arrays;
import java.util.Collections;
import java.util.Map;
@@ -68,7 +67,7 @@ class MetadataReportInstanceTest {
}
@Test
- public void test() {
+ void test() {
Assertions.assertNull(metadataReportInstance.getMetadataReport(registryId),
"the metadata report was not initialized.");
assertThat(metadataReportInstance.getMetadataReports(true),
Matchers.anEmptyMap());
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/KeyTypeEnumTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/KeyTypeEnumTest.java
index c4dae1f919..fe9a8274c8 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/KeyTypeEnumTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/KeyTypeEnumTest.java
@@ -25,13 +25,13 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
*
* @since 2.7.8
*/
-public class KeyTypeEnumTest {
+class KeyTypeEnumTest {
/**
* {@link KeyTypeEnum#build(String, String...)}
*/
@Test
- public void testBuild() {
+ void testBuild() {
assertEquals("/A/B/C", KeyTypeEnum.PATH.build("/A", "/B", "C"));
assertEquals("A:B:C", KeyTypeEnum.UNIQUE_KEY.build("A", "B", "C"));
}
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/MetadataIdentifierTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/MetadataIdentifierTest.java
index 3a58933e16..810c668a62 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/MetadataIdentifierTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/identifier/MetadataIdentifierTest.java
@@ -27,10 +27,10 @@ import static
org.apache.dubbo.common.constants.CommonConstants.PROVIDER_SIDE;
/**
* 2019/1/7
*/
-public class MetadataIdentifierTest {
+class MetadataIdentifierTest {
@Test
- public void testGetUniqueKey() {
+ void testGetUniqueKey() {
String interfaceName =
"org.apache.dubbo.metadata.integration.InterfaceNameTestService";
String version = "1.0.0.zk.md";
String group = null;
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportFactoryTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportFactoryTest.java
index 99e5cb114b..56cd80e778 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportFactoryTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportFactoryTest.java
@@ -37,7 +37,7 @@ import java.util.concurrent.ConcurrentHashMap;
/**
* 2018/9/14
*/
-public class AbstractMetadataReportFactoryTest {
+class AbstractMetadataReportFactoryTest {
private AbstractMetadataReportFactory metadataReportFactory = new
AbstractMetadataReportFactory() {
@Override
@@ -112,7 +112,7 @@ public class AbstractMetadataReportFactoryTest {
};
@Test
- public void testGetOneMetadataReport() {
+ void testGetOneMetadataReport() {
URL url = URL.valueOf("zookeeper://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.TestService?version=1.0.0&application=vic");
MetadataReport metadataReport1 =
metadataReportFactory.getMetadataReport(url);
MetadataReport metadataReport2 =
metadataReportFactory.getMetadataReport(url);
@@ -120,7 +120,7 @@ public class AbstractMetadataReportFactoryTest {
}
@Test
- public void testGetOneMetadataReportForIpFormat() {
+ void testGetOneMetadataReportForIpFormat() {
String hostName = NetUtils.getLocalAddress().getHostName();
String ip = NetUtils.getIpByHost(hostName);
URL url1 = URL.valueOf("zookeeper://" + hostName +
":4444/org.apache.dubbo.TestService?version=1.0.0&application=vic");
@@ -131,7 +131,7 @@ public class AbstractMetadataReportFactoryTest {
}
@Test
- public void testGetForDiffService() {
+ void testGetForDiffService() {
URL url1 = URL.valueOf("zookeeper://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.TestService1?version=1.0.0&application=vic");
URL url2 = URL.valueOf("zookeeper://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.TestService2?version=1.0.0&application=vic");
MetadataReport metadataReport1 =
metadataReportFactory.getMetadataReport(url1);
@@ -140,7 +140,7 @@ public class AbstractMetadataReportFactoryTest {
}
@Test
- public void testGetForDiffGroup() {
+ void testGetForDiffGroup() {
URL url1 = URL.valueOf("zookeeper://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.TestService?version=1.0.0&application=vic&group=aaa");
URL url2 = URL.valueOf("zookeeper://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.TestService?version=1.0.0&application=vic&group=bbb");
MetadataReport metadataReport1 =
metadataReportFactory.getMetadataReport(url1);
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportTest.java
index 99a0c44178..50ecefe1a3 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportTest.java
@@ -48,7 +48,7 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.junit.jupiter.api.Assertions.assertTrue;
-public class AbstractMetadataReportTest {
+class AbstractMetadataReportTest {
private NewMetadataReport abstractMetadataReport;
@@ -67,7 +67,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testGetProtocol() {
+ void testGetProtocol() {
URL url = URL.valueOf("dubbo://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.TestService?version=1.0.0&application=vic&side=provider");
String protocol = abstractMetadataReport.getProtocol(url);
assertEquals(protocol, "provider");
@@ -78,7 +78,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testStoreProviderUsual() throws ClassNotFoundException,
InterruptedException {
+ void testStoreProviderUsual() throws ClassNotFoundException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.InterfaceNameTestService";
String version = "1.0.0";
String group = null;
@@ -89,7 +89,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testStoreProviderSync() throws ClassNotFoundException,
InterruptedException {
+ void testStoreProviderSync() throws ClassNotFoundException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.InterfaceNameTestService";
String version = "1.0.0";
String group = null;
@@ -100,7 +100,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testFileExistAfterPut() throws InterruptedException,
ClassNotFoundException {
+ void testFileExistAfterPut() throws InterruptedException,
ClassNotFoundException {
//just for one method
URL singleUrl = URL.valueOf("redis://" +
NetUtils.getLocalAddress().getHostName() +
":4444/org.apache.dubbo.metadata.store.InterfaceNameTestService?version=1.0.0&application=singleTest");
NewMetadataReport singleMetadataReport = new
NewMetadataReport(singleUrl);
@@ -119,7 +119,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testRetry() throws InterruptedException,
ClassNotFoundException {
+ void testRetry() throws InterruptedException, ClassNotFoundException {
String interfaceName =
"org.apache.dubbo.metadata.store.RetryTestService";
String version = "1.0.0.retry";
String group = null;
@@ -148,7 +148,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testRetryCancel() throws InterruptedException,
ClassNotFoundException {
+ void testRetryCancel() throws InterruptedException, ClassNotFoundException
{
String interfaceName =
"org.apache.dubbo.metadata.store.RetryTestService";
String version = "1.0.0.retrycancel";
String group = null;
@@ -197,7 +197,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testPublishAll() throws ClassNotFoundException,
InterruptedException {
+ void testPublishAll() throws ClassNotFoundException, InterruptedException {
assertTrue(abstractMetadataReport.store.isEmpty());
assertTrue(abstractMetadataReport.allMetadataReports.isEmpty());
@@ -250,7 +250,7 @@ public class AbstractMetadataReportTest {
}
@Test
- public void testCalculateStartTime() {
+ void testCalculateStartTime() {
for (int i = 0; i < 300; i++) {
long t = abstractMetadataReport.calculateStartTime() +
System.currentTimeMillis();
Calendar c = Calendar.getInstance();
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/jaxrs/JAXRSServiceRestMetadataResolverTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/jaxrs/JAXRSServiceRestMetadataResolverTest.java
index 1ebaf3a86a..1e1acb1cf3 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/jaxrs/JAXRSServiceRestMetadataResolverTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/jaxrs/JAXRSServiceRestMetadataResolverTest.java
@@ -37,12 +37,12 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class JAXRSServiceRestMetadataResolverTest {
+class JAXRSServiceRestMetadataResolverTest {
private JAXRSServiceRestMetadataResolver instance = new
JAXRSServiceRestMetadataResolver();
@Test
- public void testSupports() {
+ void testSupports() {
// JAX-RS RestService class
assertTrue(instance.supports(StandardRestService.class));
// Spring MVC RestService class
@@ -56,7 +56,7 @@ public class JAXRSServiceRestMetadataResolverTest {
}
@Test
- public void testResolve() {
+ void testResolve() {
// Generated by "dubbo-metadata-processor"
ClassPathServiceRestMetadataReader reader = new
ClassPathServiceRestMetadataReader("META-INF/dubbo/jax-rs-service-rest-metadata.json");
List<ServiceRestMetadata> serviceRestMetadataList = reader.read();
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/springmvc/SpringMvcServiceRestMetadataResolverTest.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/springmvc/SpringMvcServiceRestMetadataResolverTest.java
index 4fd0e1a164..e7d0ba42ca 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/springmvc/SpringMvcServiceRestMetadataResolverTest.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/rest/springmvc/SpringMvcServiceRestMetadataResolverTest.java
@@ -37,12 +37,12 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.9
*/
-public class SpringMvcServiceRestMetadataResolverTest {
+class SpringMvcServiceRestMetadataResolverTest {
private SpringMvcServiceRestMetadataResolver instance = new
SpringMvcServiceRestMetadataResolver();
@Test
- public void testSupports() {
+ void testSupports() {
// Spring MVC RestService class
assertTrue(instance.supports(SpringRestService.class));
// JAX-RS RestService class
@@ -56,7 +56,7 @@ public class SpringMvcServiceRestMetadataResolverTest {
}
@Test
- public void testResolve() {
+ void testResolve() {
// Generated by "dubbo-metadata-processor"
ClassPathServiceRestMetadataReader reader = new
ClassPathServiceRestMetadataReader("META-INF/dubbo/spring-mvc-service-rest-metadata.json");
List<ServiceRestMetadata> serviceRestMetadataList = reader.read();
diff --git
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/test/JTestMetadataReport4Test.java
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/test/JTestMetadataReport4Test.java
index 8e94495126..ad4f052954 100644
---
a/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/test/JTestMetadataReport4Test.java
+++
b/dubbo-metadata/dubbo-metadata-api/src/test/java/org/apache/dubbo/metadata/test/JTestMetadataReport4Test.java
@@ -34,7 +34,7 @@ import java.util.concurrent.ConcurrentHashMap;
/**
* ZookeeperRegistry
*/
-public class JTestMetadataReport4Test extends AbstractMetadataReport {
+class JTestMetadataReport4Test extends AbstractMetadataReport {
private final static Logger logger =
LoggerFactory.getLogger(JTestMetadataReport4Test.class);
diff --git
a/dubbo-metadata/dubbo-metadata-definition-protobuf/src/test/java/org/apache/dubbo/metadata/definition/protobuf/ProtobufTypeBuilderTest.java
b/dubbo-metadata/dubbo-metadata-definition-protobuf/src/test/java/org/apache/dubbo/metadata/definition/protobuf/ProtobufTypeBuilderTest.java
index 426fe0cb73..559d313f46 100644
---
a/dubbo-metadata/dubbo-metadata-definition-protobuf/src/test/java/org/apache/dubbo/metadata/definition/protobuf/ProtobufTypeBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-definition-protobuf/src/test/java/org/apache/dubbo/metadata/definition/protobuf/ProtobufTypeBuilderTest.java
@@ -36,9 +36,9 @@ import static org.hamcrest.MatcherAssert.assertThat;
/**
* 2019-07-01
*/
-public class ProtobufTypeBuilderTest {
+class ProtobufTypeBuilderTest {
@Test
- public void testProtobufBuilder() {
+ void testProtobufBuilder() {
TypeDefinitionBuilder.initBuilders(FrameworkModel.defaultModel());
// TEST Pb Service metaData builder
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ArrayTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ArrayTypeDefinitionBuilderTest.java
index d3e7ff1190..cd84396e6d 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ArrayTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ArrayTypeDefinitionBuilderTest.java
@@ -41,7 +41,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class ArrayTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class ArrayTypeDefinitionBuilderTest extends AbstractAnnotationProcessingTest {
private ArrayTypeDefinitionBuilder builder;
@@ -74,7 +74,7 @@ public class ArrayTypeDefinitionBuilderTest extends
AbstractAnnotationProcessing
}
@Test
- public void testAccept() {
+ void testAccept() {
assertTrue(builder.accept(processingEnv, integersField.asType()));
assertTrue(builder.accept(processingEnv, stringsField.asType()));
assertTrue(builder.accept(processingEnv,
primitiveTypeModelsField.asType()));
@@ -83,7 +83,7 @@ public class ArrayTypeDefinitionBuilderTest extends
AbstractAnnotationProcessing
}
@Test
- public void testBuild() {
+ void testBuild() {
buildAndAssertTypeDefinition(processingEnv, integersField, "int[]",
"int", builder);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/CollectionTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/CollectionTypeDefinitionBuilderTest.java
index bd6e7a4e04..e99d417784 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/CollectionTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/CollectionTypeDefinitionBuilderTest.java
@@ -35,7 +35,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class CollectionTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class CollectionTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
private CollectionTypeDefinitionBuilder builder;
@@ -72,7 +72,7 @@ public class CollectionTypeDefinitionBuilderTest extends
AbstractAnnotationProce
}
@Test
- public void testAccept() {
+ void testAccept() {
assertTrue(builder.accept(processingEnv, stringsField.asType()));
assertTrue(builder.accept(processingEnv, colorsField.asType()));
assertTrue(builder.accept(processingEnv,
primitiveTypeModelsField.asType()));
@@ -81,7 +81,7 @@ public class CollectionTypeDefinitionBuilderTest extends
AbstractAnnotationProce
}
@Test
- public void testBuild() {
+ void testBuild() {
buildAndAssertTypeDefinition(processingEnv, stringsField,
"java.util.Collection<java.lang.String>", "java.lang.String", builder);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/EnumTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/EnumTypeDefinitionBuilderTest.java
index aaea5df876..49572e708e 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/EnumTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/EnumTypeDefinitionBuilderTest.java
@@ -36,7 +36,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class EnumTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class EnumTypeDefinitionBuilderTest extends AbstractAnnotationProcessingTest {
private EnumTypeDefinitionBuilder builder;
@@ -51,13 +51,13 @@ public class EnumTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingT
}
@Test
- public void testAccept() {
+ void testAccept() {
TypeElement typeElement = getType(Color.class);
assertTrue(builder.accept(processingEnv, typeElement.asType()));
}
@Test
- public void testBuild() {
+ void testBuild() {
TypeElement typeElement = getType(Color.class);
Map<String, TypeDefinition> typeCache = new HashMap<>();
TypeDefinition typeDefinition =
TypeDefinitionBuilder.build(processingEnv, typeElement, typeCache);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/GeneralTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/GeneralTypeDefinitionBuilderTest.java
index 33e27d3176..8ca703aa32 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/GeneralTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/GeneralTypeDefinitionBuilderTest.java
@@ -36,7 +36,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class GeneralTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class GeneralTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
private GeneralTypeDefinitionBuilder builder;
@@ -51,7 +51,7 @@ public class GeneralTypeDefinitionBuilderTest extends
AbstractAnnotationProcessi
}
@Test
- public void testAccept() {
+ void testAccept() {
assertTrue(builder.accept(processingEnv,
getType(Model.class).asType()));
assertTrue(builder.accept(processingEnv,
getType(PrimitiveTypeModel.class).asType()));
assertTrue(builder.accept(processingEnv,
getType(SimpleTypeModel.class).asType()));
@@ -61,7 +61,7 @@ public class GeneralTypeDefinitionBuilderTest extends
AbstractAnnotationProcessi
}
@Test
- public void testBuild() {
+ void testBuild() {
}
}
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/MapTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/MapTypeDefinitionBuilderTest.java
index a415277703..4f48684606 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/MapTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/MapTypeDefinitionBuilderTest.java
@@ -40,7 +40,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class MapTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class MapTypeDefinitionBuilderTest extends AbstractAnnotationProcessingTest {
private MapTypeDefinitionBuilder builder;
@@ -77,7 +77,7 @@ public class MapTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTe
}
@Test
- public void testAccept() {
+ void testAccept() {
assertTrue(builder.accept(processingEnv, stringsField.asType()));
assertTrue(builder.accept(processingEnv, colorsField.asType()));
assertTrue(builder.accept(processingEnv,
primitiveTypeModelsField.asType()));
@@ -86,7 +86,7 @@ public class MapTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTe
}
@Test
- public void testBuild() {
+ void testBuild() {
buildAndAssertTypeDefinition(processingEnv, stringsField,
"java.util.Map<java.lang.String,java.lang.String>",
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/PrimitiveTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/PrimitiveTypeDefinitionBuilderTest.java
index c259ffc270..3c9774cf6c 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/PrimitiveTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/PrimitiveTypeDefinitionBuilderTest.java
@@ -38,7 +38,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class PrimitiveTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class PrimitiveTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
private PrimitiveTypeDefinitionBuilder builder;
@@ -90,7 +90,7 @@ public class PrimitiveTypeDefinitionBuilderTest extends
AbstractAnnotationProces
}
@Test
- public void testAccept() {
+ void testAccept() {
assertTrue(builder.accept(processingEnv, zField.asType()));
assertTrue(builder.accept(processingEnv, bField.asType()));
assertTrue(builder.accept(processingEnv, cField.asType()));
@@ -102,7 +102,7 @@ public class PrimitiveTypeDefinitionBuilderTest extends
AbstractAnnotationProces
}
@Test
- public void testBuild() {
+ void testBuild() {
buildAndAssertTypeDefinition(processingEnv, zField, builder);
buildAndAssertTypeDefinition(processingEnv, bField, builder);
buildAndAssertTypeDefinition(processingEnv, cField, builder);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ServiceDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ServiceDefinitionBuilderTest.java
index a2477cc896..9906f12975 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ServiceDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/ServiceDefinitionBuilderTest.java
@@ -35,7 +35,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
*
* @since 2.7.6
*/
-public class ServiceDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class ServiceDefinitionBuilderTest extends AbstractAnnotationProcessingTest {
@Override
@@ -48,7 +48,7 @@ public class ServiceDefinitionBuilderTest extends
AbstractAnnotationProcessingTe
}
@Test
- public void testBuild() {
+ void testBuild() {
ServiceDefinition serviceDefinition = build(processingEnv,
getType(TestServiceImpl.class));
assertEquals(TestServiceImpl.class.getTypeName(),
serviceDefinition.getCanonicalName());
assertEquals("org/apache/dubbo/metadata/tools/TestServiceImpl.class",
serviceDefinition.getCodeSource());
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/SimpleTypeDefinitionBuilderTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/SimpleTypeDefinitionBuilderTest.java
index 97edf89655..08f008d394 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/SimpleTypeDefinitionBuilderTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/builder/SimpleTypeDefinitionBuilderTest.java
@@ -36,7 +36,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class SimpleTypeDefinitionBuilderTest extends
AbstractAnnotationProcessingTest {
+class SimpleTypeDefinitionBuilderTest extends AbstractAnnotationProcessingTest
{
private SimpleTypeDefinitionBuilder builder;
@@ -110,7 +110,7 @@ public class SimpleTypeDefinitionBuilderTest extends
AbstractAnnotationProcessin
}
@Test
- public void testAccept() {
+ void testAccept() {
assertTrue(builder.accept(processingEnv, vField.asType()));
assertTrue(builder.accept(processingEnv, zField.asType()));
assertTrue(builder.accept(processingEnv, cField.asType()));
@@ -129,7 +129,7 @@ public class SimpleTypeDefinitionBuilderTest extends
AbstractAnnotationProcessin
}
@Test
- public void testBuild() {
+ void testBuild() {
buildAndAssertTypeDefinition(processingEnv, vField, builder);
buildAndAssertTypeDefinition(processingEnv, zField, builder);
buildAndAssertTypeDefinition(processingEnv, cField, builder);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/rest/AnnotatedMethodParameterProcessorTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/rest/AnnotatedMethodParameterProcessorTest.java
index 825906b2f6..6db0efb5b5 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/rest/AnnotatedMethodParameterProcessorTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/rest/AnnotatedMethodParameterProcessorTest.java
@@ -51,7 +51,7 @@ public abstract class AnnotatedMethodParameterProcessorTest
extends AbstractAnno
protected abstract String getExpectedAnnotationType();
@Test
- public void testGetAnnotationType() {
+ void testGetAnnotationType() {
String expectedAnnotationType = getExpectedAnnotationType();
assertNull(processor.getAnnotationType());
assertEquals(expectedAnnotationType, processor.getAnnotationType());
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/AnnotationUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/AnnotationUtilsTest.java
index cbc3dc12d9..186b06bb4f 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/AnnotationUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/AnnotationUtilsTest.java
@@ -56,7 +56,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class AnnotationUtilsTest extends AbstractAnnotationProcessingTest {
+class AnnotationUtilsTest extends AbstractAnnotationProcessingTest {
private TypeElement testType;
@@ -70,7 +70,7 @@ public class AnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAnnotation() {
+ void testGetAnnotation() {
AnnotationMirror serviceAnnotation = getAnnotation(testType,
Service.class);
assertEquals("3.0.0", getAttribute(serviceAnnotation, "version"));
assertEquals("test", getAttribute(serviceAnnotation, "group"));
@@ -90,7 +90,7 @@ public class AnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAnnotations() {
+ void testGetAnnotations() {
List<AnnotationMirror> annotations = getAnnotations(testType);
Iterator<AnnotationMirror> iterator = annotations.iterator();
@@ -129,7 +129,7 @@ public class AnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAllAnnotations() {
+ void testGetAllAnnotations() {
List<AnnotationMirror> annotations = getAllAnnotations(testType);
assertEquals(5, annotations.size());
@@ -172,7 +172,7 @@ public class AnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
@Test
- public void testFindAnnotation() {
+ void testFindAnnotation() {
assertEquals("org.apache.dubbo.config.annotation.Service",
findAnnotation(testType, Service.class).getAnnotationType().toString());
assertEquals("com.alibaba.dubbo.config.annotation.Service",
findAnnotation(testType,
com.alibaba.dubbo.config.annotation.Service.class).getAnnotationType().toString());
@@ -193,14 +193,14 @@ public class AnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testFindMetaAnnotation() {
+ void testFindMetaAnnotation() {
getAllDeclaredMethods(getType(TestService.class)).forEach(method -> {
assertEquals("javax.ws.rs.HttpMethod", findMetaAnnotation(method,
"javax.ws.rs.HttpMethod").getAnnotationType().toString());
});
}
@Test
- public void testGetAttribute() {
+ void testGetAttribute() {
assertEquals("org.apache.dubbo.metadata.tools.TestService",
getAttribute(findAnnotation(testType, Service.class), "interfaceName"));
assertEquals("org.apache.dubbo.metadata.tools.TestService",
getAttribute(findAnnotation(testType, Service.class).getElementValues(),
"interfaceName"));
assertEquals("/echo", getAttribute(findAnnotation(testType,
Path.class), "value"));
@@ -217,13 +217,13 @@ public class AnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetValue() {
+ void testGetValue() {
AnnotationMirror pathAnnotation =
getAnnotation(getType(TestService.class), Path.class);
assertEquals("/echo", getValue(pathAnnotation));
}
@Test
- public void testIsAnnotationPresent() {
+ void testIsAnnotationPresent() {
assertTrue(isAnnotationPresent(testType,
"org.apache.dubbo.config.annotation.Service"));
assertTrue(isAnnotationPresent(testType,
"com.alibaba.dubbo.config.annotation.Service"));
assertTrue(isAnnotationPresent(testType, "javax.ws.rs.Path"));
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/FieldUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/FieldUtilsTest.java
index 3afc647843..74b69cc72d 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/FieldUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/FieldUtilsTest.java
@@ -57,7 +57,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class FieldUtilsTest extends AbstractAnnotationProcessingTest {
+class FieldUtilsTest extends AbstractAnnotationProcessingTest {
private TypeElement testType;
@@ -71,7 +71,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetDeclaredFields() {
+ void testGetDeclaredFields() {
TypeElement type = getType(Model.class);
List<VariableElement> fields = getDeclaredFields(type);
assertModelFields(fields);
@@ -88,7 +88,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAllDeclaredFields() {
+ void testGetAllDeclaredFields() {
TypeElement type = getType(Model.class);
List<VariableElement> fields = getAllDeclaredFields(type);
@@ -104,7 +104,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetDeclaredField() {
+ void testGetDeclaredField() {
TypeElement type = getType(Model.class);
testGetDeclaredField(type, "f", float.class);
testGetDeclaredField(type, "d", double.class);
@@ -124,7 +124,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testFindField() {
+ void testFindField() {
TypeElement type = getType(Model.class);
testFindField(type, "f", float.class);
testFindField(type, "d", double.class);
@@ -149,7 +149,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsEnumField() {
+ void testIsEnumField() {
TypeElement type = getType(Color.class);
VariableElement field = findField(type, "RED");
assertTrue(isEnumMemberField(field));
@@ -168,7 +168,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsNonStaticField() {
+ void testIsNonStaticField() {
TypeElement type = getType(Model.class);
assertTrue(isNonStaticField(findField(type, "f")));
@@ -177,7 +177,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsField() {
+ void testIsField() {
TypeElement type = getType(Model.class);
assertTrue(isField(findField(type, "f")));
assertTrue(isField(findField(type, "f"), PRIVATE));
@@ -191,7 +191,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetNonStaticFields() {
+ void testGetNonStaticFields() {
TypeElement type = getType(Model.class);
List<VariableElement> fields = getNonStaticFields(type);
@@ -205,7 +205,7 @@ public class FieldUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAllNonStaticFields() {
+ void testGetAllNonStaticFields() {
TypeElement type = getType(Model.class);
List<VariableElement> fields = getAllNonStaticFields(type);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/LoggerUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/LoggerUtilsTest.java
index 4a06b216b1..284ccb4231 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/LoggerUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/LoggerUtilsTest.java
@@ -27,22 +27,22 @@ import static
org.junit.jupiter.api.Assertions.assertNotNull;
*
* @since 2.7.6
*/
-public class LoggerUtilsTest {
+class LoggerUtilsTest {
@Test
- public void testLogger() {
+ void testLogger() {
assertNotNull(LoggerUtils.LOGGER);
}
@Test
- public void testInfo() {
+ void testInfo() {
info("Hello,World");
info("Hello,%s", "World");
info("%s,%s", "Hello", "World");
}
@Test
- public void testWarn() {
+ void testWarn() {
warn("Hello,World");
warn("Hello,%s", "World");
warn("%s,%s", "Hello", "World");
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MemberUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MemberUtilsTest.java
index e119f00377..59a1b4545a 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MemberUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MemberUtilsTest.java
@@ -47,7 +47,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class MemberUtilsTest extends AbstractAnnotationProcessingTest {
+class MemberUtilsTest extends AbstractAnnotationProcessingTest {
private TypeElement testType;
@@ -61,13 +61,13 @@ public class MemberUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsPublicNonStatic() {
+ void testIsPublicNonStatic() {
assertFalse(isPublicNonStatic(null));
methodsIn(getDeclaredMembers(testType.asType())).forEach(method ->
assertTrue(isPublicNonStatic(method)));
}
@Test
- public void testHasModifiers() {
+ void testHasModifiers() {
assertFalse(hasModifiers(null));
List<? extends Element> members =
getAllDeclaredMembers(testType.asType());
List<VariableElement> fields = fieldsIn(members);
@@ -75,7 +75,7 @@ public class MemberUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testDeclaredMembers() {
+ void testDeclaredMembers() {
TypeElement type = getType(Model.class);
List<? extends Element> members = getDeclaredMembers(type.asType());
List<VariableElement> fields = fieldsIn(members);
@@ -105,7 +105,7 @@ public class MemberUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testMatchParameterTypes() {
+ void testMatchParameterTypes() {
ExecutableElement method = findMethod(testType, "echo",
"java.lang.String");
assertTrue(matchParameterTypes(method.getParameters(),
"java.lang.String"));
assertFalse(matchParameterTypes(method.getParameters(),
"java.lang.Object"));
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MethodUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MethodUtilsTest.java
index db36f40957..dca4090df5 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MethodUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/MethodUtilsTest.java
@@ -47,7 +47,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class MethodUtilsTest extends AbstractAnnotationProcessingTest {
+class MethodUtilsTest extends AbstractAnnotationProcessingTest {
private TypeElement testType;
@@ -61,7 +61,7 @@ public class MethodUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testDeclaredMethods() {
+ void testDeclaredMethods() {
TypeElement type = getType(Model.class);
List<ExecutableElement> methods = getDeclaredMethods(type);
assertEquals(12, methods.size());
@@ -79,13 +79,13 @@ public class MethodUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAllDeclaredMethods() {
+ void testGetAllDeclaredMethods() {
List<? extends ExecutableElement> methods = doGetAllDeclaredMethods();
assertEquals(14, methods.size());
}
@Test
- public void testGetPublicNonStaticMethods() {
+ void testGetPublicNonStaticMethods() {
List<? extends ExecutableElement> methods =
getPublicNonStaticMethods(testType, Object.class);
assertEquals(14, methods.size());
@@ -94,19 +94,19 @@ public class MethodUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsMethod() {
+ void testIsMethod() {
List<? extends ExecutableElement> methods =
getPublicNonStaticMethods(testType, Object.class);
assertEquals(14, methods.stream().map(MethodUtils::isMethod).count());
}
@Test
- public void testIsPublicNonStaticMethod() {
+ void testIsPublicNonStaticMethod() {
List<? extends ExecutableElement> methods =
getPublicNonStaticMethods(testType, Object.class);
assertEquals(14,
methods.stream().map(MethodUtils::isPublicNonStaticMethod).count());
}
@Test
- public void testFindMethod() {
+ void testFindMethod() {
TypeElement type = getType(Model.class);
// Test methods from java.lang.Object
// Object#toString()
@@ -161,7 +161,7 @@ public class MethodUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetOverrideMethod() {
+ void testGetOverrideMethod() {
List<? extends ExecutableElement> methods = doGetAllDeclaredMethods();
ExecutableElement overrideMethod = getOverrideMethod(processingEnv,
testType, methods.get(0));
@@ -174,21 +174,21 @@ public class MethodUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetMethodName() {
+ void testGetMethodName() {
ExecutableElement method = findMethod(testType, "echo",
"java.lang.String");
assertEquals("echo", getMethodName(method));
assertNull(getMethodName(null));
}
@Test
- public void testReturnType() {
+ void testReturnType() {
ExecutableElement method = findMethod(testType, "echo",
"java.lang.String");
assertEquals("java.lang.String", getReturnType(method));
assertNull(getReturnType(null));
}
@Test
- public void testMatchParameterTypes() {
+ void testMatchParameterTypes() {
ExecutableElement method = findMethod(testType, "echo",
"java.lang.String");
assertArrayEquals(new String[]{"java.lang.String"},
getMethodParameterTypes(method));
assertTrue(getMethodParameterTypes(null).length == 0);
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/ServiceAnnotationUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/ServiceAnnotationUtilsTest.java
index e0e95a45c9..1bca2bc027 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/ServiceAnnotationUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/ServiceAnnotationUtilsTest.java
@@ -52,7 +52,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest {
+class ServiceAnnotationUtilsTest extends AbstractAnnotationProcessingTest {
@Override
protected void addCompiledClasses(Set<Class<?>> classesToBeCompiled) {
@@ -65,7 +65,7 @@ public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest
}
@Test
- public void testConstants() {
+ void testConstants() {
assertEquals("org.apache.dubbo.config.annotation.DubboService",
DUBBO_SERVICE_ANNOTATION_TYPE);
assertEquals("org.apache.dubbo.config.annotation.Service",
SERVICE_ANNOTATION_TYPE);
assertEquals("com.alibaba.dubbo.config.annotation.Service",
LEGACY_SERVICE_ANNOTATION_TYPE);
@@ -77,7 +77,7 @@ public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest
}
@Test
- public void testIsServiceAnnotationPresent() {
+ void testIsServiceAnnotationPresent() {
assertTrue(isServiceAnnotationPresent(getType(TestServiceImpl.class)));
assertTrue(isServiceAnnotationPresent(getType(GenericTestService.class)));
@@ -87,7 +87,7 @@ public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest
}
@Test
- public void testGetAnnotation() {
+ void testGetAnnotation() {
TypeElement type = getType(TestServiceImpl.class);
assertEquals("org.apache.dubbo.config.annotation.Service",
getAnnotation(type).getAnnotationType().toString());
@@ -101,7 +101,7 @@ public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest
}
@Test
- public void testResolveServiceInterfaceName() {
+ void testResolveServiceInterfaceName() {
TypeElement type = getType(TestServiceImpl.class);
assertEquals("org.apache.dubbo.metadata.tools.TestService",
resolveServiceInterfaceName(type, getAnnotation(type)));
@@ -113,7 +113,7 @@ public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest
}
@Test
- public void testGetVersion() {
+ void testGetVersion() {
TypeElement type = getType(TestServiceImpl.class);
assertEquals("3.0.0", getVersion(getAnnotation(type)));
@@ -125,7 +125,7 @@ public class ServiceAnnotationUtilsTest extends
AbstractAnnotationProcessingTest
}
@Test
- public void testGetGroup() {
+ void testGetGroup() {
TypeElement type = getType(TestServiceImpl.class);
assertEquals("test", getGroup(getAnnotation(type)));
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/TypeUtilsTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/TypeUtilsTest.java
index fc5995b7f3..04e86e77ff 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/TypeUtilsTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/annotation/processing/util/TypeUtilsTest.java
@@ -82,7 +82,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
*
* @since 2.7.6
*/
-public class TypeUtilsTest extends AbstractAnnotationProcessingTest {
+class TypeUtilsTest extends AbstractAnnotationProcessingTest {
private TypeElement testType;
@@ -98,7 +98,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsSimpleType() {
+ void testIsSimpleType() {
assertTrue(isSimpleType(getType(Void.class)));
assertTrue(isSimpleType(getType(Boolean.class)));
@@ -121,7 +121,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsSameType() {
+ void testIsSameType() {
assertTrue(isSameType(getType(Void.class).asType(), "java.lang.Void"));
assertFalse(isSameType(getType(String.class).asType(),
"java.lang.Void"));
@@ -133,7 +133,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsArrayType() {
+ void testIsArrayType() {
TypeElement type = getType(ArrayTypeModel.class);
assertTrue(isArrayType(findField(type.asType(), "integers").asType()));
assertTrue(isArrayType(findField(type.asType(), "strings").asType()));
@@ -146,7 +146,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsEnumType() {
+ void testIsEnumType() {
TypeElement type = getType(Color.class);
assertTrue(isEnumType(type.asType()));
@@ -158,7 +158,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsClassType() {
+ void testIsClassType() {
TypeElement type = getType(ArrayTypeModel.class);
assertTrue(isClassType(type.asType()));
@@ -170,7 +170,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsPrimitiveType() {
+ void testIsPrimitiveType() {
TypeElement type = getType(PrimitiveTypeModel.class);
getDeclaredFields(type.asType())
.stream()
@@ -184,7 +184,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsInterfaceType() {
+ void testIsInterfaceType() {
TypeElement type = getType(CharSequence.class);
assertTrue(isInterfaceType(type));
assertTrue(isInterfaceType(type.asType()));
@@ -198,7 +198,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsAnnotationType() {
+ void testIsAnnotationType() {
TypeElement type = getType(Override.class);
assertTrue(isAnnotationType(type));
@@ -213,7 +213,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetHierarchicalTypes() {
+ void testGetHierarchicalTypes() {
Set hierarchicalTypes = getHierarchicalTypes(testType.asType(), true,
true, true);
Iterator iterator = hierarchicalTypes.iterator();
assertEquals(8, hierarchicalTypes.size());
@@ -288,7 +288,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
@Test
- public void testGetInterfaces() {
+ void testGetInterfaces() {
TypeElement type = getType(Model.class);
List<TypeMirror> interfaces = getInterfaces(type);
assertTrue(interfaces.isEmpty());
@@ -305,7 +305,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAllInterfaces() {
+ void testGetAllInterfaces() {
Set<? extends TypeMirror> interfaces =
getAllInterfaces(testType.asType());
assertEquals(4, interfaces.size());
Iterator<? extends TypeMirror> iterator = interfaces.iterator();
@@ -328,7 +328,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetType() {
+ void testGetType() {
TypeElement element = TypeUtils.getType(processingEnv, String.class);
assertEquals(element, TypeUtils.getType(processingEnv,
element.asType()));
assertEquals(element, TypeUtils.getType(processingEnv,
"java.lang.String"));
@@ -340,7 +340,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetSuperType() {
+ void testGetSuperType() {
TypeElement gtsTypeElement = getSuperType(testType);
assertEquals(gtsTypeElement, getType(GenericTestService.class));
TypeElement dtsTypeElement = getSuperType(gtsTypeElement);
@@ -356,7 +356,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetAllSuperTypes() {
+ void testGetAllSuperTypes() {
Set<?> allSuperTypes = getAllSuperTypes(testType);
Iterator<?> iterator = allSuperTypes.iterator();
assertEquals(3, allSuperTypes.size());
@@ -376,7 +376,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsDeclaredType() {
+ void testIsDeclaredType() {
assertTrue(isDeclaredType(testType));
assertTrue(isDeclaredType(testType.asType()));
assertFalse(isDeclaredType((Element) null));
@@ -387,7 +387,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testOfDeclaredType() {
+ void testOfDeclaredType() {
assertEquals(testType.asType(), ofDeclaredType(testType));
assertEquals(testType.asType(), ofDeclaredType(testType.asType()));
assertEquals(ofDeclaredType(testType),
ofDeclaredType(testType.asType()));
@@ -397,7 +397,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testIsTypeElement() {
+ void testIsTypeElement() {
assertTrue(isTypeElement(testType));
assertTrue(isTypeElement(testType.asType()));
@@ -406,7 +406,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testOfTypeElement() {
+ void testOfTypeElement() {
assertEquals(testType, ofTypeElement(testType));
assertEquals(testType, ofTypeElement(testType.asType()));
@@ -415,7 +415,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testOfDeclaredTypes() {
+ void testOfDeclaredTypes() {
Set<DeclaredType> declaredTypes =
ofDeclaredTypes(asList(getType(String.class), getType(TestServiceImpl.class),
getType(Color.class)));
assertTrue(declaredTypes.contains(getType(String.class).asType()));
assertTrue(declaredTypes.contains(getType(TestServiceImpl.class).asType()));
@@ -425,7 +425,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testListDeclaredTypes() {
+ void testListDeclaredTypes() {
List<DeclaredType> types = listDeclaredTypes(asList(testType,
testType, testType));
assertEquals(1, types.size());
assertEquals(ofDeclaredType(testType), types.get(0));
@@ -435,7 +435,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testListTypeElements() {
+ void testListTypeElements() {
List<TypeElement> typeElements =
listTypeElements(asList(testType.asType(), ofDeclaredType(testType)));
assertEquals(1, typeElements.size());
assertEquals(testType, typeElements.get(0));
@@ -463,7 +463,7 @@ public class TypeUtilsTest extends
AbstractAnnotationProcessingTest {
}
@Test
- public void testGetResourceName() {
+ void testGetResourceName() {
assertEquals("java/lang/String.class",
getResourceName("java.lang.String"));
assertNull(getResourceName(null));
}
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/CompilerTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/CompilerTest.java
index e633e0fba8..071e1b26c8 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/CompilerTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/CompilerTest.java
@@ -23,10 +23,10 @@ import java.io.IOException;
/**
* The Compiler test case
*/
-public class CompilerTest {
+class CompilerTest {
@Test
- public void testCompile() throws IOException {
+ void testCompile() throws IOException {
Compiler compiler = new Compiler();
compiler.compile(
TestServiceImpl.class,
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/DefaultRestServiceTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/DefaultRestServiceTest.java
index 6921bbd96a..d7b73c4abc 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/DefaultRestServiceTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/DefaultRestServiceTest.java
@@ -31,10 +31,10 @@ import java.io.IOException;
*
* @since 2.7.6
*/
-public class DefaultRestServiceTest {
+class DefaultRestServiceTest {
@Test
- public void test() throws IOException {
+ void test() throws IOException {
Compiler compiler = new Compiler();
compiler.compile(User.class,
RestService.class,
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/RestServiceTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/RestServiceTest.java
index fcb444e251..5f0ed6e093 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/RestServiceTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/RestServiceTest.java
@@ -30,10 +30,10 @@ import java.io.IOException;
*
* @since 2.7.6
*/
-public class RestServiceTest {
+class RestServiceTest {
@Test
- public void test() throws IOException {
+ void test() throws IOException {
Compiler compiler = new Compiler();
compiler.compile(User.class, RestService.class,
StandardRestService.class,
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/SpringRestServiceTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/SpringRestServiceTest.java
index f2ebe1d3dd..ce17abd592 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/SpringRestServiceTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/SpringRestServiceTest.java
@@ -29,10 +29,10 @@ import java.io.IOException;
*
* @since 2.7.6
*/
-public class SpringRestServiceTest {
+class SpringRestServiceTest {
@Test
- public void test() throws IOException {
+ void test() throws IOException {
Compiler compiler = new Compiler();
compiler.compile(User.class, RestService.class,
SpringRestService.class);
}
diff --git
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/StandardRestServiceTest.java
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/StandardRestServiceTest.java
index bba4d54d2d..e55fdf3da4 100644
---
a/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/StandardRestServiceTest.java
+++
b/dubbo-metadata/dubbo-metadata-processor/src/test/java/org/apache/dubbo/metadata/tools/StandardRestServiceTest.java
@@ -29,10 +29,10 @@ import java.io.IOException;
*
* @since 2.7.6
*/
-public class StandardRestServiceTest {
+class StandardRestServiceTest {
@Test
- public void test() throws IOException {
+ void test() throws IOException {
Compiler compiler = new Compiler();
compiler.compile(User.class, RestService.class,
StandardRestService.class);
}
diff --git
a/dubbo-metadata/dubbo-metadata-report-redis/src/test/java/org/apache/dubbo/metadata/store/redis/RedisMetadataReportTest.java
b/dubbo-metadata/dubbo-metadata-report-redis/src/test/java/org/apache/dubbo/metadata/store/redis/RedisMetadataReportTest.java
index 445363cb79..c9b4e125de 100644
---
a/dubbo-metadata/dubbo-metadata-report-redis/src/test/java/org/apache/dubbo/metadata/store/redis/RedisMetadataReportTest.java
+++
b/dubbo-metadata/dubbo-metadata-report-redis/src/test/java/org/apache/dubbo/metadata/store/redis/RedisMetadataReportTest.java
@@ -46,7 +46,7 @@ import static
org.apache.dubbo.common.constants.CommonConstants.PROVIDER_SIDE;
import static
org.apache.dubbo.common.constants.CommonConstants.SYNC_REPORT_KEY;
import static redis.embedded.RedisServer.newRedisServer;
-public class RedisMetadataReportTest {
+class RedisMetadataReportTest {
private static final String
REDIS_URL_TEMPLATE = "redis://%slocalhost:%d",
@@ -107,12 +107,12 @@ public class RedisMetadataReportTest {
}
@Test
- public void testAsyncStoreProvider() throws ClassNotFoundException {
+ void testAsyncStoreProvider() throws ClassNotFoundException {
testStoreProvider(redisMetadataReport, "1.0.0.redis.md.p1", 3000);
}
@Test
- public void testSyncStoreProvider() throws ClassNotFoundException {
+ void testSyncStoreProvider() throws ClassNotFoundException {
testStoreProvider(syncRedisMetadataReport, "1.0.0.redis.md.p2", 3);
}
@@ -146,12 +146,12 @@ public class RedisMetadataReportTest {
}
@Test
- public void testAsyncStoreConsumer() throws ClassNotFoundException {
+ void testAsyncStoreConsumer() throws ClassNotFoundException {
testStoreConsumer(redisMetadataReport, "1.0.0.redis.md.c1", 3000);
}
@Test
- public void testSyncStoreConsumer() throws ClassNotFoundException {
+ void testSyncStoreConsumer() throws ClassNotFoundException {
testStoreConsumer(syncRedisMetadataReport, "1.0.0.redis.md.c2", 3);
}
@@ -216,12 +216,12 @@ public class RedisMetadataReportTest {
}
@Test
- public void testAuthRedisMetadata() throws ClassNotFoundException {
+ void testAuthRedisMetadata() throws ClassNotFoundException {
testStoreProvider(redisMetadataReport, "1.0.0.redis.md.p1", 3000);
}
@Test
- public void testWrongAuthRedisMetadata() throws ClassNotFoundException {
+ void testWrongAuthRedisMetadata() throws ClassNotFoundException {
registryUrl = registryUrl.setPassword("123456");
redisMetadataReport = (RedisMetadataReport) new
RedisMetadataReportFactory().createMetadataReport(registryUrl);
try {
diff --git
a/dubbo-metadata/dubbo-metadata-report-zookeeper/src/test/java/org/apache/dubbo/metadata/store/zookeeper/ZookeeperMetadataReportTest.java
b/dubbo-metadata/dubbo-metadata-report-zookeeper/src/test/java/org/apache/dubbo/metadata/store/zookeeper/ZookeeperMetadataReportTest.java
index c585db9083..183185514b 100644
---
a/dubbo-metadata/dubbo-metadata-report-zookeeper/src/test/java/org/apache/dubbo/metadata/store/zookeeper/ZookeeperMetadataReportTest.java
+++
b/dubbo-metadata/dubbo-metadata-report-zookeeper/src/test/java/org/apache/dubbo/metadata/store/zookeeper/ZookeeperMetadataReportTest.java
@@ -53,7 +53,7 @@ import static
org.apache.dubbo.metadata.ServiceNameMapping.DEFAULT_MAPPING_GROUP
/**
* 2018/10/9
*/
-public class ZookeeperMetadataReportTest {
+class ZookeeperMetadataReportTest {
private ZookeeperMetadataReport zookeeperMetadataReport;
private URL registryUrl;
private ZookeeperMetadataReportFactory zookeeperMetadataReportFactory;
@@ -78,7 +78,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testStoreProvider() throws ClassNotFoundException,
InterruptedException {
+ void testStoreProvider() throws ClassNotFoundException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0.zk.md";
String group = null;
@@ -106,7 +106,7 @@ public class ZookeeperMetadataReportTest {
@Test
- public void testConsumer() throws ClassNotFoundException,
InterruptedException {
+ void testConsumer() throws ClassNotFoundException, InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0.zk.md";
String group = null;
@@ -130,7 +130,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testDoSaveMetadata() throws ExecutionException,
InterruptedException {
+ void testDoSaveMetadata() throws ExecutionException, InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0";
String group = null;
@@ -149,7 +149,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testDoRemoveMetadata() throws ExecutionException,
InterruptedException {
+ void testDoRemoveMetadata() throws ExecutionException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0";
String group = null;
@@ -172,7 +172,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testDoGetExportedURLs() throws ExecutionException,
InterruptedException {
+ void testDoGetExportedURLs() throws ExecutionException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0";
String group = null;
@@ -194,7 +194,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testDoSaveSubscriberData() throws ExecutionException,
InterruptedException {
+ void testDoSaveSubscriberData() throws ExecutionException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0";
String group = null;
@@ -214,7 +214,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testDoGetSubscribedURLs() throws ExecutionException,
InterruptedException {
+ void testDoGetSubscribedURLs() throws ExecutionException,
InterruptedException {
String interfaceName =
"org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReport4TstService";
String version = "1.0.0";
String group = null;
@@ -279,7 +279,7 @@ public class ZookeeperMetadataReportTest {
@Test
- public void testMapping() throws InterruptedException {
+ void testMapping() throws InterruptedException {
String serviceKey = ZookeeperMetadataReportTest.class.getName();
URL url = URL.valueOf("test://127.0.0.1:8888/" + serviceKey);
String appNames = "demo1,demo2";
@@ -308,7 +308,7 @@ public class ZookeeperMetadataReportTest {
}
@Test
- public void testAppMetadata() {
+ void testAppMetadata() {
String serviceKey = ZookeeperMetadataReportTest.class.getName();
String appName = "demo";
URL url = URL.valueOf("test://127.0.0.1:8888/" + serviceKey);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/ListenerRegistryWrapperTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/ListenerRegistryWrapperTest.java
index 1d9c21455d..363adec7fa 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/ListenerRegistryWrapperTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/ListenerRegistryWrapperTest.java
@@ -36,10 +36,10 @@ import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
-public class ListenerRegistryWrapperTest {
+class ListenerRegistryWrapperTest {
@Test
- public void testSubscribe() {
+ void testSubscribe() {
Map<String, String> parameters = new HashMap<>();
parameters.put(INTERFACE_KEY, DemoService.class.getName());
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/PerformanceRegistryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/PerformanceRegistryTest.java
index ce8b8643b5..a4dca95b0c 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/PerformanceRegistryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/PerformanceRegistryTest.java
@@ -28,12 +28,12 @@ import org.junit.jupiter.api.Test;
* RegistryPerformanceTest
*
*/
-public class PerformanceRegistryTest {
+class PerformanceRegistryTest {
private static final Logger logger =
LoggerFactory.getLogger(PerformanceRegistryTest.class);
@Test
- public void testRegistry() {
+ void testRegistry() {
// read server info from property
if (PerformanceUtils.getProperty("server", null) == null) {
logger.warn("Please set -Dserver=127.0.0.1:9090");
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/RegistryFactoryWrapperTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/RegistryFactoryWrapperTest.java
index cc797438b4..c87b31c750 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/RegistryFactoryWrapperTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/RegistryFactoryWrapperTest.java
@@ -23,11 +23,11 @@ import org.apache.dubbo.common.extension.ExtensionLoader;
import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
-public class RegistryFactoryWrapperTest {
+class RegistryFactoryWrapperTest {
private RegistryFactory registryFactory =
ExtensionLoader.getExtensionLoader(RegistryFactory.class).getAdaptiveExtension();
@Test
- public void test() throws Exception {
+ void test() throws Exception {
RegistryServiceListener listener1 =
Mockito.mock(RegistryServiceListener.class);
RegistryServiceListener1.delegate = listener1;
RegistryServiceListener listener2 =
Mockito.mock(RegistryServiceListener.class);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/AbstractServiceDiscoveryFactoryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/AbstractServiceDiscoveryFactoryTest.java
index c15f086da0..65b39fc83a 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/AbstractServiceDiscoveryFactoryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/AbstractServiceDiscoveryFactoryTest.java
@@ -28,10 +28,10 @@ import java.util.List;
/**
* {@link AbstractServiceDiscoveryFactory}
*/
-public class AbstractServiceDiscoveryFactoryTest {
+class AbstractServiceDiscoveryFactoryTest {
@Test
- public void testGetServiceDiscoveryWithCache() {
+ void testGetServiceDiscoveryWithCache() {
ApplicationModel.defaultModel().getApplicationConfigManager().setApplication(new
ApplicationConfig("AbstractServiceDiscoveryFactoryTest"));
URL url = URL.valueOf("mock://127.0.0.1:8888");
ServiceDiscoveryFactory factory =
ServiceDiscoveryFactory.getExtension(url);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/DefaultServiceInstanceTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/DefaultServiceInstanceTest.java
index ea33a98477..11c618d651 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/DefaultServiceInstanceTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/DefaultServiceInstanceTest.java
@@ -37,7 +37,7 @@ import static org.junit.jupiter.api.Assertions.assertNotSame;
*
* @since 2.7.5
*/
-public class DefaultServiceInstanceTest {
+class DefaultServiceInstanceTest {
public DefaultServiceInstance instance;
@@ -61,7 +61,7 @@ public class DefaultServiceInstanceTest {
}
@Test
- public void testSetAndGetValues() {
+ void testSetAndGetValues() {
instance.setEnabled(false);
instance.setHealthy(false);
@@ -74,7 +74,7 @@ public class DefaultServiceInstanceTest {
}
@Test
- public void testInstanceOperations() {
+ void testInstanceOperations() {
// test multiple protocols
assertEquals(2, instance.getEndpoints().size());
DefaultServiceInstance.Endpoint endpoint = getEndpoint(instance,
"rest");
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/InstanceAddressURLTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/InstanceAddressURLTest.java
index 15442b29fd..1667dba8e0 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/InstanceAddressURLTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/InstanceAddressURLTest.java
@@ -50,7 +50,7 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNull;
import static org.junit.jupiter.api.Assertions.assertTrue;
-public class InstanceAddressURLTest {
+class InstanceAddressURLTest {
private static URL url =
URL.valueOf("dubbo://30.225.21.30:20880/org.apache.dubbo.registry.service.DemoService?"
+
"REGISTRY_CLUSTER=registry1&anyhost=true&application=demo-provider2&delay=1000&deprecated=false&dubbo=2.0.2"
+
"&dynamic=true&generic=false&group=greeting&interface=org.apache.dubbo.registry.service.DemoService"
+
@@ -113,7 +113,7 @@ public class InstanceAddressURLTest {
}
@Test
- public void test1() {
+ void test1() {
// test reading of keys in instance and metadata work fine
assertEquals("value1", instanceURL.getParameter("key1"));//return
instance key
assertNull(instanceURL.getParameter("delay"));// no service key
specified
@@ -169,7 +169,7 @@ public class InstanceAddressURLTest {
}
@Test
- public void test2() {
+ void test2() {
RpcServiceContext.getServiceContext().setConsumerUrl(null);
Assertions.assertNull(instanceURL.getScopeModel());
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/ServiceDiscoveryRegistryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/ServiceDiscoveryRegistryTest.java
index 40546c73c2..b54f1590fd 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/ServiceDiscoveryRegistryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/ServiceDiscoveryRegistryTest.java
@@ -60,7 +60,7 @@ import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
-public class ServiceDiscoveryRegistryTest {
+class ServiceDiscoveryRegistryTest {
public static final String APP_NAME1 = "app1";
public static final String APP_NAME2 = "app2";
public static final String APP_NAME3 = "app3";
@@ -121,7 +121,7 @@ public class ServiceDiscoveryRegistryTest {
* - check=false
*/
@Test
- public void testDoSubscribe() {
+ void testDoSubscribe() {
ApplicationModel applicationModel =
spy(ApplicationModel.defaultModel());
when(applicationModel.getDefaultExtension(ServiceNameMapping.class)).thenReturn(mapping);
// Exceptional case, no interface-app mapping found
@@ -170,7 +170,7 @@ public class ServiceDiscoveryRegistryTest {
* - instance listener and service listener rightly mapped
*/
@Test
- public void testSubscribeURLs() {
+ void testSubscribeURLs() {
// interface to single app mapping
Set<String> singleApp = new TreeSet<>();
singleApp.add(APP_NAME1);
@@ -230,12 +230,12 @@ public class ServiceDiscoveryRegistryTest {
* repeat of {@link this#testSubscribeURLs()} with multi threads
*/
@Test
- public void testConcurrencySubscribe() {
+ void testConcurrencySubscribe() {
// TODO
}
@Test
- public void testUnsubscribe() {
+ void testUnsubscribe() {
// do subscribe to prepare for unsubscribe verification
Set<String> multiApps = new TreeSet<>();
multiApps.add(APP_NAME1);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/event/listener/ServiceInstancesChangedListenerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/event/listener/ServiceInstancesChangedListenerTest.java
index 7bab0b8554..9a0b676cc8 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/event/listener/ServiceInstancesChangedListenerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/event/listener/ServiceInstancesChangedListenerTest.java
@@ -73,7 +73,7 @@ import static org.mockito.Mockito.when;
* @since 2.7.5
*/
@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
-public class ServiceInstancesChangedListenerTest {
+class ServiceInstancesChangedListenerTest {
static List<ServiceInstance> app1Instances;
static List<ServiceInstance> app2Instances;
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceNameMappingTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceNameMappingTest.java
index 3f096fd930..36bd64a49e 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceNameMappingTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceNameMappingTest.java
@@ -48,7 +48,7 @@ import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.when;
-public class MetadataServiceNameMappingTest {
+class MetadataServiceNameMappingTest {
private MetadataServiceNameMapping mapping;
private URL url;
@@ -74,7 +74,7 @@ public class MetadataServiceNameMappingTest {
}
@Test
- public void testMap() {
+ void testMap() {
ApplicationModel mockedApplicationModel = spy(applicationModel);
when(configManager.getMetadataConfigs()).thenReturn(Collections.emptyList());
@@ -123,7 +123,7 @@ public class MetadataServiceNameMappingTest {
* This test currently doesn't make any sense
*/
@Test
- public void testGet() {
+ void testGet() {
Set<String> set = new HashSet<>();
set.add("app1");
@@ -140,7 +140,7 @@ public class MetadataServiceNameMappingTest {
* Same situation as testGet, so left empty.
*/
@Test
- public void testGetAndListen() {
+ void testGetAndListen() {
// TODO
}
}
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceURLBuilderTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceURLBuilderTest.java
index 1dd2d98eb7..4a59066288 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceURLBuilderTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/MetadataServiceURLBuilderTest.java
@@ -25,7 +25,7 @@ import org.apache.dubbo.rpc.model.ApplicationModel;
*
* @since 2.7.5
*/
-public class MetadataServiceURLBuilderTest {
+class MetadataServiceURLBuilderTest {
static ServiceInstance serviceInstance = new
DefaultServiceInstance("test", "127.0.0.1", 8080,
ApplicationModel.defaultModel());
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ProtocolPortsMetadataCustomizerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ProtocolPortsMetadataCustomizerTest.java
index f0ba7cb82f..d5bf6ba89e 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ProtocolPortsMetadataCustomizerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ProtocolPortsMetadataCustomizerTest.java
@@ -41,7 +41,7 @@ import static org.hamcrest.Matchers.hasProperty;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertNotNull;
-public class ProtocolPortsMetadataCustomizerTest {
+class ProtocolPortsMetadataCustomizerTest {
public DefaultServiceInstance instance;
private MetadataService mockedMetadataService;
@@ -80,7 +80,7 @@ public class ProtocolPortsMetadataCustomizerTest {
}
@Test
- public void test() {
+ void test() {
ProtocolPortsMetadataCustomizer customizer = new
ProtocolPortsMetadataCustomizer();
customizer.customize(instance, ApplicationModel.defaultModel());
String endpoints = instance.getMetadata().get(ENDPOINTS);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ServiceInstanceMetadataCustomizerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ServiceInstanceMetadataCustomizerTest.java
index 01d77c06a0..81362c384f 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ServiceInstanceMetadataCustomizerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/ServiceInstanceMetadataCustomizerTest.java
@@ -31,7 +31,7 @@ import static
org.apache.dubbo.common.constants.CommonConstants.SIDE_KEY;
import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.spy;
-public class ServiceInstanceMetadataCustomizerTest {
+class ServiceInstanceMetadataCustomizerTest {
private static ServiceInstanceMetadataCustomizer
serviceInstanceMetadataCustomizer;
@BeforeAll
@@ -49,7 +49,7 @@ public class ServiceInstanceMetadataCustomizerTest {
* Only 'include' policy spicified in Customized Filter will take effect
*/
@Test
- public void testCustomizeWithIncludeFilters() {
+ void testCustomizeWithIncludeFilters() {
ApplicationModel applicationModel =
spy(ApplicationModel.defaultModel());
ApplicationConfig applicationConfig = new ApplicationConfig("aa");
doReturn(applicationConfig).when(applicationModel).getCurrentConfig();
@@ -69,7 +69,7 @@ public class ServiceInstanceMetadataCustomizerTest {
* Only 'exclude' policies specified in Exclude Filters will take effect
*/
@Test
- public void testCustomizeWithExcludeFilters() {
+ void testCustomizeWithExcludeFilters() {
ApplicationModel applicationModel =
spy(ApplicationModel.defaultModel());
ApplicationConfig applicationConfig = new ApplicationConfig("aa");
doReturn(applicationConfig).when(applicationModel).getCurrentConfig();
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/SpringCloudMetadataServiceURLBuilderTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/SpringCloudMetadataServiceURLBuilderTest.java
index 9f0395db02..91aba6b4cf 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/SpringCloudMetadataServiceURLBuilderTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/SpringCloudMetadataServiceURLBuilderTest.java
@@ -32,12 +32,12 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
*
* @since 2.7.5
*/
-public class SpringCloudMetadataServiceURLBuilderTest {
+class SpringCloudMetadataServiceURLBuilderTest {
private SpringCloudMetadataServiceURLBuilder builder = new
SpringCloudMetadataServiceURLBuilder();
@Test
- public void testBuild() {
+ void testBuild() {
List<URL> urls = builder.build(new DefaultServiceInstance("127.0.0.1",
"test", 8080, ApplicationModel.defaultModel()));
assertEquals(0, urls.size());
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/StandardMetadataServiceURLBuilderTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/StandardMetadataServiceURLBuilderTest.java
index 91350da89e..d6bd958d1f 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/StandardMetadataServiceURLBuilderTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/StandardMetadataServiceURLBuilderTest.java
@@ -35,7 +35,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
/**
* {@link StandardMetadataServiceURLBuilder} Test
*/
-public class StandardMetadataServiceURLBuilderTest {
+class StandardMetadataServiceURLBuilderTest {
@BeforeAll
public static void setUp() {
@@ -50,7 +50,7 @@ public class StandardMetadataServiceURLBuilderTest {
}
@Test
- public void testBuild() {
+ void testBuild() {
ExtensionLoader<MetadataServiceURLBuilder> loader =
ApplicationModel.defaultModel()
.getExtensionLoader(MetadataServiceURLBuilder.class);
MetadataServiceURLBuilder builder =
loader.getExtension(StandardMetadataServiceURLBuilder.NAME);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/store/MetaCacheManagerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/store/MetaCacheManagerTest.java
index daa5ff4264..f8dc8ca82a 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/store/MetaCacheManagerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/metadata/store/MetaCacheManagerTest.java
@@ -34,7 +34,7 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.junit.jupiter.api.Assertions.assertNull;
import static org.junit.jupiter.api.Assertions.fail;
-public class MetaCacheManagerTest {
+class MetaCacheManagerTest {
@BeforeEach
public void setup() throws URISyntaxException {
@@ -50,7 +50,7 @@ public class MetaCacheManagerTest {
}
@Test
- public void testCache() {
+ void testCache() {
// ScheduledExecutorService cacheRefreshExecutor =
Executors.newSingleThreadScheduledExecutor(new
NamedThreadFactory("Dubbo-cache-refresh"));
// ExecutorRepository executorRepository =
Mockito.mock(ExecutorRepository.class);
//
when(executorRepository.getCacheRefreshExecutor()).thenReturn(cacheRefreshExecutor);
@@ -85,7 +85,7 @@ public class MetaCacheManagerTest {
@Test
- public void testCacheDump() {
+ void testCacheDump() {
System.setProperty("dubbo.meta.cache.fileName",
"not-exist.dubbo.cache");
MetadataInfo metadataInfo3 =
JsonUtils.getJson().toJavaObject("{\"app\":\"demo3\",\"services\":{\"greeting/org.apache.dubbo.registry.service.DemoService2:1.0.0:dubbo\":{\"name\":\"org.apache.dubbo.registry.service.DemoService2\",\"group\":\"greeting\",\"version\":\"1.0.0\",\"protocol\":\"dubbo\",\"path\":\"org.apache.dubbo.registry.service.DemoService2\",\"params\":{\"application\":\"demo-provider2\",\"sayHello.timeout\":\"7000\",\"version\":\"1.0.0\",\"timeout\":\"5000\",\"group
[...]
MetaCacheManager cacheManager = new MetaCacheManager();
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/DefaultMigrationAddressComparatorTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/DefaultMigrationAddressComparatorTest.java
index 0314ff7ed2..b6c43baa2c 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/DefaultMigrationAddressComparatorTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/DefaultMigrationAddressComparatorTest.java
@@ -33,10 +33,10 @@ import java.util.List;
import static
org.apache.dubbo.registry.client.migration.DefaultMigrationAddressComparator.NEW_ADDRESS_SIZE;
import static
org.apache.dubbo.registry.client.migration.DefaultMigrationAddressComparator.OLD_ADDRESS_SIZE;
-public class DefaultMigrationAddressComparatorTest {
+class DefaultMigrationAddressComparatorTest {
@Test
- public void test() {
+ void test() {
DefaultMigrationAddressComparator comparator = new
DefaultMigrationAddressComparator();
ClusterInvoker newInvoker = Mockito.mock(ClusterInvoker.class);
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationInvokerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationInvokerTest.java
index dee3ca5e2d..5bed016013 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationInvokerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationInvokerTest.java
@@ -39,7 +39,7 @@ import org.mockito.Mockito;
import java.util.LinkedList;
import java.util.List;
-public class MigrationInvokerTest {
+class MigrationInvokerTest {
@BeforeEach
public void before() {
FrameworkModel.destroyAll();
@@ -55,7 +55,7 @@ public class MigrationInvokerTest {
}
@Test
- public void test() {
+ void test() {
RegistryProtocol registryProtocol =
Mockito.mock(RegistryProtocol.class);
ClusterInvoker invoker = Mockito.mock(ClusterInvoker.class);
@@ -226,7 +226,7 @@ public class MigrationInvokerTest {
}
@Test
- public void testDecide() {
+ void testDecide() {
RegistryProtocol registryProtocol =
Mockito.mock(RegistryProtocol.class);
ClusterInvoker invoker = Mockito.mock(ClusterInvoker.class);
@@ -286,7 +286,7 @@ public class MigrationInvokerTest {
}
@Test
- public void testConcurrency() {
+ void testConcurrency() {
// 独立线程
// 独立线程invoker状态切换
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleHandlerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleHandlerTest.java
index b5ae5dd8c1..a98a4f4385 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleHandlerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleHandlerTest.java
@@ -25,9 +25,9 @@ import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
-public class MigrationRuleHandlerTest {
+class MigrationRuleHandlerTest {
@Test
- public void test() {
+ void test() {
MigrationClusterInvoker invoker =
Mockito.mock(MigrationClusterInvoker.class);
URL url = Mockito.mock(URL.class);
Mockito.when(url.getDisplayServiceKey()).thenReturn("test");
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleListenerTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleListenerTest.java
index 72a0775fc1..9a31e66ea4 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleListenerTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/MigrationRuleListenerTest.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
import org.mockito.ArgumentCaptor;
import org.mockito.Mockito;
-public class MigrationRuleListenerTest {
+class MigrationRuleListenerTest {
private String localRule = "key: demo-consumer\n" +
"step: APPLICATION_FIRST\n" +
@@ -75,7 +75,7 @@ public class MigrationRuleListenerTest {
* Check local rule take effect
*/
@Test
- public void test() throws InterruptedException {
+ void test() throws InterruptedException {
DynamicConfiguration dynamicConfiguration =
Mockito.mock(DynamicConfiguration.class);
ApplicationModel.reset();
@@ -110,7 +110,7 @@ public class MigrationRuleListenerTest {
* Test listener started without local rule and config center, INIT should
be used and no scheduled task should be started.
*/
@Test
- public void testWithInitAndNoLocalRule() {
+ void testWithInitAndNoLocalRule() {
ApplicationModel.defaultModel().getDefaultModule().getModelEnvironment().setDynamicConfiguration(null);
ApplicationModel.defaultModel().getDefaultModule().getModelEnvironment().setLocalMigrationRule("");
ApplicationConfig applicationConfig = new ApplicationConfig();
@@ -145,7 +145,7 @@ public class MigrationRuleListenerTest {
* 2. remote rule change and all invokers gets notified
*/
@Test
- public void testWithConfigurationListenerAndLocalRule() throws
InterruptedException {
+ void testWithConfigurationListenerAndLocalRule() throws
InterruptedException {
DynamicConfiguration dynamicConfiguration =
Mockito.mock(DynamicConfiguration.class);
Mockito.doReturn(remoteRule).when(dynamicConfiguration).getConfig(Mockito.anyString(),
Mockito.anyString());
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/model/MigrationRuleTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/model/MigrationRuleTest.java
index 412c597589..82ae140f03 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/model/MigrationRuleTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/client/migration/model/MigrationRuleTest.java
@@ -35,11 +35,11 @@ import static org.mockito.ArgumentMatchers.anyString;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
-public class MigrationRuleTest {
+class MigrationRuleTest {
private static ServiceNameMapping mapping = mock(ServiceNameMapping.class);
@Test
- public void test_parse() {
+ void test_parse() {
when(mapping.getCachedMapping(any(URL.class))).thenReturn(Collections.emptySet());
String rule = "key: demo-consumer\n" +
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/DynamicDirectoryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/DynamicDirectoryTest.java
index 485f1a3940..f181de31b2 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/DynamicDirectoryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/DynamicDirectoryTest.java
@@ -40,13 +40,13 @@ import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
-public class DynamicDirectoryTest {
+class DynamicDirectoryTest {
/**
* verify simplified consumer url information that needs to be registered
*/
@Test
- public void testSimplifiedUrl() {
+ void testSimplifiedUrl() {
// verify that the consumer url information that needs to be
registered is not simplified by default
Map<String, String> parameters = new HashMap<>();
@@ -112,7 +112,7 @@ public class DynamicDirectoryTest {
@Test
- public void testSubscribe() {
+ void testSubscribe() {
Map<String, String> parameters = new HashMap<>();
parameters.put(INTERFACE_KEY, DemoService.class.getName());
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/RegistryProtocolTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/RegistryProtocolTest.java
index c1eb5da4f6..a4669373d9 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/RegistryProtocolTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/integration/RegistryProtocolTest.java
@@ -64,7 +64,7 @@ import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
-public class RegistryProtocolTest {
+class RegistryProtocolTest {
@AfterEach
public void tearDown() throws IOException {
@@ -76,7 +76,7 @@ public class RegistryProtocolTest {
* verify the generated consumer url information
*/
@Test
- public void testConsumerUrlWithoutProtocol() {
+ void testConsumerUrlWithoutProtocol() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
@@ -143,7 +143,7 @@ public class RegistryProtocolTest {
* verify that when the protocol is configured, the protocol of consumer
url is the configured protocol
*/
@Test
- public void testConsumerUrlWithProtocol() {
+ void testConsumerUrlWithProtocol() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
@@ -214,7 +214,7 @@ public class RegistryProtocolTest {
* @see FailoverCluster
*/
@Test
- public void testReferWithoutGroup() {
+ void testReferWithoutGroup() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
@@ -275,7 +275,7 @@ public class RegistryProtocolTest {
* @see MergeableCluster
*/
@Test
- public void testReferWithGroup() {
+ void testReferWithGroup() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
@@ -340,7 +340,7 @@ public class RegistryProtocolTest {
* @see MigrationRuleListener
*/
@Test
- public void testInterceptInvokerForMigrationRuleListener() {
+ void testInterceptInvokerForMigrationRuleListener() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
@@ -405,7 +405,7 @@ public class RegistryProtocolTest {
* @see CountRegistryProtocolListener
*/
@Test
- public void testInterceptInvokerForCustomRegistryProtocolListener() {
+ void testInterceptInvokerForCustomRegistryProtocolListener() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
@@ -468,7 +468,7 @@ public class RegistryProtocolTest {
* verify the registered consumer url
*/
@Test
- public void testRegisterConsumerUrl() {
+ void testRegisterConsumerUrl() {
ApplicationConfig applicationConfig = new ApplicationConfig();
applicationConfig.setName("application1");
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryFactoryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryFactoryTest.java
index 6347b78602..ae66edc2cf 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryFactoryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryFactoryTest.java
@@ -33,7 +33,7 @@ import java.util.List;
/**
* AbstractRegistryFactoryTest
*/
-public class AbstractRegistryFactoryTest {
+class AbstractRegistryFactoryTest {
private AbstractRegistryFactory registryFactory;
@@ -91,7 +91,7 @@ public class AbstractRegistryFactoryTest {
}
@Test
- public void testRegistryFactoryCache() throws Exception {
+ void testRegistryFactoryCache() throws Exception {
URL url = URL.valueOf("dubbo://" +
NetUtils.getLocalAddress().getHostAddress() + ":2233");
Registry registry1 = registryFactory.getRegistry(url);
Registry registry2 = registryFactory.getRegistry(url);
@@ -109,14 +109,14 @@ public class AbstractRegistryFactoryTest {
}
@Test
- public void testRegistryFactoryGroupCache() throws Exception {
+ void testRegistryFactoryGroupCache() throws Exception {
Registry registry1 =
registryFactory.getRegistry(URL.valueOf("dubbo://" + NetUtils.getLocalHost() +
":2233?group=aaa"));
Registry registry2 =
registryFactory.getRegistry(URL.valueOf("dubbo://" + NetUtils.getLocalHost() +
":2233?group=bbb"));
Assertions.assertNotSame(registry1, registry2);
}
@Test
- public void testDestroyAllRegistries() {
+ void testDestroyAllRegistries() {
Registry registry1 =
registryFactory.getRegistry(URL.valueOf("dubbo://" + NetUtils.getLocalHost() +
":8888?group=xxx"));
Registry registry2 =
registryFactory.getRegistry(URL.valueOf("dubbo://" + NetUtils.getLocalHost() +
":9999?group=yyy"));
Registry registry3 = new AbstractRegistry(URL.valueOf("dubbo://" +
NetUtils.getLocalHost() + ":2020?group=yyy")) {
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryTest.java
index 79d937942c..ce7ddd2690 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/AbstractRegistryTest.java
@@ -40,7 +40,7 @@ import static
org.apache.dubbo.common.constants.RegistryConstants.EMPTY_PROTOCOL
/**
* AbstractRegistryTest
*/
-public class AbstractRegistryTest {
+class AbstractRegistryTest {
private URL testUrl;
private URL mockUrl;
@@ -93,7 +93,7 @@ public class AbstractRegistryTest {
* @throws Exception
*/
@Test
- public void testRegister() throws Exception {
+ void testRegister() throws Exception {
//test one url
abstractRegistry.register(mockUrl);
assert abstractRegistry.getRegistered().contains(mockUrl);
@@ -109,7 +109,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testRegisterIfURLNULL() {
+ void testRegisterIfURLNULL() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
abstractRegistry.register(null);
Assertions.fail("register url == null");
@@ -122,7 +122,7 @@ public class AbstractRegistryTest {
*
*/
@Test
- public void testUnregister() {
+ void testUnregister() {
//test one unregister
URL url = new ServiceConfigURL("dubbo", "192.168.0.1", 2200);
abstractRegistry.register(url);
@@ -143,7 +143,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testUnregisterIfUrlNull() {
+ void testUnregisterIfUrlNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
abstractRegistry.unregister(null);
Assertions.fail("unregister url == null");
@@ -154,7 +154,7 @@ public class AbstractRegistryTest {
* test subscribe and unsubscribe
*/
@Test
- public void testSubscribeAndUnsubscribe() {
+ void testSubscribeAndUnsubscribe() {
//test subscribe
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener = urls -> notified.set(Boolean.TRUE);
@@ -169,7 +169,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testSubscribeIfUrlNull() {
+ void testSubscribeIfUrlNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener = urls -> notified.set(Boolean.TRUE);
@@ -180,7 +180,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testSubscribeIfListenerNull() {
+ void testSubscribeIfListenerNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener = urls -> notified.set(Boolean.TRUE);
@@ -191,7 +191,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testUnsubscribeIfUrlNull() {
+ void testUnsubscribeIfUrlNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener = urls -> notified.set(Boolean.TRUE);
@@ -201,7 +201,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testUnsubscribeIfNotifyNull() {
+ void testUnsubscribeIfNotifyNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
URL url = new ServiceConfigURL("dubbo", "192.168.0.1", 2200);
@@ -216,7 +216,7 @@ public class AbstractRegistryTest {
*
*/
@Test
- public void testSubscribe() {
+ void testSubscribe() {
// check parameters
try {
abstractRegistry.subscribe(testUrl, null);
@@ -244,7 +244,7 @@ public class AbstractRegistryTest {
*
*/
@Test
- public void testUnsubscribe() {
+ void testUnsubscribe() {
// check parameters
try {
abstractRegistry.unsubscribe(testUrl, null);
@@ -274,7 +274,7 @@ public class AbstractRegistryTest {
* {@link org.apache.dubbo.registry.support.AbstractRegistry#recover()}.
*/
@Test
- public void testRecover() throws Exception {
+ void testRecover() throws Exception {
// test recover nothing
abstractRegistry.recover();
Assertions.assertFalse(abstractRegistry.getRegistered().contains(testUrl));
@@ -292,7 +292,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testRecover2() throws Exception {
+ void testRecover2() throws Exception {
List<URL> list = getList();
abstractRegistry.recover();
Assertions.assertEquals(0, abstractRegistry.getRegistered().size());
@@ -309,7 +309,7 @@ public class AbstractRegistryTest {
* {@link org.apache.dubbo.registry.support.AbstractRegistry#notify(List)}.
*/
@Test
- public void testNotify() {
+ void testNotify() {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener1 = urls -> notified.set(Boolean.TRUE);
URL url1 = new ServiceConfigURL("dubbo", "192.168.0.1", 2200,
parametersConsumer);
@@ -335,7 +335,7 @@ public class AbstractRegistryTest {
* test notifyList
*/
@Test
- public void testNotifyList() {
+ void testNotifyList() {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener1 = urls -> notified.set(Boolean.TRUE);
URL url1 = new ServiceConfigURL("dubbo", "192.168.0.1", 2200,
parametersConsumer);
@@ -358,7 +358,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testNotifyIfURLNull() {
+ void testNotifyIfURLNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener1 = urls -> notified.set(Boolean.TRUE);
@@ -380,7 +380,7 @@ public class AbstractRegistryTest {
}
@Test
- public void testNotifyIfNotifyNull() {
+ void testNotifyIfNotifyNull() {
Assertions.assertThrows(IllegalArgumentException.class, () -> {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener1 = urls -> notified.set(Boolean.TRUE);
@@ -408,7 +408,7 @@ public class AbstractRegistryTest {
*
*/
@Test
- public void testNotifyArgs() {
+ void testNotifyArgs() {
// check parameters
try {
abstractRegistry.notify(null, null, null);
@@ -444,7 +444,7 @@ public class AbstractRegistryTest {
}
@Test
- public void filterEmptyTest() {
+ void filterEmptyTest() {
// check parameters
try {
AbstractRegistry.filterEmpty(null, null);
@@ -477,7 +477,7 @@ public class AbstractRegistryTest {
@Test
- public void lookupTest() {
+ void lookupTest() {
// loop up before registry
try {
abstractRegistry.lookup(null);
@@ -497,7 +497,7 @@ public class AbstractRegistryTest {
}
@Test
- public void destroyTest() {
+ void destroyTest() {
abstractRegistry.register(testUrl);
abstractRegistry.subscribe(testUrl, listener);
Assertions.assertEquals(1, abstractRegistry.getRegistered().size());
@@ -509,7 +509,7 @@ public class AbstractRegistryTest {
}
@Test
- public void allTest() {
+ void allTest() {
// test all methods
List<URL> urls = new ArrayList<>();
urls.add(testUrl);
@@ -539,7 +539,7 @@ public class AbstractRegistryTest {
}
@Test
- public void getCacheUrlsTest() {
+ void getCacheUrlsTest() {
List<URL> urls = new ArrayList<>();
urls.add(testUrl);
// check if notify successfully
diff --git
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/FailbackRegistryTest.java
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/FailbackRegistryTest.java
index 391a80e97b..b63ae3f1e6 100644
---
a/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/FailbackRegistryTest.java
+++
b/dubbo-registry/dubbo-registry-api/src/test/java/org/apache/dubbo/registry/support/FailbackRegistryTest.java
@@ -34,7 +34,7 @@ import static
org.apache.dubbo.registry.Constants.CONSUMER_PROTOCOL;
import static org.apache.dubbo.registry.Constants.REGISTRY_RETRY_PERIOD_KEY;
import static org.junit.jupiter.api.Assertions.assertEquals;
-public class FailbackRegistryTest {
+class FailbackRegistryTest {
protected final Logger logger = LoggerFactory.getLogger(getClass());
private URL serviceUrl;
@@ -60,7 +60,7 @@ public class FailbackRegistryTest {
* @throws Exception
*/
@Test
- public void testDoRetry() throws Exception {
+ void testDoRetry() throws Exception {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
@@ -95,7 +95,7 @@ public class FailbackRegistryTest {
}
@Test
- public void testDoRetryRegister() throws Exception {
+ void testDoRetryRegister() throws Exception {
final CountDownLatch latch = new CountDownLatch(1);//All of them are
called 4 times. A successful attempt to lose 1. subscribe will not be done
@@ -115,7 +115,7 @@ public class FailbackRegistryTest {
}
@Test
- public void testDoRetrySubscribe() throws Exception {
+ void testDoRetrySubscribe() throws Exception {
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
final CountDownLatch latch = new CountDownLatch(1);//All of them are
called 4 times. A successful attempt to lose 1. subscribe will not be done
@@ -143,7 +143,7 @@ public class FailbackRegistryTest {
}
@Test
- public void testRecover() throws Exception {
+ void testRecover() throws Exception {
CountDownLatch countDownLatch = new CountDownLatch(6);
final AtomicReference<Boolean> notified = new
AtomicReference<Boolean>(false);
NotifyListener listener = urls -> notified.set(Boolean.TRUE);
diff --git
a/dubbo-registry/dubbo-registry-multicast/src/test/java/org/apache/dubbo/registry/multicast/MulticastRegistryFactoryTest.java
b/dubbo-registry/dubbo-registry-multicast/src/test/java/org/apache/dubbo/registry/multicast/MulticastRegistryFactoryTest.java
index c2f5520bc6..8f0c7a5df5 100644
---
a/dubbo-registry/dubbo-registry-multicast/src/test/java/org/apache/dubbo/registry/multicast/MulticastRegistryFactoryTest.java
+++
b/dubbo-registry/dubbo-registry-multicast/src/test/java/org/apache/dubbo/registry/multicast/MulticastRegistryFactoryTest.java
@@ -26,9 +26,9 @@ import static org.hamcrest.CoreMatchers.not;
import static org.hamcrest.CoreMatchers.nullValue;
import static org.hamcrest.MatcherAssert.assertThat;
-public class MulticastRegistryFactoryTest {
+class MulticastRegistryFactoryTest {
@Test
- public void shouldCreateRegistry() {
+ void shouldCreateRegistry() {
Registry registry = new
MulticastRegistryFactory().createRegistry(URL.valueOf("multicast://239.255.255.255/"));
assertThat(registry, not(nullValue()));
assertThat(registry.isAvailable(), is(true));
diff --git
a/dubbo-registry/dubbo-registry-multiple/src/test/java/org/apache/dubbo/registry/multiple/MultipleRegistry2S2RTest.java
b/dubbo-registry/dubbo-registry-multiple/src/test/java/org/apache/dubbo/registry/multiple/MultipleRegistry2S2RTest.java
index dfa04ecae8..4d16b2d600 100644
---
a/dubbo-registry/dubbo-registry-multiple/src/test/java/org/apache/dubbo/registry/multiple/MultipleRegistry2S2RTest.java
+++
b/dubbo-registry/dubbo-registry-multiple/src/test/java/org/apache/dubbo/registry/multiple/MultipleRegistry2S2RTest.java
@@ -34,7 +34,7 @@ import java.util.List;
/**
* 2019-04-30
*/
-public class MultipleRegistry2S2RTest {
+class MultipleRegistry2S2RTest {
private static final String SERVICE_NAME =
"org.apache.dubbo.registry.MultipleService2S2R";
private static final String SERVICE2_NAME =
"org.apache.dubbo.registry.MultipleService2S2R2";
@@ -68,7 +68,7 @@ public class MultipleRegistry2S2RTest {
}
@Test
- public void testParamConfig() {
+ void testParamConfig() {
Assertions.assertEquals(2,
multipleRegistry.origReferenceRegistryURLs.size());
Assertions.assertTrue(multipleRegistry.origReferenceRegistryURLs.contains(zookeeperConnectionAddress1));
@@ -107,7 +107,7 @@ public class MultipleRegistry2S2RTest {
}
@Test
- public void testRegistryAndUnRegistry() throws InterruptedException {
+ void testRegistryAndUnRegistry() throws InterruptedException {
URL serviceUrl = URL.valueOf("http2://multiple/" + SERVICE_NAME +
"?notify=false&methods=test1,test2&category=providers");
// URL serviceUrl2 = URL.valueOf("http2://multiple2/" + SERVICE_NAME +
"?notify=false&methods=test1,test2&category=providers");
multipleRegistry.register(serviceUrl);
@@ -138,7 +138,7 @@ public class MultipleRegistry2S2RTest {
}
@Test
- public void testSubscription() throws InterruptedException {
+ void testSubscription() throws InterruptedException {
URL serviceUrl = URL.valueOf("http2://multiple/" + SERVICE2_NAME +
"?notify=false&methods=test1,test2&category=providers");
// URL serviceUrl2 = URL.valueOf("http2://multiple2/" + SERVICE_NAME +
"?notify=false&methods=test1,test2&category=providers");
multipleRegistry.register(serviceUrl);
@@ -176,7 +176,7 @@ public class MultipleRegistry2S2RTest {
}
@Test
- public void testAggregation() {
+ void testAggregation() {
List<URL> result = new ArrayList<URL>();
List<URL> listToAggregate = new ArrayList<URL>();
URL url1= URL.valueOf("dubbo://127.0.0.1:20880/service1");
diff --git
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryFactoryTest.java
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryFactoryTest.java
index effabf2193..1df7d4d760 100644
---
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryFactoryTest.java
+++
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryFactoryTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
/**
* Test for NacosRegistryFactory
*/
-public class NacosRegistryFactoryTest {
+class NacosRegistryFactoryTest {
private NacosRegistryFactory nacosRegistryFactory;
@@ -41,7 +41,7 @@ public class NacosRegistryFactoryTest {
}
@Test
- public void testCreateRegistryCacheKey() {
+ void testCreateRegistryCacheKey() {
URL url = URL.valueOf("dubbo://" +
NetUtils.getLocalAddress().getHostAddress() + ":8080");
String registryCacheKey1 =
nacosRegistryFactory.createRegistryCacheKey(url);
String registryCacheKey2 =
nacosRegistryFactory.createRegistryCacheKey(url);
@@ -49,7 +49,7 @@ public class NacosRegistryFactoryTest {
}
@Test
- public void testCreateRegistryCacheKeyWithNamespace() {
+ void testCreateRegistryCacheKeyWithNamespace() {
URL url = URL.valueOf("dubbo://" +
NetUtils.getLocalAddress().getHostAddress() + ":8080?namespace=test");
String registryCacheKey1 =
nacosRegistryFactory.createRegistryCacheKey(url);
String registryCacheKey2 =
nacosRegistryFactory.createRegistryCacheKey(url);
diff --git
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryTest.java
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryTest.java
index 9e61a84f2b..b5c9ee55ff 100644
---
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryTest.java
+++
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosRegistryTest.java
@@ -51,7 +51,7 @@ import static org.mockito.Mockito.when;
/**
* Test for NacosRegistry
*/
-public class NacosRegistryTest {
+class NacosRegistryTest {
private static final String serviceInterface =
"org.apache.dubbo.registry.nacos.NacosService";
@@ -83,7 +83,7 @@ public class NacosRegistryTest {
}
@Test
- public void testRegister() {
+ void testRegister() {
NamingService namingService = mock(NacosNamingService.class);
try {
@@ -118,7 +118,7 @@ public class NacosRegistryTest {
}
@Test
- public void testUnRegister() {
+ void testUnRegister() {
NamingService namingService = mock(NacosNamingService.class);
try {
@@ -158,7 +158,7 @@ public class NacosRegistryTest {
}
@Test
- public void testSubscribe() {
+ void testSubscribe() {
NamingService namingService = mock(NacosNamingService.class);
try {
@@ -195,7 +195,7 @@ public class NacosRegistryTest {
}
@Test
- public void testUnSubscribe() {
+ void testUnSubscribe() {
NamingService namingService = mock(NacosNamingService.class);
try {
@@ -240,7 +240,7 @@ public class NacosRegistryTest {
@Test
- public void testIsConformRules() {
+ void testIsConformRules() {
NamingService namingService = mock(NacosNamingService.class);
URL serviceUrlWithoutCategory = URL.valueOf("nacos://127.0.0.1:3333/"
+ serviceInterface + "?interface=" +
serviceInterface +
"¬ify=false&methods=test1,test2&version=1.0.0&group=default");
diff --git
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryFactoryTest.java
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryFactoryTest.java
index a8d1c11ec6..c52466d9a6 100644
---
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryFactoryTest.java
+++
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryFactoryTest.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
/**
* Test for NacosServiceDiscoveryFactory
*/
-public class NacosServiceDiscoveryFactoryTest {
+class NacosServiceDiscoveryFactoryTest {
private NacosServiceDiscoveryFactory nacosServiceDiscoveryFactory;
@@ -41,7 +41,7 @@ public class NacosServiceDiscoveryFactoryTest {
}
@Test
- public void testGetServiceDiscoveryWithCache() {
+ void testGetServiceDiscoveryWithCache() {
URL url = URL.valueOf("dubbo://test:8080");
ServiceDiscovery discovery =
nacosServiceDiscoveryFactory.createDiscovery(url);
diff --git
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryTest.java
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryTest.java
index 87f8cd1399..962acca8af 100644
---
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryTest.java
+++
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/NacosServiceDiscoveryTest.java
@@ -57,7 +57,7 @@ import static org.mockito.Mockito.when;
/**
* Test for NacosServiceDiscovery
*/
-public class NacosServiceDiscoveryTest {
+class NacosServiceDiscoveryTest {
private static final String SERVICE_NAME = "NACOS_SERVICE";
@@ -132,7 +132,7 @@ public class NacosServiceDiscoveryTest {
}
@Test
- public void testDoRegister() throws NacosException {
+ void testDoRegister() throws NacosException {
DefaultServiceInstance serviceInstance =
createServiceInstance(SERVICE_NAME, LOCALHOST, NetUtils.getAvailablePort());
// register
nacosServiceDiscovery.doRegister(serviceInstance);
@@ -147,7 +147,7 @@ public class NacosServiceDiscoveryTest {
}
@Test
- public void testDoUnRegister() throws NacosException {
+ void testDoUnRegister() throws NacosException {
// register
DefaultServiceInstance serviceInstance =
createServiceInstance(SERVICE_NAME, LOCALHOST, NetUtils.getAvailablePort());
// register
@@ -166,7 +166,7 @@ public class NacosServiceDiscoveryTest {
}
@Test
- public void testGetServices() throws NacosException {
+ void testGetServices() throws NacosException {
DefaultServiceInstance serviceInstance =
createServiceInstance(SERVICE_NAME, LOCALHOST, NetUtils.getAvailablePort());
// register
nacosServiceDiscovery.doRegister(serviceInstance);
@@ -187,7 +187,7 @@ public class NacosServiceDiscoveryTest {
}
@Test
- public void testAddServiceInstancesChangedListener() {
+ void testAddServiceInstancesChangedListener() {
List<ServiceInstance> serviceInstances = new LinkedList<>();
// Add Listener
nacosServiceDiscovery.addServiceInstancesChangedListener(
diff --git
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/util/NacosNamingServiceUtilsTest.java
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/util/NacosNamingServiceUtilsTest.java
index fc8203590b..701d97cffa 100644
---
a/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/util/NacosNamingServiceUtilsTest.java
+++
b/dubbo-registry/dubbo-registry-nacos/src/test/java/org/apache/dubbo/registry/nacos/util/NacosNamingServiceUtilsTest.java
@@ -34,18 +34,18 @@ import static org.mockito.Mockito.mock;
/**
* Test for NacosNamingServiceUtils
*/
-public class NacosNamingServiceUtilsTest {
+class NacosNamingServiceUtilsTest {
private static MetadataReport metadataReport =
Mockito.mock(MetadataReport.class);
@Test
- public void testToInstance() {
+ void testToInstance() {
ServiceInstance serviceInstance = mock(ServiceInstance.class);
Instance instance =
NacosNamingServiceUtils.toInstance(serviceInstance);
Assertions.assertNotNull(instance);
}
@Test
- public void testToServiceInstance() {
+ void testToServiceInstance() {
URL registryUrl = URL.valueOf("test://test:8080/test");
Instance instance = new Instance();
instance.setServiceName("serviceName");
@@ -66,7 +66,7 @@ public class NacosNamingServiceUtilsTest {
}
@Test
- public void testCreateNamingService() {
+ void testCreateNamingService() {
URL url = URL.valueOf("test://test:8080/test?backup=backup");
NacosNamingServiceWrapper namingService =
NacosNamingServiceUtils.createNamingService(url);
Assertions.assertNotNull(namingService);
diff --git
a/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperRegistryTest.java
b/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperRegistryTest.java
index c1e58cd0c8..3f36ad395b 100644
---
a/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperRegistryTest.java
+++
b/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperRegistryTest.java
@@ -43,7 +43,7 @@ import static org.hamcrest.Matchers.containsString;
import static org.junit.jupiter.api.Assertions.fail;
import static org.mockito.Mockito.mock;
-public class ZookeeperRegistryTest {
+class ZookeeperRegistryTest {
private static String zookeeperConnectionAddress1;
private ZookeeperRegistry zookeeperRegistry;
private String service = "org.apache.dubbo.test.injvmServie";
@@ -66,7 +66,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testAnyHost() {
+ void testAnyHost() {
Assertions.assertThrows(IllegalStateException.class, () -> {
URL errorUrl = URL.valueOf("multicast://0.0.0.0/");
new
ZookeeperRegistryFactory(ApplicationModel.defaultModel()).createRegistry(errorUrl);
@@ -74,7 +74,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testRegister() {
+ void testRegister() {
Set<URL> registered;
for (int i = 0; i < 2; i++) {
@@ -88,7 +88,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testSubscribe() {
+ void testSubscribe() {
NotifyListener listener = mock(NotifyListener.class);
zookeeperRegistry.subscribe(serviceUrl, listener);
@@ -103,7 +103,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testAvailable() {
+ void testAvailable() {
zookeeperRegistry.register(serviceUrl);
assertThat(zookeeperRegistry.isAvailable(), is(true));
@@ -112,7 +112,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testLookup() {
+ void testLookup() {
List<URL> lookup = zookeeperRegistry.lookup(serviceUrl);
assertThat(lookup.size(), is(1));
@@ -122,7 +122,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testLookupIllegalUrl() {
+ void testLookupIllegalUrl() {
try {
zookeeperRegistry.lookup(null);
fail();
@@ -133,7 +133,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testLookupWithException() {
+ void testLookupWithException() {
URL errorUrl = URL.valueOf("multicast://0.0.0.0/");
Assertions.assertThrows(RpcException.class, () ->
zookeeperRegistry.lookup(errorUrl));
}
@@ -161,7 +161,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testSubscribeAnyValue() throws InterruptedException {
+ void testSubscribeAnyValue() throws InterruptedException {
final CountDownLatch latch = new CountDownLatch(1);
zookeeperRegistry.register(serviceUrl);
zookeeperRegistry.subscribe(anyUrl, urls -> latch.countDown());
@@ -170,14 +170,14 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testDestroy() {
+ void testDestroy() {
zookeeperRegistry.destroy();
assertThat(zookeeperRegistry.isAvailable(), is(false));
}
@Test
- public void testDoRegisterWithException() {
+ void testDoRegisterWithException() {
Assertions.assertThrows(RpcException.class, () -> {
URL errorUrl = URL.valueOf("multicast://0.0.0.0/");
zookeeperRegistry.doRegister(errorUrl);
@@ -185,7 +185,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testDoUnregisterWithException() {
+ void testDoUnregisterWithException() {
Assertions.assertThrows(RpcException.class, () -> {
URL errorUrl = URL.valueOf("multicast://0.0.0.0/");
zookeeperRegistry.doUnregister(errorUrl);
@@ -193,7 +193,7 @@ public class ZookeeperRegistryTest {
}
@Test
- public void testDoSubscribeWithException() {
+ void testDoSubscribeWithException() {
Assertions.assertThrows(RpcException.class,
() -> zookeeperRegistry.doSubscribe(anyUrl, listener));
}
diff --git
a/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperServiceDiscoveryTest.java
b/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperServiceDiscoveryTest.java
index 2de256ec68..f3420bb596 100644
---
a/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperServiceDiscoveryTest.java
+++
b/dubbo-registry/dubbo-registry-zookeeper/src/test/java/org/apache/dubbo/registry/zookeeper/ZookeeperServiceDiscoveryTest.java
@@ -47,7 +47,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
* @since 2.7.5
*/
@DisabledForJreRange(min = JRE.JAVA_16)
-public class ZookeeperServiceDiscoveryTest {
+class ZookeeperServiceDiscoveryTest {
private static final String SERVICE_NAME = "A";
@@ -78,7 +78,7 @@ public class ZookeeperServiceDiscoveryTest {
}
@Test
- public void testRegistration() throws InterruptedException {
+ void testRegistration() throws InterruptedException {
CountDownLatch latch = new CountDownLatch(1);
