Fix TypeError: Cannot read property 'text' of undefined error.

[heavenshell]

TL;DR

Fix TypeError: Cannot read property 'text' of undefined error.

Detail

When importing relative path like following, typescript-simple raised exception.

• https://github.com/9renpoto/ts/blob/fix/filename/test/index.test.ts#L1

  TypeError: Cannot read property 'text' of undefined
      at TypeScriptSimple.toJavaScript (/tmp/src/ts/node_modules/typescript-simple/index.js:162:28)
      at TypeScriptSimple.compile (/tmp/src/ts/node_modules/typescript-simple/index.js:71:25)
      at Object.loadTypeScript (/tmp/src/ts/node_modules/espower-typescript/index.js:19:22)
  

Actual behavior

service.getEmitOutput(fileName) returns relative path and absorute path.

eg.
#: see also power-assert-js/espower-typescript#16 (comment)

/tmp/src/sample/test.ts

import {test} from './dep'

test()

/tmp/src/sample/dep.ts

export function test() {
    console.log('from ts')
}

• service.getEmitOutput('test.ts') returns relative path
  • test.js
• service.getEmitOutput('dep.ts') returns absolute path
  • /tmp/src/sample/deps.js

So, this line compared relative path(outputFileName) with absolute path(file.name) and returns undefined.

Solution

Convert relative path to absolute path.

PoC

Please see our sample project.

1. git clone https://github.com/9renpoto/ts.git
2. git checkout fix/filename
3. npm install
4. npm test

note

package.json's typescript-simple pointed to our PR branch.

If you delete this line, exception would raised.

Misc

This PR would also solve following issue.

• [bug] error if test file import other typescript power-assert-js/espower-typescript#16

Please check and include this PR.

Thank you.

[teppeis]

@heavenshell Thanks!
Your patch didn't fix this issue in Windows, but I released it with "known issues" in v5.0.3.

[heavenshell]

@teppeis Thank you for solving this issue.