[ISSUE #673] Add unit tests for TopicController. (#718)

Co-authored-by: zhangjidi2016 <zhangjidi@cmss.chinamobile.com>

src/main/java/org/apache/rocketmq/console/service/impl/TopicServiceImpl.java

@@ -210,13 +210,17 @@ public class TopicServiceImpl extends AbstractCommonService implements TopicServ
         return true;
     }
 
+    public DefaultMQProducer buildDefaultMQProducer(String producerGroup, RPCHook rpcHook) {
+        return new DefaultMQProducer(producerGroup, rpcHook);
+    }
+
     private TopicList  getSystemTopicList() {
         RPCHook rpcHook = null;
         boolean isEnableAcl = !StringUtils.isEmpty(rMQConfigure.getAccessKey()) && !StringUtils.isEmpty(rMQConfigure.getSecretKey());
         if (isEnableAcl) {
             rpcHook = new AclClientRPCHook(new SessionCredentials(rMQConfigure.getAccessKey(),rMQConfigure.getSecretKey()));
         }
-        DefaultMQProducer producer = new DefaultMQProducer(MixAll.SELF_TEST_PRODUCER_GROUP,rpcHook);
+        DefaultMQProducer producer = buildDefaultMQProducer(MixAll.SELF_TEST_PRODUCER_GROUP, rpcHook);
         producer.setInstanceName(String.valueOf(System.currentTimeMillis()));
         producer.setNamesrvAddr(rMQConfigure.getNamesrvAddr());
 
@@ -237,13 +241,13 @@ public class TopicServiceImpl extends AbstractCommonService implements TopicServ
     public SendResult sendTopicMessageRequest(SendTopicMessageRequest sendTopicMessageRequest) {
         DefaultMQProducer producer = null;
         if (rMQConfigure.isACLEnabled()) {
-            producer = new DefaultMQProducer(new AclClientRPCHook(new SessionCredentials(
+            AclClientRPCHook rpcHook = new AclClientRPCHook(new SessionCredentials(
                 rMQConfigure.getAccessKey(),
                 rMQConfigure.getSecretKey()
-            )));
-            producer.setProducerGroup(MixAll.SELF_TEST_PRODUCER_GROUP);
+            ));
+            producer = buildDefaultMQProducer(MixAll.SELF_TEST_PRODUCER_GROUP, rpcHook);
         } else {
-            producer = new DefaultMQProducer(MixAll.SELF_TEST_PRODUCER_GROUP);
+            producer = buildDefaultMQProducer(MixAll.SELF_TEST_PRODUCER_GROUP, null);
         }
 
         producer.setInstanceName(String.valueOf(System.currentTimeMillis()));

src/test/java/org/apache/rocketmq/console/BaseTest.java

@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.rocketmq.console;
+
+import java.lang.reflect.Field;
+import java.util.ArrayList;
+import java.util.List;
+import org.mockito.internal.util.MockUtil;
+import org.springframework.util.ReflectionUtils;
+
+public class BaseTest {
+    /**
+     * Inject the corresponding mock class automatically
+     */
+    protected void autoInjection() {
+        Object container = this;
+        final List<Field> localFields = getAllFields(container.getClass());
+        for (Field field : localFields) {
+            Object destObj = ReflectionUtils.getField(field, container);
+            if (MockUtil.isSpy(destObj)) {
+                processInjection(destObj, container, localFields);
+            }
+        }
+    }
+
+    protected void processInjection(Object target, final Object localDest, final List<Field> localFields) {
+        final List<Field> fields = getAllFields(target.getClass());
+        for (Field localField : localFields) {
+            for (Field field : fields) {
+                if (field.getName()
+                    .equals(localField.getName())) {
+                    Object obj = ReflectionUtils.getField(field, target);
+                    if (obj == null) {
+                        Object destObj = ReflectionUtils.getField(localField, localDest);
+                        if (MockUtil.isSpy(destObj)) {
+                            // Recursive processing
+                            processInjection(destObj, localDest, localFields);
+                        }
+                        // injection
+                        ReflectionUtils.setField(field, target, ReflectionUtils.getField(localField, localDest));
+                    }
+                    break;
+                }
+            }
+        }
+    }
+
+    protected List<Field> getAllFields(Class<?> leafClass) {
+        final List<Field> fields = new ArrayList<>(32);
+        ReflectionUtils.FieldCallback fc = (field) -> {
+            ReflectionUtils.makeAccessible(field);
+            fields.add(field);
+        };
+        ReflectionUtils.doWithFields(leafClass, fc);
+        return fields;
+    }
+}

src/test/java/org/apache/rocketmq/console/controller/BaseControllerTest.java

@@ -0,0 +1,89 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.rocketmq.console.controller;
+
+import org.apache.rocketmq.console.BaseTest;
+import org.apache.rocketmq.console.config.RMQConfigure;
+import org.apache.rocketmq.console.support.GlobalExceptionHandler;
+import org.apache.rocketmq.console.support.GlobalRestfulResponseBodyAdvice;
+import org.apache.rocketmq.console.util.MyPrintingResultHandler;
+import org.apache.rocketmq.tools.admin.MQAdminExt;
+import org.junit.Before;
+import org.mockito.Mock;
+import org.mockito.MockitoAnnotations;
+import org.springframework.test.web.servlet.MockMvc;
+import org.springframework.test.web.servlet.ResultActions;
+import org.springframework.test.web.servlet.setup.MockMvcBuilders;
+
+import static org.mockito.Mockito.when;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+public abstract class BaseControllerTest extends BaseTest {
+
+    protected MockMvc mockMvc;
+
+    @Mock
+    protected MQAdminExt mqAdminExt;
+
+    @Mock
+    protected RMQConfigure rMQConfigure;
+
+    protected abstract Object getTestController();
+
+    @Before
+    public void beforeInit() throws Exception {
+        MockitoAnnotations.initMocks(this);
+        autoInjection();
+        createMockMvc();
+    }
+
+    protected void mockRmqConfigure() {
+        when(rMQConfigure.getAccessKey()).thenReturn("12345678");
+        when(rMQConfigure.getSecretKey()).thenReturn("rocketmq");
+        when(rMQConfigure.getNamesrvAddr()).thenReturn("127.0.0.1:9876");
+        when(rMQConfigure.isACLEnabled()).thenReturn(true);
+    }
+
+    protected ResultActions performOkExpect(ResultActions perform) throws Exception {
+        return perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$").exists())
+            .andExpect(jsonPath("$").isMap())
+            .andExpect(jsonPath("$.data").exists())
+            .andExpect(jsonPath("$.status").value(0))
+            .andExpect(jsonPath("$.errMsg").doesNotExist());
+    }
+
+    protected ResultActions performErrorExpect(ResultActions perform) throws Exception {
+        return perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$").exists())
+            .andExpect(jsonPath("$.data").doesNotExist())
+            .andExpect(jsonPath("$.status").value(-1))
+            .andExpect(jsonPath("$.errMsg").isNotEmpty());
+    }
+
+    protected MockMvc createMockMvc() {
+        MockMvc innerMockMvc = MockMvcBuilders.standaloneSetup(getTestController())
+            .alwaysDo(MyPrintingResultHandler.me())
+            .setControllerAdvice(new GlobalExceptionHandler(), new GlobalRestfulResponseBodyAdvice())
+            .build();
+        this.mockMvc = innerMockMvc;
+        return innerMockMvc;
+    }
+
+}

src/test/java/org/apache/rocketmq/console/controller/TopicControllerTest.java

@@ -0,0 +1,392 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.rocketmq.console.controller;
+
+import com.alibaba.fastjson.JSON;
+import com.google.common.collect.Lists;
+import com.google.common.collect.Sets;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+import org.apache.rocketmq.client.impl.MQClientAPIImpl;
+import org.apache.rocketmq.client.impl.factory.MQClientInstance;
+import org.apache.rocketmq.client.impl.producer.DefaultMQProducerImpl;
+import org.apache.rocketmq.client.producer.DefaultMQProducer;
+import org.apache.rocketmq.client.producer.SendResult;
+import org.apache.rocketmq.client.producer.SendStatus;
+import org.apache.rocketmq.common.MixAll;
+import org.apache.rocketmq.common.TopicConfig;
+import org.apache.rocketmq.common.admin.ConsumeStats;
+import org.apache.rocketmq.common.admin.OffsetWrapper;
+import org.apache.rocketmq.common.admin.TopicOffset;
+import org.apache.rocketmq.common.admin.TopicStatsTable;
+import org.apache.rocketmq.common.message.Message;
+import org.apache.rocketmq.common.message.MessageQueue;
+import org.apache.rocketmq.common.protocol.body.ClusterInfo;
+import org.apache.rocketmq.common.protocol.body.ConsumerConnection;
+import org.apache.rocketmq.common.protocol.body.ConsumerRunningInfo;
+import org.apache.rocketmq.common.protocol.body.GroupList;
+import org.apache.rocketmq.common.protocol.body.TopicList;
+import org.apache.rocketmq.common.protocol.route.BrokerData;
+import org.apache.rocketmq.common.protocol.route.QueueData;
+import org.apache.rocketmq.common.protocol.route.TopicRouteData;
+import org.apache.rocketmq.console.model.request.SendTopicMessageRequest;
+import org.apache.rocketmq.console.model.request.TopicConfigInfo;
+import org.apache.rocketmq.console.service.impl.ConsumerServiceImpl;
+import org.apache.rocketmq.console.service.impl.TopicServiceImpl;
+import org.junit.Before;
+import org.junit.Test;
+import org.mockito.InjectMocks;
+import org.mockito.Spy;
+import org.springframework.http.MediaType;
+import org.springframework.test.web.servlet.ResultActions;
+import org.springframework.test.web.servlet.request.MockHttpServletRequestBuilder;
+import org.springframework.test.web.servlet.request.MockMvcRequestBuilders;
+
+import static org.hamcrest.Matchers.hasSize;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyBoolean;
+import static org.mockito.ArgumentMatchers.anyLong;
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.mockito.Mockito.doNothing;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+public class TopicControllerTest extends BaseControllerTest {
+
+    @InjectMocks
+    private TopicController topicController;
+
+    private MockHttpServletRequestBuilder requestBuilder = null;
+
+    private ResultActions perform;
+
+    @Spy
+    private TopicServiceImpl topicService;
+
+    @Spy
+    private ConsumerServiceImpl consumerService;
+
+    private String topicName = "topic_test";
+
+    @Before
+    public void init() {
+        super.mockRmqConfigure();
+    }
+
+    @Test
+    public void testList() throws Exception {
+        {
+            // mock all topics
+            TopicList topicList = new TopicList();
+            Set<String> topicSet = new HashSet<>();
+            topicSet.add("common_topic1");
+            topicSet.add("common_topic2");
+            topicSet.add("system_topic1");
+            topicSet.add("system_topic2");
+            topicList.setTopicList(topicSet);
+            when(mqAdminExt.fetchAllTopicList()).thenReturn(topicList);
+            // mock system topics
+            TopicList sysTopicList = new TopicList();
+            Set<String> sysTopicSet = new HashSet<>();
+            sysTopicSet.add("system_topic1");
+            sysTopicSet.add("system_topic2");
+            sysTopicList.setTopicList(sysTopicSet);
+            DefaultMQProducer producer = mock(DefaultMQProducer.class);
+            doNothing().when(producer).start();
+            doNothing().when(producer).shutdown();
+            DefaultMQProducerImpl defaultMQProducer = mock(DefaultMQProducerImpl.class);
+            MQClientInstance mqClientInstance = mock(MQClientInstance.class);
+            MQClientAPIImpl mqClientAPIImpl = mock(MQClientAPIImpl.class);
+            when(producer.getDefaultMQProducerImpl()).thenReturn(defaultMQProducer);
+            when(defaultMQProducer.getmQClientFactory()).thenReturn(mqClientInstance);
+            when(mqClientInstance.getMQClientAPIImpl()).thenReturn(mqClientAPIImpl);
+            when(mqClientAPIImpl.getSystemTopicList(anyLong())).thenReturn(sysTopicList);
+            when(topicService.buildDefaultMQProducer(anyString(), any())).thenReturn(producer);
+        }
+        final String url = "/topic/list.query";
+
+        // 1、list all topic
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        requestBuilder.param("skipSysProcess", String.valueOf(true));
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data.topicList", hasSize(4)));
+
+        // 2、filter system topic
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data.topicList[2]").value("%SYS%system_topic1"));
+    }
+
+    @Test
+    public void testStat() throws Exception {
+        {
+            TopicStatsTable topicStatsTable = createTopicStatsTable();
+            when(mqAdminExt.examineTopicStats(anyString())).thenReturn(topicStatsTable);
+        }
+        final String url = "/topic/stats.query";
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        requestBuilder.param("topic", topicName);
+        perform = mockMvc.perform(requestBuilder);
+        performOkExpect(perform);
+    }
+
+    @Test
+    public void testRoute() throws Exception {
+        {
+            TopicRouteData topicRouteData = createTopicRouteData();
+            when(mqAdminExt.examineTopicRouteInfo(anyString())).thenReturn(topicRouteData);
+        }
+        final String url = "/topic/route.query";
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        requestBuilder.param("topic", topicName);
+        perform = mockMvc.perform(requestBuilder);
+        performOkExpect(perform);
+    }
+
+    @Test
+    public void testCreateOrUpdate() throws Exception {
+        final String url = "/topic/createOrUpdate.do";
+
+        // 1、clusterName and brokerName all blank
+        requestBuilder = MockMvcRequestBuilders.post(url);
+        requestBuilder.contentType(MediaType.APPLICATION_JSON_UTF8);
+        TopicConfigInfo info = new TopicConfigInfo();
+        requestBuilder.content(JSON.toJSONString(info));
+        perform = mockMvc.perform(requestBuilder);
+        performErrorExpect(perform);
+
+        {
+            ClusterInfo clusterInfo = createClusterInfo();
+            when(mqAdminExt.examineBrokerClusterInfo()).thenReturn(clusterInfo);
+            doNothing().when(mqAdminExt).createAndUpdateTopicConfig(anyString(), any());
+        }
+
+        List<String> clusterNameList = Lists.newArrayList("DefaultCluster");
+        info.setTopicName("topic_test");
+        info.setReadQueueNums(4);
+        info.setWriteQueueNums(4);
+        info.setPerm(6);
+        info.setClusterNameList(clusterNameList);
+        // 2、create topic
+        requestBuilder = MockMvcRequestBuilders.post(url);
+        requestBuilder.contentType(MediaType.APPLICATION_JSON_UTF8);
+        requestBuilder.content(JSON.toJSONString(info));
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data").value(true));
+    }
+
+    @Test
+    public void testExamineTopicConfig() throws Exception {
+        final String url = "/topic/examineTopicConfig.query";
+        {
+            TopicRouteData topicRouteData = createTopicRouteData();
+            when(mqAdminExt.examineTopicRouteInfo(anyString())).thenReturn(topicRouteData);
+            ClusterInfo clusterInfo = createClusterInfo();
+            when(mqAdminExt.examineBrokerClusterInfo()).thenReturn(clusterInfo);
+            when(mqAdminExt.examineTopicConfig(anyString(), anyString())).thenReturn(new TopicConfig(topicName));
+        }
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        requestBuilder.param("topic", topicName);
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data[0].topicName").value(topicName));
+    }
+
+    @Test
+    public void testQueryConsumerByTopic() throws Exception {
+        final String url = "/topic/queryConsumerByTopic.query";
+        {
+            GroupList list = new GroupList();
+            list.setGroupList(Sets.newHashSet("group1"));
+            when(mqAdminExt.queryTopicConsumeByWho(anyString())).thenReturn(list);
+            ConsumeStats consumeStats = createConsumeStats();
+            when(mqAdminExt.examineConsumeStats(anyString(), anyString())).thenReturn(consumeStats);
+            when(mqAdminExt.examineConsumerConnectionInfo(anyString())).thenReturn(new ConsumerConnection());
+            when(mqAdminExt.getConsumerRunningInfo(anyString(), anyString(), anyBoolean())).thenReturn(new ConsumerRunningInfo());
+        }
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        requestBuilder.param("topic", topicName);
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data").isMap());
+    }
+
+    @Test
+    public void testQueryTopicConsumerInfo() throws Exception {
+        final String url = "/topic/queryTopicConsumerInfo.query";
+        {
+            GroupList list = new GroupList();
+            list.setGroupList(Sets.newHashSet("group1", "group2", "group3"));
+            when(mqAdminExt.queryTopicConsumeByWho(anyString())).thenReturn(list);
+        }
+        requestBuilder = MockMvcRequestBuilders.get(url);
+        requestBuilder.param("topic", topicName);
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data.groupList", hasSize(3)));
+    }
+
+    @Test
+    public void testSendTopicMessage() throws Exception {
+        final String url = "/topic/sendTopicMessage.do";
+        {
+            DefaultMQProducer producer = mock(DefaultMQProducer.class);
+            doNothing().when(producer).start();
+            doNothing().when(producer).shutdown();
+            SendResult result = new SendResult(SendStatus.SEND_OK, "7F000001E41A2E5D6D978B82C20F003D",
+                "0A8E83C300002A9F00000000000013D3", new MessageQueue(), 1000L);
+            when(producer.send(any(Message.class))).thenReturn(result);
+            when(topicService.buildDefaultMQProducer(anyString(), any())).thenReturn(producer);
+        }
+
+        SendTopicMessageRequest request = new SendTopicMessageRequest();
+        request.setTopic(topicName);
+        request.setMessageBody("hello world");
+        requestBuilder = MockMvcRequestBuilders.post(url);
+        requestBuilder.contentType(MediaType.APPLICATION_JSON_UTF8);
+        requestBuilder.content(JSON.toJSONString(request));
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data.sendStatus").value(SendStatus.SEND_OK.name()))
+            .andExpect(jsonPath("$.data.msgId").value("7F000001E41A2E5D6D978B82C20F003D"));
+
+    }
+
+    @Test
+    public void testDelete() throws Exception {
+        final String url = "/topic/deleteTopic.do";
+        {
+            ClusterInfo clusterInfo = createClusterInfo();
+            when(mqAdminExt.examineBrokerClusterInfo()).thenReturn(clusterInfo);
+            doNothing().when(mqAdminExt).deleteTopicInBroker(any(), anyString());
+            doNothing().when(mqAdminExt).deleteTopicInNameServer(any(), anyString());
+        }
+
+        // 1、clusterName is blank
+        requestBuilder = MockMvcRequestBuilders.post(url);
+        requestBuilder.param("topic", topicName);
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data").value(true));
+
+        // 2、clusterName is not blank
+        requestBuilder.param("clusterName", "DefaultCluster");
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data").value(true));
+    }
+
+    @Test
+    public void testDeleteTopicByBroker() throws Exception {
+        {
+            ClusterInfo clusterInfo = createClusterInfo();
+            when(mqAdminExt.examineBrokerClusterInfo()).thenReturn(clusterInfo);
+            doNothing().when(mqAdminExt).deleteTopicInBroker(any(), anyString());
+        }
+        final String url = "/topic/deleteTopicByBroker.do";
+        requestBuilder = MockMvcRequestBuilders.post(url);
+        requestBuilder.param("topic", topicName);
+        requestBuilder.param("brokerName", "broker-a");
+        perform = mockMvc.perform(requestBuilder);
+        perform.andExpect(status().isOk())
+            .andExpect(jsonPath("$.data").value(true));
+    }
+
+
+    private ConsumeStats createConsumeStats(){
+        ConsumeStats stats = new ConsumeStats();
+        HashMap<MessageQueue, OffsetWrapper> offsetTable = new HashMap<MessageQueue, OffsetWrapper>();
+        OffsetWrapper wrapper = new OffsetWrapper();
+        wrapper.setBrokerOffset(10);
+        wrapper.setConsumerOffset(7);
+        wrapper.setLastTimestamp(System.currentTimeMillis());
+        offsetTable.put(new MessageQueue(topicName, "broker-a", 1), wrapper);
+        offsetTable.put(new MessageQueue(topicName, "broker-a", 2), wrapper);
+        stats.setOffsetTable(offsetTable);
+        return stats;
+    }
+
+    private ClusterInfo createClusterInfo() {
+        ClusterInfo clusterInfo = new ClusterInfo();
+        HashMap<String, Set<String>> clusterAddrTable = new HashMap<>(3);
+        Set<String> brokerNameSet = new HashSet<>(3);
+        brokerNameSet.add("broker-a");
+        clusterAddrTable.put("DefaultCluster", brokerNameSet);
+        clusterInfo.setClusterAddrTable(clusterAddrTable);
+        HashMap<String, BrokerData> brokerAddrTable = new HashMap<>(3);
+        BrokerData brokerData = new BrokerData();
+        brokerData.setBrokerName("broker-a");
+        HashMap<Long, String> brokerAddrs = new HashMap<>(2);
+        brokerAddrs.put(MixAll.MASTER_ID, "127.0.0.1:10911");
+        brokerData.setBrokerAddrs(brokerAddrs);
+        brokerAddrTable.put("broker-a", brokerData);
+        clusterInfo.setBrokerAddrTable(brokerAddrTable);
+        return clusterInfo;
+    }
+
+    private TopicStatsTable createTopicStatsTable() {
+        TopicStatsTable topicStatsTable = new TopicStatsTable();
+        HashMap<MessageQueue, TopicOffset> offsetTable = new HashMap<>();
+        MessageQueue queue = new MessageQueue("topic_test", "broker-a", 2);
+        TopicOffset offset = new TopicOffset();
+        offset.setMinOffset(0);
+        offset.setMaxOffset(100);
+        offset.setLastUpdateTimestamp(System.currentTimeMillis());
+        offsetTable.put(queue, offset);
+        topicStatsTable.setOffsetTable(offsetTable);
+        return topicStatsTable;
+    }
+
+    private TopicRouteData createTopicRouteData() {
+        TopicRouteData topicRouteData = new TopicRouteData();
+
+        topicRouteData.setFilterServerTable(new HashMap<>());
+        List<BrokerData> brokerDataList = new ArrayList<>();
+        BrokerData brokerData = new BrokerData();
+        brokerData.setBrokerName("broker-a");
+        brokerData.setCluster("DefaultCluster");
+        HashMap<Long, String> brokerAddrs = new HashMap<>();
+        brokerAddrs.put(0L, "127.0.0.1:10911");
+        brokerData.setBrokerAddrs(brokerAddrs);
+        brokerDataList.add(brokerData);
+        topicRouteData.setBrokerDatas(brokerDataList);
+
+        List<QueueData> queueDataList = new ArrayList<>();
+        QueueData queueData = new QueueData();
+        queueData.setBrokerName("broker-a");
+        queueData.setPerm(6);
+        queueData.setReadQueueNums(4);
+        queueData.setWriteQueueNums(4);
+        queueData.setTopicSynFlag(0);
+        queueDataList.add(queueData);
+        topicRouteData.setQueueDatas(queueDataList);
+        return topicRouteData;
+    }
+
+    @Override protected Object getTestController() {
+        return topicController;
+    }
+}

src/test/java/org/apache/rocketmq/console/util/MyPrintingResultHandler.java

@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.rocketmq.console.util;
+
+import java.lang.reflect.Method;
+import org.apache.commons.lang3.StringUtils;
+import org.springframework.mock.web.MockHttpServletResponse;
+import org.springframework.test.web.servlet.result.PrintingResultHandler;
+import org.springframework.util.CollectionUtils;
+import org.springframework.util.ReflectionUtils;
+
+public class MyPrintingResultHandler extends PrintingResultHandler {
+    public static MyPrintingResultHandler me() {
+        return new MyPrintingResultHandler();
+    }
+
+    private static ResultValuePrinter printer;
+
+    static {
+        printer = new ResultValuePrinter() {
+            private boolean isPrint = true;
+
+            @Override
+            public void printHeading(String heading) {
+                if (isPrint) {
+                    System.out.println();
+                    System.out.println(String.format("%s:", heading));
+                }
+            }
+
+            @Override
+            public void printValue(String label, Object value) {
+                if (isPrint) {
+                    if (value != null && value.getClass().isArray()) {
+                        value = CollectionUtils.arrayToList(value);
+                    }
+                    System.out.println(String.format("%17s = %s", label, value));
+                }
+            }
+        };
+    }
+
+    public MyPrintingResultHandler() {
+        super(printer);
+    }
+
+    @Override
+    protected void printResponse(MockHttpServletResponse response) throws Exception {
+        this.getPrinter().printValue("Status", response.getStatus());
+        this.getPrinter().printValue("Error message", response.getErrorMessage());
+        this.getPrinter().printValue("Headers", getResponseHeaders(response));
+        this.getPrinter().printValue("Content type", response.getContentType());
+        String disposition = response.getHeader("Content-Disposition");
+        if (StringUtils.isBlank(disposition)) {
+            this.getPrinter().printValue("Body", response.getContentAsString());
+        } else {
+            this.getPrinter().printValue("Body", "this a file");
+        }
+        this.getPrinter().printValue("Forwarded URL", response.getForwardedUrl());
+        this.getPrinter().printValue("Redirected URL", response.getRedirectedUrl());
+        Method method = ReflectionUtils.findMethod(getClass(), "printCookies");
+        if (method != null) {
+            ReflectionUtils.makeAccessible(method);
+            method.invoke(this, response.getCookies());
+        }
+    }
+}