GeezerVison

Description

1. added pre-replace unnecessary part of the line instead of slicing it after splitting
2. fixed removing last round bracket during throwing out eval information
3. removed regexp group ':(\d+):(\d+)' because we should get any location in brackets
4. removed splitting 'sanitizedLine' and reorganized using 'sanitizedLine' and 'location'
5. added a new option for node error test when the function name isn't there

Motivation and Context

I have a problem with the Node.js product. In some ways error looks like this:

Error
    at /var/app/scratch/my project/index.js:2:9
    at Object.<anonymous> (/var/app/scratch/my project/index.js:2:9)
    at Module._compile (internal/modules/cjs/loader.js:774:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:785:10)
    at Module.load (internal/modules/cjs/loader.js:641:32)
    at Function.Module._load (internal/modules/cjs/loader.js:556:12)
    at Function.Module.runMain (internal/modules/cjs/loader.js:837:10)
    at internal/main/run_main_module.js:17:11

First-line doesn't have a function name and round brackets and it loses full file name after splitting.

How Has This Been Tested?

Added new option for test 'should handle spaces in Node.js stacks'

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• node_modules/.bin/jscs -c .jscsrc error-stack-parser.js passes without errors
• npm test passes without errors
• I have read the contribution guidelines
• I have updated the documentation accordingly
• I have added tests to cover my changes