Skip to content

Integration tests don't check screenshots #2918

Open

Description

@bitsandfoxes

The integration tests don't check screenshots. They fetch the event the test app printed and check its fields, but not its screenshot.jpg, so a broken or empty screenshot goes unnoticed. This let the Metal readback error fixed in #2917 pass CI: the iOS simulator and macOS runs logged it on every capture.

The screenshots are empty in CI on every platform anyway. The test app calls Application.Quit in the frame it captures in, so the end-of-frame screenshot never gets taken and the SDK sends a 0-byte screenshot.jpg ("Screenshot was not taken in time").

What we could do:

  • Let IntegrationTester wait for the screenshot before quitting in the message and exception tests.
  • Assert that the event's screenshot.jpg isn't empty, skipped where no screenshot is taken (batch mode, WebGL). Reading attachments needs a function in app-runner's sentry-api-client, which has none.

Activity

  1. linear-code commented on Oct 29, 2026

    @linear-code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Teststesting, CIUnityplatform label

    Projects

    • Status
      No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions