As in my other PR for halo I've added pretty much the same test for halo notebook. _get_test_output has been refactored to take a no_ansi arg, which defaults to True. We then use this in the new test to keep the ANSI escape characters and compare them to those supplied by colorama.
I also cleared up an un-needed import in the other test file.
Sorry it took me a while to get to!
Checklist
[x] Your branch is up-to-date with the base branch
[x] You've included at least one test if this is a new feature
Description of new feature, or changes
As in my other PR for halo I've added pretty much the same test for halo notebook.
_get_test_output
has been refactored to take ano_ansi
arg, which defaults toTrue
. We then use this in the new test to keep the ANSI escape characters and compare them to those supplied by colorama.I also cleared up an un-needed import in the other test file.
Sorry it took me a while to get to!
Checklist
Related Issues and Discussions
81
People to notify
@manrajgrover