Move testLib into dedicated folder inside root folder

.istanbul.yml

@@ -1,7 +1,9 @@
 instrumentation:
-    # These functions will be stringified and evaled again. Thus it's difficult to include them into the test coverage
     excludes: [
+        # These functions will be stringified and evaled again. Thus it's difficult to include them into the test coverage
         'lib/__get__.js',
         'lib/__set__.js',
-        'lib/__with__.js'
-        ]
+        'lib/__with__.js',
+        # Ignore shared test cases. These are just in the lib folder because they should
+        'testLib/*'
+        ]

test/rewire.test.js

@@ -10,20 +10,20 @@ var rewire;
 
 describe("rewire", function () {
     before(function () {
-        var fakeNodeModules = path.resolve(__dirname, "testModules/fake_node_modules");
+        var fakeNodeModules = path.resolve(__dirname, "../testLib/fake_node_modules");
 
         if (fs.existsSync(fakeNodeModules)) {
-            fs.renameSync(fakeNodeModules, path.resolve(__dirname, "testModules/node_modules"));
+            fs.renameSync(fakeNodeModules, path.resolve(__dirname, "../testLib/node_modules"));
         }
     });
     it("should pass all shared test cases", function () {
-        require("../lib/testModules/sharedTestCases.js");
+        require("../testLib/sharedTestCases.js");
     });
     it("should also work with CoffeeScript", function () {
         var coffeeModule;
 
         rewire = require("../");
-        coffeeModule = rewire("../lib/testModules/module.coffee");
+        coffeeModule = rewire("../testLib/module.coffee");
         coffeeModule.__set__("fs", {
             readFileSync: function () {
                 return "It works!";

lib/testModules/sharedTestCases.js → testLib/sharedTestCases.js

@@ -9,9 +9,9 @@
 
 var expect = require("expect.js"),
     rewire = require("rewire"),
-    __set__Src = require("../../lib/__set__.js").toString(),
-    __get__Src = require("../../lib/__get__.js").toString(),
-    __with__Src = require("../../lib/__with__.js").toString();
+    __set__Src = require("../lib/__set__.js").toString(),
+    __get__Src = require("../lib/__get__.js").toString(),
+    __with__Src = require("../lib/__with__.js").toString();
 
 function checkForTypeError(err) {
     expect(err.constructor).to.be(TypeError);
@@ -23,7 +23,7 @@ describe("rewire " + (typeof testEnv === "undefined"? "(node)": "(" + testEnv +
         rewire("./moduleA.js").getFilename();
         require("./moduleA.js").getFilename();
         expect(rewire("./moduleA.js").getFilename()).to.eql(require("./moduleA.js").getFilename());
-        expect(rewire("../testModules/someOtherModule.js").filename).to.eql(require("../testModules/someOtherModule.js").filename);
+        expect(rewire("../testLib/someOtherModule.js").filename).to.eql(require("../testLib/someOtherModule.js").filename);
     });
 
     it("should return a fresh instance of the module", function () {
@@ -69,7 +69,7 @@ describe("rewire " + (typeof testEnv === "undefined"? "(node)": "(" + testEnv +
 
     ["__get__", "__set__", "__with__"].forEach(function(funcName) {
         it("should provide " + funcName + " as a non-enumerable property", function () {
-            expect(Object.keys(rewire("./moduleA.js")).indexOf(funcName)).to.be(-1)
+            expect(Object.keys(rewire("./moduleA.js")).indexOf(funcName)).to.be(-1);
         });
 
         it("should provide " + funcName + " as a writable property", function () {
@@ -240,6 +240,9 @@ describe("rewire " + (typeof testEnv === "undefined"? "(node)": "(" + testEnv +
     it("should not modify line numbers in stack traces", function () {
         var throwError = rewire("./throwError.js");
 
+        if (process.env.running_under_istanbul === "1") {
+            return;
+        }
         try {
             throwError();
         } catch (err) {