去除无用的被mock的map

Author: SweetWuXiaoMei

core/src/test/java/org/apache/servicecomb/core/TestInvocation.java

@@ -17,9 +17,7 @@
 package org.apache.servicecomb.core;
 
 import java.util.Arrays;
-import java.util.Map;
 
-import org.apache.logging.log4j.core.util.internal.InternalLoggerRegistry;
 import org.apache.servicecomb.core.definition.InvocationRuntimeType;
 import org.apache.servicecomb.core.definition.OperationMeta;
 import org.apache.servicecomb.core.event.InvocationBaseEvent;
@@ -40,19 +38,16 @@
 import org.junit.BeforeClass;
 import org.junit.Test;
 import org.junit.jupiter.api.Assertions;
-import org.mockito.Mockito;
 
 import com.google.common.eventbus.EventBus;
 import com.google.common.eventbus.Subscribe;
 
-import mockit.Deencapsulation;
 import mockit.Expectations;
 import mockit.Mock;
 import mockit.MockUp;
 import mockit.Mocked;
 
 public class TestInvocation {
-  Invocation invocation;
 
   @Mocked
   Endpoint endpoint;
@@ -63,9 +58,6 @@ public class TestInvocation {
   @Mocked
   InvocationRuntimeType invocationRuntimeType;
 
-  @Mocked
-  Map<String, Object> arguments;
-
   static long nanoTime = 123;
 
   @BeforeClass
@@ -98,7 +90,7 @@ public void onStart(InvocationStartEvent event) {
     };
     EventManager.register(subscriber);
 
-    Invocation invocation = new Invocation(endpoint, operationMeta, arguments);
+    Invocation invocation = new Invocation(endpoint, operationMeta, null);
     invocation.onStart();
 
     Assertions.assertSame(invocation, result.value);
@@ -118,7 +110,7 @@ public void onStart(InvocationFinishEvent event) {
     };
     EventManager.register(subscriber);
 
-    Invocation invocation = new Invocation(endpoint, operationMeta, arguments);
+    Invocation invocation = new Invocation(endpoint, operationMeta, null);
     Assertions.assertFalse(invocation.isFinished());
     Response response = Response.succResp(null);
     invocation.onFinish(response);
@@ -137,19 +129,19 @@ public void onStart(InvocationFinishEvent event) {
 
   @Test
   public void isConsumer_yes() {
-    Invocation invocation = new Invocation(endpoint, operationMeta, arguments);
+    Invocation invocation = new Invocation(endpoint, operationMeta, null);
     Assertions.assertFalse(invocation.isConsumer());
   }
 
   @Test
   public void isConsumer_no(@Mocked ReferenceConfig referenceConfig) {
-    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, arguments);
+    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, null);
     Assertions.assertTrue(invocation.isConsumer());
   }
 
   @Test
   public void localContext(@Mocked ReferenceConfig referenceConfig) {
-    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, arguments);
+    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, null);
 
     invocation.addLocalContext("k", 1);
     Assertions.assertSame(invocation.getHandlerContext(), invocation.getLocalContext());
@@ -158,7 +150,7 @@ public void localContext(@Mocked ReferenceConfig referenceConfig) {
 
   @Test
   public void traceId_fromContext(@Mocked ReferenceConfig referenceConfig) {
-    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, arguments);
+    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, null);
     invocation.addContext(CoreConst.TRACE_ID_NAME, "abc");
 
     invocation.onStart();
@@ -176,7 +168,7 @@ public void traceId_consumerCreateTraceId(@Mocked ReferenceConfig referenceConfi
         result = "abc";
       }
     };
-    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, arguments);
+    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, null);
 
     invocation.onStart();
 
@@ -192,7 +184,7 @@ public void traceId_fromRequest(@Mocked Endpoint endpoint, @Mocked HttpServletRe
         result = "abc";
       }
     };
-    Invocation invocation = new Invocation(endpoint, operationMeta, arguments);
+    Invocation invocation = new Invocation(endpoint, operationMeta, null);
 
     invocation.onStart(requestEx);
 
@@ -210,7 +202,7 @@ public void traceId_producerCreateTraceId(@Mocked Endpoint endpoint, @Mocked Htt
         result = "abc";
       }
     };
-    Invocation invocation = new Invocation(endpoint, operationMeta, arguments);
+    Invocation invocation = new Invocation(endpoint, operationMeta, null);
 
     invocation.onStart(requestEx);
 
@@ -254,7 +246,7 @@ public void onBusinessMethodStart(InvocationBusinessMethodStartEvent event) {
       }
     };
     EventManager.getEventBus().register(listener);
-    Invocation invocation = new Invocation(endpoint, operationMeta, arguments);
+    Invocation invocation = new Invocation(endpoint, operationMeta, null);
     mockNonaTime();
     invocation.onBusinessMethodStart();
     EventManager.getEventBus().unregister(listener);
@@ -269,12 +261,12 @@ public void onBusinessMethodStart(InvocationBusinessMethodStartEvent event) {
   public void marker(@Mocked ReferenceConfig referenceConfig) {
     Invocation.INVOCATION_ID.set(0);
 
-    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, arguments);
+    Invocation invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, null);
     invocation.addContext(CoreConst.TRACE_ID_NAME, "abc");
     invocation.onStart();
     Assertions.assertEquals("abc", invocation.getTraceIdLogger().getName());
 
-    invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, arguments);
+    invocation = new Invocation(referenceConfig, operationMeta, invocationRuntimeType, null);
     invocation.addContext(CoreConst.TRACE_ID_NAME, "abc");
     invocation.onStart();
     Assertions.assertEquals("abc", invocation.getTraceIdLogger().getName());