Add test for implicit global vars

jhnns · jhnns · commit 9597e469871b · 2014-12-14T02:21:21.000+01:00
jhnns#35
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);

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+implicitGlobal = "this is an implicit global var ..." +`
	`2`	`+ "yes, it's bad coding style but there are still some libs out there";`