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

Unified Diff: test/filterStorage.js

Issue 29870577: Issue 6916 - Encapsulate filter subscriptions (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Add JSDoc for addSubscription and removeSubscription Created Sept. 1, 2018, 2:10 p.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 | « test/filterListener.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/filterStorage.js
===================================================================
--- a/test/filterStorage.js
+++ b/test/filterStorage.js
@@ -75,17 +75,17 @@
let result = FilterStorage.subscriptions.map(
subscription => subscription.filters.map(
filter => filter.text));
test.deepEqual(result, list, testMessage);
}
function compareFilterSubscriptions(test, testMessage, filter, list)
{
- let result = [...filter.subscriptions].map(subscription => subscription.url);
+ let result = [...filter.subscriptions()].map(subscription => subscription.url);
let expected = list.map(subscription => subscription.url);
test.deepEqual(result, expected, testMessage);
}
exports.testAddingSubscriptions = function(test)
{
let subscription1 = Subscription.fromURL("http://test1/");
let subscription2 = Subscription.fromURL("http://test2/");
« no previous file with comments | « test/filterListener.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld