Issue 2664 - Remove ignoreHeaders argument from parseFilter(s) and move the logic to the UI

qunit/tests/filterValidation.js

 (function()
 {
   var filterValidation = require("filterValidation");
   var parseFilter = filterValidation.parseFilter;
   var parseFilters = filterValidation.parseFilters;
 
   var filterClasses = require("filterClasses");
   var BlockingFilter = filterClasses.BlockingFilter;
   var ElemHideFilter = filterClasses.ElemHideFilter;
   var CommentFilter = filterClasses.CommentFilter;
 
   module("Filter validation");
 
   test("Detecting invalid filters", function()
   {
-    ok(parseFilter("||example.com^$unknown").error, "unknown option");
-    ok(parseFilter("[foobar]").error, "filter list header");
-    ok(parseFilter("##[foo").error, "invalid selector");
-    ok(/\b4\b/.test(parseFilters("!comment\r\n||example.com^\n\n##/").error), "error contains corresponding line number");
+    equal(parseFilter("||example.com^$unknown").error.type, "invalid-filter", "unknown option");
+    equal(parseFilter("[foobar]").error.type, "unexpected-filter-list-header", "filter list header");
+    equal(parseFilter("##[foo").error.type, "invalid-css-selector", "invalid selector");
+    ok(/\b4\b/.test(parseFilters("!comment\r\n||example.com^\n\n##/").errors[0]), "error contains corresponding line number");

[Wladimir Palant, 2015/06/08 18:15:47]

Maybe test for line number field here instead of parsing the text message? Now
we can do that.

[Sebastian Noack, 2015/06/08 18:36:46]

I considered that. But then decided to keep it as it is to  also test the code
path adding the line number to the error message. However, I wouldn't mind
changing it if you disagree.

[Wladimir Palant, 2015/06/08 19:14:16]

Well, I'm not going to insist - I merely dislike parsing localizable strings.

   });
 
   test("Allowing valid filters", function()
   {
     var text, filter;
 
     text = "||example.com^";
     filter = parseFilter(text).filter;
     ok(filter instanceof BlockingFilter, "blocking filter parsed");
     equal(filter.text, text, "blocking filter text matches");
@@ skipping 14 matching lines @@
   test("Normalizing filters", function()
   {
     var ws = " \t\r\n";
 
     equal(parseFilter(ws + "@@" + ws + "||" + ws + "example.com" + ws + "^" + ws).filter.text, "@@||example.com^", "unnecessary spaces");
     equal(parseFilter(ws).filter, null, "only spaces");
   });
 
   test("Parsing multiple filters", function()
   {
-    var filters = parseFilters("||example.com^\n \n###foobar\r\n! foo bar\n").filters;
+    var result = parseFilters("||example.com^\n \n###foobar\r\n! foo bar\n");
 
-    equal(filters.length, 3, "all filters parsed");
+    equal(result.errors.length, 0, "no error occurred");
+    equal(result.filters.length, 3, "all filters parsed");
 
-    ok(filters[0] instanceof BlockingFilter, "1st filter is blocking");
-    equal(filters[0].text, "||example.com^", "1st filter text matches");
+    ok(result.filters[0] instanceof BlockingFilter, "1st filter is blocking");
+    equal(result.filters[0].text, "||example.com^", "1st filter text matches");
 
-    ok(filters[1] instanceof ElemHideFilter, "2nd filter is elemhide");
-    equal(filters[1].text, "###foobar",      "2nd filter text matches");
+    ok(result.filters[1] instanceof ElemHideFilter, "2nd filter is elemhide");
+    equal(result.filters[1].text, "###foobar",      "2nd filter text matches");
 
-    ok(filters[2] instanceof CommentFilter,  "3rd filter is comment");
-    equal(filters[2].text, "! foo bar",      "3rd filter text matches");
-  });
-
-  test("Parsing multiple filters, stripping filter list headers", function()
-  {
-    var filters = parseFilters("[foobar]\n \n||example.com^\r\n! foo bar\n", true).filters;
-
-    equal(filters.length, 2, "all filters parsed");
-
-    ok(filters[0] instanceof BlockingFilter, "1st filter is blocking");
-    equal(filters[0].text, "||example.com^", "1st filter text matches");
-
-    ok(filters[1] instanceof CommentFilter,  "2nd filter is comment");
-    equal(filters[1].text, "! foo bar",      "2nd filter text matches");
+    ok(result.filters[2] instanceof CommentFilter,  "3rd filter is comment");
+    equal(result.filters[2].text, "! foo bar",      "3rd filter text matches");
   });
 })();