Fix report() error message responsibility for a missing node or lin…

…e number (#7474)

Including a rule name in the error message helps us understand why an error happens.

Co-authored-by: Richard Hallows <jeddy3@users.noreply.github.com>

.changeset/weak-maps-tease.md

@@ -0,0 +1,5 @@
+---
+"stylelint": patch
+---
+
+Fixed: `report()` error message responsibility for a missing node or line number

lib/utils/__tests__/report.test.mjs

@@ -419,3 +419,14 @@ test('with custom message function', () => {
 	expect(spyArgs[0]).toBe('a=str, b=123');
 	expect(spyArgs[1].node).toBe(v.node);
 });
+
+test('without node nor line', () => {
+	const v = {
+		ruleName: 'foo',
+		result: {},
+	};
+
+	expect(() => report(v)).toThrow(
+		'The "foo" rule failed to pass either a node or a line number to the `report()` function.',
+	);
+});

lib/utils/report.cjs

@@ -47,7 +47,9 @@ function report(problem) {
 	const startLine = line || (start && start.line);
 
 	if (!startLine) {
-		throw new Error('You must pass either a node or a line number');
+		throw new Error(
+			`The "${ruleName}" rule failed to pass either a node or a line number to the \`report()\` function.`,
+		);
 	}
 
 	const { ignoreDisables } = result.stylelint.config || {};

lib/utils/report.mjs

@@ -43,7 +43,9 @@ export default function report(problem) {
 	const startLine = line || (start && start.line);
 
 	if (!startLine) {
-		throw new Error('You must pass either a node or a line number');
+		throw new Error(
+			`The "${ruleName}" rule failed to pass either a node or a line number to the \`report()\` function.`,
+		);
 	}
 
 	const { ignoreDisables } = result.stylelint.config || {};