search

IntelliTect / TestTools

A collection of tools for aiding in test automation
MIT License
10 stars 7 forks source link

Fixes a Regex that was overly aggressive #87

Closed thornegarvin closed 4 years ago

thornegarvin commented 4 years ago

This improves a Regex that incorrectly removed "+" characters when it wasn't necessary. This also fixes an issue where " characters would be incorrectly inserted into formatted output when the printed strings were very long.

Closes #59

Keboo commented 4 years ago

Can we add a unit test for the described failure case here?

thornegarvin commented 4 years ago

Not sure the naming convention on this test is exactly correct. I also considered adding a success case but injecting into the parser method would be pretty difficult as-is. If that case is important I can add a test but it would require refactoring.