add tests

Ffloriel · Ffloriel · commit dcc062507e0d · 2017-12-17T20:37:17.000Z
add integration tests based on extract-text-webpack-plugin tests
diff --git a/.babelrc b/.babelrc
@@ -1,20 +1,26 @@
 {
-  "presets": [
-    [
-      "env",
-      {
-        "modules": false
-      }
-    ]
-  ],
-  "plugins": [
-    "external-helpers",
-    "syntax-object-rest-spread",
-    "transform-object-rest-spread"
-  ]
-
-  // {
-  //   "presets": ["es2015-rollup"],
-  //   "plugins": ["syntax-object-rest-spread", "transform-object-rest-spread"]
-  // }
+  "env": {
+    "test": {
+      "presets": ["env"],
+      "plugins": ["syntax-object-rest-spread", "transform-object-rest-spread"]
+    },
+    "development": {
+      "presets": [
+        [
+          "env",
+          {
+            "targets": {
+              "browsers": ["last 2 versions"]
+            },
+            "modules": false
+          }
+        ]
+      ],
+      "plugins": [
+        "external-helpers",
+        "syntax-object-rest-spread",
+        "transform-object-rest-spread"
+      ]
+    }
+  }
 }
diff --git a/.eslintrc b/.eslintrc
@@ -1,13 +1,14 @@
 {
-    "extends": "eslint:recommended",
-    "parserOptions": {
-        "ecmaVersion": 6,
-        "sourceType": "module"
-    },
-    "env": {
-        "browser": true,
-        "commonjs": true,
-        "node": true,
-        "es6": true
-    }
-}
+  "extends": "eslint:recommended",
+  "parserOptions": {
+    "ecmaVersion": 6,
+    "sourceType": "module"
+  },
+  "env": {
+    "browser": true,
+    "commonjs": true,
+    "node": true,
+    "es6": true
+  },
+  "parser": "babel-eslint"
+}
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,5 @@
+/__tests__/js
+
 # Logs
 logs
 *.log
diff --git a/__tests__/.eslintrc b/__tests__/.eslintrc
@@ -0,0 +1,5 @@
+{
+  "env": {
+    "jest": true
+  }
+}
diff --git a/__tests__/__snapshots__/webpack-integration.test.js.snap b/__tests__/__snapshots__/webpack-integration.test.js.snap
@@ -0,0 +1,16 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Webpack Integration Tests simple 1`] = `
+".hello {
+  color: red;
+}
+
+.whitelisted {
+  color: green;
+}
+
+md\\\\:w-2\\\\/3 {
+  color: red;
+}
+"
+`;
diff --git a/__tests__/cases/simple/expected/style.css b/__tests__/cases/simple/expected/style.css
@@ -0,0 +1,11 @@
+.hello {
+  color: red;
+}
+
+.whitelisted {
+  color: green;
+}
+
+md\:w-2\/3 {
+  color: red;
+}
diff --git a/__tests__/cases/simple/src/content.html b/__tests__/cases/simple/src/content.html
@@ -0,0 +1,13 @@
+<html>
+
+<head>
+  <title>Purgecss webpack test</title>
+</head>
+
+<body>
+  <div class="md:w-2/3"></div>
+  <div class="hello"></div>
+  <div></div>
+</body>
+
+</html>
diff --git a/__tests__/cases/simple/src/index.js b/__tests__/cases/simple/src/index.js
@@ -0,0 +1 @@
+import './../style.css'
diff --git a/__tests__/cases/simple/style.css b/__tests__/cases/simple/style.css
@@ -0,0 +1,15 @@
+.hello {
+  color: red;
+}
+
+.unused {
+  color: blue;
+}
+
+.whitelisted {
+  color: green;
+}
+
+md\:w-2\/3 {
+  color: red;
+}
diff --git a/__tests__/cases/simple/webpack.config.js b/__tests__/cases/simple/webpack.config.js
@@ -0,0 +1,46 @@
+const path = require('path')
+const glob = require('glob')
+const ExtractTextPlugin = require('extract-text-webpack-plugin')
+const PurgecssPlugin = require('../../../src/').default
+
+class CustomExtractor {
+  static extract(content) {
+    return content.match(/[A-z0-9-:/]+/g)
+  }
+}
+
+const PATHS = {
+  src: path.join(__dirname, 'src')
+}
+
+module.exports = {
+  entry: './src/index.js',
+  // output: {
+  //   filename: 'bundle.js',
+  //   path: path.join(__dirname, 'dist')
+  // },
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: ExtractTextPlugin.extract({
+          fallback: 'style-loader',
+          use: 'css-loader?sourceMap'
+        })
+      }
+    ]
+  },
+  plugins: [
+    new ExtractTextPlugin('style.css'),
+    new PurgecssPlugin({
+      paths: glob.sync(`${PATHS.src}/*`),
+      whitelist: ['whitelisted'],
+      extractors: [
+        {
+          extractor: CustomExtractor,
+          extensions: ['html', 'js']
+        }
+      ]
+    })
+  ]
+}
diff --git a/__tests__/webpack-integration.test.js b/__tests__/webpack-integration.test.js
@@ -0,0 +1,62 @@
+import fs from 'fs'
+import path from 'path'
+import webpack from 'webpack'
+import ExtractPlugin from 'extract-text-webpack-plugin'
+
+const cases = process.env.CASES
+  ? process.env.CASES.split(',')
+  : fs.readdirSync(path.join(__dirname, 'cases'))
+
+describe('Webpack Integration Tests', () => {
+  cases.forEach(testCase => {
+    it(testCase, done => {
+      let options = { entry: { test: './index.js' } }
+      const testDirectory = path.join(__dirname, 'cases', testCase)
+      const outputDirectory = path.join(__dirname, 'js', testCase)
+      const configFile = path.join(testDirectory, 'webpack.config.js')
+
+      if (fs.existsSync(configFile)) {
+        options = require(configFile)
+      }
+      options.context = testDirectory
+      if (!options.module) options.module = {}
+      if (!options.module.loaders) {
+        options.module.loaders = [
+          { test: /\.txt$/, loader: ExtractPlugin.extract('raw-loader') }
+        ]
+      }
+      if (!options.output) options.output = { filename: '[name].js' }
+      if (!options.output.path) options.output.path = outputDirectory
+      if (process.env.CASES) {
+        console.log(
+          `\nwebpack.${testCase}.config.js ${JSON.stringify(options, null, 2)}`
+        )
+      }
+
+      webpack(options, (err, stats) => {
+        if (err) return done(err)
+        if (stats.hasErrors()) return done(new Error(stats.toString()))
+        const testFile = path.join(outputDirectory, 'test.js')
+        if (fs.existsSync(testFile)) {
+          require(testFile)(suite)
+        }
+        const expectedDirectory = path.join(testDirectory, 'expected')
+        fs.readdirSync(expectedDirectory).forEach(file => {
+          const filePath = path.join(expectedDirectory, file)
+          const actualPath = path.join(outputDirectory, file)
+          expect(readFileOrEmpty(actualPath)).toEqual(readFileOrEmpty(filePath))
+          expect(readFileOrEmpty(actualPath)).toMatchSnapshot()
+        })
+        done()
+      })
+    })
+  })
+})
+
+function readFileOrEmpty(path) {
+  try {
+    return fs.readFileSync(path, 'utf-8')
+  } catch (e) {
+    return ''
+  }
+}
diff --git a/package.json b/package.json
@@ -12,32 +12,29 @@
   "scripts": {
     "build": "rollup -c rollup.config.js",
     "dev": "rollup -c rollup.config.js -w",
-    "test": "jest",
+    "test": "npx jest .+.test.js",
     "lint": "eslint -c .eslintrc src/"
   },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/FullHuman/purgecss-webpack-plugin.git"
   },
-  "keywords": [
-    "webpack-plugin",
-    "purgecss",
-    "remove",
-    "unused",
-    "css"
-  ],
+  "keywords": ["webpack-plugin", "purgecss", "remove", "unused", "css"],
   "author": "Ffloriel",
   "license": "MIT",
   "bugs": {
     "url": "https://github.com/FullHuman/purgecss-webpack-plugin/issues"
   },
   "homepage": "https://github.com/FullHuman/purgecss-webpack-plugin#readme",
   "devDependencies": {
+    "babel-eslint": "^8.0.3",
     "babel-plugin-external-helpers": "^6.22.0",
     "babel-plugin-syntax-object-rest-spread": "^6.13.0",
     "babel-plugin-transform-object-rest-spread": "^6.26.0",
     "babel-preset-env": "^1.6.1",
+    "css-loader": "^0.28.7",
     "eslint": "^4.13.1",
+    "extract-text-webpack-plugin": "^3.0.2",
     "jest": "^21.2.1",
     "rollup": "^0.52.1",
     "rollup-plugin-babel": "^3.0.2",

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+/__tests__/js`
	`2`	`+`
`1`	`3`	`# Logs`
`2`	`4`	`logs`
`3`	`5`	`*.log`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +{
 +  "env": {
 +    "jest": true
 +  }
 +}