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..ddc5044e89554d4ca7b7b984c57567da42288d1c 100644
--- a/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
+++ b/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
@@ -73,17 +73,58 @@ 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(), "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");
+        results = suite.run(new TestLogger.NullLogger(), "TestClass*:*test1-*test2");
         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());
+
+        results = suite.run(new TestLogger.NullLogger(), "-TestClass.test1-TestClass.test2");

[cjhopman, 2015/03/31 20:27:54]

Ah, here's a bit of the confusion. From my reading of
https://code.google.com/p/googletest/wiki/AdvancedGuide#Running_a_Subset_of_t...,
the format for the filter is:

pos1:pos2:pos3-neg1:neg2:neg3

[nyquist, 2015/03/31 23:03:05]

Yeah, that's how I understand it as well.

[wychen, 2015/04/01 15:44:07]

Oh! I misread it. Fixed.
I thought it was something like this: "pos1-neg1:pos2:pos3-neg2-neg3".

+        assertTrue(results.get("TestClass").getResults().get("test1").skipped());
+        assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+        assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
     }
 }