warn, don't error

Author: cmrn

index.js

@@ -40,7 +40,7 @@ const replaceValueSymbols = (valueString, replacements) => {
 const getDefinition = (atRule, existingDefinitions, requiredDefinitions) => {
   const matches = matchValueDefinition.exec(atRule.params);
   if (!matches) {
-    throw atRule.error('Invalid @value definition');
+    return null;
   }
 
   const [/* match */, name, middle, value, end] = matches;
@@ -115,6 +115,9 @@ const walk = async (requiredDefinitions, walkFile, root, result) => {
     }
 
     const newDefinitions = getDefinition(atRule, existingDefinitions, requiredDefinitions);
+    if (!newDefinitions) {
+      result.warn(`Invalid value definition: ${atRule.params}`);
+    }
     return Object.assign(existingDefinitions, newDefinitions);
   };
 

index.test.js

@@ -51,7 +51,7 @@ test('should remove exports if noEmitExports is true', async (t) => {
   await run(t, '@value red blue;', '', { noEmitExports: true });
 });
 
-test('gives an error when there is no semicolon between lines', async (t) => {
+test('gives a warning when there is no semicolon between lines', async (t) => {
   const input = '@value red blue\n@value green yellow';
   const processor = postcss([plugin]);
   const result = await processor.process(input, { from: undefined });
@@ -61,6 +61,16 @@ test('gives an error when there is no semicolon between lines', async (t) => {
   t.expect(warnings[0].text).toBe('Invalid value definition: red blue\n@value green yellow');
 });
 
+test('gives a warning when @value definition is invalid', async (t) => {
+  const input = '@value oops:;';
+  const processor = postcss([plugin]);
+  const result = await processor.process(input, { from: undefined });
+  const warnings = result.warnings();
+
+  t.expect(warnings.length).toBe(1);
+  t.expect(warnings[0].text).toBe('Invalid value definition: oops:');
+});
+
 test('gives an error when path to imported file is wrong', async (t) => {
   const input = '@value red from "./non-existent-file.css"';
   const processor = postcss([plugin]);
@@ -73,12 +83,6 @@ test('gives an error when @value import statement is invalid', async (t) => {
   await t.expect(processor.process(input, parserOpts)).rejects.toThrow('@value statement "" is invalid!');
 });
 
-test('gives an error when @value declaration is invalid', async (t) => {
-  const input = '@value oops;';
-  const processor = postcss([plugin]);
-  await t.expect(processor.process(input, parserOpts)).rejects.toThrow('Invalid @value definition');
-});
-
 test('shouldn\'t break on draft spec syntax', async (t) => {
   await run(
     t,