test: refactor (#299)

webpack-contrib · Oct 10, 2017 · e066f34 · e066f34
1 parent aaf1672
commit e066f34
Show file tree

Hide file tree

Showing 70 changed files with 593 additions and 4,822 deletions.
diff --git a/.npmrc b/.npmrc
@@ -0,0 +1 @@
+package-lock=false
diff --git a/.travis.yml b/.travis.yml
@@ -1,12 +1,12 @@
-cache:
-  directories:
-    - node_modules
-
 language: node_js
+
 node_js:
   - "node"
   - 6
-  - 4
+
+after_success:
+  - npm i coveralls
+  - cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js
 
 notifications:
   email: false
diff --git a/README.md b/README.md
@@ -2,10 +2,9 @@
 [![node][node]][node-url]
 [![deps][deps]][deps-url]
 [![tests][tests]][tests-url]
+[![coverage][cover]][cover-url]
 [![chat][chat]][chat-url]
 
-<!-- [![coverage][cover]][cover-url] -->
-
 <div align="center">
   <img width="180" height="180" hspace="10"
     alt="PostCSS Logo"

diff --git a/lib/index.js b/lib/index.js
@@ -40,13 +40,13 @@ const SyntaxError = require('./Error')
  * @return {cb} cb      Result
  */
 module.exports = function loader (css, map) {
-  const cb = this.async()
-  const file = this.resourcePath
-
   const options = loaderUtils.getOptions(this) || {}
 
   validateOptions(require('./options.json'), options, 'PostCSS Loader')
 
+  const cb = this.async()
+  const file = this.resourcePath
+
   const sourceMap = options.sourceMap
 
   Promise.resolve().then(() => {