remove unneeded code from rewire for coderoad usage

Author: ShMcK

lib/__get__.js

@@ -11,10 +11,10 @@
  */
 function __get__(varName) {
     try {
-        if (varName && typeof varName === "string") {
+        if (varName && typeof varName === 'string') {
             return eval(varName);
         } else {
-            throw new TypeError("__get__ expects a non-empty string");
+            throw new TypeError('__get__ expects a non-empty string');
         }
     } catch(e) {
         // does not exist

lib/detectImports.js

@@ -0,0 +1,9 @@
+const import = /\bimport\s+(?:.+\s+from\s+)?[\'"]([^"\']+)["\']/;
+
+// detects imports
+// adds imports to global scope of rewired object
+function detectImports(file) {
+
+}
+
+module.exports = detectImports;

lib/detectStrictMode.js

@@ -1,6 +1,6 @@
-var multiLineComment = /^\s*\/\*.*?\*\//;
-var singleLineComment = /^\s*\/\/.*?[\r\n]/;
-var strictMode = /^\s*(?:"use strict"|'use strict')[ \t]*(?:[\r\n]|;)/;
+const multiLineComment = /^\s*\/\*.*?\*\//;
+const singleLineComment = /^\s*\/\/.*?[\r\n]/;
+const strictMode = /^\s*(?:["']use strict["'])[ \t]*(?:[\r\n]|;)/;
 
 /**
  * Returns true if the source code is intended to run in strict mode. Does not detect
@@ -13,15 +13,17 @@ function detectStrictMode(src) {
     var singleLine;
     var multiLine;
 
-    while ((singleLine = singleLineComment.test(src)) || (multiLine = multiLineComment.test(src))) {
+    while (
+      (singleLine = singleLineComment.test(src))
+      || (multiLine = multiLineComment.test(src))
+    ) {
         if (singleLine) {
-            src = src.replace(singleLineComment, "");
+            src = src.replace(singleLineComment, '');
         }
         if (multiLine) {
-            src = src.replace(multiLineComment, "");
+            src = src.replace(multiLineComment, '');
         }
     }
-
     return strictMode.test(src);
 }
 

lib/fileExists.js

@@ -0,0 +1,16 @@
+var fs = require('fs');
+
+function fileExists(path) {
+  try {
+      fs.accessSync(path, fs.F_OK);
+  }
+  catch (e) {
+    if (e) {
+      console.log(e);
+    }
+    return false;
+  }
+  return true;
+}
+
+module.exports = fileExists;

lib/getDefinePropertySrc.js

@@ -1,27 +1,20 @@
-"use strict";
+'use strict';
 
-var __get__ = require("./__get__.js");
+const __get__ = require("./__get__.js");
 
-var srcs = {
+const srcs = {
     "__get__": __get__.toString(),
 };
 
 function getDefinePropertySrc() {
-    var src = "if (typeof(module.exports) === 'function' || \n" +
-            "(typeof(module.exports) === 'object' && module.exports !== null && Object.isExtensible(module.exports))) {\n";
-
-    src += Object.keys(srcs).reduce(function forEachSrc(preValue, value) {
-        return preValue += "Object.defineProperty(module.exports, '" +
-            value +
-            "', {enumerable: false, value: " +
-            srcs[value] +
-            ", "+
-            "writable: true}); ";
-    }, "");
-
-    src += "\n}";
-
-    return src;
+  return `
+if (typeof(module.exports) === 'function' ||
+(typeof(module.exports) === 'object' && module.exports !== null && Object.isExtensible(module.exports))) {
+  ${Object.keys(srcs).reduce((preValue, value) => {
+    return preValue += `Object.defineProperty(module.exports, '${value}', {enumerable: false, value: ${srcs[value]}, writable: true});`;
+    }, '')
+  }
+}`;
 }
 
 module.exports = getDefinePropertySrc;

lib/getImportGlobalsSrc.js

@@ -10,16 +10,16 @@
 function getImportGlobalsSrc(ignore) {
     var key,
         value,
-        src = "",
-        globalObj = typeof global === "undefined"? window: global;
+        src = '',
+        globalObj = typeof global === 'undefined' ? window : global;
 
     ignore = ignore || [];
     // global itself can't be overridden because it's the only reference to our real global objects
-    ignore.push("global");
+    ignore.push('global');
     // ignore 'module', 'exports' and 'require' on the global scope, because otherwise our code would
     // shadow the module-internal variables
     // @see https://github.com/jhnns/rewire-webpack/pull/6
-    ignore.push("module", "exports", "require");
+    ignore.push('module', 'exports', 'require');
 
     for (key in globalObj) { /* jshint forin: false */
         if (ignore.indexOf(key) !== -1) {
@@ -29,8 +29,8 @@ function getImportGlobalsSrc(ignore) {
 
         // key may be an invalid variable name (e.g. 'a-b')
         try {
-          eval("var " + key + ";");
-          src += "var " + key + " = global." + key + "; ";
+          eval(`var ${key};`);
+          src += `var ${key} = global.${key};`;
         } catch(e) {}
     }
 

lib/index.js

@@ -1,21 +1,5 @@
-var rewireModule = require("./rewire.js");
-var fs = require('fs');
-
-function fileExists(path, silent) {
-    if (silent === void 0) { silent = true; }
-    try {
-        fs.accessSync(path, fs.F_OK);
-    }
-    catch (e) {
-        if (e) {
-            if (!silent) {
-                console.log(e);
-            }
-            return false;
-        }
-    }
-    return true;
-}
+var rewireModule = require('./rewire.js');
+var fileExists = require('./fileExists');
 
 /**
  * Adds a special setter and getter to the module located at filename. After the module has been rewired, you can
@@ -28,7 +12,7 @@ function rewire(filename) {
   // is not a package path
   if (!filename.match(/^[a-zA-Z]/)) {
     // if the file doesn't exist yet,
-    // set to get undefined
+    // create a __get__ mock to prevent tests breaking
     if (!fileExists(filename)) {
       return {
         __get__: () => {}
@@ -41,4 +25,5 @@ function rewire(filename) {
 
 module.exports = rewire;
 
-delete require.cache[__filename];   // deleting self from module cache so the parent module is always up to date
+delete require.cache[__filename];
+// deleting self from module cache so the parent module is always up to date

lib/moduleEnv.js

@@ -1,8 +1,7 @@
-"use strict";
+'use strict';
 
-var Module = require("module"),
-    fs = require("fs"),
-    coffee;
+var Module = require('module'),
+    fs = require('fs');
 
     // caching original wrapper
 var moduleWrapper0 = Module.wrapper[0],
@@ -16,7 +15,6 @@ function load(targetModule) {
     targetModule.require = requireProxy;
     currentModule = targetModule;
 
-    registerExtensions();
     targetModule.load(targetModule.id);
 
     // This is only necessary if nothing has been required within the module
@@ -26,7 +24,6 @@ function load(targetModule) {
 function reset() {
     Module.wrapper[0] = moduleWrapper0;
     Module.wrapper[1] = moduleWrapper1;
-    restoreExtensions();
 }
 
 function inject(prelude, appendix) {
@@ -46,49 +43,5 @@ function requireProxy(path) {
     return nodeRequire.call(currentModule, path);  // node's require only works when "this" points to the module
 }
 
-function registerExtensions() {
-    var originalCoffeeExtension = require.extensions[".coffee"];
-
-    if (originalCoffeeExtension) {
-        originalExtensions.coffee = originalCoffeeExtension;
-    }
-    require.extensions[".coffee"] = coffeeExtension;
-}
-
-function restoreExtensions() {
-    if ("coffee" in originalExtensions) {
-        require.extensions[".coffee"] = originalExtensions.coffee;
-    }
-}
-
-function coffeeExtension(module, filename) {
-    var content = stripBOM(fs.readFileSync(filename, "utf8"));
-
-    content = coffee.compile(content, {
-        filename: filename,
-        bare: true
-    });
-    module._compile(content, filename);
-}
-
-/**
- * @see https://github.com/joyent/node/blob/master/lib/module.js
- */
-function stripBOM(content) {
-    // Remove byte order marker. This catches EF BB BF (the UTF-8 BOM)
-    // because the buffer-to-string conversion in `fs.readFileSync()`
-    // translates it to FEFF, the UTF-16 BOM.
-    if (content.charCodeAt(0) === 0xFEFF) {
-        content = content.slice(1);
-    }
-    return content;
-}
-
-try {
-    coffee = require("coffee-script");
-} catch (err) {
-    // We are not able to provide coffee-script support, but that's ok as long as the user doesn't want it.
-}
-
 exports.load = load;
 exports.inject = inject;

lib/rewire.js

@@ -1,9 +1,9 @@
-var Module = require("module"),
-    fs = require("fs"),
-    getImportGlobalsSrc = require("./getImportGlobalsSrc.js"),
-    getDefinePropertySrc = require("./getDefinePropertySrc.js"),
-    detectStrictMode = require("./detectStrictMode.js"),
-    moduleEnv = require("./moduleEnv.js");
+var Module = require('module'),
+    fs = require('fs'),
+    getImportGlobalsSrc = require('./getImportGlobalsSrc.js'),
+    getDefinePropertySrc = require('./getDefinePropertySrc.js'),
+    detectStrictMode = require('./detectStrictMode.js'),
+    moduleEnv = require('./moduleEnv.js');
 
 /**
  * Does actual rewiring the module. For further documentation @see index.js
@@ -15,8 +15,8 @@ function internalRewire(parentModulePath, targetPath) {
         src;
 
     // Checking params
-    if (typeof targetPath !== "string") {
-        throw new TypeError("Filename must be a string");
+    if (typeof targetPath !== 'string') {
+        throw new TypeError('Filename must be a string');
     }
 
     // Resolve full filename relative to the parent module
@@ -38,19 +38,19 @@ function internalRewire(parentModulePath, targetPath) {
 
     // Wrap module src inside IIFE so that function declarations do not clash with global variables
     // @see https://github.com/jhnns/rewire/issues/56
-    prelude += "(function () { ";
+    prelude += '(function () { ';
 
     // We append our special setter and getter.
-    appendix = "\n" + getDefinePropertySrc();
+    appendix = '\n' + getDefinePropertySrc();
 
     // End of IIFE
-    appendix += "})();";
+    appendix += '})();';
 
     // Check if the module uses the strict mode.
     // If so we must ensure that "use strict"; stays at the beginning of the module.
-    src = fs.readFileSync(targetPath, "utf8");
+    src = fs.readFileSync(targetPath, 'utf8');
     if (detectStrictMode(src) === true) {
-        prelude = ' "use strict"; ' + prelude;
+        prelude = ` "use strict"; ${prelude}`;
     }
 
     moduleEnv.inject(prelude, appendix);