Skip to content

Improve snapshot specs error messages#64

Merged
mdesantis merged 1 commit intomasterfrom
mdesantis/improve-snapshot-specs-error-messages
Jan 24, 2020
Merged

Improve snapshot specs error messages#64
mdesantis merged 1 commit intomasterfrom
mdesantis/improve-snapshot-specs-error-messages

Conversation

@mdesantis
Copy link
Copy Markdown

@mdesantis mdesantis commented Jan 17, 2020

It lets the snapshot specs print a usable diff in case of not matching response.

BEFORE

Schermata del 2020-01-17 10-09-24

AFTER

Schermata del 2020-01-17 10-05-33

@mdesantis mdesantis added the enhancement New feature or request label Jan 17, 2020
@mdesantis mdesantis self-assigned this Jan 17, 2020
@mdesantis mdesantis removed the enhancement New feature or request label Jan 17, 2020
@mdesantis mdesantis force-pushed the mdesantis/improve-snapshot-specs-error-messages branch 2 times, most recently from 74b00d2 to 75bdedd Compare January 17, 2020 12:17
@mdesantis mdesantis force-pushed the mdesantis/improve-snapshot-specs-error-messages branch 2 times, most recently from 0296196 to b2fe18d Compare January 24, 2020 10:48
@mdesantis mdesantis force-pushed the mdesantis/improve-snapshot-specs-error-messages branch 2 times, most recently from cfebb6f to 377809b Compare January 24, 2020 11:08
AlessioRocco
AlessioRocco previously approved these changes Jan 24, 2020
Copy link
Copy Markdown
Contributor

@AlessioRocco AlessioRocco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mdesantis mdesantis force-pushed the mdesantis/improve-snapshot-specs-error-messages branch from 377809b to c2c9959 Compare January 24, 2020 11:11
AlessioRocco
AlessioRocco previously approved these changes Jan 24, 2020
It lets the snapshot specs print a usable diff in case of not matching
response.
@mdesantis mdesantis force-pushed the mdesantis/improve-snapshot-specs-error-messages branch from c2c9959 to 0d9191f Compare January 24, 2020 11:41
@mdesantis mdesantis merged commit 6f29810 into master Jan 24, 2020
@mdesantis mdesantis deleted the mdesantis/improve-snapshot-specs-error-messages branch January 24, 2020 11:50
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.

3 participants