[Pkg-mozext-commits] [adblock-plus] 17/87: Issue 3659 - Remove locale dependency from filter classes
David Prévot
taffit at moszumanska.debian.org
Sat Apr 30 17:59:03 UTC 2016
This is an automated email from the git hooks/post-receive script.
taffit pushed a commit to branch master
in repository adblock-plus.
commit adb6d087085237f8acbbfd5849297f25b8b88dfd
Author: Wladimir Palant <trev at adblockplus.org>
Date: Wed Feb 17 16:51:15 2016 +0100
Issue 3659 - Remove locale dependency from filter classes
---
.gitignore | 1 +
.hgignore | 1 +
README.md | 13 ++
lib/filterClasses.js | 11 +-
package.json | 12 ++
test/filterClasses.js | 345 ++++++++++++++++++++++++++++++++++++++++++++++++++
test_wrapper.js | 5 +
7 files changed, 382 insertions(+), 6 deletions(-)
diff --git a/.gitignore b/.gitignore
new file mode 100644
index 0000000..2ccbe46
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1 @@
+/node_modules/
diff --git a/.hgignore b/.hgignore
new file mode 100644
index 0000000..a4a0084
--- /dev/null
+++ b/.hgignore
@@ -0,0 +1 @@
+^node_modules$
diff --git a/README.md b/README.md
index 50d3d14..ffddf46 100644
--- a/README.md
+++ b/README.md
@@ -5,3 +5,16 @@ This repository contains the generic Adblock Plus code that's shared between
platforms. This repository is not designed to be used directly, but instead to
serve as a dependency for `adblockplus`, `adblockpluschrome` and
`libadblockplus`.
+
+Running the unit tests
+----------------------
+
+*Note*: The unit test suite isn't complete yet, it is in the process of being
+migrated from the
+[adblockplustests repository](https://hg.adblockplus.org/adblockplustests/).
+
+In order to run the unit test suite you need a reasonably recent
+[Node.js version](https://nodejs.org/). Once Node.js is installed please run
+`npm install` in the repository directory in order to install the required
+dependencies. After that you can run `npm test` which will execute all tests
+in the `test` directory of the repository.
diff --git a/lib/filterClasses.js b/lib/filterClasses.js
index e192192..2f68235 100644
--- a/lib/filterClasses.js
+++ b/lib/filterClasses.js
@@ -20,7 +20,6 @@
*/
let {FilterNotifier} = require("filterNotifier");
-let {Utils} = require("utils");
/**
* Abstract base class for filters
@@ -737,7 +736,7 @@ RegExpFilter.fromText = function(text)
else if (option == "SITEKEY" && typeof value != "undefined")
sitekeys = value;
else
- return new InvalidFilter(origText, "Unknown option " + option.toLowerCase());
+ return new InvalidFilter(origText, "filter_unknown_option");
}
}
@@ -750,7 +749,7 @@ RegExpFilter.fromText = function(text)
}
catch (e)
{
- return new InvalidFilter(origText, e);
+ return new InvalidFilter(origText, "filter_invalid_regexp");
}
};
@@ -927,7 +926,7 @@ ElemHideBase.fromText = function(text, domain, isException, tagName, attrRules,
else
{
if (id)
- return new InvalidFilter(text, Utils.getString("filter_elemhide_duplicate_id"));
+ return new InvalidFilter(text, "filter_elemhide_duplicate_id");
id = rule;
}
@@ -939,7 +938,7 @@ ElemHideBase.fromText = function(text, domain, isException, tagName, attrRules,
else if (tagName || additional)
selector = tagName + additional;
else
- return new InvalidFilter(text, Utils.getString("filter_elemhide_nocriteria"));
+ return new InvalidFilter(text, "filter_elemhide_nocriteria");
}
if (isException)
@@ -951,7 +950,7 @@ ElemHideBase.fromText = function(text, domain, isException, tagName, attrRules,
// CSS property filters are inefficient so we need to make sure that
// they're only applied if they specify active domains
if (!/,[^~][^,.]*\.[^,]/.test("," + domain))
- return new InvalidFilter(text, Utils.getString("filter_cssproperty_nodomain"));
+ return new InvalidFilter(text, "filter_cssproperty_nodomain");
return new CSSPropertyFilter(text, domain, selector, match[2],
selector.substr(0, match.index),
diff --git a/package.json b/package.json
new file mode 100644
index 0000000..533d44e
--- /dev/null
+++ b/package.json
@@ -0,0 +1,12 @@
+{
+ "name": "adblockpluscore",
+ "license": "GPL-3.0",
+ "dependencies": {},
+ "devDependencies": {
+ "node-babel": "0.1.2",
+ "nodeunit": "0.9.1"
+ },
+ "scripts": {
+ "test": "NODE_PATH=lib node --harmony test_wrapper.js"
+ }
+}
diff --git a/test/filterClasses.js b/test/filterClasses.js
new file mode 100644
index 0000000..0ec40c1
--- /dev/null
+++ b/test/filterClasses.js
@@ -0,0 +1,345 @@
+/*
+ * This file is part of Adblock Plus <https://adblockplus.org/>,
+ * Copyright (C) 2006-2016 Eyeo GmbH
+ *
+ * Adblock Plus is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * Adblock Plus is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+"use strict";
+
+let {
+ Filter, InvalidFilter, CommentFilter, ActiveFilter, RegExpFilter,
+ BlockingFilter, WhitelistFilter, ElemHideBase, ElemHideFilter,
+ ElemHideException, CSSPropertyFilter
+} = require("../lib/filterClasses");
+
+function serializeFilter(filter)
+{
+ // Filter serialization only writes out essential properties, need to do a full serialization here
+ let result = [];
+ result.push("text=" + filter.text);
+ if (filter instanceof InvalidFilter)
+ {
+ result.push("type=invalid");
+ result.push("reason=" + filter.reason);
+ }
+ else if (filter instanceof CommentFilter)
+ {
+ result.push("type=comment");
+ }
+ else if (filter instanceof ActiveFilter)
+ {
+ result.push("disabled=" + filter.disabled);
+ result.push("lastHit=" + filter.lastHit);
+ result.push("hitCount=" + filter.hitCount);
+
+ let domains = [];
+ if (filter.domains)
+ {
+ for (let domain in filter.domains)
+ if (domain != "")
+ domains.push(filter.domains[domain] ? domain : "~" + domain);
+ }
+ result.push("domains=" + domains.sort().join("|"));
+
+ if (filter instanceof RegExpFilter)
+ {
+ result.push("regexp=" + filter.regexp.source);
+ result.push("contentType=" + filter.contentType);
+ result.push("matchCase=" + filter.matchCase);
+
+ let sitekeys = filter.sitekeys || [];
+ result.push("sitekeys=" + sitekeys.slice().sort().join("|"));
+
+ result.push("thirdParty=" + filter.thirdParty);
+ if (filter instanceof BlockingFilter)
+ {
+ result.push("type=filterlist");
+ result.push("collapse=" + filter.collapse);
+ }
+ else if (filter instanceof WhitelistFilter)
+ {
+ result.push("type=whitelist");
+ }
+ }
+ else if (filter instanceof ElemHideBase)
+ {
+ if (filter instanceof ElemHideFilter)
+ result.push("type=elemhide");
+ else if (filter instanceof ElemHideException)
+ result.push("type=elemhideexception");
+ else if (filter instanceof CSSPropertyFilter)
+ {
+ result.push("type=cssrule");
+ result.push("prefix=" + (filter.selectorPrefix || ""));
+ result.push("regexp=" + filter.regexpString);
+ result.push("suffix=" + (filter.selectorSuffix || ""));
+ }
+
+ result.push("selectorDomain=" + (filter.selectorDomain || ""));
+ result.push("selector=" + filter.selector);
+ }
+ }
+ return result;
+}
+
+function addDefaults(expected)
+{
+ let type = null;
+ let hasProperty = {};
+ for (let entry of expected)
+ {
+ if (/^type=(.*)/.test(entry))
+ type = RegExp.$1;
+ else if (/^(\w+)/.test(entry))
+ hasProperty[RegExp.$1] = true;
+ }
+
+ function addProperty(prop, value)
+ {
+ if (!(prop in hasProperty))
+ expected.push(prop + "=" + value);
+ }
+
+ if (type == "whitelist" || type == "filterlist" || type == "elemhide" || type == "elemhideexception" || type == "cssrule")
+ {
+ addProperty("disabled", "false");
+ addProperty("lastHit", "0");
+ addProperty("hitCount", "0");
+ }
+ if (type == "whitelist" || type == "filterlist")
+ {
+ addProperty("contentType", 0x7FFFFFFF & ~(
+ RegExpFilter.typeMap.DOCUMENT | RegExpFilter.typeMap.ELEMHIDE |
+ RegExpFilter.typeMap.POPUP | RegExpFilter.typeMap.GENERICHIDE |
+ RegExpFilter.typeMap.GENERICBLOCK
+ ));
+ addProperty("matchCase", "false");
+ addProperty("thirdParty", "null");
+ addProperty("domains", "");
+ addProperty("sitekeys", "");
+ }
+ if (type == "filterlist")
+ {
+ addProperty("collapse", "null");
+ }
+ if (type == "elemhide" || type == "elemhideexception" || type == "cssrule")
+ {
+ addProperty("selectorDomain", "");
+ addProperty("domains", "");
+ }
+ if (type == "cssrule")
+ {
+ addProperty("regexp", "");
+ addProperty("prefix", "");
+ addProperty("suffix", "");
+ }
+}
+
+function compareFilter(test, text, expected, postInit)
+{
+ addDefaults(expected);
+
+ let filter = Filter.fromText(text);
+ if (postInit)
+ postInit(filter)
+ let result = serializeFilter(filter);
+ test.equal(result.sort().join("\n"), expected.sort().join("\n"), text);
+
+ // Test round-trip
+ let filter2;
+ let buffer = [];
+ filter.serialize(buffer);
+ if (buffer.length)
+ {
+ let map = {__proto__: null};
+ for (let line of buffer.slice(1))
+ {
+ if (/(.*?)=(.*)/.test(line))
+ map[RegExp.$1] = RegExp.$2;
+ }
+ filter2 = Filter.fromObject(map);
+ }
+ else
+ {
+ filter2 = Filter.fromText(filter.text);
+ }
+
+ test.equal(serializeFilter(filter).join("\n"), serializeFilter(filter2).join("\n"), text + " deserialization");
+}
+
+exports.testFilterClassDefinitions = function(test)
+{
+ test.equal(typeof Filter, "function", "typeof Filter");
+ test.equal(typeof InvalidFilter, "function", "typeof InvalidFilter");
+ test.equal(typeof CommentFilter, "function", "typeof CommentFilter");
+ test.equal(typeof ActiveFilter, "function", "typeof ActiveFilter");
+ test.equal(typeof RegExpFilter, "function", "typeof RegExpFilter");
+ test.equal(typeof BlockingFilter, "function", "typeof BlockingFilter");
+ test.equal(typeof WhitelistFilter, "function", "typeof WhitelistFilter");
+ test.equal(typeof ElemHideBase, "function", "typeof ElemHideBase");
+ test.equal(typeof ElemHideFilter, "function", "typeof ElemHideFilter");
+ test.equal(typeof ElemHideException, "function", "typeof ElemHideException");
+ test.equal(typeof CSSPropertyFilter, "function", "typeof CSSPropertyFilter");
+
+ test.done();
+};
+
+exports.testComments = function(test)
+{
+ compareFilter(test, "!asdf", ["type=comment", "text=!asdf"]);
+ compareFilter(test, "!foo#bar", ["type=comment", "text=!foo#bar"]);
+ compareFilter(test, "!foo##bar", ["type=comment", "text=!foo##bar"]);
+
+ test.done();
+};
+
+exports.testInvalidFilters = function(test)
+{
+ compareFilter(test, "/??/", ["type=invalid", "text=/??/", "reason=filter_invalid_regexp"]);
+ compareFilter(test, "asd$foobar", ["type=invalid", "text=asd$foobar", "reason=filter_unknown_option"]);
+ compareFilter(test, "#dd(asd)(ddd)", ["type=invalid", "text=#dd(asd)(ddd)", "reason=filter_elemhide_duplicate_id"]);
+ compareFilter(test, "#*", ["type=invalid", "text=#*", "reason=filter_elemhide_nocriteria"]);
+
+ function compareCSSRule(domains)
+ {
+ let filterText = domains + "##[-abp-properties='abc']";
+ compareFilter(test, filterText, ["type=invalid", "text=" + filterText, "reason=filter_cssproperty_nodomain"]);
+ }
+ compareCSSRule("");
+ compareCSSRule("~foo.com");
+ compareCSSRule("~foo.com,~bar.com");
+ compareCSSRule("foo");
+ compareCSSRule("~foo.com,bar");
+
+ test.done();
+};
+
+exports.testFiltersWithState = function(test)
+{
+ compareFilter(test, "blabla", ["type=filterlist", "text=blabla", "regexp=blabla"]);
+ compareFilter(test, "blabla_default", ["type=filterlist", "text=blabla_default", "regexp=blabla_default"], function(filter)
+ {
+ filter.disabled = false;
+ filter.hitCount = 0;
+ filter.lastHit = 0;
+ });
+ compareFilter(test, "blabla_non_default", ["type=filterlist", "text=blabla_non_default", "regexp=blabla_non_default", "disabled=true", "hitCount=12", "lastHit=20"], function(filter)
+ {
+ filter.disabled = true;
+ filter.hitCount = 12;
+ filter.lastHit = 20;
+ });
+
+ test.done();
+};
+
+let t = RegExpFilter.typeMap;
+let defaultTypes = 0x7FFFFFFF & ~(t.ELEMHIDE | t.DOCUMENT | t.POPUP | t.GENERICHIDE | t.GENERICBLOCK);
+
+exports.testSpecialCharacters = function(test)
+{
+ compareFilter(test, "/ddd|f?a[s]d/", ["type=filterlist", "text=/ddd|f?a[s]d/", "regexp=ddd|f?a[s]d"]);
+ compareFilter(test, "*asdf*d**dd*", ["type=filterlist", "text=*asdf*d**dd*", "regexp=asdf.*d.*dd"]);
+ compareFilter(test, "|*asd|f*d**dd*|", ["type=filterlist", "text=|*asd|f*d**dd*|", "regexp=^.*asd\\|f.*d.*dd.*$"]);
+ compareFilter(test, "dd[]{}$%<>&()d", ["type=filterlist", "text=dd[]{}$%<>&()d", "regexp=dd\\[\\]\\{\\}\\$\\%\\<\\>\\&\\(\\)d"]);
+
+ compareFilter(test, "@@/ddd|f?a[s]d/", ["type=whitelist", "text=@@/ddd|f?a[s]d/", "regexp=ddd|f?a[s]d", "contentType=" + defaultTypes]);
+ compareFilter(test, "@@*asdf*d**dd*", ["type=whitelist", "text=@@*asdf*d**dd*", "regexp=asdf.*d.*dd", "contentType=" + defaultTypes]);
+ compareFilter(test, "@@|*asd|f*d**dd*|", ["type=whitelist", "text=@@|*asd|f*d**dd*|", "regexp=^.*asd\\|f.*d.*dd.*$", "contentType=" + defaultTypes]);
+ compareFilter(test, "@@dd[]{}$%<>&()d", ["type=whitelist", "text=@@dd[]{}$%<>&()d", "regexp=dd\\[\\]\\{\\}\\$\\%\\<\\>\\&\\(\\)d", "contentType=" + defaultTypes]);
+
+ test.done();
+};
+
+exports.testFilterOptions = function(test)
+{
+ compareFilter(test, "bla$match-case,script,other,third-party,domain=foo.com,sitekey=foo", ["type=filterlist", "text=bla$match-case,script,other,third-party,domain=foo.com,sitekey=foo", "regexp=bla", "matchCase=true", "contentType=" + (t.SCRIPT | t.OTHER), "thirdParty=true", "domains=FOO.COM", "sitekeys=FOO"]);
+ compareFilter(test, "bla$~match-case,~script,~other,~third-party,domain=~bar.com", ["type=filterlist", "text=bla$~match-case,~script,~other,~third-party,domain=~bar.com", "regexp=bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER)), "thirdParty=false", "domains=~BAR.COM"]);
+ compareFilter(test, "@@bla$match-case,script,other,third-party,domain=foo.com|bar.com|~bar.foo.com|~foo.bar.com,sitekey=foo|bar", ["type=whitelist", "text=@@bla$match-case,script,other,third-party,domain=foo.com|bar.com|~bar.foo.com|~foo.bar.com,sitekey=foo|bar", "regexp=bla", "matchCase=true", "contentType=" + (t.SCRIPT | t.OTHER), "thirdParty=true", "domains=BAR.COM|FOO.COM|~BAR.FOO.COM|~FOO.BAR.COM", "sitekeys=BAR|FOO"]);
+
+ // background and image should be the same for backwards compatibility
+ compareFilter(test, "bla$image", ["type=filterlist", "text=bla$image", "regexp=bla", "contentType=" + (t.IMAGE)]);
+ compareFilter(test, "bla$background", ["type=filterlist", "text=bla$background", "regexp=bla", "contentType=" + (t.IMAGE)]);
+ compareFilter(test, "bla$~image", ["type=filterlist", "text=bla$~image", "regexp=bla", "contentType=" + (defaultTypes & ~t.IMAGE)]);
+ compareFilter(test, "bla$~background", ["type=filterlist", "text=bla$~background", "regexp=bla", "contentType=" + (defaultTypes & ~t.IMAGE)]);
+
+ compareFilter(test, "@@bla$~script,~other", ["type=whitelist", "text=@@bla$~script,~other", "regexp=bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER))]);
+ compareFilter(test, "@@http://bla$~script,~other", ["type=whitelist", "text=@@http://bla$~script,~other", "regexp=http\\:\\/\\/bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER))]);
+ compareFilter(test, "@@|ftp://bla$~script,~other", ["type=whitelist", "text=@@|ftp://bla$~script,~other", "regexp=^ftp\\:\\/\\/bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER))]);
+ compareFilter(test, "@@bla$~script,~other,document", ["type=whitelist", "text=@@bla$~script,~other,document", "regexp=bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER) | t.DOCUMENT)]);
+ compareFilter(test, "@@bla$~script,~other,~document", ["type=whitelist", "text=@@bla$~script,~other,~document", "regexp=bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER))]);
+ compareFilter(test, "@@bla$document", ["type=whitelist", "text=@@bla$document", "regexp=bla", "contentType=" + t.DOCUMENT]);
+ compareFilter(test, "@@bla$~script,~other,elemhide", ["type=whitelist", "text=@@bla$~script,~other,elemhide", "regexp=bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER) | t.ELEMHIDE)]);
+ compareFilter(test, "@@bla$~script,~other,~elemhide", ["type=whitelist", "text=@@bla$~script,~other,~elemhide", "regexp=bla", "contentType=" + (defaultTypes & ~(t.SCRIPT | t.OTHER))]);
+ compareFilter(test, "@@bla$elemhide", ["type=whitelist", "text=@@bla$elemhide", "regexp=bla", "contentType=" + t.ELEMHIDE]);
+
+ compareFilter(test, "@@bla$~script,~other,donottrack", ["type=invalid", "text=@@bla$~script,~other,donottrack", "reason=filter_unknown_option"]);
+ compareFilter(test, "@@bla$~script,~other,~donottrack", ["type=invalid", "text=@@bla$~script,~other,~donottrack", "reason=filter_unknown_option"]);
+ compareFilter(test, "@@bla$donottrack", ["type=invalid", "text=@@bla$donottrack", "reason=filter_unknown_option"]);
+ compareFilter(test, "@@bla$foobar", ["type=invalid", "text=@@bla$foobar", "reason=filter_unknown_option"]);
+ compareFilter(test, "@@bla$image,foobar", ["type=invalid", "text=@@bla$image,foobar", "reason=filter_unknown_option"]);
+ compareFilter(test, "@@bla$foobar,image", ["type=invalid", "text=@@bla$foobar,image", "reason=filter_unknown_option"]);
+
+ test.done();
+};
+
+exports.testElementHidingRules = function(test)
+{
+ compareFilter(test, "#ddd", ["type=elemhide", "text=#ddd", "selector=ddd"]);
+ compareFilter(test, "#ddd(fff)", ["type=elemhide", "text=#ddd(fff)", "selector=ddd.fff,ddd#fff"]);
+ compareFilter(test, "#ddd(foo=bar)(foo2^=bar2)(foo3*=bar3)(foo4$=bar4)", ["type=elemhide", "text=#ddd(foo=bar)(foo2^=bar2)(foo3*=bar3)(foo4$=bar4)", 'selector=ddd[foo="bar"][foo2^="bar2"][foo3*="bar3"][foo4$="bar4"]']);
+ compareFilter(test, "#ddd(fff)(foo=bar)", ["type=elemhide", "text=#ddd(fff)(foo=bar)", 'selector=ddd.fff[foo="bar"],ddd#fff[foo="bar"]']);
+ compareFilter(test, "#*(fff)", ["type=elemhide", "text=#*(fff)", "selector=.fff,#fff"]);
+ compareFilter(test, "#*(foo=bar)", ["type=elemhide", "text=#*(foo=bar)", 'selector=[foo="bar"]']);
+ compareFilter(test, "##body > div:first-child", ["type=elemhide", "text=##body > div:first-child", "selector=body > div:first-child"]);
+ compareFilter(test, "foo#ddd", ["type=elemhide", "text=foo#ddd", "selectorDomain=foo", "selector=ddd", "domains=FOO"]);
+ compareFilter(test, "foo,bar#ddd", ["type=elemhide", "text=foo,bar#ddd", "selectorDomain=foo,bar", "selector=ddd", "domains=BAR|FOO"]);
+ compareFilter(test, "foo,~bar#ddd", ["type=elemhide", "text=foo,~bar#ddd", "selectorDomain=foo", "selector=ddd", "domains=FOO|~BAR"]);
+ compareFilter(test, "foo,~baz,bar#ddd", ["type=elemhide", "text=foo,~baz,bar#ddd", "selectorDomain=foo,bar", "selector=ddd", "domains=BAR|FOO|~BAZ"]);
+
+ test.done();
+};
+
+exports.testElementHidingExceptions = function(test)
+{
+ compareFilter(test, "#@ddd", ["type=elemhideexception", "text=#@ddd", "selector=ddd"]);
+ compareFilter(test, "#@ddd(fff)", ["type=elemhideexception", "text=#@ddd(fff)", "selector=ddd.fff,ddd#fff"]);
+ compareFilter(test, "#@ddd(foo=bar)(foo2^=bar2)(foo3*=bar3)(foo4$=bar4)", ["type=elemhideexception", "text=#@ddd(foo=bar)(foo2^=bar2)(foo3*=bar3)(foo4$=bar4)", 'selector=ddd[foo="bar"][foo2^="bar2"][foo3*="bar3"][foo4$="bar4"]']);
+ compareFilter(test, "#@ddd(fff)(foo=bar)", ["type=elemhideexception", "text=#@ddd(fff)(foo=bar)", 'selector=ddd.fff[foo="bar"],ddd#fff[foo="bar"]']);
+ compareFilter(test, "#@*(fff)", ["type=elemhideexception", "text=#@*(fff)", "selector=.fff,#fff"]);
+ compareFilter(test, "#@*(foo=bar)", ["type=elemhideexception", "text=#@*(foo=bar)", 'selector=[foo="bar"]']);
+ compareFilter(test, "#@#body > div:first-child", ["type=elemhideexception", "text=#@#body > div:first-child", "selector=body > div:first-child"]);
+ compareFilter(test, "foo#@ddd", ["type=elemhideexception", "text=foo#@ddd", "selectorDomain=foo", "selector=ddd", "domains=FOO"]);
+ compareFilter(test, "foo,bar#@ddd", ["type=elemhideexception", "text=foo,bar#@ddd", "selectorDomain=foo,bar", "selector=ddd", "domains=BAR|FOO"]);
+ compareFilter(test, "foo,~bar#@ddd", ["type=elemhideexception", "text=foo,~bar#@ddd", "selectorDomain=foo", "selector=ddd", "domains=FOO|~BAR"]);
+ compareFilter(test, "foo,~baz,bar#@ddd", ["type=elemhideexception", "text=foo,~baz,bar#@ddd", "selectorDomain=foo,bar", "selector=ddd", "domains=BAR|FOO|~BAZ"]);
+
+ test.done();
+};
+
+exports.testCSSPropertyFilters = function(test)
+{
+ // Check valid domain combinations
+ compareFilter(test, "foo.com##[-abp-properties='abc']", ["type=cssrule", "text=foo.com##[-abp-properties='abc']", "selectorDomain=foo.com", "selector=[-abp-properties='abc']", "domains=FOO.COM", "regexp=abc"]);
+ compareFilter(test, "foo.com,~bar.com##[-abp-properties='abc']", ["type=cssrule", "text=foo.com,~bar.com##[-abp-properties='abc']", "selectorDomain=foo.com", "selector=[-abp-properties='abc']", "domains=FOO.COM|~BAR.COM", "regexp=abc"]);
+ compareFilter(test, "foo.com,~bar##[-abp-properties='abc']", ["type=cssrule", "text=foo.com,~bar##[-abp-properties='abc']", "selectorDomain=foo.com", "selector=[-abp-properties='abc']", "domains=FOO.COM|~BAR", "regexp=abc"]);
+ compareFilter(test, "~foo.com,bar.com##[-abp-properties='abc']", ["type=cssrule", "text=~foo.com,bar.com##[-abp-properties='abc']", "selectorDomain=bar.com", "selector=[-abp-properties='abc']", "domains=BAR.COM|~FOO.COM", "regexp=abc"]);
+
+ compareFilter(test, "##[-abp-properties='']", ["type=elemhide", "text=##[-abp-properties='']", "selector=[-abp-properties='']"]);
+ compareFilter(test, "foo.com#@#[-abp-properties='abc']", ["type=elemhideexception", "text=foo.com#@#[-abp-properties='abc']", "selectorDomain=foo.com", "selector=[-abp-properties='abc']", "domains=FOO.COM"]);
+ compareFilter(test, "foo.com##aaa [-abp-properties='abc'] bbb", ["type=cssrule", "text=foo.com##aaa [-abp-properties='abc'] bbb", "selectorDomain=foo.com", "selector=aaa [-abp-properties='abc'] bbb", "domains=FOO.COM", "prefix=aaa ", "regexp=abc", "suffix= bbb"]);
+ compareFilter(test, "foo.com##[-abp-properties='|background-image: url(data:*)']", ["type=cssrule", "text=foo.com##[-abp-properties='|background-image: url(data:*)']", "selectorDomain=foo.com", "selector=[-abp-properties='|background-image: url(data:*)']", "domains=FOO.COM", "regexp=^background\\-image\\:\\ url\\(data\\:.*\\)"]);
+
+ test.done();
+};
diff --git a/test_wrapper.js b/test_wrapper.js
new file mode 100644
index 0000000..4bbf604
--- /dev/null
+++ b/test_wrapper.js
@@ -0,0 +1,5 @@
+"use strict";
+
+require("node-babel")();
+
+require("./node_modules/.bin/nodeunit");
--
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/adblock-plus.git
More information about the Pkg-mozext-commits
mailing list