Noissue - Add criteria test for ~example.com##world

test/elemHide.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 161 matching lines @@
 
   removeFilter("~foo.example.com,example.com##foo");
 
   // Test criteria
   addFilter("##hello");
   addFilter("~example.com##world");
   addFilter("foo.com##specific");
   testResult(test, "foo.com", ["specific"], true);
   testResult(test, "foo.com", ["hello", "specific", "world"], false);
   testResult(test, "foo.com", ["hello", "specific", "world"]);
+  testResult(test, "foo.com.", ["hello", "specific", "world"]);
   testResult(test, "example.com", [], true);

[Manish Jethani, 2018/05/26 00:26:33]

Surely the purpose of adding the filter ~example.com##world is to actually test
it, otherwise it makes no sense.

   removeFilter("foo.com##specific");
   removeFilter("~example.com##world");
   removeFilter("##hello");
   testResult(test, "foo.com", []);
 
   addFilter("##hello");
   testResult(test, "foo.com", [], true);
   testResult(test, "foo.com", ["hello"], false);
   testResult(test, "foo.com", ["hello"]);
   testResult(test, "bar.com", [], true);
@@ skipping 39 matching lines @@
 };
 
 exports.testZeroFilterKey = function(test)
 {
   ElemHide.add(Filter.fromText("##test"));
   ElemHide.add(Filter.fromText("foo.com#@#test"));
   testResult(test, "foo.com", []);
   testResult(test, "bar.com", ["test"]);
   test.done();
 };