Fix end positions for at-rule-*-list (#6032)

Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
stylelint · Apr 24, 2022 · 2962bd7 · 2962bd7
1 parent 9b82c8f
commit 2962bd7
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 0 deletions.
diff --git a/lib/rules/at-rule-allowed-list/__tests__/index.js b/lib/rules/at-rule-allowed-list/__tests__/index.js
@@ -70,6 +70,8 @@ testRule({
     `,
 			line: 2,
 			column: 7,
+			endLine: 2,
+			endColumn: 13,
 			message: messages.rejected('mixin'),
 			description: '@rule not from an allowed list; independent rule.',
 		},
@@ -128,20 +130,26 @@ testRule({
 			message: messages.rejected('mixin'),
 			line: 2,
 			column: 7,
+			endLine: 2,
+			endColumn: 13,
 			description: '@rule not from an allowed list.',
 		},
 		{
 			code: "@import 'path/to/file.css';",
 			message: messages.rejected('import'),
 			line: 1,
 			column: 1,
+			endLine: 1,
+			endColumn: 8,
 			description: '@rule not from an allowed list.',
 		},
 		{
 			code: '@media screen and (max-witdh: 1000px) {}',
 			message: messages.rejected('media'),
 			line: 1,
 			column: 1,
+			endLine: 1,
+			endColumn: 7,
 			description: '@rule not from an allowed list.',
 		},
 	],

diff --git a/lib/rules/at-rule-allowed-list/index.js b/lib/rules/at-rule-allowed-list/index.js
@@ -47,6 +47,7 @@ const rule = (primary) => {
 				node: atRule,
 				result,
 				ruleName,
+				word: `@${name}`,
 			});
 		});
 	};

diff --git a/lib/rules/at-rule-disallowed-list/__tests__/index.js b/lib/rules/at-rule-disallowed-list/__tests__/index.js
@@ -24,6 +24,8 @@ testRule({
 			message: messages.rejected('extend'),
 			line: 1,
 			column: 5,
+			endLine: 1,
+			endColumn: 12,
 			description: '@rule from a disallowed list, is a Sass directive.',
 		},
 		{
@@ -36,6 +38,8 @@ testRule({
 			message: messages.rejected('extend'),
 			line: 3,
 			column: 9,
+			endLine: 3,
+			endColumn: 16,
 			description: '@rule from a disallowed list; newline after its name.',
 		},
 		{
@@ -47,6 +51,9 @@ testRule({
     `,
 			message: messages.rejected('keyframes'),
 			line: 2,
+			column: 7,
+			endLine: 2,
+			endColumn: 17,
 			description: '@rule from a disallowed list; independent rule.',
 		},
 		{

diff --git a/lib/rules/at-rule-disallowed-list/index.js b/lib/rules/at-rule-disallowed-list/index.js
@@ -47,6 +47,7 @@ const rule = (primary) => {
 				node: atRule,
 				result,
 				ruleName,
+				word: `@${atRule.name}`,
 			});
 		});
 	};