Add test for implicit global vars

#35
jhnns · Dec 14, 2014 · 9597e46 · 9597e46
1 parent be744f2
commit 9597e46
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/test/testModules/implicitGlobal.js b/test/testModules/implicitGlobal.js
@@ -0,0 +1,2 @@
+implicitGlobal = "this is an implicit global var ..." +
+    "yes, it's bad coding style but there are still some libs out there";
diff --git a/test/testModules/sharedTestCases.js b/test/testModules/sharedTestCases.js
@@ -248,6 +248,16 @@ describe("rewire " + (typeof testEnv === "undefined"? "(node)": "(" + testEnv +
         }
     });
 
+    it("should be possible to set implicit globals", function () {
+        var implicitGlobalModule = rewire("./implicitGlobal.js");
+
+        implicitGlobalModule.__set__("implicitGlobal", true);
+        expect(implicitGlobalModule.__get__("implicitGlobal")).to.be(true);
+        // setting implicit global vars will change them globally instead of locally.
+        // that's a shortcoming of the current implementation which can't be solved easily.
+        //expect(implicitGlobal).to.be.a("string");
+    });
+
     it("should throw a TypeError if the path is not a string", function () {
         expect(function () {
             rewire(null);