Unified Diff: test/filterListener.js
Issue 29926557:
Issue 6994 - Use shortcut matching for location-only filters (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Fix comment
Created Oct. 25, 2018, 9:27 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/matcher.js ('K')
|
« lib/matcher.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: test/filterListener.js |
| =================================================================== |
| --- a/test/filterListener.js |
| +++ b/test/filterListener.js |
| @@ -61,23 +61,27 @@ |
| function checkKnownFilters(test, text, expected) |
| { |
| let result = {}; |
| for (let type of ["blacklist", "whitelist"]) |
| { |
| let matcher = defaultMatcher["_" + type]; |
| let filters = []; |
| - for (let [keyword, set] of matcher._filterByKeyword) |
| + for (let map of [matcher._simpleFiltersByKeyword, |
| + matcher._complexFiltersByKeyword]) |
| { |
| - for (let filter of set) |
| + for (let [keyword, set] of map) |
| { |
| - test.equal(matcher.findKeyword(filter), keyword, |
| - "Keyword of filter " + filter.text); |
| - filters.push(filter.text); |
| + for (let filter of set) |
| + { |
| + test.equal(matcher.findKeyword(filter), keyword, |
| + "Keyword of filter " + filter.text); |
| + filters.push(filter.text); |
| + } |
| } |
| } |
| result[type] = filters; |
| } |
| let elemHide = sandboxedRequire("../lib/elemHide"); |
| result.elemhide = []; |
| for (let filter of elemHide.knownFilters) |
