Unified Diff: test/filterClasses.js
Issue 29900557:
Issue 7016 - Convert serialization functions into generators (Closed)
Patch Set: Actually Address PS2 Comments
Created Oct. 21, 2018, 5:25 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/subscriptionClasses.js ('K')
|
« lib/subscriptionClasses.js ('k')
|
test/subscriptionClasses.js » ('j')
|
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/filterClasses.js |
| =================================================================== |
| --- a/test/filterClasses.js |
| +++ b/test/filterClasses.js |
| @@ -191,18 +191,17 @@ |
| 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); |
| + let buffer = [...filter.serialize()]; |
| if (buffer.length) |
| { |
| let map = Object.create(null); |
| for (let line of buffer.slice(1)) |
| { |
| if (/(.*?)=(.*)/.test(line)) |
| map[RegExp.$1] = RegExp.$2; |
| } |
« lib/subscriptionClasses.js ('K')
|
« lib/subscriptionClasses.js ('k')
|
test/subscriptionClasses.js » ('j')
|
no next file with comments »
