Support source mapping of stack traces in the Debug Console #6

[nojvek]

Will add some tests tomorrow. Still figuring out how tests work with sourcemap mocks.

Let me know if the code section looks good by you.

@auchenberg @roblourens

[roblourens]

Thanks, this looks great! Tests are always welcome.

[auchenberg]

TSLint failes due to missing radex param.

[nojvek]

Just installed tslint plugin for vscode. Just realized that that it now has tslint.autofix like eslint.

I press save and everything fixes itself. So awesome 😎 🎉

[nojvek]

@roblourens I am trying to get the unit test for exceptions to work.

Firing mockEventEmitter.emit('Runtime.exceptionThrown', ... but that never ends up calling onExceptionThrown even thought the handler is run this.chrome.Runtime.onExceptionThrown(params => this.onExceptionThrown(params));. Weird, am I doing something obviously wrong that I can't see?

My test strategy is to fire a bunch of Runtime.exceptionThrown events, mock the sourcemapper and pathmapper and check the final exception string sent to client is source mapped.

[roblourens]

Looks like there's a typo in my mock event handling code - https://github.com/Microsoft/vscode-chrome-debug-core/blob/master/test/mocks/debugProtocolMocks.ts#L51 - should be Runtime.exceptionThrown not onExceptionThrown.

[nojvek]

@roblourens Thanks for the pointer.

Making mocks work properly was a mission but I think I have it all working now.

[nojvek]

Tested on dummy and webpack projects. I'm lovin it

[roblourens]

Awesome, thanks for the tests