Snapshot test the terminal UI (!400) · Merge requests · Warren Gifford / src-cli

Warren Gifford requested to merge mrn/tui-integration-test into main Dec 01, 2020

Created by: mrnugget

I started with one thought: "I just wish I had one test to make sure that this whole thing [the terminal UI] doesn't blow up every time I make a change" What I ended with is this. It includes a tiny VT100 "emulator".

And here's the kicker: I actually like this. It's fast to run and easy to extend and it gives me confidence that the basic rendering functionality works. Of course, the hacky injections need to be cleaned up before merging. And maybe we can get it to pass on Windows.

Opinions?

(This fixes https://github.com/sourcegraph/sourcegraph/issues/16330)

Snapshot test the terminal UI

Merge request reports