Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: qunit/tests/cssEscaping.js

Issue 29713631: Issue 5760 - Use relative require paths (Closed)
Patch Set: Address PS3 comment Created April 3, 2018, 6:03 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 "use strict"; 1 "use strict";
2 2
3 { 3 {
4 const {Filter, ElemHideFilter} = require("filterClasses"); 4 const {Filter, ElemHideFilter} =
5 require("../../adblockpluscore/lib/filterClasses");
5 const {escapeCSS, quoteCSS} = require("filterComposer"); 6 const {escapeCSS, quoteCSS} = require("filterComposer");
kzar 2018/04/05 11:14:57 What about this one?
Jon Sonesen 2018/04/05 23:01:20 Acknowledged.
6 7
7 QUnit.module("CSS escaping"); 8 QUnit.module("CSS escaping");
8 9
9 test("CSS escaping", () => 10 test("CSS escaping", () =>
10 { 11 {
11 function testSelector(opts) 12 function testSelector(opts)
12 { 13 {
13 let mustMatch = opts.mustMatch !== false; 14 let mustMatch = opts.mustMatch !== false;
14 let doc = document.implementation.createHTMLDocument(); 15 let doc = document.implementation.createHTMLDocument();
15 16
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 93
93 // Leading dashes must be escaped, when followed by certain characters. 94 // Leading dashes must be escaped, when followed by certain characters.
94 testEscape("-" + chr); 95 testEscape("-" + chr);
95 } 96 }
96 97
97 // Test some non-ASCII characters. However, those shouldn't 98 // Test some non-ASCII characters. However, those shouldn't
98 // require escaping. 99 // require escaping.
99 testEscape("\uD83D\uDE3B\u2665\u00E4"); 100 testEscape("\uD83D\uDE3B\u2665\u00E4");
100 }); 101 });
101 } 102 }
OLDNEW

Powered by Google App Engine
This is Rietveld