Skip to content

Conversation

cmcculloh-kr
Copy link
Contributor

Instead of continually appending new suffixes, replace the previous testId suffix with a new one.

Uses a . instead of a space so that we aren't introducing spaces into filenames where the user may not want them.

Instead of continually appending new suffixes, replace the previous testId suffix with a new one.

Uses a `.` instead of a space so that we aren't introducing spaces into filenames where the user may not want them.
@NimaSoroush
Copy link
Owner

Hi @CormacMcCarthy,
Thanks for putting this PR. I believe this will break some unit-tests and some integration-tests. Did you run them locally?

@@ -204,14 +204,52 @@ describe('Differencify', () => {
.goto('http://example.com/')
.waitFor(1000)
.screenshot()
.toMatchSnapshot()
.toMatchSnapshot((resultDetail) => {
Copy link
Owner

Choose a reason for hiding this comment

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

Probably better to write a new test instead of updating the existing one. These tests are good examples for differencify usage

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