dgarlitt / karma-nyan-reporter

Nyan Cat style reporter based on the Mocha version
MIT License
169 stars 19 forks source link

Remove errors that contain matching strings #28

Closed whyboris closed 7 years ago

whyboris commented 7 years ago

Adds a user-editable array of strings which dictate what lines to omit from final error report.

For example: removeLinesContaining: ['@angular', 'zone.js'] will remove useless error stack traces like this:

webpack:///~/@angular/core/@angular/core/testing.es5.js:105:0 <- src/test.ts:16390:29
onInvoke@webpack:///~/zone.js/dist/proxy.js:76:0 <- src/test.ts:95814:47
drainMicroTaskQueue@webpack:///~/zone.js/dist/zone.js:611:0 <- src/polyfills.ts:1766:28

Good riddance!

This PR does not have appropriate tests written. If you think this enhancement to the project is worthwhile, I will write the tests and resubmit, including whatever recommendations/suggestions you have.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-15.07%) to 84.932% when pulling 3dca56a3b0744d4556af3ca8cd8a645f06a6e570 on whyboris:removelinescontaining into 68da8261defe424c4ecf7805034f2de778c06d96 on dgarlitt:master.

whyboris commented 7 years ago

I recommend closing this PR in lieu of https://github.com/dgarlitt/karma-nyan-reporter/pull/33