tests(migration): typescript

commitlint.config.js

@@ -12,7 +12,6 @@ module.exports = {
 		"body-leading-blank": [1, "always"],
 		"footer-leading-blank": [1, "always"],
 		"header-max-length": [2, "always", 80],
-		lang: [0, "always", "eng"],
 		"scope-case": [2, "always", "lowerCase"],
 		"scope-empty": [0, "never"],
 		"subject-case": [2, "never", ["sentence-case", "start-case", "pascal-case", "upper-case"]],

package.json

@@ -43,7 +43,8 @@
     "travis:integration": "npm run build && npm run test && npm run reportCoverage",
     "travis:lint": "lerna bootstrap && npm run build && npm run lint && npm run tslint && npm run bundlesize",
     "tslint": "tslint -c tslint.json \"packages/**/*.ts\"",
-    "watch": "npm run build && tsc -w"
+    "watch": "npm run build && tsc -w",
+    "test:local": "jest --watch"
   },
   "husky": {
     "hooks": {
@@ -69,9 +70,15 @@
       "html",
       "cobertura"
     ],
-    "moduleNameMapper": {
+    "transform": {
       "^.+\\.(ts)?$": "ts-jest"
-    }
+    },
+    "testRegex": "/__tests__/.*\\.(test.js|test.ts)$",
+    "moduleFileExtensions": [
+      "ts",
+      "js",
+      "json"
+    ]
   },
   "nyc": {
     "include": [

packages/add/README.md

@@ -10,17 +10,25 @@ This package contains the logic to add new properties in a webpack configuration
 npm i -D webpack-cli @webpack-cli/add
 ```
 
+or
+
+```bash
+yarn add --dev  webpack-cli @webpack-cli/add
+```
+
 ## Usage
 
 To run the scaffolding instance programmatically, install it as a dependency. When using the package programmatically, one does not have to install webpack-cli.
 
 ### Node
+
 ```js
 const add = require("@webpack-cli/add").default;
 add();
 ```
 
 ### CLI (via `webpack-cli`)
+
 ```bash
 npx webpack-cli add
-```
+```

packages/add/index.ts

@@ -9,15 +9,14 @@ import modifyConfigHelper from "@webpack-cli/utils/modify-config-helper";
  * 	we're given on a generator
  *
  */
+const DEFAULT_WEBPACK_CONFIG_FILENAME: string = "webpack.config.js";
 
 export default function add(...args: string[]): Function {
-	const DEFAULT_WEBPACK_CONFIG_FILENAME: string = "webpack.config.js";
+  const filePaths: string[] = args.slice(3);
+  let configFile: string = DEFAULT_WEBPACK_CONFIG_FILENAME;
+  if (filePaths.length) {
+	configFile = filePaths[0];
+  }
 
-	const filePaths: string[] = args.slice(3);
-	let configFile: string = DEFAULT_WEBPACK_CONFIG_FILENAME;
-	if (filePaths.length) {
-		configFile = filePaths[0];
-	}
-
-	return modifyConfigHelper("add", defaultGenerator, configFile);
+  return modifyConfigHelper("add", defaultGenerator, configFile);
 }

packages/migrate/__tests__/migrate.test.ts

@@ -0,0 +1,63 @@
+import { transform, transformations } from "../migrate";
+
+const input = `
+module.exports = {
+	devtool: 'eval',
+	entry: [
+		'./src/index'
+	],
+	output: {
+		path: path.join(__dirname, 'dist'),
+		filename: 'index.js'
+	},
+	module: {
+		loaders: [{
+			test: /.js$/,
+			loaders: ['babel'],
+			include: path.join(__dirname, 'src')
+		}]
+	},
+	resolve: {
+		root: path.resolve('/src'),
+		modules: ['node_modules']
+	},
+	plugins: [
+		new webpack.optimize.UglifyJsPlugin(),
+		new webpack.optimize.OccurrenceOrderPlugin()
+	],
+	debug: true
+};
+`;
+
+describe("transform", () => {
+  it("should not transform if no transformations defined", (done) => {
+	transform(input, []).then((output) => {
+		expect(output).toMatchSnapshot(input);
+		done();
+	});
+  });
+
+  it("should transform using all transformations", (done) => {
+	transform(input).then((output) => {
+		expect(output).toMatchSnapshot();
+		done();
+	});
+  });
+
+  it("should transform only using specified transformations", (done) => {
+	transform(input, [transformations.loadersTransform]).then((output) => {
+		expect(output).toMatchSnapshot();
+		done();
+	});
+  });
+
+  it("should respect recast options", (done) => {
+	transform(input, undefined, {
+		quote: "double",
+		trailingComma: true,
+	}).then((output) => {
+		expect(output).toMatchSnapshot();
+		done();
+	});
+  });
+});

packages/migrate/bannerPlugin/__tests__/bannerPlugin.test.ts

@@ -0,0 +1,8 @@
+import defineTest from "@webpack-cli/utils/defineTest";
+import { join } from "path";
+
+const dirName: string = join(__dirname, "..");
+
+defineTest(dirName, "bannerPlugin", "bannerPlugin-0");
+defineTest(dirName, "bannerPlugin", "bannerPlugin-1");
+defineTest(dirName, "bannerPlugin", "bannerPlugin-2");

packages/migrate/commonsChunkPlugin/__tests__/commonsChunkPlugin.test.ts

@@ -0,0 +1,32 @@
+import defineTest from "@webpack-cli/utils/defineTest";
+import { join } from "path";
+
+const dirName: string = join(__dirname, "..");
+
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-0");
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-1");
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-2");
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-3");
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-4");
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-5");
+defineTest(
+  dirName,
+  "commonsChunkPlugin",
+  "commonsChunkPlugin-6a",
+);
+defineTest(
+  dirName,
+  "commonsChunkPlugin",
+  "commonsChunkPlugin-6b",
+);
+defineTest(
+  dirName,
+  "commonsChunkPlugin",
+  "commonsChunkPlugin-6c",
+);
+defineTest(
+  dirName,
+  "commonsChunkPlugin",
+  "commonsChunkPlugin-6d",
+);
+defineTest(dirName, "commonsChunkPlugin", "commonsChunkPlugin-7");

packages/migrate/extractTextPlugin/__tests__/__snapshots__/extractTextPlugin.test.ts.snap

@@ -0,0 +1,21 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`extractTextPlugin transforms correctly 1`] = `
+"const ExtractTextPlugin = require(\\"extract-text-webpack-plugin\\");
+
+module.exports = {
+	module: {
+		rules: [
+			{
+				test: /\\\\.css$/,
+				use: ExtractTextPlugin.extract({
+                    fallback: 'style-loader',
+                    use: 'css-loader'
+                })
+			}
+		]
+	},
+	plugins: [new ExtractTextPlugin(\\"styles.css\\")]
+};
+"
+`;

packages/migrate/extractTextPlugin/__tests__/__testfixtures__/extractTextPlugin.input.js

@@ -0,0 +1,13 @@
+const ExtractTextPlugin = require("extract-text-webpack-plugin");
+
+module.exports = {
+	module: {
+		rules: [
+			{
+				test: /\.css$/,
+				use: ExtractTextPlugin.extract("style-loader", "css-loader")
+			}
+		]
+	},
+	plugins: [new ExtractTextPlugin("styles.css")]
+};

packages/migrate/extractTextPlugin/__tests__/extractTextPlugin.test.ts

@@ -0,0 +1,5 @@
+import defineTest from "@webpack-cli/utils/defineTest";
+import { join } from "path";
+
+const dirName: string = join(__dirname, "..");
+defineTest(dirName, "extractTextPlugin");

packages/migrate/index.ts

@@ -17,7 +17,7 @@ declare var process: {
 	webpackModule: {
 		validate: Function;
 		WebpackOptionsValidationError: {
-			new(errors: string[]): {
+			new: (errors: string[]) => {
 				message: string;
 			};
 		};

packages/migrate/loaderOptionsPlugin/__tests__/loaderOptionsPlugin.test.ts

@@ -0,0 +1,25 @@
+import defineTest from "@webpack-cli/utils/defineTest";
+import { join } from "path";
+
+const dirName: string = join(__dirname, "..");
+
+defineTest(
+  dirName,
+  "loaderOptionsPlugin",
+  "loaderOptionsPlugin-0",
+);
+defineTest(
+  dirName,
+  "loaderOptionsPlugin",
+  "loaderOptionsPlugin-1",
+);
+defineTest(
+  dirName,
+  "loaderOptionsPlugin",
+  "loaderOptionsPlugin-2",
+);
+defineTest(
+  dirName,
+  "loaderOptionsPlugin",
+  "loaderOptionsPlugin-3",
+);