Change testcase to clarify that it covers the peewee behavior of on_conflict_ignore

This commit is contained in:
Gdub 2026-09-01 11:19:26 -07:00
parent b535fb0b27
commit e05e5c0e34

View File

@ -497,8 +497,8 @@ class TestHttpReview(BaseTestHttp):
)
assert user_review.has_been_reviewed == True
def test_post_reviews_viewed_concurrent_duplicate_does_not_raise(self):
"""Regression: concurrent requests marking the same review must not 500.
def test_reviews_concurrent_insert_peewee_ignore(self):
"""Validates that on_conflict_ignore() silently skips a duplicate insert
Two requests can both SELECT and find no existing status, then both try
to INSERT, hitting the unique (user_id, review_segment) constraint.