Use inlineSourceMaps for better test coverage checks

[campionfellin]

General testing improvement... we can now see more accurate coverage numbers and missing lines.

Previously:

Now

(note this is just one test). We can see now it covers the .ts files instead of .js files.

Signed-off-by: campionfellin campionfellin@gmail.com

• npm run test succeeds.
• npm run lint succeeds.
• Appropriate changes to README are included in PR.

[grant]

But won't this add source maps to the version published to npm? We probably want this tsconfig option just to be applied when testing.

[campionfellin]

I think that should be fine. What would the problem be in publishing source maps to npm?

[grant]

I guess it's fine. I thought it would make the build step take longer or add extra bloat.

After a brief search, it seems like it should be fine.
rails/webpacker#769 (comment)

[campionfellin]

Sounds great!

[grant]

Also, thanks for the PRs! I sometimes forget to say thanks.

[campionfellin]

Np! It is a good learning opportunity for me!