Fix #1200 show errors on source files with paths lengths over 255 chars

[angelorpa]

Changes

Add functions to work with lines of the events file and get the real path of the source file that was compiled taking care of files with paths lengths more than 255 characters to fix #1200

Checklist

• have tested my change
• updated relevant documentation
• Remove any/all console.logs I added
• eslint is not complaining
• have added myself to the contributors' list in CONTRIBUTING.md
• for feature PRs: PR only includes one feature enhancement.

[worksofliam]

Works as expected - thanks for a great change.

Before I merge, please see my comment about formatting and then we should be good to go.

Liam

[worksofliam]

Functionally works good and looks like the code cleanup is good too!

@chrjorgensen @sebjulliand This is a pretty impact change. Any chance you two can have a play with compiling code to ensure errors continue to show as they did before?

I plan on writing some tests for this code in another PR.

[sebjulliand]

I played around with it and everything worked fine. I compiled from a local project, a remote PF and a remote streamfile (with a long name) and errors were correctly reported.

Great job; code looks good too! 😃

[worksofliam]

@angelorpa Before we can merge this, we need you to pull in the latest changes from master and add a test for this change into this file: https://github.com/halcyon-tech/vscode-ibmi/blob/master/src/testing/ileErrors.ts

[angelorpa]

@angelorpa Before we can merge this, we need you to pull in the latest changes from master and add a test for this change into this file: https://github.com/halcyon-tech/vscode-ibmi/blob/master/src/testing/ileErrors.ts

Hi, I did pulled the latest changes and add two cases for handler errors. Please check it, and let me know if something else is needed.

[worksofliam]

Tests are excellent! Thank you! This is a great addition.