Merge remote-tracking branch 'origin/warning-format'

Author: pirxpilot

Makefile

@@ -3,7 +3,7 @@ all: clean lint test
 lint:
 	./node_modules/.bin/jshint *.js
 
-TESTS = opts source-maps source-maps-file stdout stdin config config-all js-config js-config-all invalid
+TESTS = opts source-maps source-maps-file stdout stdin config config-all js-config js-config-all invalid warning
 
 
 DIFF = diff -q
@@ -69,6 +69,9 @@ test/build/stdin.css: test/in.css
 test/build/invalid.css: test/in-force-error.css
 	./bin/postcss --use ./test/dummy-plugin --dummy-plugin.fail=true -o $@ $< || echo Error is OK here....
 
+test/build/warning.css: test/in-warning.css
+	./bin/postcss --use ./test/dummy-plugin -o $@ $< && echo Warning is OK here....
+
 test/build/config.css: test/in.css
 	./bin/postcss -u postcss-url -c test/config.json -o $@ $<
 	$(DIFF) $@ $(subst build,ref,$@)

index.js

@@ -187,7 +187,7 @@ function processCSS(processor, input, output, fn) {
   function doProcess(css, fn) {
     function onResult(result) {
       if (typeof result.warnings === 'function') {
-        result.warnings().forEach(console.error);
+        result.warnings().forEach(function(w) { console.warn(w.toString()); });
       }
       fn(null, result);
     }

test/dummy-plugin.js

@@ -1,14 +1,15 @@
-module.exports = function(opts) {
+var postcss = require('postcss');
+
+module.exports = postcss.plugin('dummy-plugin', function(opts) {
   opts = opts || {};
 
-  return function (css) {
+  return function (css, result) {
     // throw something so that we can test error messages
     if (css.nodes[0].selector === 'a') {
       throw css.nodes[0].error('Dummy error');
     }
+    if (css.nodes[0].selector === 'figure') {
+    	result.warn('Dummy warning', { node: css.nodes[0] });
+    }
   };
-};
-
-module.exports.postcss = function (css) {
-  return module.exports()(css);
-};
+});

test/in-warning.css

@@ -0,0 +1,3 @@
+figure {
+  display: none;
+}