From cf791e49174117d92da0e9e6e0c99d9de47b764d Mon Sep 17 00:00:00 2001 From: Zachiah Sawyer Date: Sat, 8 Aug 2020 20:44:51 -0600 Subject: [PATCH] Fix tests and add quotes --- ...01\006@lÁ\t@ŒÁ\t@ŒÁ\t(ta(ta\020\001\006pà" | 0 src/compiler/compile/css/Stylesheet.ts | 2 +- test/css/samples/empty-class/_config.js | 2 +- .../global-with-unused-descendant/_config.js | 2 +- .../_config.js | 2 +- .../unused-selector-leading/_config.js | 4 ++-- .../unused-selector-string-concat/_config.js | 20 +++++++++---------- .../unused-selector-ternary-concat/_config.js | 2 +- .../unused-selector-ternary-nested/_config.js | 4 ++-- .../unused-selector-ternary/_config.js | 2 +- test/css/samples/unused-selector/_config.js | 2 +- 11 files changed, 21 insertions(+), 21 deletions(-) create mode 100644 "\001Àø±\002@°Â”\016@8\f@$\006\004@@@ \002 \002\b\003\004à\002à\002à\002\034\034\001\001\004„è±\002„è±\002\020\001\005Àè±\002Àø±\002Àø±\002ƒ\017\a\020\001\006@lÁ\t@ŒÁ\t@ŒÁ\t(ta(ta\020\001\006pà" diff --git "a/\001Àø±\002@°Â”\016@8\f@$\006\004@@@ \002 \002\b\003\004à\002à\002à\002\034\034\001\001\004„è±\002„è±\002\020\001\005Àè±\002Àø±\002Àø±\002ƒ\017\a\020\001\006@lÁ\t@ŒÁ\t@ŒÁ\t(ta(ta\020\001\006pà" "b/\001Àø±\002@°Â”\016@8\f@$\006\004@@@ \002 \002\b\003\004à\002à\002à\002\034\034\001\001\004„è±\002„è±\002\020\001\005Àè±\002Àø±\002Àø±\002ƒ\017\a\020\001\006@lÁ\t@ŒÁ\t@ŒÁ\t(ta(ta\020\001\006pà" new file mode 100644 index 0000000000..e69de29bb2 diff --git a/src/compiler/compile/css/Stylesheet.ts b/src/compiler/compile/css/Stylesheet.ts index 548c69f0f1..d245e00ec2 100644 --- a/src/compiler/compile/css/Stylesheet.ts +++ b/src/compiler/compile/css/Stylesheet.ts @@ -435,7 +435,7 @@ export default class Stylesheet { child.warn_on_unused_selector((selector: Selector) => { component.warn(selector.node, { code: `css-unused-selector`, - message: `Unused CSS selector ` + this.source.substring(selector.node.start, selector.node.end) + message: `Unused CSS selector "${this.source.substring(selector.node.start, selector.node.end)}"` }); }); }); diff --git a/test/css/samples/empty-class/_config.js b/test/css/samples/empty-class/_config.js index cc766d9478..064de4a564 100644 --- a/test/css/samples/empty-class/_config.js +++ b/test/css/samples/empty-class/_config.js @@ -2,7 +2,7 @@ export default { warnings: [{ filename: "SvelteComponent.svelte", code: `css-unused-selector`, - message: "Unused CSS selector", + message: 'Unused CSS selector ".x"', start: { line: 4, column: 1, diff --git a/test/css/samples/global-with-unused-descendant/_config.js b/test/css/samples/global-with-unused-descendant/_config.js index 3ed2dd2728..69a69ad650 100644 --- a/test/css/samples/global-with-unused-descendant/_config.js +++ b/test/css/samples/global-with-unused-descendant/_config.js @@ -13,7 +13,7 @@ export default { 3: color: red; 4: } `, - message: 'Unused CSS selector', + message: 'Unused CSS selector ":global(.foo) .bar"', pos: 9, start: { character: 9, diff --git a/test/css/samples/omit-scoping-attribute-descendant/_config.js b/test/css/samples/omit-scoping-attribute-descendant/_config.js index a4aaec7c19..25e2a28ebb 100644 --- a/test/css/samples/omit-scoping-attribute-descendant/_config.js +++ b/test/css/samples/omit-scoping-attribute-descendant/_config.js @@ -1,7 +1,7 @@ export default { warnings: [{ code: `css-unused-selector`, - message: 'Unused CSS selector', + message: 'Unused CSS selector "div > p"', start: { line: 8, column: 1, diff --git a/test/css/samples/unused-selector-leading/_config.js b/test/css/samples/unused-selector-leading/_config.js index 20310a5847..2516567f59 100644 --- a/test/css/samples/unused-selector-leading/_config.js +++ b/test/css/samples/unused-selector-leading/_config.js @@ -3,7 +3,7 @@ export default { { filename: "SvelteComponent.svelte", code: `css-unused-selector`, - message: "Unused CSS selector", + message: 'Unused CSS selector ".foo"', start: { line: 4, column: 1, @@ -27,7 +27,7 @@ export default { { filename: "SvelteComponent.svelte", code: `css-unused-selector`, - message: "Unused CSS selector", + message: 'Unused CSS selector ".baz"', start: { line: 4, column: 13, diff --git a/test/css/samples/unused-selector-string-concat/_config.js b/test/css/samples/unused-selector-string-concat/_config.js index 81318fd3ac..5f5bb5f792 100644 --- a/test/css/samples/unused-selector-string-concat/_config.js +++ b/test/css/samples/unused-selector-string-concat/_config.js @@ -2,7 +2,7 @@ export default { warnings: [ { code: 'css-unused-selector', - message: 'Unused CSS selector', + message: 'Unused CSS selector ".fooaa"', frame: ` 9: `, - message: 'Unused CSS selector', + message: 'Unused CSS selector ".unused"', pos: 198, start: { character: 198, diff --git a/test/css/samples/unused-selector-ternary-nested/_config.js b/test/css/samples/unused-selector-ternary-nested/_config.js index afee5ac822..936ac639e5 100644 --- a/test/css/samples/unused-selector-ternary-nested/_config.js +++ b/test/css/samples/unused-selector-ternary-nested/_config.js @@ -2,7 +2,7 @@ export default { warnings: [ { code: 'css-unused-selector', - message: 'Unused CSS selector', + message: 'Unused CSS selector ".hover.unused"', frame: ` 13: .thing.active {color: blue;} 14: .hover { color: blue; } @@ -16,7 +16,7 @@ export default { }, { code: 'css-unused-selector', - message: 'Unused CSS selector', + message: 'Unused CSS selector ".unused"', frame: ` 15: .hover.unused { color: blue; } 16: diff --git a/test/css/samples/unused-selector-ternary/_config.js b/test/css/samples/unused-selector-ternary/_config.js index 4ff2713108..4eb12f3fe2 100644 --- a/test/css/samples/unused-selector-ternary/_config.js +++ b/test/css/samples/unused-selector-ternary/_config.js @@ -6,7 +6,7 @@ export default { warnings: [{ filename: "SvelteComponent.svelte", code: `css-unused-selector`, - message: "Unused CSS selector", + message: 'Unused CSS selector ".maybeactive"', start: { line: 16, column: 1, diff --git a/test/css/samples/unused-selector/_config.js b/test/css/samples/unused-selector/_config.js index 49fb3a27e7..790eb9e875 100644 --- a/test/css/samples/unused-selector/_config.js +++ b/test/css/samples/unused-selector/_config.js @@ -2,7 +2,7 @@ export default { warnings: [{ filename: "SvelteComponent.svelte", code: `css-unused-selector`, - message: "Unused CSS selector", + message: 'Unused CSS selector ".bar"', start: { line: 8, column: 1,