- added a test for the case where a comment is on the last line

jhnns · jhnns · commit f5fd56e1e22a · 2013-03-02T16:41:37.000+01:00
diff --git a/test/testModules/emptyModule.js b/test/testModules/emptyModule.js
@@ -1 +1,5 @@
 "use strict"; // run code in ES5 strict mode
+
+var someVar;
+
+// Comment on file end. Hope this won't break anything
diff --git a/test/testModules/sharedTestCases.js b/test/testModules/sharedTestCases.js
@@ -145,6 +145,12 @@ describe("rewire " + (typeof testEnv === "undefined"? "(node)": "(" + testEnv +
             }
         }
     });
+    it("should not be a problem to have a comment on file end", function () {
+        var rewired = rewire("./emptyModule.js");
+
+        rewired.__set__("someVar", "hello");
+        expect(rewired.__get__("someVar")).to.be("hello");
+    });
     it("should not influence the original require if nothing has been required within the rewired module", function () {
         rewire("./emptyModule.js"); // nothing happens here because emptyModule doesn't require anything
         expect(require("./moduleA.js").__set__).to.be(undefined); // if restoring the original node require didn't worked, the module would have a setter

Original file line number	Diff line number	Diff line change
`@@ -145,6 +145,12 @@ describe("rewire " + (typeof testEnv === "undefined"? "(node)": "(" + testEnv +`
`145`	`145`	`}`
`146`	`146`	`}`
`147`	`147`	`});`
	`148`	`+ it("should not be a problem to have a comment on file end", function () {`
	`149`	`+ var rewired = rewire("./emptyModule.js");`
	`150`	`+`
	`151`	`+ rewired.__set__("someVar", "hello");`
	`152`	`+ expect(rewired.__get__("someVar")).to.be("hello");`
	`153`	`+ });`
`148`	`154`	`it("should not influence the original require if nothing has been required within the rewired module", function () {`
`149`	`155`	`rewire("./emptyModule.js"); // nothing happens here because emptyModule doesn't require anything`
`150`	`156`	`expect(require("./moduleA.js").__set__).to.be(undefined); // if restoring the original node require didn't worked, the module would have a setter`