Update ANSI_ESCAPE_PATTERN for discussion #313

[denisbondar]

Description

Modify the ANSI_ESCAPE_PATTERN regular expression so that it honors the control characters described in #313.

Also added comments inside the regular expression to simplify future edits.

Type of change

Please delete options that are not relevant.

• 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 not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration

Checklist:

• My code follows the style guidelines of this project (no GitHub actions complaints! run make lint before
  committing!)
• I have commented my code, pydocstyle and darglint are happy, docstrings are in google docstring format, and all
  docstrings include a summary, args, returns and raises fields (even if N/A)
• I have added tests that prove my fix is effective or that my feature works, if adding "functional" tests please
  note if there are any considerations for the vrnetlab setup
• New and existing unit tests pass locally with my changes

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e02100d) 90.50% compared to head (a426179) 90.50%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #316   +/-   ##
=======================================
  Coverage   90.50%   90.50%           
=======================================
  Files          60       60           
  Lines        3487     3487           
=======================================
  Hits         3156     3156           
  Misses        331      331           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[carlmontanari]

👋 just firing up the lab stuff to run against real gear to test too, but think this looks great. I found this and stole from here for some more test cases. this did show one case (the "match clear tabs sequence in a string" one) failing, adding g to the control chars section sorted that.

will try to get the tests ran against real boxes tonight. will holler if anything breaks there, otherwise lemme know if this update/addition looks ok to you and we can get it merged! thanks again for all the help and work @denisbondar !

[carlmontanari]

update: all the tests against container lab boxes worked so, if you're happy we'll get this merged :D

[denisbondar]

Lets go! 🚀