getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
36.96k stars 3.97k forks source link

fix(replays): normalize replay_id assertion in test #69281

Closed JoshFerge closed 1 week ago

JoshFerge commented 1 week ago

We are changing the replay_id field to have the uuid processing like all of our other uuid fields, which strips dashes. This test caused a CI failure when snuba was updated.

This PR simply normalizes the replay_id when it checks the equality. I can revert this commit after the Snuba PR is merged. This is only an issue on the issue platform dataset, and I have high confidence that removing dashes from this field will not cause problems.