Use gtest_filter syntax in jstest filter

javatests/org/chromium/distiller/JsTestSuiteBaseTest.java

Index: javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
diff --git a/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java b/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
index 1cac261befdd3671e75555777da2c0d3feef666a..718edaf5f61e2e36436df2dbbe12142e7c0392b8 100644
--- a/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
+++ b/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
@@ -73,17 +73,53 @@ public class JsTestSuiteBaseTest extends JsTestCase {
         suite.addTestCase(null, "TestClass")
             .addTest(null, "test1")
             .addTest(null, "test2");
+        suite.addTestCase(null, "Test2Class")
+            .addTest(null, "test1");
+
         Map<String, JsTestSuiteBase.TestCaseResults> results = suite.run(
-                new TestLogger.NullLogger(), ".*TestClass.*");
+                new TestLogger.NullLogger(), "TestClass.*");
         assertFalse(results.get("TestClass").getResults().get("test1").skipped());
         assertFalse(results.get("TestClass").getResults().get("test2").skipped());
+        assertTrue(results.get("Test2Class").getResults().get("test1").skipped());
 
-        results = suite.run(new TestLogger.NullLogger(), ".*NotTestClass.*");
+        results = suite.run(new TestLogger.NullLogger(), "*NotTestClass*");
         assertTrue(results.get("TestClass").getResults().get("test1").skipped());
         assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+        assertTrue(results.get("Test2Class").getResults().get("test1").skipped());
+
+        results = suite.run(new TestLogger.NullLogger(), "TestClass.test1");
+        assertFalse(results.get("TestClass").getResults().get("test1").skipped());
+        assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+        assertTrue(results.get("Test2Class").getResults().get("test1").skipped());
+
+        results = suite.run(new TestLogger.NullLogger(), "TestClass.*-*.test1");
+        assertTrue(results.get("TestClass").getResults().get("test1").skipped());
+        assertFalse(results.get("TestClass").getResults().get("test2").skipped());
+        assertTrue(results.get("Test2Class").getResults().get("test1").skipped());
+
+        results = suite.run(new TestLogger.NullLogger(), ":*.test1::*.test2:");
+        assertFalse(results.get("TestClass").getResults().get("test1").skipped());
+        assertFalse(results.get("TestClass").getResults().get("test2").skipped());
+        assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
 
-        results = suite.run(new TestLogger.NullLogger(), ".*TestClass.test1");
+        results = suite.run(new TestLogger.NullLogger(), "T*C*-*tes?2");
         assertFalse(results.get("TestClass").getResults().get("test1").skipped());
         assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+        assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
+
+        results = suite.run(new TestLogger.NullLogger(), "TestClass*:*test1-*test2");

[cjhopman, 2015/03/31 17:55:41]

Should add a test with multiple negative patterns.

[wychen, 2015/03/31 18:26:49]

Done.

+        assertFalse(results.get("TestClass").getResults().get("test1").skipped());
+        assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+        assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
+
+        results = suite.run(new TestLogger.NullLogger(), "-*1");
+        assertTrue(results.get("TestClass").getResults().get("test1").skipped());
+        assertFalse(results.get("TestClass").getResults().get("test2").skipped());
+        assertTrue(results.get("Test2Class").getResults().get("test1").skipped());
+
+        results = suite.run(new TestLogger.NullLogger(), "-*.e*");
+        assertFalse(results.get("TestClass").getResults().get("test1").skipped());
+        assertFalse(results.get("TestClass").getResults().get("test2").skipped());
+        assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
     }
 }