-
Notifications
You must be signed in to change notification settings - Fork 3
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
Test for actual output, not just exit code #35
Comments
How do we test a solution? Pushing and checking? |
Maybe using https://github.com/nektos/act? |
Not sure what would be a good way to do this, maybe @sverhoeven can help? |
You could capture the stdout and compare it with a expected output using https://man7.org/linux/man-pages/man1/comm.1.html |
We are having issues capturing the stdout. |
The |
Might need a |
Testing @nlescgt |
The tests in https://github.com/citation-file-format/cffconvert-github-action/blob/a73af040d6aeb196ca0f696e6fdcc72663c5a96b/.github/workflows/selftest.yml test for exit code. It would be better if we can test for their actual output.
The text was updated successfully, but these errors were encountered: