[
https://issues.apache.org/jira/browse/ROCKETMQ-264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16127015#comment-16127015
]
ASF GitHub Bot commented on ROCKETMQ-264:
-----------------------------------------
Github user vongosling commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/145#discussion_r133153891
--- Diff:
broker/src/test/java/org/apache/rocketmq/broker/filter/MessageStoreWithFilterTest.java
---
@@ -76,6 +86,24 @@
}
}
+ @Before
+ public void init() {
+ filterManager = ConsumerFilterManagerTest.gen(topicCount,
msgPerTopic);
+ try {
+ master = gen(filterManager);
+ } catch (Exception e) {
+ e.printStackTrace();
--- End diff --
redundant code, when you use assert in your exception scenario, right ?
> 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)
