-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support automated UX testing #847
base: main
Are you sure you want to change the base?
Conversation
CI tends to run tests in unpredictable PWD.
What is your plan for the |
I think Reedline can just pin to a specific commit, like this PR is currently doing. PTY-related tests can be flaky, so I think that's the best way to ensure things don't break randomly. I made |
It's useful, but probably doesn't belong to this PR.
…to automate-ux-testing
This PR adds support for automated UX testing using a terminal emulator. It implements most of the checkboxes in
UX_TESTING.md
as automated tests.This is a reworked and improved version of my previous PR on automated UX tests. I've made a new crate
alacritty-test
containing the tools. The new implementation usespolling
to read from PTYs efficiently and reliably. If everything works out, we can adopt this in Nushell as well.