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

Sauce Visual Espresso docs #2988

Merged
merged 16 commits into from
Dec 10, 2024
Merged

Sauce Visual Espresso docs #2988

merged 16 commits into from
Dec 10, 2024

Conversation

kb-kerem
Copy link
Contributor

No description provided.

Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

1 similar comment
Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

docs/visual-testing/integrations/espresso.md Outdated Show resolved Hide resolved
docs/visual-testing/integrations/espresso.md Outdated Show resolved Hide resolved
Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

Copy link
Contributor

@konraddysput konraddysput left a comment

Choose a reason for hiding this comment

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

Can we add a description to the clip to element support?

docs/visual-testing/integrations/espresso.md Show resolved Hide resolved
docs/visual-testing/integrations/espresso.md Show resolved Hide resolved
docs/visual-testing/integrations/espresso.md Outdated Show resolved Hide resolved
docs/visual-testing/integrations/espresso.md Show resolved Hide resolved
Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

1 similar comment
Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

Copy link

github-actions bot commented Dec 6, 2024

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

1 similar comment
Copy link

github-actions bot commented Dec 6, 2024

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

@kb-kerem
Copy link
Contributor Author

kb-kerem commented Dec 6, 2024

Your review for the newly added FPS section would be much appreciated
@konraddysput @omacranger

Copy link
Member

@omacranger omacranger left a comment

Choose a reason for hiding this comment

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

There are some Vale warnings for title case in heading. May be worth updating before merging. Otherwise looks good.

```

To enhance efficiency in managing tests, it's important to provide a specific test name and suite name for each test. This practice allows Sauce Visual to effectively organize snapshots into coherent groups. As a result, it simplifies the review process, saving time and effort in navigating through test results and understanding the context of each snapshot.

Moreover, our Espresso binding offers an automated solution to this process. By integrating the following code snippet into your tests, the Java Binding can automatically assign appropriate test names and suite names, streamlining your testing workflow.
Moreover, our Espresso binding offers an automated solution to this process. By integrating the following code snippet into your tests, the Sauce Visual can automatically assign appropriate test names and suite names, streamlining your testing workflow.
Copy link
Member

Choose a reason for hiding this comment

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

the Sauce Visual

Would remove 'the' here

Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

1 similar comment
Copy link

Deploy preview ready for 2988!
https://docs.dev.saucelabs.net/pr-preview/pr-2988

@kb-kerem kb-kerem merged commit 5b27082 into main Dec 10, 2024
3 checks passed
@kb-kerem kb-kerem deleted the sauce-visual-espresso branch December 10, 2024 19:20
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.

4 participants