Remove and Polish unstable UT

common/src/test/java/org/apache/rocketmq/common/filter/FilterAPITest.java

@@ -17,11 +17,12 @@
 
 package org.apache.rocketmq.common.filter;
 
-import java.util.HashSet;
-import java.util.Set;
 import org.apache.rocketmq.common.protocol.heartbeat.SubscriptionData;
 import org.junit.Test;
 
+import java.util.HashSet;
+import java.util.Set;
+
 import static org.assertj.core.api.Assertions.assertThat;
 
 public class FilterAPITest {
@@ -32,7 +33,7 @@ public class FilterAPITest {
     @Test
     public void testBuildSubscriptionData() throws Exception {
         SubscriptionData subscriptionData =
-            FilterAPI.buildSubscriptionData(group, topic, subString);
+                FilterAPI.buildSubscriptionData(group, topic, subString);
         assertThat(subscriptionData.getTopic()).isEqualTo(topic);
         assertThat(subscriptionData.getSubString()).isEqualTo(subString);
         String[] tags = subString.split("\\|\\|");
@@ -47,7 +48,7 @@ public void testBuildSubscriptionData() throws Exception {
     public void testBuildTagSome() {
         try {
             SubscriptionData subscriptionData = FilterAPI.build(
-                "TOPIC", "A || B", ExpressionType.TAG
+                    "TOPIC", "A || B", ExpressionType.TAG
             );
 
             assertThat(subscriptionData).isNotNull();
@@ -67,7 +68,7 @@ public void testBuildTagSome() {
     public void testBuildSQL() {
         try {
             SubscriptionData subscriptionData = FilterAPI.build(
-                "TOPIC", "a is not null", ExpressionType.SQL92
+                    "TOPIC", "a is not null", ExpressionType.SQL92
             );
 
             assertThat(subscriptionData).isNotNull();
@@ -79,16 +80,8 @@ public void testBuildSQL() {
         }
     }
 
-    @Test
-    public void testBuildSQLWithNullSubString() {
-        try {
-            FilterAPI.build(
-                "TOPIC", null, ExpressionType.SQL92
-            );
-
-            assertThat(Boolean.FALSE).isTrue();
-        } catch (Exception e) {
-            e.printStackTrace();
-        }
+    @Test(expected = IllegalArgumentException.class)
+    public void testBuildSQLWithNullSubString() throws Exception {
+        FilterAPI.build("TOPIC", null, ExpressionType.SQL92);
     }
 }