Fixed: replacement of postcss-log-warnings (deprecated) by postcss-reporter

Close #162

Author: MoOx

CHANGELOG.md

@@ -1,3 +1,7 @@
+
+- Fixed: replacement of `postcss-log-warnings` (deprecated) by `postcss-reporter`
+([#162](https://github.com/cssnext/cssnext/issues/162))
+
 # 1.7.1 - 2015-06-19
 
 - Fixed: bullet for browser messages should be visible on Chrome Windows.

CONTRIBUTING.md

@@ -14,6 +14,16 @@ $ npm test
 $ git checkout -b fix.bug423
 ```
 
+`npm test` will compile what is need to be and run all tests.
+If you want to work on one test only, you can run something like
+
+```console
+$ babel-tape-runner src/__tests__/option.browsers.js
+```
+
+_Be sure to have in your PATH `./node_modules/.bin` so you can use local
+module directly in console._
+
 ## Details
 
 ### Add a feature

package.json

@@ -47,11 +47,11 @@
     "postcss-custom-selectors": "^2.0.1",
     "postcss-font-variant": "^1.0.0",
     "postcss-import": "^6.0.0",
-    "postcss-log-warnings": "^0.3.1",
     "postcss-media-minmax": "^1.1.0",
     "postcss-messages": "^0.2.2",
     "postcss-pseudo-class-any-link": "^0.2.1",
     "postcss-pseudoelements": "^2.1.1",
+    "postcss-reporter": "^0.1.0",
     "postcss-selector-matches": "^1.0.1",
     "postcss-selector-not": "^1.0.1",
     "postcss-url": "^3.0.0",
@@ -65,6 +65,7 @@
     "babel-core": "^5.4.7",
     "babel-eslint": "^3.1.17",
     "babel-loader": "^5.1.3",
+    "babel-tape-runner": "^1.1.0",
     "classnames": "^2.1.1",
     "css-loader": "^0.13.1",
     "cssnext-loader": "^1.0.1",

src/__tests__/options.browsers.js renamed to src/__tests__/option.browsers.js

@@ -3,27 +3,39 @@ const test = require("tape")
 const cssnext = require("..")
 
 test("cssnext browsers option", function(t) {
-  const input = ":root{--foo:bar}baz{qux:var(--foo)}"
-  const output = "baz{qux: bar}"
+
+  // no recent browser need pixrem
+  const remInput = "body{font-size:2rem}"
+  t.equal(
+    cssnext(
+      remInput,
+      {browsers: "last 1 version"}
+    ),
+    remInput,
+    "should not enable px fallback when all browsers support it"
+  )
+
+  const customPropsInput = ":root{--foo:bar}baz{qux:var(--foo)}"
+  const customPropsOutput = "baz{qux: bar}"
 
   // fx 30 doesn't handle custom prop
   t.equal(
-    cssnext(input, {browsers: "Firefox >= 30"}),
-    output,
+    cssnext(customPropsInput, {browsers: "Firefox >= 30"}),
+    customPropsOutput,
     "should enable custom properties when browsers do not support it"
   )
 
   // fx 31 handle custom prop
   t.equal(
-    cssnext(input, {browsers: "Firefox >= 31"}),
-    input,
+    cssnext(customPropsInput, {browsers: "Firefox >= 31"}),
+    customPropsInput,
     "should NOT enable custom properties when browsers support it"
   )
 
   // fx 31 support but not IE 8
   t.equal(
-    cssnext(input, {browsers: "Firefox >= 31, IE 8"}),
-    output,
+    cssnext(customPropsInput, {browsers: "Firefox >= 31, IE 8"}),
+    customPropsOutput,
     "should enable custom properties when at least one browsers do not " +
       "support it"
   )

src/__tests__/option.messages.js

@@ -67,6 +67,18 @@ test("cssnext option: messages", (t) => {
 
   // we should do the same for "console" value, but we need to test stdout
   // and I am lazy atm. Finger crossed.
+  // t.ok(
+  //   cssnext({
+  //     ...defaultOptions,
+  //     messages: {
+  //       console: true,
+  //     },
+  //   })
+  //     .process("test{}")
+  //     .css
+  //   ,
+  //   "should show messages in the console"
+  // )
 
   t.end()
 })

src/option.messages.js

@@ -1,4 +1,4 @@
-import postcssMessagesConsole from "postcss-log-warnings"
+import postcssMessagesConsole from "postcss-reporter"
 // https://github.com/postcss/postcss-messages/issues/16
 // import postcssMessagesCSS from "postcss-messages"
 import postcssMessagesCSS from "./plugins/messages"