Include query string in MatchPattern URL test

[NoneGiven]

Super simple fix prescribed by @arantius, fixes #2715.

Tested with this branch loaded as a temporary webext in FF 57 and it does indeed fix the issue (obviously).

[arantius]

Please include a unit test.

[NoneGiven]

Added unit test.

Without fix:

Firefox 57.0.0 (Windows 7 0.0.0) bg/user-script-registry match pattern respects query string FAILED
        expected 'lowlight' to equal 'highlight'
        AssertionError@node_modules/chai/chai.js:9320:13
        [3]</module.exports/Assertion.prototype.assert@node_modules/chai/chai.js:239:13
        [6]</module.exports/assert.equal@node_modules/chai/chai.js:4218:5
        @test/bg/user-script-registry.test.js:50:5

Firefox 57.0.0 (Windows 7 0.0.0): Executed 28 of 28 (1 FAILED) (0.609 secs / 0.568 secs)

With fix:

Firefox 57.0.0 (Windows 7 0.0.0): Executed 28 of 28 SUCCESS (0.617 secs / 0.585 secs)

[arantius]

Instead of it('match pattern ..., this should be a test of MatchPattern.doMatch() directly. That's the code you're changing. And it would also need less setup code, tested that way.

[NoneGiven]

Should I add a test/third-party directory and MatchPattern.test.js file then?

[arantius]

Yes

[NoneGiven]

Added doMatch unit test.

[arantius]

Great! Thanks for all the updates!