Add Firefox compatibility to stack trace test

lib/moduleEnv.js

@@ -44,7 +44,7 @@ var moduleWrapper0 = Module.wrapper[0],
     // Test the regular expresssion at https://regex101.com/r/dvnZPv/2 and also check out testLib/constModule.js.
     matchConst = /(^|\s|\}|;)const(\/\*|\s|{)/gm,
     // Required for importing modules with shebang declarations, since NodeJS 12.16.0
-    shebang = /^(#!).+/,
+    shebang = /^#!.+/,
     nodeRequire,
     currentModule;
 

testLib/sharedTestCases.js

@@ -284,7 +284,18 @@ module.exports = function () {
         try {
             throwError();
         } catch (err) {
-            expect(err.stack.split("\n")[1]).to.match(/:6:26/);
+
+            // Firefox implements a different error-stack format,
+            // but does offer line and column numbers on errors: we use
+            // those instead.
+            if (err.lineNumber !== undefined && err.columnNumber !== undefined) {
+                expect(err.lineNumber).to.equal(6)
+                expect(err.columnNumber).to.equal(26)
+            }
+            // This is for the V8 stack trace format (Node, Chrome)
+            else {
+                expect(err.stack.split("\n")[1]).to.match(/:6:26/);
+            }
         }
     });
 
@@ -409,8 +420,10 @@ module.exports = function () {
     });
 
     it("should be possible to rewire shebang modules", function () {
-        expect(function () {
-            rewire("./shebangModule");
-        }).to.not.throwError();
+        var shebangModule = rewire("./shebangModule");
+        var shebangs = shebangModule.__get__("shebangs");
+
+        expect(typeof shebangs).to.be("function");
+        expect(shebangModule.shebangs()).to.be(true);
     });
 };

testLib/shebangModule.js

@@ -4,4 +4,4 @@ function shebangs() {
     return true;
 }
 
-module.exports.shebangs = shebangs;
+exports.shebangs = shebangs;