Fix selector-pseudo-class-allowed-list end positions and message (#…

…6262)
stylelint · Aug 14, 2022 · 9afc6f3 · 9afc6f3
1 parent 3eb4549
commit 9afc6f3
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 12 deletions.
diff --git a/lib/rules/selector-pseudo-class-allowed-list/__tests__/index.js b/lib/rules/selector-pseudo-class-allowed-list/__tests__/index.js
@@ -55,51 +55,67 @@ testRule({
 	reject: [
 		{
 			code: 'a:HOVER {}',
-			message: messages.rejected('HOVER'),
+			message: messages.rejected(':HOVER'),
 			line: 1,
 			column: 2,
+			endLine: 1,
+			endColumn: 8,
 		},
 		{
 			code: 'a:-MOZ-PLACEholder {}',
-			message: messages.rejected('-MOZ-PLACEholder'),
+			message: messages.rejected(':-MOZ-PLACEholder'),
 			line: 1,
 			column: 2,
+			endLine: 1,
+			endColumn: 19,
 		},
 		{
 			code: 'a:focus {}',
-			message: messages.rejected('focus'),
+			message: messages.rejected(':focus'),
 			line: 1,
 			column: 2,
+			endLine: 1,
+			endColumn: 8,
 		},
 		{
 			code: 'div:nth-LAST-child {}',
-			message: messages.rejected('nth-LAST-child'),
+			message: messages.rejected(':nth-LAST-child'),
 			line: 1,
 			column: 4,
+			endLine: 1,
+			endColumn: 19,
 		},
 		{
 			code: 'a,\n:global {}',
-			message: messages.rejected('global'),
+			message: messages.rejected(':global'),
 			line: 2,
 			column: 1,
+			endLine: 2,
+			endColumn: 8,
 		},
 		{
 			code: 'input:-ms-input-placeholder {}',
-			message: messages.rejected('-ms-input-placeholder'),
+			message: messages.rejected(':-ms-input-placeholder'),
 			line: 1,
 			column: 6,
+			endLine: 1,
+			endColumn: 28,
 		},
 		{
 			code: 'input:-Ms-INPUT-placeholder {}',
-			message: messages.rejected('-Ms-INPUT-placeholder'),
+			message: messages.rejected(':-Ms-INPUT-placeholder'),
 			line: 1,
 			column: 6,
+			endLine: 1,
+			endColumn: 28,
 		},
 		{
 			code: 'a:not(::selection) {}',
-			message: messages.rejected('not'),
+			message: messages.rejected(':not'),
 			line: 1,
 			column: 2,
+			endLine: 1,
+			endColumn: 6,
 		},
 	],
 });
@@ -123,9 +139,11 @@ testRule({
 	reject: [
 		{
 			code: 'a:hover {}',
-			message: messages.rejected('hover'),
+			message: messages.rejected(':hover'),
 			line: 1,
 			column: 2,
+			endLine: 1,
+			endColumn: 8,
 		},
 	],
 });
@@ -143,9 +161,11 @@ testRule({
 	reject: [
 		{
 			code: 'a:hover {}',
-			message: messages.rejected('hover'),
+			message: messages.rejected(':hover'),
 			line: 1,
 			column: 2,
+			endLine: 1,
+			endColumn: 8,
 		},
 	],
 });

diff --git a/lib/rules/selector-pseudo-class-allowed-list/index.js b/lib/rules/selector-pseudo-class-allowed-list/index.js
@@ -58,8 +58,8 @@ const rule = (primary) => {
 					}
 
 					report({
-						index: pseudoNode.sourceIndex,
-						message: messages.rejected(name),
+						word: value,
+						message: messages.rejected(value),
 						node: ruleNode,
 						result,
 						ruleName,