Properly set the errored property for --report options

lib/prepareReturnValue.js

@@ -34,7 +34,10 @@ function prepareReturnValue(stylelintResults, options, formatter) {
 	if (reportDescriptionlessDisables) descriptionlessDisables(stylelintResults);
 
 	const errored = stylelintResults.some(
-		(result) => result.errored || result.parseErrors.length > 0,
+		(result) =>
+			result.errored ||
+			result.parseErrors.length > 0 ||
+			result.warnings.some((warning) => warning.severity === 'error'),
 	);
 
 	/** @type {StylelintStandaloneReturnValue} */

system-tests/004/__snapshots__/fs.test.js.snap

@@ -0,0 +1,69 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`no-fs - errored state for reportNeedlessDisables 1`] = `
+Object {
+  "errored": true,
+  "output": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [
+        Object {
+          "column": 1,
+          "line": 1,
+          "rule": "--report-needless-disables",
+          "severity": "error",
+          "text": "Needless disable for \\"block-no-empty\\"",
+        },
+      ],
+    },
+  ],
+  "reportedDisables": Array [],
+  "results": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "ignored": undefined,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [
+        Object {
+          "column": 1,
+          "line": 1,
+          "rule": "--report-needless-disables",
+          "severity": "error",
+          "text": "Needless disable for \\"block-no-empty\\"",
+        },
+      ],
+    },
+  ],
+}
+`;
+
+exports[`no-fs - no errored state 1`] = `
+Object {
+  "errored": false,
+  "output": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [],
+    },
+  ],
+  "reportedDisables": Array [],
+  "results": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "ignored": undefined,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [],
+    },
+  ],
+}
+`;

system-tests/004/__snapshots__/no-fs.test.js.snap

@@ -0,0 +1,69 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`no-fs - errored state for reportNeedlessDisables 1`] = `
+Object {
+  "errored": true,
+  "output": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [
+        Object {
+          "column": 1,
+          "line": 1,
+          "rule": "--report-needless-disables",
+          "severity": "error",
+          "text": "Needless disable for \\"block-no-empty\\"",
+        },
+      ],
+    },
+  ],
+  "reportedDisables": Array [],
+  "results": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "ignored": undefined,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [
+        Object {
+          "column": 1,
+          "line": 1,
+          "rule": "--report-needless-disables",
+          "severity": "error",
+          "text": "Needless disable for \\"block-no-empty\\"",
+        },
+      ],
+    },
+  ],
+}
+`;
+
+exports[`no-fs - no errored state 1`] = `
+Object {
+  "errored": false,
+  "output": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [],
+    },
+  ],
+  "reportedDisables": Array [],
+  "results": Array [
+    Object {
+      "deprecations": Array [],
+      "errored": false,
+      "ignored": undefined,
+      "invalidOptionWarnings": Array [],
+      "parseErrors": Array [],
+      "warnings": Array [],
+    },
+  ],
+}
+`;

system-tests/004/config.json

@@ -0,0 +1,5 @@
+{
+	"rules": {
+		"block-no-empty": true
+	}
+}

system-tests/004/fs.test.js

@@ -0,0 +1,29 @@
+'use strict';
+
+const stylelint = require('../../lib');
+const { caseConfigFile, caseFilesForFix, prepForSnapshot } = require('../systemTestUtils');
+
+const CASE_NUMBER = '004';
+
+it('no-fs - errored state for reportNeedlessDisables', async () => {
+	expect(
+		prepForSnapshot(
+			await stylelint.lint({
+				files: await caseFilesForFix(CASE_NUMBER),
+				configFile: caseConfigFile(CASE_NUMBER),
+				reportNeedlessDisables: true,
+			}),
+		),
+	).toMatchSnapshot();
+}, 10000);
+
+it('no-fs - no errored state', async () => {
+	expect(
+		prepForSnapshot(
+			await stylelint.lint({
+				files: await caseFilesForFix(CASE_NUMBER),
+				configFile: caseConfigFile(CASE_NUMBER),
+			}),
+		),
+	).toMatchSnapshot();
+}, 10000);

system-tests/004/no-fs.test.js

@@ -0,0 +1,29 @@
+'use strict';
+
+const stylelint = require('../../lib');
+const { caseConfig, caseCode, prepForSnapshot } = require('../systemTestUtils');
+
+const CASE_NUMBER = '004';
+
+it('no-fs - errored state for reportNeedlessDisables', async () => {
+	expect(
+		prepForSnapshot(
+			await stylelint.lint({
+				code: await caseCode(CASE_NUMBER),
+				config: await caseConfig(CASE_NUMBER),
+				reportNeedlessDisables: true,
+			}),
+		),
+	).toMatchSnapshot();
+}, 10000);
+
+it('no-fs - no errored state', async () => {
+	expect(
+		prepForSnapshot(
+			await stylelint.lint({
+				code: await caseCode(CASE_NUMBER),
+				config: await caseConfig(CASE_NUMBER),
+			}),
+		),
+	).toMatchSnapshot();
+}, 10000);

system-tests/004/stylesheet.css

@@ -0,0 +1,4 @@
+/* stylelint-disable block-no-empty */
+a {
+  color: red;
+}