diff options
| author | Ian Foote <python@ian.feete.org> | 2015-11-06 20:18:00 +0100 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2015-11-10 12:12:07 -0500 |
| commit | 4608573788c04fc047da42b4b7b48fdee8136ad3 (patch) | |
| tree | febbc76c12922323106be53aee742c3e9295b355 /tests | |
| parent | dbbae2cead5ca1f15177415d4bfe795d4402be9f (diff) | |
Fixed #25693 -- Prevented data loss with Prefetch and ManyToManyField.
Thanks to Jamie Matthews for finding and explaining the bug.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/prefetch_related/tests.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/tests/prefetch_related/tests.py b/tests/prefetch_related/tests.py index 7aed238bc0..cd763bb950 100644 --- a/tests/prefetch_related/tests.py +++ b/tests/prefetch_related/tests.py @@ -223,6 +223,17 @@ class PrefetchRelatedTests(TestCase): self.assertIn('prefetch_related', str(cm.exception)) self.assertIn("name", str(cm.exception)) + def test_m2m_shadow(self): + msg = 'to_attr=books conflicts with a field on the Author model.' + poems = Book.objects.filter(title='Poems') + with self.assertRaisesMessage(ValueError, msg): + list(Author.objects.prefetch_related( + Prefetch('books', queryset=poems, to_attr='books'), + )) + # Without the ValueError, a book was deleted due to the implicit + # save of reverse relation assignment. + self.assertEqual(self.author1.books.count(), 2) + class CustomPrefetchTests(TestCase): @classmethod |
