Issue 4394 - Create a filter class for element hiding emulation filters

test/elemHideEmulation.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 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 10 matching lines @@
 
 let ElemHideEmulationFilter = null;
 let ElemHideEmulation = null;
 let ElemHide = null;
 let Filter = null;
 
 exports.setUp = function(callback)
 {
   let sandboxedRequire = createSandbox();
   (
-    {Filter, ElemHideEmulationFilter} = sandboxedRequire("../lib/filterClasses"),

[kzar, 2016/11/04 15:45:56]

Nit: Mind fixing the few long lines here?

[Felix Dahlke, 2016/11/04 16:43:35]

It bothered me too, but I don't see an obvious way of doing that here. From the
look of it we generally seem to ignore the 80 column rule in require lines with
destructuring assignments. How would you do it?

[kzar, 2016/11/07 17:19:26]

Well this line is not the worst, but none the less I would probably do it like
so:

{Filter,
 ElemHideEmulationFilter} = sandboxedRequire("../lib/filterClasses"),

I suppose I would adjust the later line like so:

let matches = ElemHideEmulation.getRulesForDomain(domain).map(
  filter => filter.text
);

[Felix Dahlke, 2016/11/08 17:45:35]

Yeah, that's nice actually. 

[Felix Dahlke, 2016/11/11 11:51:42]

Done.

+    {Filter,
+     ElemHideEmulationFilter} = sandboxedRequire("../lib/filterClasses"),
     {ElemHideEmulation} = sandboxedRequire("../lib/elemHideEmulation"),
     {ElemHide} = sandboxedRequire("../lib/elemHide")
   );
 
   callback();
 };
 
 exports.testDomainRestrictions = function(test)
 {
   function testSelectorMatches(description, filters, domain, expectedMatches)
   {
     for (let filter of filters)
     {
       filter = Filter.fromText(filter);
       if (filter instanceof ElemHideEmulationFilter)
         ElemHideEmulation.add(filter);
       else
         ElemHide.add(filter);
     }
 
-    let matches = ElemHideEmulation.getRulesForDomain(domain).map(filter => filter.text);
+    let matches = ElemHideEmulation.getRulesForDomain(domain)
+        .map(filter => filter.text);
     test.deepEqual(matches.sort(), expectedMatches.sort(), description);
 
     ElemHideEmulation.clear();
     ElemHide.clear();
   }
 
   testSelectorMatches(
     "Ignore generic filters",
     ["##[-abp-properties='foo']", "example.com##[-abp-properties='foo']",
      "~example.com##[-abp-properties='foo']"],
@@ skipping 64 matching lines @@
 
   ElemHideEmulation.clear();
   compareRules(
     "Return no filters after clearing",
     "www.example.com",
     []
   );
 
   test.done();
 };