fix test coverage

Author: cspotcode

src/index.spec.ts

@@ -724,7 +724,7 @@ describe('ts-node', function () {
   describe('esm', () => {
     this.slow(1000)
 
-    const cmd = `node --loader ts-node/esm.mjs`
+    const cmd = `node --loader ts-node/esm`
 
     if (semver.gte(process.version, '13.0.0')) {
       it('should compile and execute as ESM', (done) => {
@@ -735,6 +735,19 @@ describe('ts-node', function () {
           return done()
         })
       })
+      it('should use source maps', function (done) {
+        exec(`${cmd} throw.ts`, { cwd: join(__dirname, '../tests/esm') }, function (err, stdout) {
+          expect(err).not.to.equal(null)
+          expect(err!.message).to.contain([
+            `${join(__dirname, '../tests/esm/throw.ts')}:100`,
+            '  bar () { throw new Error(\'this is a demo\') }',
+            '                 ^',
+            'Error: this is a demo'
+          ].join('\n'))
+
+          return done()
+        })
+      })
       it('supports --experimental-specifier-resolution=node', (done) => {
         exec(`${cmd} --experimental-specifier-resolution=node index.ts`, { cwd: join(__dirname, '../tests/esm-node-resolver') }, function (err, stdout) {
           expect(err).to.equal(null)

src/index.ts

@@ -451,7 +451,7 @@ export function create (rawOptions: CreateOptions = {}): Register {
       // If it's a file URL, convert to local path
       // Note: fileURLToPath does not exist on early node v10
       // I could not find a way to handle non-URLs except to swallow an error
-      if (fileURLToPath && path.startsWith('file://')) {
+      if (options.experimentalEsmLoader && path.startsWith('file://')) {
         try {
           path = fileURLToPath(path)
         } catch (e) {/* swallow error */}

tests/esm/throw.ts

@@ -0,0 +1,102 @@
+// intentional whitespace to prove that sourcemaps are working.  Throw should happen on line 100.
+// 100 lines is meant to be far more space than the helper functions would take.
+class Foo {
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+  constructor () { this.bar() }
+  bar () { throw new Error('this is a demo') }
+}
+new Foo()