-
Notifications
You must be signed in to change notification settings - Fork 0
Improve snapshot test coverage #74
Copy link
Copy link
Open
Labels
area: clicrates/tryke CLI surfacecrates/tryke CLI surfacegood first issueGood for newcomersGood for newcomershelp wantedContributions especially welcomeContributions especially welcomeinternalRefactor or cleanup with no user-visible behavior changeRefactor or cleanup with no user-visible behavior changerustPull requests that update rust codePull requests that update rust code
Metadata
Metadata
Assignees
Labels
area: clicrates/tryke CLI surfacecrates/tryke CLI surfacegood first issueGood for newcomersGood for newcomershelp wantedContributions especially welcomeContributions especially welcomeinternalRefactor or cleanup with no user-visible behavior changeRefactor or cleanup with no user-visible behavior changerustPull requests that update rust codePull requests that update rust code
Tryke uses insta for snapshot tests
Right now snapshot tests are pretty narrow - they fire
on_test_completeto one of the reporters and assert the outputThese should be more end-to-end tests that:
Examples here
Additionally, I think having the explicit snapshot definitions in code in string literals (like the examples above) would be nice.
Once the tests are more end-to-end, add more test coverage. Some ideas:
graphandgraph --fixtures