-
Notifications
You must be signed in to change notification settings - Fork 4
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
automated integration tests #4
Comments
Played around a bit, very impressive, have heard about before but didn't really understand what it was about. I'm using some expect for some basic fq CLI/REPL tests but it is a mess and very shakey. |
Yeah. I am using expect also for testing. This looks promising. |
I tried vhs a bit for testing fq but i'm having trouble making the "golden" output stable, maybe i'm doing something wrong. @tpodowd Having any success using expect to test readline completion? having trouble with that too :) Example tape that i tried that changes a bit with every run:
|
Hi @wader - I'm going to be no help and say that for completion testing, I'm not actually testing the terminal output but instead I'm constructing tests around the AutoCompleter interface Do() function which I implement and ensuring my behaviour is right at that level as that is where my complexity is. |
@tpodowd I see, no worries 👍 Yeah i do something similar but turned out the test script version of the readline works a tiny bit different so didn't catch a regression. Can post here if i figure how to do it, maybe useful for someone |
Investigate https://github.com/charmbracelet/vhs for this: https://www.youtube.com/watch?v=dtVezRX8OEM
The text was updated successfully, but these errors were encountered: