diff options
| author | Hasan <hasan.r67@gmail.com> | 2016-01-04 12:20:08 +0330 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2016-01-29 13:03:39 -0500 |
| commit | 253adc2b8a52982139d40c4f55b3fd446e1cb8f3 (patch) | |
| tree | c508d48636f5b37e97c8078737d398d7475ff8cc /tests/get_earliest_or_latest | |
| parent | 3d0dcd7f5af378d3ab6adb303b913e6c7b2e0ee5 (diff) | |
| download | django-253adc2b8a52982139d40c4f55b3fd446e1cb8f3.tar.gz | |
Refs #26022 -- Used context manager version of assertRaisesMessage in tests.
Diffstat (limited to 'tests/get_earliest_or_latest')
| -rw-r--r-- | tests/get_earliest_or_latest/tests.py | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/tests/get_earliest_or_latest/tests.py b/tests/get_earliest_or_latest/tests.py index d74a0724d7..e03f701da6 100644 --- a/tests/get_earliest_or_latest/tests.py +++ b/tests/get_earliest_or_latest/tests.py @@ -58,12 +58,12 @@ class EarliestOrLatestTests(TestCase): # Ensure that error is raised if the user forgot to add a get_latest_by # in the Model.Meta Article.objects.model._meta.get_latest_by = None - self.assertRaisesMessage( + with self.assertRaisesMessage( AssertionError, "earliest() and latest() require either a field_name parameter or " - "'get_latest_by' in the model", - lambda: Article.objects.earliest(), - ) + "'get_latest_by' in the model" + ): + Article.objects.earliest() def test_latest(self): # Because no Articles exist yet, latest() raises ArticleDoesNotExist. @@ -109,12 +109,11 @@ class EarliestOrLatestTests(TestCase): # Ensure that error is raised if the user forgot to add a get_latest_by # in the Model.Meta Article.objects.model._meta.get_latest_by = None - self.assertRaisesMessage( + with self.assertRaisesMessage( AssertionError, - "earliest() and latest() require either a field_name parameter or " - "'get_latest_by' in the model", - lambda: Article.objects.latest(), - ) + "earliest() and latest() require either a field_name parameter or " + "'get_latest_by' in the model"): + Article.objects.latest() def test_latest_manual(self): # You can still use latest() with a model that doesn't have |