Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: test/FilterEngine.cpp

Issue 10420020: Made sure FilterEngine instances are always initialized (Closed)
Patch Set: Using a generic messaging mechanism Created May 23, 2013, 6:35 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/BaseJsTest.h ('k') | test/JsEngine.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/FilterEngine.cpp
===================================================================
--- a/test/FilterEngine.cpp
+++ b/test/FilterEngine.cpp
@@ -155,15 +155,8 @@ TEST_F(FilterEngineTest, Matches)
AdblockPlus::FilterPtr match7 = filterEngine->Matches("http://example.org/tpbanner.gif", "IMAGE", "http://example.com/");
ASSERT_TRUE(match7);
ASSERT_EQ(AdblockPlus::Filter::TYPE_BLOCKING, match6->GetType());
AdblockPlus::FilterPtr match8 = filterEngine->Matches("http://example.org/fpbanner.gif", "IMAGE", "http://example.com/");
ASSERT_FALSE(match8);
}
-
-TEST_F(FilterEngineTest, IsInitialized)
-{
- ASSERT_FALSE(filterEngine->IsInitialized());
- jsEngine->Evaluate("require('filterNotifier').FilterNotifier.triggerListeners( 'load')");
- ASSERT_TRUE(filterEngine->IsInitialized());
-}
« no previous file with comments | « test/BaseJsTest.h ('k') | test/JsEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld