Issue 6886 - Return snippet filters instead of just scripts

test/snippets.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 30 matching lines @@
 exports.testDomainRestrictions = function(test)
 {
   function testScriptMatches(description, filters, domain, expectedMatches)
   {
     for (let filter of filters.map(Filter.fromText))
     {
       if (filter instanceof SnippetFilter)
         Snippets.add(filter);
     }
 
-    let matches = Snippets.getFiltersForDomain(domain).map(elem => elem.script);

[Manish Jethani, 2018/08/27 12:36:21]

Nit: Can we keep it `filter => filter.script` similar to the way it was
previously in `compareRules`? I feel like more explicit naming is usually
better. We could just split it into two lines if required.

[hub, 2018/08/27 12:53:35]

Done.

+    let matches = Snippets.getFiltersForDomain(domain).map(
+      filter => filter.script
+    );
     test.deepEqual(matches.sort(), expectedMatches.sort(), description);
 
     Snippets.clear();
   }
 
   testScriptMatches(
     "Ignore generic filters",
     [
       "#$#foo-1", "example.com#$#foo-2",
       "~example.com#$#foo-3"
@@ skipping 218 matching lines @@
   // between executions. In the example below, assertFoo does not find 456 but
   // it doesn't find 123 either. It's the initial value 0.
   new Function(
     compileScript("setFoo 456; assertFoo 0", [
       ...libraries, "let foo = 1; exports.setFoo = value => { foo = value; };"
     ])
   )();
 
   test.done();
 };