Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1800#discussion_r163257179
--- Diff:
artemis-commons/src/test/java/org/apache/activemq/artemis/utils/ThreadLeakCheckRule.java
---
@@ -257,11 +262,13 @@ private boolean isExpectedThread(Thread thread) {
@Override
public void testSucceeded(Description description) {
this.enabled = true;
+ this.testFailed = false;
}
@Override
public void testFailed(Throwable e, Description description) {
- this.enabled = false;
+ this.testFailed = true;
--- End diff --
remove this.enabled from here... just this.testFailed
---
