diff options
| author | Simon Charette <charette.s@gmail.com> | 2015-11-13 15:54:05 -0500 |
|---|---|---|
| committer | Simon Charette <charette.s@gmail.com> | 2015-11-14 11:33:28 -0500 |
| commit | 1c5f4e86bc9c7bae5d5f3830d9cfbb0043297261 (patch) | |
| tree | fe79a6afb78716fad02cd0659f336e459b1576b6 /tests/model_regress | |
| parent | d95b22bd56764c71047ec16f628c00c0fd997cfd (diff) | |
Fixed #25745 -- Promoted RuntimeWarnings to errors in the test suite.
Diffstat (limited to 'tests/model_regress')
| -rw-r--r-- | tests/model_regress/test_pickle.py | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/tests/model_regress/test_pickle.py b/tests/model_regress/test_pickle.py index 517646c407..393c53733b 100644 --- a/tests/model_regress/test_pickle.py +++ b/tests/model_regress/test_pickle.py @@ -3,13 +3,11 @@ import os import pickle import subprocess import sys -import warnings from django.core.files.temp import NamedTemporaryFile from django.db import DJANGO_VERSION_PICKLE_KEY, models from django.test import TestCase, mock from django.utils._os import npath, upath -from django.utils.encoding import force_text from django.utils.version import get_version from .models import Article @@ -31,11 +29,9 @@ class ModelPickleTestCase(TestCase): return reduce_list p = MissingDjangoVersion(title="FooBar") - with warnings.catch_warnings(record=True) as recorded: + msg = "Pickled model instance's Django version is not specified." + with self.assertRaisesMessage(RuntimeWarning, msg): pickle.loads(pickle.dumps(p)) - msg = force_text(recorded.pop().message) - self.assertEqual(msg, - "Pickled model instance's Django version is not specified.") def test_unsupported_unpickle(self): """ @@ -52,14 +48,9 @@ class ModelPickleTestCase(TestCase): return reduce_list p = DifferentDjangoVersion(title="FooBar") - with warnings.catch_warnings(record=True) as recorded: + msg = "Pickled model instance's Django version 1.0 does not match the current version %s." % get_version() + with self.assertRaisesMessage(RuntimeWarning, msg): pickle.loads(pickle.dumps(p)) - msg = force_text(recorded.pop().message) - self.assertEqual( - msg, - "Pickled model instance's Django version 1.0 does not " - "match the current version %s." % get_version() - ) def test_unpickling_when_appregistrynotready(self): """ |
