fix @test annotation (apache#9596)

zeusbee · Jan 23, 2022 · 3339e88 · 3339e88
1 parent c589064
commit 3339e88
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 16 deletions.
diff --git a/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/AbstractClusterInvokerTest.java b/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/AbstractClusterInvokerTest.java
@@ -499,7 +499,7 @@ private void initDic() {
         dic.buildRouterChain();
     }
 
-    @Test()
+    @Test
     public void testTimeoutExceptionCode() {
         List<Invoker<DemoService>> invokers = new ArrayList<Invoker<DemoService>>();
         invokers.add(new Invoker<DemoService>() {

diff --git a/...uster/src/test/java/org/apache/dubbo/rpc/cluster/support/AvailableClusterInvokerTest.java b/...uster/src/test/java/org/apache/dubbo/rpc/cluster/support/AvailableClusterInvokerTest.java
@@ -86,7 +86,7 @@ private void resetInvokerToNoException() {
         given(invoker3.getInterface()).willReturn(AvailableClusterInvokerTest.class);
     }
 
-    @Test()
+    @Test
     public void testInvokeNoException() {
 
         resetInvokerToNoException();
@@ -96,7 +96,7 @@ public void testInvokeNoException() {
         Assertions.assertSame(result, ret);
     }
 
-    @Test()
+    @Test
     public void testInvokeWithException() {
 
         // remove invokers for test exception

diff --git a/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/FailSafeClusterInvokerTest.java b/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/FailSafeClusterInvokerTest.java
@@ -89,7 +89,7 @@ public void testInvokeExceptoin() {
         Assertions.assertNull(RpcContext.getServiceContext().getInvoker());
     }
 
-    @Test()
+    @Test
     public void testInvokeNoExceptoin() {
 
         resetInvokerToNoException();
@@ -99,7 +99,7 @@ public void testInvokeNoExceptoin() {
         Assertions.assertSame(result, ret);
     }
 
-    @Test()
+    @Test
     public void testNoInvoke() {
         dic = mock(Directory.class);
 

diff --git a/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/FailfastClusterInvokerTest.java b/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/FailfastClusterInvokerTest.java
@@ -108,7 +108,7 @@ public void testInvokeBizException() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvokeNoException() {
 
         resetInvoker1ToNoException();
@@ -118,7 +118,7 @@ public void testInvokeNoException() {
         assertSame(result, ret);
     }
 
-    @Test()
+    @Test
     public void testNoInvoke() {
         dic = mock(Directory.class);
 

diff --git a/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/FailoverClusterInvokerTest.java b/...luster/src/test/java/org/apache/dubbo/rpc/cluster/support/FailoverClusterInvokerTest.java
@@ -101,7 +101,7 @@ public void testInvokeWithRuntimeException() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvokeWithRPCException() {
         given(invoker1.invoke(invocation)).willThrow(new RpcException());
         given(invoker1.isAvailable()).willReturn(true);
@@ -120,7 +120,7 @@ public void testInvokeWithRPCException() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvoke_retryTimes() {
         given(invoker1.invoke(invocation)).willThrow(new RpcException(RpcException.TIMEOUT_EXCEPTION));
         given(invoker1.isAvailable()).willReturn(false);
@@ -143,7 +143,7 @@ public void testInvoke_retryTimes() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvoke_retryTimes2() {
         int finalRetries = 1;
         given(invoker1.invoke(invocation)).willThrow(new RpcException(RpcException.TIMEOUT_EXCEPTION));
@@ -170,7 +170,7 @@ public void testInvoke_retryTimes2() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvoke_retryTimes_withBizException() {
         given(invoker1.invoke(invocation)).willThrow(new RpcException(RpcException.BIZ_EXCEPTION));
         given(invoker1.isAvailable()).willReturn(false);
@@ -192,7 +192,7 @@ public void testInvoke_retryTimes_withBizException() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvoke_without_retry() {
         int withoutRetry = 0;
         final URL url = URL.valueOf("test://localhost/" + Demo.class.getName() + "?loadbalance=roundrobin&retries=" + withoutRetry);
@@ -219,7 +219,7 @@ public void testInvoke_without_retry() {
         }
     }
 
-    @Test()
+    @Test
     public void testInvoke_when_retry_illegal() {
         int illegalRetry = -1;
         final URL url = URL.valueOf("test://localhost/" + Demo.class.getName() + "?loadbalance=roundrobin&retries=" + illegalRetry);
@@ -246,7 +246,7 @@ public void testInvoke_when_retry_illegal() {
         }
     }
 
-    @Test()
+    @Test
     public void testNoInvoke() {
         dic = mock(Directory.class);
 

diff --git a/...cluster/src/test/java/org/apache/dubbo/rpc/cluster/support/ForkingClusterInvokerTest.java b/...cluster/src/test/java/org/apache/dubbo/rpc/cluster/support/ForkingClusterInvokerTest.java
@@ -141,7 +141,7 @@ public void testClearRpcContext() {
         Assertions.assertTrue(afterInvoke != null && afterInvoke.size() == 0, "clear attachment failed!");
     }
 
-    @Test()
+    @Test
     public void testInvokeNoException() {
 
         resetInvokerToNoException();
@@ -151,7 +151,7 @@ public void testInvokeNoException() {
         Assertions.assertSame(result, ret);
     }
 
-    @Test()
+    @Test
     public void testInvokeWithIllegalForksParam() {
         URL url = URL.valueOf("test://test:11/test?forks=-1");
         given(dic.getUrl()).willReturn(url);