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

Support expectations in snapshot files #1513

Merged
merged 1 commit into from
Nov 10, 2024
Merged

Conversation

plypaul
Copy link
Contributor

@plypaul plypaul commented Nov 7, 2024

To make snapshot files easier to review, this PR adds an expectation_description field to the headers. This way, reviewers can read the description to check that the snapshot contains the expected result. The field was named expectation_description instead of expectation to make it more clear that it's supposed to be human-readable text vs. a specific value in the snapshot.

@cla-bot cla-bot bot added the cla:yes label Nov 7, 2024
@plypaul plypaul force-pushed the p--misc-improvements--03 branch from 9091615 to 33132e0 Compare November 7, 2024 20:42
@plypaul plypaul marked this pull request as ready for review November 8, 2024 03:20
@plypaul plypaul force-pushed the p--misc-improvements--02 branch from 63c86ce to eb18b9b Compare November 8, 2024 18:25
@plypaul plypaul force-pushed the p--misc-improvements--03 branch from 33132e0 to dbb45c6 Compare November 8, 2024 18:25
@plypaul plypaul force-pushed the p--misc-improvements--02 branch from eb18b9b to 7e4e2eb Compare November 9, 2024 23:57
Base automatically changed from p--misc-improvements--02 to main November 10, 2024 00:06
@plypaul plypaul force-pushed the p--misc-improvements--03 branch from dbb45c6 to 4b087c4 Compare November 10, 2024 00:11
@plypaul plypaul merged commit 5b5d0fe into main Nov 10, 2024
15 checks passed
@plypaul plypaul deleted the p--misc-improvements--03 branch November 10, 2024 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants