search

gdborton / webpack-parallel-uglify-plugin

A faster uglifyjs plugin.
466 stars 34 forks source link

Fix source map test #34

Closed gdborton closed 7 years ago

gdborton commented 7 years ago

The tests for source map support were failing silently, this corrects that.

[tests] Add tests to check that the compilation object is updated with errors. I noticed that the sourcemap test was passing when it should not have been. That test needs to be corrected, but this additional test would have surfaced the build error easier.

[tests] Correct failing sourceMap tests. These were previously failing due to the provided SourceMap being a string and not an object as webpack would pass. This update also introduces acorn and escodegen, which are used together to create an input sourceMap rather than creating one by hand.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.8%) to 95.868% when pulling ecd5b8353355a6fa1f343da4f251ef2a9be6db78 on fix-source-map-test into 4b584c452731f501a2fe1b60f84000edfa84e94c on master.