Fix parseError for double-slash comments within selector lists in no-…

…descending-specificity (#5891)
stylelint · Feb 7, 2022 · e81473d · e81473d
1 parent 12a35ba
commit e81473d
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 1 deletion.
diff --git a/lib/rules/no-descending-specificity/__tests__/index.js b/lib/rules/no-descending-specificity/__tests__/index.js
@@ -1,6 +1,8 @@
 'use strict';
 
 const postcssScss = require('postcss-scss');
+const postcssLess = require('postcss-less');
+const stripIndent = require('common-tags').stripIndent;
 
 const { messages, ruleName } = require('..');
 
@@ -172,6 +174,22 @@ testRule({
 	],
 });
 
+testRule({
+	ruleName,
+	customSyntax: postcssLess,
+	config: [true],
+
+	accept: [
+		{
+			code: stripIndent`
+				a,
+				// comment2
+				b {}
+			`,
+		},
+	],
+});
+
 testRule({
 	skip: true,
 	ruleName,

diff --git a/lib/utils/__tests__/isStandardSyntaxSelector.test.js b/lib/utils/__tests__/isStandardSyntaxSelector.test.js
@@ -80,7 +80,10 @@ describe('isStandardSyntaxSelector', () => {
 		expect(isStandardSyntaxSelector('.foo(@a)')).toBeFalsy();
 		expect(isStandardSyntaxSelector('.foo(@a: 5px)')).toBeFalsy();
 	});
-
+	it('SCSS or Less comments', () => {
+		expect(isStandardSyntaxSelector('a\n// comment\nb')).toBeFalsy();
+		expect(isStandardSyntaxSelector('a\n//comment\nb')).toBeFalsy();
+	});
 	it('ERB templates', () => {
 		// E. g. like in https://github.com/stylelint/stylelint/issues/4489
 		expect(isStandardSyntaxSelector('<% COLORS.each do |color| %>\na')).toBe(false);

diff --git a/lib/utils/isStandardSyntaxSelector.js b/lib/utils/isStandardSyntaxSelector.js
@@ -49,5 +49,10 @@ module.exports = function (selector) {
 		return false;
 	}
 
+	//  SCSS and Less comments
+	if (selector.includes('//')) {
+		return false;
+	}
+
 	return true;
 };