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

Unified Diff: test/domainRestrictions.js

Issue 29790629: Issue 6690 - Always ignore trailing dot in document domain (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Ignore trailing dot in ElemHide.getSelectorsForDomain Created May 26, 2018, 12:10 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 | « lib/filterClasses.js ('k') | test/elemHide.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/domainRestrictions.js
===================================================================
--- a/test/domainRestrictions.js
+++ b/test/domainRestrictions.js
@@ -65,32 +65,32 @@
};
exports.testDomainRestrictedBlockingFilters = function(test)
{
testActive(test, "foo$domain=example.com", null, false, false);
testActive(test, "foo$domain=example.com", "com", false, true);
testActive(test, "foo$domain=example.com", "example.com", true, true);
testActive(test, "foo$domain=example.com", "example.com.", true, true);
- testActive(test, "foo$domain=example.com.", "example.com", true, true);
- testActive(test, "foo$domain=example.com.", "example.com.", true, true);
+ testActive(test, "foo$domain=example.com.", "example.com", false, false);
+ testActive(test, "foo$domain=example.com.", "example.com.", false, false);
testActive(test, "foo$domain=example.com", "foo.example.com", true, false);
testActive(test, "foo$domain=example.com", "mple.com", false, false);
test.done();
};
exports.testDomainRestrictedHidingRules = function(test)
{
testActive(test, "example.com##foo", null, false, false);
testActive(test, "example.com##foo", "com", false, true);
testActive(test, "example.com##foo", "example.com", true, true);
- testActive(test, "example.com##foo", "example.com.", false, false);
+ testActive(test, "example.com##foo", "example.com.", true, true);
testActive(test, "example.com.##foo", "example.com", false, false);
- testActive(test, "example.com.##foo", "example.com.", true, true);
+ testActive(test, "example.com.##foo", "example.com.", false, false);
testActive(test, "example.com##foo", "foo.example.com", true, false);
testActive(test, "example.com##foo", "mple.com", false, false);
test.done();
};
exports.testBlockingFiltersRestrictedToDomainAndItsSubdomain = function(test)
{
@@ -104,17 +104,17 @@
test.done();
};
exports.testHidingRulesRestrictedToDomainAndItsSubdomain = function(test)
{
testActive(test, "example.com,foo.example.com##foo", null, false, false);
testActive(test, "example.com,foo.example.com##foo", "com", false, true);
testActive(test, "example.com,foo.example.com##foo", "example.com", true, true);
- testActive(test, "example.com,foo.example.com##foo", "example.com.", false, false);
+ testActive(test, "example.com,foo.example.com##foo", "example.com.", true, true);
testActive(test, "example.com,foo.example.com##foo", "foo.example.com", true, false);
testActive(test, "example.com,foo.example.com##foo", "mple.com", false, false);
test.done();
};
exports.testBlockingFiltersWithExceptionForASubdomain = function(test)
{
@@ -152,17 +152,17 @@
test.done();
};
exports.testHidingRulesForDomainButNotItsSubdomain = function(test)
{
testActive(test, "example.com,~foo.example.com##foo", null, false, false);
testActive(test, "example.com,~foo.example.com##foo", "com", false, true);
testActive(test, "example.com,~foo.example.com##foo", "example.com", true, true);
- testActive(test, "example.com,~foo.example.com##foo", "example.com.", false, false);
+ testActive(test, "example.com,~foo.example.com##foo", "example.com.", true, true);
testActive(test, "example.com,~foo.example.com##foo", "foo.example.com", false, false);
testActive(test, "example.com,~foo.example.com##foo", "mple.com", false, false);
test.done();
};
exports.testBlockingFiltersForDomainButNotItsTLD = function(test)
{
@@ -176,17 +176,17 @@
test.done();
};
exports.testHidingRulesForDomainButNotItsTLD = function(test)
{
testActive(test, "example.com,~com##foo", null, false, false);
testActive(test, "example.com,~com##foo", "com", false, true);
testActive(test, "example.com,~com##foo", "example.com", true, true);
- testActive(test, "example.com,~com##foo", "example.com.", false, false);
+ testActive(test, "example.com,~com##foo", "example.com.", true, true);
testActive(test, "example.com,~com##foo", "foo.example.com", true, false);
testActive(test, "example.com,~com##foo", "mple.com", false, false);
test.done();
};
exports.testBlockingFiltersRestrictedToAnUnrelatedDomain = function(test)
{
« no previous file with comments | « lib/filterClasses.js ('k') | test/elemHide.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld