Unified Diff: qunit/tests/cssEscaping.js
Issue 29374674:
Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Patch Set: Stop using commonjs
Created Feb. 21, 2017, 5:08 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« qunit/common.js ('k')
|
qunit/tests/filterValidation.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: qunit/tests/cssEscaping.js |
| diff --git a/qunit/tests/cssEscaping.js b/qunit/tests/cssEscaping.js |
| index 1ba05486dca37b0f970f9522b44b01d19317935e..d7cb056e305380a7bd372bc972c71647f97d0bf2 100644 |
| --- a/qunit/tests/cssEscaping.js |
| +++ b/qunit/tests/cssEscaping.js |
| @@ -43,16 +43,11 @@ |
| let filter = Filter.fromText("##" + opts.selector); |
| if (!(filter instanceof ElemHideFilter)) |
| - { |
| ok(false, opts.selector + " (not allowed in elemhide filters)"); |
| - } |
| + else if (mustMatch) |
| + equal(foundElement, element, opts.selector); |
| else |
| - { |
| - if (mustMatch) |
| - equal(foundElement, element, opts.selector); |
| - else |
| - ok(true, opts.selector); |
| - } |
| + ok(true, opts.selector); |
| } |
| function testEscape(s) |
| @@ -99,7 +94,8 @@ |
| testEscape("-" + chr); |
| } |
| - // Test some non-ASCII characters. However, those shouldn't require escaping. |
| + // Test some non-ASCII characters. However, those shouldn't |
| + // require escaping. |
| testEscape("\uD83D\uDE3B\u2665\u00E4"); |
| }); |
| } |
« no previous file with comments
|
« qunit/common.js ('k')
|
qunit/tests/filterValidation.js » ('j')
|
no next file with comments »
