Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(297)

Unified Diff: javatests/org/chromium/distiller/JsTestSuiteBaseTest.java

Issue 1047853004: Use gtest_filter syntax in jstest filter (Closed) Base URL: git@github.com:chromium/dom-distiller.git@master
Patch Set: address comments Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « javatests/org/chromium/distiller/JsTestSuiteBase.java ('k') | run_jstests.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..7745653b303155e0ae991a3456c5ae7be43c8a41 100644
--- a/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
+++ b/javatests/org/chromium/distiller/JsTestSuiteBaseTest.java
@@ -73,17 +73,81 @@ 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*");
+ 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(), ".*NotTestClass.*");
+ 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-*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(), ".*TestClass.test1");
+ 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");
+ assertTrue(results.get("TestClass").getResults().get("test1").skipped());
assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+ assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
+
+ suite.addTestCase(null, "Test3Class")
+ .addTest(null, "test1")
+ .addTest(null, "test2")
+ .addTest(null, "test3");
+
+ results = suite.run(new TestLogger.NullLogger(),
+ "Test?Class.test*-Test2Class.test1:Test*Class.test3");
+ assertTrue(results.get("TestClass").getResults().get("test1").skipped());
+ assertTrue(results.get("TestClass").getResults().get("test2").skipped());
+ assertTrue(results.get("Test2Class").getResults().get("test1").skipped());
+ assertFalse(results.get("Test3Class").getResults().get("test1").skipped());
+ assertFalse(results.get("Test3Class").getResults().get("test2").skipped());
+ assertTrue(results.get("Test3Class").getResults().get("test3").skipped());
+
+ results = suite.run(new TestLogger.NullLogger(),
+ "TestClass.*:Test?Class.test1:Test3Class.test3-TestClass.test1:Test3Class.test*");
+ assertTrue(results.get("TestClass").getResults().get("test1").skipped());
+ assertFalse(results.get("TestClass").getResults().get("test2").skipped());
+ assertFalse(results.get("Test2Class").getResults().get("test1").skipped());
+ assertTrue(results.get("Test3Class").getResults().get("test1").skipped());
+ assertTrue(results.get("Test3Class").getResults().get("test2").skipped());
+ assertTrue(results.get("Test3Class").getResults().get("test3").skipped());
}
}
« no previous file with comments | « javatests/org/chromium/distiller/JsTestSuiteBase.java ('k') | run_jstests.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698