Unified Diff: test/subscriptionClasses.js
Issue 29900557:
Issue 7016 - Convert serialization functions into generators (Closed)
Patch Set: Actually address nits
Created Oct. 23, 2018, 3:21 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« test/filterClasses.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/subscriptionClasses.js |
| =================================================================== |
| --- a/test/subscriptionClasses.js |
| +++ b/test/subscriptionClasses.js |
| @@ -38,18 +38,17 @@ |
| }; |
| function compareSubscription(test, url, expected, postInit) |
| { |
| expected.push("[Subscription]"); |
| let subscription = Subscription.fromURL(url); |
| if (postInit) |
| postInit(subscription); |
| - let result = []; |
| - subscription.serialize(result); |
| + let result = [...subscription.serialize()]; |
| test.equal(result.sort().join("\n"), expected.sort().join("\n"), url); |
| let map = Object.create(null); |
| for (let line of result.slice(1)) |
| { |
| if (/(.*?)=(.*)/.test(line)) |
| map[RegExp.$1] = RegExp.$2; |
| } |
« no previous file with comments
|
« test/filterClasses.js ('k')
|
no next file »
|
no next file with comments »
