Add throw-validate-errors option #71

Author: hudochenkov

README.md

@@ -27,6 +27,7 @@ The plugin has no default options. Everything is disabled by default.
 - [`order`](./lib/order/README.md): Specify the order of content within declaration blocks.
 - [`properties-order`](./lib/properties-order/README.md): Specify the order of properties within declaration blocks.
 - [`unspecified-properties-position`](./lib/properties-order/unspecified-properties-position.md): Specify position for properties not specified in `properties-order`.
+- `throw-validate-errors`: Throw config validation errors instead of just showing and ignoring them.
 
 ## Handling comments
 

index.js

@@ -17,12 +17,22 @@ function plugin(css, opts) {
 	const validatedOptions = validateOptions(opts);
 
 	if (validatedOptions !== true) {
-		// eslint-disable-next-line no-console
-		if (console && console.warn && _.isString(validatedOptions)) {
-			console.warn(validatedOptions); // eslint-disable-line no-console
-		}
+		const throwValidateErrors = _.get(opts, 'throw-validate-errors', false);
+
+		if (throwValidateErrors) {
+			if (_.isString(validatedOptions)) {
+				throw new Error(validatedOptions);
+			}
 
-		return;
+			throw new Error(`postcss-sorting: Invalid config.`);
+		} else {
+			// eslint-disable-next-line no-console
+			if (console && console.warn && _.isString(validatedOptions)) {
+				console.warn(validatedOptions); // eslint-disable-line no-console
+			}
+
+			return;
+		}
 	}
 
 	if (opts.order) {

lib/__tests__/test.js

@@ -1,4 +1,26 @@
 'use strict';
 
-test(`Should do nothing if config is undefined`, () =>
-	runTest('empty-lines-preserve', undefined, __dirname));
+const plugin = require('../../');
+const postcss = require('postcss');
+
+test(`Should do nothing if config is undefined`, () => {
+	runTest('empty-lines-preserve', undefined, __dirname);
+});
+
+test(`Should throw an error if config has error`, () => {
+	const opts = {
+		'throw-validate-errors': true,
+		order: 'Justice Rains From Above',
+	};
+
+	const pluginRun = postcss([plugin(opts)])
+		.process('')
+		.then(() => {
+			expect('Plugin should throw an error').toBeFalst();
+		})
+		.catch(err => {
+			throw err;
+		});
+
+	expect(pluginRun).rejects.toBeTruthy();
+});