Skip to content

[Java] Integration tests #233

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

Merged
merged 9 commits into from
May 13, 2025
Merged

[Java] Integration tests #233

merged 9 commits into from
May 13, 2025

Conversation

omacranger
Copy link
Member

@omacranger omacranger commented May 9, 2025

Description

Adds integration tests to cover the various features we expect the visual SDK to either handle or pass to the backend to handle. Uses the returned visual check snapshot ID to re-query the snapshot and assert that the result matches on following runs (including all properties, ignore region positions, etc). This also exposes a few pieces of functionality to the client to allow others to also make use of some of the features.

  • sauceVisualCheck now returns the ID of the snapshot taken, so it can be queried and asserted on if needed
  • visualApi.execute is now exposed, allowing GraphQL calls through our client so consumers can make additional queries without needing spin up the same configuration

Types of Changes

  • New feature (non-breaking change which adds functionality)
  • Configuration change
  • Refactor/improvements

@perf2711
Copy link
Contributor

Can you also update the PR's description?

@omacranger omacranger force-pushed the feat/java-integration-tests branch from 3f07128 to ac78396 Compare May 13, 2025 13:31
@omacranger omacranger merged commit 3636a30 into main May 13, 2025
4 checks passed
@omacranger omacranger deleted the feat/java-integration-tests branch May 13, 2025 14:03
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