[ 
https://issues.apache.org/jira/browse/ROCKETMQ-264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16131559#comment-16131559
 ] 

ASF GitHub Bot commented on ROCKETMQ-264:
-----------------------------------------

Github user shroman commented on a diff in the pull request:

    https://github.com/apache/incubator-rocketmq/pull/145#discussion_r133862502
  
    --- Diff: 
broker/src/test/java/org/apache/rocketmq/broker/BrokerControllerTest.java ---
    @@ -37,16 +37,14 @@
          */
         @Test
         public void testBrokerRestart() throws Exception {
    -        for (int i = 0; i < 2; i++) {
    -            BrokerController brokerController = new BrokerController(
    -                new BrokerConfig(),
    -                new NettyServerConfig(),
    -                new NettyClientConfig(),
    -                new MessageStoreConfig());
    -            assertThat(brokerController.initialize());
    -            brokerController.start();
    -            brokerController.shutdown();
    -        }
    +        BrokerController brokerController = new BrokerController(
    --- End diff --
    
    By changing this you change the expected behavior -- it is a check for 
proper `restart`.


> Unit test cost too long and there is exception in unit test
> -----------------------------------------------------------
>
>                 Key: ROCKETMQ-264
>                 URL: https://issues.apache.org/jira/browse/ROCKETMQ-264
>             Project: Apache RocketMQ
>          Issue Type: Improvement
>            Reporter: lindzh
>            Assignee: vongosling
>
> When run mvn test , it cost too much time and some times there is exception 
> in unit test and the test result is pass



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to