Issue 3659 - Remove locale dependency from filter classes

I migrated the required unit tests from adblockplustests repository. The test itself is unchanged except for the logic around invalid filters, reason being tested explicitly now.

[Thomas Greiner, 2016-02-17 15:32:50]

https://codereview.adblockplus.org/29336430/diff/29336483/test_wrapper.js
File test_wrapper.js (right):

https://codereview.adblockplus.org/29336430/diff/29336483/test_wrapper.js#new...
test_wrapper.js:1: #!/usr/bin/env node --harmony
Detail: Seems a bit redundant to specify it here as well since you already did
in package.json. Making this file executable doesn't appear to be a requirement.

https://codereview.adblockplus.org/29336430/diff/29336483/test_wrapper.js#new...
test_wrapper.js:9: // resolves modules right now.
You can simply set the "NODE_PATH" environment variable in package.json to let
Node do the job for us.

  "test": "NODE_PATH=lib node --harmony test_wrapper.js"

Alternatively, you could do it on runtime if you prefer that:

  process.env.NODE_PATH = "lib";
  require("module").Module._initPaths();

Source: https://gist.github.com/branneman/8048520

[Wladimir Palant, 2016-02-17 15:53:02]

https://codereview.adblockplus.org/29336430/diff/29336483/test_wrapper.js
File test_wrapper.js (right):

https://codereview.adblockplus.org/29336430/diff/29336483/test_wrapper.js#new...
test_wrapper.js:1: #!/usr/bin/env node --harmony
On 2016/02/17 15:32:50, Thomas Greiner wrote:
> Detail: Seems a bit redundant to specify it here as well since you already did
> in package.json. Making this file executable doesn't appear to be a
requirement.

Right, this file was originally run directly - it no longer is.

https://codereview.adblockplus.org/29336430/diff/29336483/test_wrapper.js#new...
test_wrapper.js:9: // resolves modules right now.
On 2016/02/17 15:32:50, Thomas Greiner wrote:
> You can simply set the "NODE_PATH" environment variable in package.json to let
> Node do the job for us.
> 
>   "test": "NODE_PATH=lib node --harmony test_wrapper.js"

Nice catch, originally this wasn't possible as the file would be run directly.

[Thomas Greiner, 2016-02-17 16:04:40]

LGTM