search

danvk / source-map-explorer

Analyze and debug space usage through source maps
Apache License 2.0
3.82k stars 100 forks source link

Add performance test #166

Closed nikolay-borzov closed 4 years ago

nikolay-borzov commented 4 years ago

158 has shown that some unobvious changes could reduce performance greatly.

We should add a simple performance test to track such cases.

danvk commented 4 years ago

I thought about the performance test but haven't figure out how to implement it yet. Should it be just measuring big source map processing time (outside main code) or adding measures to several places in code?

I think running source-map-explorer --json with a single large JS file (large enough to have shown the regression in the bug) would be sufficient. We could run that either as part of CI or as part of the release process.