From 28712d8acfffa9cdabb88cb610bae14913fa185d Mon Sep 17 00:00:00 2001 From: Simon Charette Date: Fri, 15 Feb 2019 01:01:25 -0500 Subject: Refs #19544 -- Ignored auto-created through additions conflicts if supported. This prevents IntegrityError caused by race conditions between missing ids retrieval and bulk insertions. --- tests/many_to_many/tests.py | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'tests') diff --git a/tests/many_to_many/tests.py b/tests/many_to_many/tests.py index 933eb23a7a..adde2ac563 100644 --- a/tests/many_to_many/tests.py +++ b/tests/many_to_many/tests.py @@ -117,6 +117,16 @@ class ManyToManyTests(TestCase): ] ) + @skipUnlessDBFeature('supports_ignore_conflicts') + def test_add_ignore_conflicts(self): + manager_cls = self.a1.publications.__class__ + # Simulate a race condition between the missing ids retrieval and + # the bulk insertion attempt. + missing_target_ids = {self.p1.id} + with mock.patch.object(manager_cls, '_get_missing_target_ids', return_value=missing_target_ids) as mocked: + self.a1.publications.add(self.p1) + mocked.assert_called_once() + def test_related_sets(self): # Article objects have access to their related Publication objects. self.assertQuerysetEqual(self.a1.publications.all(), ['']) -- cgit v1.3