Fix end positions for function-no-unknown (#6038)

stylelint · Apr 26, 2022 · 9716062 · 9716062
1 parent dcba172
commit 9716062
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/lib/rules/function-no-unknown/__tests__/index.js b/lib/rules/function-no-unknown/__tests__/index.js
@@ -27,18 +27,24 @@ testRule({
 			message: messages.rejected('unknown'),
 			line: 1,
 			column: 16,
+			endLine: 1,
+			endColumn: 23,
 		},
 		{
 			code: 'a { transform: UNKNOWN(4); }',
 			message: messages.rejected('UNKNOWN'),
 			line: 1,
 			column: 16,
+			endLine: 1,
+			endColumn: 23,
 		},
 		{
 			code: 'a { width: calc(10% * unknown(1)); }',
 			message: messages.rejected('unknown'),
 			line: 1,
 			column: 23,
+			endLine: 1,
+			endColumn: 30,
 		},
 	],
 });

diff --git a/lib/rules/function-no-unknown/index.js b/lib/rules/function-no-unknown/index.js
@@ -49,6 +49,8 @@ const rule = (primary, secondaryOptions) => {
 			const { value } = decl;
 
 			valueParser(value).walk((node) => {
+				const name = node.value;
+
 				if (node.type !== 'function') {
 					return;
 				}
@@ -57,24 +59,25 @@ const rule = (primary, secondaryOptions) => {
 					return;
 				}
 
-				if (isCustomFunction(node.value)) {
+				if (isCustomFunction(name)) {
 					return;
 				}
 
-				if (optionsMatches(secondaryOptions, 'ignoreFunctions', node.value)) {
+				if (optionsMatches(secondaryOptions, 'ignoreFunctions', name)) {
 					return;
 				}
 
-				if (functionsList.includes(node.value.toLowerCase())) {
+				if (functionsList.includes(name.toLowerCase())) {
 					return;
 				}
 
 				report({
-					message: messages.rejected(node.value),
+					message: messages.rejected(name),
 					node: decl,
 					index: declarationValueIndex(decl) + node.sourceIndex,
 					result,
 					ruleName,
+					word: name,
 				});
 			});
 		});