feat: Adds a resolved path for output

lib/transformations/__snapshots__/utils.test.js.snap

@@ -6,7 +6,7 @@ Object {
   "loc": null,
   "regex": null,
   "type": "Literal",
-  "value": "strintLiteral",
+  "value": "stringLiteral",
 }
 `;
 
@@ -71,3 +71,47 @@ exports[`utils createProperty should create properties for non-literal keys 1`]
     1: \\"bar\\"
 }"
 `;
+
+exports[`utils getRequire should create a require statement 1`] = `
+Object {
+  "comments": null,
+  "declarations": Array [
+    Object {
+      "comments": null,
+      "id": Object {
+        "comments": null,
+        "loc": null,
+        "name": "filesys",
+        "type": "Identifier",
+        "typeAnnotation": null,
+      },
+      "init": Object {
+        "arguments": Array [
+          Object {
+            "comments": null,
+            "loc": null,
+            "regex": null,
+            "type": "Literal",
+            "value": "fs",
+          },
+        ],
+        "callee": Object {
+          "comments": null,
+          "loc": null,
+          "name": "require",
+          "type": "Identifier",
+          "typeAnnotation": null,
+        },
+        "comments": null,
+        "loc": null,
+        "type": "CallExpression",
+      },
+      "loc": null,
+      "type": "VariableDeclarator",
+    },
+  ],
+  "kind": "const",
+  "loc": null,
+  "type": "VariableDeclaration",
+}
+`;

lib/transformations/outputPath/__snapshots__/outputPath.test.js.snap

@@ -0,0 +1,21 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`outputPath transforms correctly using "outputPath-0" data 1`] = `
+"const path = require('path');
+module.exports = {
+	output: {
+		path: path.join(__dirname, 'dist')
+	}
+}
+"
+`;
+
+exports[`outputPath transforms correctly using "outputPath-1" data 1`] = `
+"const path = require('path');
+module.exports = {
+	output: {
+		path: path.join(__dirname, 'dist')
+	}
+}
+"
+`;

lib/transformations/outputPath/__testfixtures__/outputPath-0.input.js

@@ -0,0 +1,5 @@
+module.exports = {
+	output: {
+		path: 'dist'
+	}
+}

lib/transformations/outputPath/__testfixtures__/outputPath-1.input.js

@@ -0,0 +1,6 @@
+const path = require('path');
+module.exports = {
+	output: {
+		path: path.join(__dirname, 'dist')
+	}
+}

lib/transformations/outputPath/outputPath.js

@@ -0,0 +1,34 @@
+const utils = require('../utils');
+
+module.exports = function(j, ast) {
+	const literalOutputPath = ast.find(j.ObjectExpression)
+		.filter(p => utils.safeTraverse(p, ['parentPath', 'value', 'key', 'name']) === 'output')
+		.find(j.Property)
+		.filter(p => utils.safeTraverse(p, ['value', 'key', 'name']) === 'path' && utils.safeTraverse(p, ['value', 'value', 'type']) === 'Literal');
+	if (literalOutputPath) {
+		literalOutputPath.find(j.Literal)
+			.replaceWith(p => replaceWithPath(j, p));
+
+		const isPathPresent = ast
+			.find(j.VariableDeclarator)
+			.filter(p => utils.safeTraverse(p, ['value', 'id', 'name']) === 'path').size();
+		if(!isPathPresent){
+			const pathRequire = utils.getRequire(j, 'path', 'path');
+			return ast.find(j.Program)
+			.replaceWith(p => j.program([].concat(pathRequire).concat(p.value.body)));
+		}
+
+	}
+	return ast;
+};
+
+function replaceWithPath(j, p) {
+	const convertedPath = j.callExpression(
+		j.memberExpression(
+		j.identifier('path'),
+		j.identifier('join'),
+		false),
+		[j.identifier('__dirname'), p.value]);
+	return convertedPath;
+}
+

lib/transformations/outputPath/outputPath.test.js

@@ -0,0 +1,4 @@
+const defineTest = require('../defineTest');
+
+defineTest(__dirname, 'outputPath', 'outputPath-0');
+defineTest(__dirname, 'outputPath', 'outputPath-1');

lib/transformations/utils.js

@@ -224,6 +224,28 @@ function findObjWithOneOfKeys (p, keyNames) {
 		}, false);
 }
 
+/*
+* @function getRequire
+*
+* Returns constructed require symbol
+* @param j — jscodeshift API
+* @param { string } constName - Name of require
+* @param { string } packagePath - path of required package
+* @returns {NodePath} - the created ast
+*/
+
+function getRequire(j, constName, packagePath) {
+	return j.variableDeclaration('const', [
+		j.variableDeclarator(
+        j.identifier(constName),
+        j.callExpression(
+          j.identifier('require'),
+          [j.literal(packagePath)]
+          )
+        )
+	]);
+}
+
 module.exports = {
 	safeTraverse,
 	createProperty,
@@ -233,5 +255,6 @@ module.exports = {
 	findVariableToPlugin,
 	isType,
 	createLiteral,
-	findObjWithOneOfKeys
+	findObjWithOneOfKeys,
+	getRequire
 };

lib/transformations/utils.test.js

@@ -136,7 +136,7 @@ var a = { plugs: [] }
 
 	describe('createLiteral', () => {
 		it('should create basic literal', () => {
-			const literal = utils.createLiteral(j, 'strintLiteral');
+			const literal = utils.createLiteral(j, 'stringLiteral');
 			expect(literal).toMatchSnapshot();
 		});
 		it('should create boolean', () => {
@@ -157,4 +157,11 @@ var a = { plugs: [] }
 			.filter(p => utils.findObjWithOneOfKeys(p, ['a'])).size()).toEqual(1);
 		});
 	});
+
+	describe('getRequire', () => {
+		it('should create a require statement', () => {
+			const require = utils.getRequire(j, 'filesys', 'fs');
+			expect(require).toMatchSnapshot();
+		});
+	});
 });