Fixed tests to run on Windows

[kostya-misura]

No description provided.

[samccone]

👍

[samccone]

I also thing we will want to add a unit test to this to ensure the fix works and we do not regress.

[kostya-misura]

it might be wide to use something like https://www.npmjs.org/package/normalize-newline To reduce to complexity of this line.

Do you think we really need an additional dependency just for call to "str.replace(/\r\n/g, '\n');"?

I also thing we will want to add a unit test to this to ensure the fix works and we do not regress.

I'm gladly do this, but I want to note that this fix does not related to functionality. This is fix for tests themselves. Of couse method "_normalizeString" can be considered as "helper" that can be moved somewhere and then tested, but in this case I would appreciate if you can suggest how to organize things out. (Where to move it, where to place the tests, and so forth). I haven't found anything like that in the project, probably I've missed it.

[jbpros]

Thanks @kostya-misura!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.