Unified Diff: test/filterListener.js
Issue 29907586:
Issue 6994 - Use shortcut matching for location only filters (Closed)
Patch Set: Reduce ternary use
Created Oct. 21, 2018, 4:26 a.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,17 +61,19 @@ |
| 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) |
| + let allFilters = Object.assign(matcher._fastFilterByKeyword, |
| + matcher.filterByKeyword); |
| + for (let [keyword, set] of allFilters) |
| { |
| for (let filter of set) |
| { |
| test.equal(matcher.findKeyword(filter), keyword, |
| "Keyword of filter " + filter.text); |
| filters.push(filter.text); |
| } |
| } |
