test: refactor

Author: michael-ciniawsky

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(() => {

package.json

@@ -17,25 +17,29 @@
   },
   "devDependencies": {
     "coveralls": "^2.0.0",
-    "fs-extra": "^3.0.0",
     "jest": "^21.0.0",
     "jsdoc-to-markdown": "^3.0.0",
+    "memory-fs": "^0.4.0",
     "postcss-js": "^1.0.0",
     "standard": "^10.0.0",
     "standard-version": "^4.0.0",
     "sugarss": "^1.0.0",
     "webpack": "^3.0.0"
   },
   "scripts": {
-    "clean": "rm -rf dmd jsdoc-api jest* coverage test/builds",
+    "clean": "rm -rf dmd jest* coverage jsdoc-api test/results",
     "lint": "standard",
     "docs": "jsdoc2md lib/index.js > LOADER.md",
-    "pretest": "npm run lint && npm run test:build",
-    "test": "jest",
-    "test:build": "node test/webpack.build.js",
+    "postdocs": "npm run clean",
+    "test": "jest --verbose --coverage",
     "posttest": "npm run clean",
     "release": "standard-version"
   },
+  "jest": {
+    "collectCoverageFrom": [
+      "lib/*.js"
+    ]
+  },
   "standard": {
     "env": {
       "jest": true

test/Errors.test.js

@@ -0,0 +1,31 @@
+'use strict'
+
+const webpack = require('./helpers/compiler')
+const { loader } = require('./helpers/compilation')
+
+describe('Errors', () => {
+  test('Validation Error', () => {
+    const loader = require('../lib')
+
+    const error = () => loader.call({ query: { sourceMap: 1 } })
+
+    expect(error).toThrow()
+    expect(error).toThrowErrorMatchingSnapshot()
+  })
+
+  test('Syntax Error', () => {
+    const config = {
+      loader: {
+        options: {
+          parser: 'sugarss'
+        }
+      }
+    }
+
+    return webpack('css/index.js', config).then((stats) => {
+      const error = loader(stats).err
+
+      expect(error[0]).toMatchSnapshot()
+    })
+  })
+})

test/__snapshots__/Errors.test.js.snap

@@ -0,0 +1,21 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Errors Syntax Error 1`] = `
+[ModuleBuildError: Module build failed: Syntax Error 
+
+(1:3) Unexpected separator in property
+
+> 1 | a { color: black }
+    |   ^
+  2 | 
+]
+`;
+
+exports[`Errors Validation Error 1`] = `
+"Validation Error
+
+PostCSS Loader Invalid Options
+
+options.sourceMap should be string,boolean
+"
+`;

test/__snapshots__/loader.test.js.snap

@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Loader Default 1`] = `"module.exports = \\"a { color: black }\\\\n\\""`;

test/configs/config.config.js

@@ -1,5 +1,5 @@
 module.exports = (ctx) => ({
   plugins: [
-    ctx.options.plugin ? require('../../plugin')() : false
+    ctx.options.plugin ? require('./plugin')() : false
   ]
 })

test/configs/config.js

@@ -0,0 +1,3 @@
+import style from './style.css'
+
+export default style

test/configs/error.config.js

@@ -0,0 +1,3 @@
+import style from './style.exec.js'
+
+export default style

test/configs/error.js

@@ -0,0 +1,3 @@
+import style from './style.js'
+
+export default style

test/configs/exec.config.js

@@ -1,5 +1,5 @@
 module.exports = {
   plugins: [
-    require('../plugin')()
+    require('./config/plugin')()
   ]
 }

test/configs/exec.js

@@ -0,0 +1,3 @@
+import style from './style.sss'
+
+export default style