Fix --report-invalid-scope-disables crash when no rules specified (#4498

)
stylelint · Dec 29, 2019 · 0e2439d · 0e2439d
1 parent 1dfaee8
commit 0e2439d
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 4 deletions.
diff --git a/lib/__tests__/invalidScopeDisables.test.js b/lib/__tests__/invalidScopeDisables.test.js
@@ -91,3 +91,15 @@ it('invalidScopeDisables ignored case', () => {
 		]);
 	});
 });
+
+it('invalidScopeDisables without config', () => {
+	return standalone({
+		config: {
+			rules: {},
+		},
+		code: 'a {}',
+		ignoreDisables: true,
+	}).then((linted) => {
+		expect(invalidScopeDisables(linted.results)).toEqual([]);
+	});
+});
diff --git a/lib/invalidScopeDisables.js b/lib/invalidScopeDisables.js
@@ -6,10 +6,10 @@
 
 /**
  * @param {import('stylelint').StylelintResult[]} results
- * @param {import('stylelint').StylelintConfig} config
+ * @param {import('stylelint').StylelintConfig|undefined} config
  * @returns {StylelintDisableOptionsReport}
  */
-module.exports = function(results, config) {
+module.exports = function(results, config = {}) {
 	/** @type {StylelintDisableOptionsReport} */
 	const report = [];
 	const usedRules = new Set(Object.keys(config.rules || {}));

diff --git a/lib/standalone.js b/lib/standalone.js
@@ -306,8 +306,7 @@ module.exports = function(options) {
 		if (reportInvalidScopeDisables) {
 			returnValue.invalidScopeDisables = invalidScopeDisables(
 				stylelintResults,
-				// TODO TYPES possible undefined
-				/** @type {import('stylelint').StylelintConfig} */ (stylelint._options.config),
+				stylelint._options.config,
 			);
 		}