Issue 7094 - Encapsulate management of subscription filters

test/subscriptionClasses.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 52 matching lines @@
 }
 
 function compareSubscriptionFilters(test, subscription, expected)
 {
   test.deepEqual([...subscription.filterText()], expected);
   test.deepEqual([...subscription.filters()], expected.map(f$));
 
   test.equal(subscription.filterCount, expected.length);
 
   for (let i = 0; i < subscription.filterCount; i++)
-  {
     test.equal(subscription.filterAt(i).text, expected[i]);
-    test.ok(subscription.hasFilter(f$(expected[i])));
-  }
 
   test.ok(!subscription.filterAt(subscription.filterCount));
   test.ok(!subscription.filterAt(-1));
-
-  subscription.clearCaches();

[Manish Jethani, 2018/11/18 02:25:51]

Test again after clearing caches.

-
-  for (let i = 0; i < subscription.filterCount; i++)
-    test.ok(subscription.hasFilter(f$(expected[i])));
 }
 
 exports.testSubscriptionClassDefinitions = function(test)
 {
   test.equal(typeof Subscription, "function", "typeof Subscription");
   test.equal(typeof SpecialSubscription, "function", "typeof SpecialSubscription");
   test.equal(typeof RegularSubscription, "function", "typeof RegularSubscription");
   test.equal(typeof ExternalSubscription, "function", "typeof ExternalSubscription");
   test.equal(typeof DownloadableSubscription, "function", "typeof DownloadableSubscription");
 
@@ skipping 65 matching lines @@
   subscription.addFilter(f$("##.bar"));
   compareSubscriptionFilters(test, subscription, ["##.foo", "##.bar",
                                                   "##.bar"]);
 
   // The first occurrence is found.
   test.equal(subscription.searchFilter(f$("##.bar")), 1);
 
   subscription.deleteFilterAt(0);
   compareSubscriptionFilters(test, subscription, ["##.bar", "##.bar"]);
   test.equal(subscription.searchFilter(f$("##.bar")), 0);
-  test.ok(!subscription.hasFilter(f$("##.foo")));
 
   subscription.insertFilterAt(f$("##.foo"), 0);
   compareSubscriptionFilters(test, subscription, ["##.foo", "##.bar",
                                                   "##.bar"]);
   test.equal(subscription.searchFilter(f$("##.bar")), 1);
 
   subscription.deleteFilterAt(1);
   compareSubscriptionFilters(test, subscription, ["##.foo", "##.bar"]);
   test.equal(subscription.searchFilter(f$("##.bar")), 1);
 
   subscription.deleteFilterAt(1);
   compareSubscriptionFilters(test, subscription, ["##.foo"]);
   test.equal(subscription.searchFilter(f$("##.bar")), -1);
-  test.ok(!subscription.hasFilter(f$("##.bar")));
 
   subscription.addFilter(f$("##.bar"));
   compareSubscriptionFilters(test, subscription, ["##.foo", "##.bar"]);
   test.equal(subscription.searchFilter(f$("##.bar")), 1);
 
   subscription.clearFilters();
   compareSubscriptionFilters(test, subscription, []);
-  test.ok(!subscription.hasFilter(f$("##.foo")));
-  test.ok(!subscription.hasFilter(f$("##.bar")));
   test.equal(subscription.searchFilter(f$("##.foo")), -1);
   test.equal(subscription.searchFilter(f$("##.bar")), -1);
 
   subscription.addFilter(f$("##.bar"));
   compareSubscriptionFilters(test, subscription, ["##.bar"]);
 
   subscription.addFilter(f$("##.foo"));
   compareSubscriptionFilters(test, subscription, ["##.bar", "##.foo"]);
   test.equal(subscription.searchFilter(f$("##.bar")), 0);
   test.equal(subscription.searchFilter(f$("##.foo")), 1);
@@ skipping 17 matching lines @@
   test.equal(subscription.searchFilter(f$("##.lambda")), 0);
 
   // Delete outside of bounds (negative).
   subscription.deleteFilterAt(-1000);
   compareSubscriptionFilters(test, subscription, ["##.lambda", "##.bar",
                                                   "##.foo", "##.lambda"]);
   test.equal(subscription.searchFilter(f$("##.lambda")), 0);
 
   test.done();
 };