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

Unified Diff: test/filterClasses.js

Issue 29333474: Issue 4125 - [emscripten] Convert filter classes to C++ (Closed)
Patch Set: Fixed some bugs and exposed/tested more properties Created Jan. 15, 2016, 3:40 p.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
« compiled/Filter.cpp ('K') | « test.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/filterClasses.js
===================================================================
new file mode 100644
--- /dev/null
+++ b/test/filterClasses.js
@@ -0,0 +1,80 @@
+"use strict";
+
+let filterClasses = require("../lib/filterClassesNew");
+let Filter = filterClasses.Filter;
+let InvalidFilter = filterClasses.InvalidFilter;
+let CommentFilter = filterClasses.CommentFilter;
+let RegExpFilter = filterClasses.RegExpFilter;
+let ElemHideFilter = filterClasses.ElemHideFilter;
+let ElemHideException = filterClasses.ElemHideException;
+
+exports.testFromText = function(test)
+{
+ let tests = [
+ ["!asdf", CommentFilter, "comment"],
+ ["asdf", RegExpFilter, "blocking"],
+ ["##asdf", ElemHideFilter, "elemhide"],
+ ["#@#asdf", ElemHideException, "elemhideexception"],
+ ["foobar##asdf", ElemHideFilter, "elemhide"],
+ ["foobar#@#asdf", ElemHideException, "elemhideexception"],
+ ["foobar##a", ElemHideFilter, "elemhide"],
+ ["foobar#@#a", ElemHideException, "elemhideexception"],
+
+ ["foobar#asdf", RegExpFilter, "blocking"],
+ ["foobar|foobas##asdf", RegExpFilter, "blocking"],
+ ["foobar##asdf{asdf}", RegExpFilter, "blocking"],
+ ["foobar##", RegExpFilter, "blocking"],
+ ["foobar#@#", RegExpFilter, "blocking"],
+ ];
+ for (let [text, type, typeName] of tests)
+ {
+ let filter = Filter.fromText(text);
+ test.ok(filter instanceof Filter, "Got filter for " + text);
+ test.equal(filter.text, text, "Correct filter text for " + text);
+ test.ok(filter instanceof type, "Correct filter type for " + text);
+ test.equal(filter.type, typeName, "Type name for " + text + " is " + typeName);
+ }
+ test.done();
+}
+
+exports.testNormalize = function(test)
+{
+ let tests = [
+ [" foo bar ", "foobar"],
+ ["foobar", "foobar"],
+ [" ! comment something ", "! comment something"],
+ [" ! \n comment something ", "! comment something"],
+ [" foo , bar ## foo > bar ", "foo,bar##foo > bar"],
+ [" foo , bar #@# foo > bar ", "foo,bar#@#foo > bar"],
+ ];
+ for (let [text, expected] of tests)
+ test.equal(Filter.normalize(text), expected);
+ test.done();
+}
+
+exports.testActiveFilter = function(test)
+{
+ let filter1 = Filter.fromText("asdf");
+ let filter1copy = Filter.fromText("asdf");
+ let filter2 = Filter.fromText("##foobar");
+
+ test.ok(!filter1.disabled && !filter1copy.disabled && !filter2.disabled, "Filters are initially enabled");
+ filter1.disabled = true;
+ test.ok(filter1.disabled, "Disabling filter works");
+ test.ok(filter1copy.disabled, "Filter copies are also disabled");
+ test.ok(!filter2.disabled, "Disabling one filter doesn't disable others");
+
+ test.ok(filter1.hitCount === 0 && filter1copy.hitCount === 0 && filter2.hitCount === 0, "Filters have no hit initially");
+ filter1.hitCount = 5;
+ test.equal(filter1.hitCount, 5, "Changing hit count works");
+ test.equal(filter1copy.hitCount, 5, "Hit count of filter copies is also changed");
+ test.equal(filter2.hitCount, 0, "Hit count of other filters isn't affected");
+
+ test.ok(filter1.lastHit === 0 && filter1copy.lastHit === 0 && filter2.lastHit === 0, "Filters have no last hit time initially");
+ filter1.lastHit = 10;
+ test.equal(filter1.lastHit, 10, "Changing last hit time works");
+ test.equal(filter1copy.lastHit, 10, "Last hit time of filter copies is also changed");
+ test.equal(filter2.lastHit, 0, "Last hit time of other filters isn't affected");
+
+ test.done();
+}
« compiled/Filter.cpp ('K') | « test.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld