Normalize paths when looking up items on outputCache (#1038)

* Normalize paths when looking up items on outputCache * Tweak tests to be absolutely sure sourcemaps are working

Author: cspotcode

src/index.spec.ts

@@ -189,7 +189,7 @@ describe('ts-node', function () {
  }
 
  expect(err.message).to.contain([
- `${join(__dirname, '../tests/throw.ts')}:3`,
+ `${join(__dirname, '../tests/throw.ts')}:100`,
  ' bar () { throw new Error(\'this is a demo\') }',
  ' ^',
  'Error: this is a demo'
@@ -206,7 +206,7 @@ describe('ts-node', function () {
  }
 
  expect(err.message).to.contain([
- `${join(__dirname, '../tests/throw.ts')}:3`,
+ `${join(__dirname, '../tests/throw.ts')}:100`,
  ' bar () { throw new Error(\'this is a demo\') }',
  ' ^'
  ].join('\n'))
@@ -611,7 +611,7 @@ describe('ts-node', function () {
  } catch (error) {
  expect(error.stack).to.contain([
  'Error: this is a demo',
- ` at Foo.bar (${join(__dirname, '../tests/throw.ts')}:3:18)`
+ ` at Foo.bar (${join(__dirname, '../tests/throw.ts')}:100:18)`
  ].join('\n'))
 
  done()

src/index.ts

@@ -420,7 +420,7 @@ export function create (rawOptions: CreateOptions = {}): Register {
  sourceMapSupport.install({
  environment: 'node',
  retrieveFile (path: string) {
- return outputCache.get(path)?.content || ''
+ return outputCache.get(normalizeSlashes(path))?.content || ''
  }
  })
 

tests/throw.ts

@@ -1,4 +1,101 @@
+// 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') }
 }