Skip to content
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

Move tests to snapshot-tests and enhance them #141

Merged

Conversation

jt-traub
Copy link
Contributor

Snapshot tests kind of suck in general and are more fragile, but they are all we really have right now for testing before the unit test changes and they can still serve some purpose for doing end-to-end testing.

This moves the directory to be better named, updates the github workflow to use the new directory/driver script and enhances that script to compare the output to 'expected' output each step of the way.

Snapshot tests kind of suck in general and are more fragile, but
they are all we really have right now for testing before the unit
test changes and they can still serve some purpose for doing
end-to-end testing.

This moves the directory to be better named, updates the github
workflow to use the new directory/driver script and enhances that
script to compare the output to 'expected' output each step of
the way.
@artyomtrityak artyomtrityak merged commit 4ae2adc into Atlantis-PBEM:neworigins-v6 Oct 10, 2023
4 checks passed
@jt-traub jt-traub deleted the jt-enhance-snapshot-tests branch October 10, 2023 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants