Skip to content

Commit e93ef6a

Browse files
authored
bug(flakey test): fix assertion (#100643)
found through https://sentry.sentry.io/issues/6856563923/ and https://sentry.sentry.io/issues/6864905942/ assertion is definitely a typo and would pass in most situations, making this look flakey. Fixed the test so it is asserting on the correct value, didn't see this typo elsewhere in code.
1 parent 335bbf6 commit e93ef6a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/sentry/mail/test_adapter.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -868,7 +868,7 @@ def test_notify_with_suspect_commits(self) -> None:
868868

869869
assert "Suspect Commits" in msg.body
870870
assert self.user.email in msg.body
871-
assert commit.key[-7] in msg.body
871+
assert commit.key[:7] in msg.body
872872

873873
def test_notify_with_replay_id(self) -> None:
874874
project = self.project

0 commit comments

Comments
 (0)