Add contents_formatted to InMemoryTerm

[dfaust]

This method is almost identical to contents, but it includes VT-100 codes.

[djc]

Why do you need this?

[dfaust]

For testing. This allows seeing the colored output directly in the tests.

[djc]

Sorry for being so slow to follow up. Adding this API seems reasonable, though I'd prefer to add it in a way that doesn't duplicate as much code. How about we move most of the logic into a new InMemoryTermState::contents() method which can take a formatted: bool argument?

[dfaust]

I just pushed an updated version, which adds a reset code at the end of each line.
This is necessary because rows_formatted returns each line in an "undefined state".

Yes, there is some code duplication, the problem is, that contents returns a String, while contents_formatted returns a Vec. And this starts basically with the first call, since rows returns a String and rows_formatted returns a Vec.

One way would be for the contents method to turn the strings into vecs, process them and at the end turn the result vec back into a string.

[djc]

Okay, thanks!