-
-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sourcemaps not included in karma configuration #812
Comments
I did a little research on this. A generated project using webpack/karma/babel already generates sourcemaps. But by the time the So setting the coverage parameter to false ( Regarding generated projects using webpack/karma/typescript, you have to make some configuration changes to get sourcemaps, but in the end the istanbul coverage plugin still does something to them so they're not usable. So in addition to setting the webpack coverage parameter to false in
Here's my working typescript repo just in case I missed anything in the above. I spent a little time looking into the problem with |
So it looks like istanbul making sourcemaps unusable is a known issue according to gotwarlost/istanbul#212. From the looks of it a few projects have split their test scripts to either run code coverage with istanbul or run tests without it to keep working sourcemaps. |
I'm submitting a bug report
0.32.0
Please tell us about your environment:
Operating System:
Windows 10
Node Version:
6.11.3
NPM Version:
3.10.10
Browser:
all
Language:
all
Loader/bundler:
Webpack
Current behavior:
Creating a new app using Webpack and Karma does not include sourcemaps while debugging karma tests
To use sourcemaps while debugging karma tests
It appears there was an attempt to include this feature according to the issue here: #656.
The related PR updated a karma.conf.js file located here https://github.com/aurelia/cli/blob/master/lib/resources/content/karma.conf.js to use sourcemaps.
But the cli is still creating new apps using the karma configs located here
The text was updated successfully, but these errors were encountered: