diff options
author | Daniyal <abbasi.daniyal98@gmail.com> | 2021-03-16 21:11:27 +0530 |
---|---|---|
committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2021-03-19 08:04:37 +0100 |
commit | 474cc420bf6bc1067e2aaa4b40cf6a08d62096f7 (patch) | |
tree | 1281ae03beab376bbc08c0741da3b91d285cc2e3 /tests/check_framework | |
parent | 37044817f9a57126d655f216019e8c8cca7c151b (diff) | |
download | django-474cc420bf6bc1067e2aaa4b40cf6a08d62096f7.tar.gz |
Refs #32508 -- Raised Type/ValueError instead of using "assert" in django.core.
Diffstat (limited to 'tests/check_framework')
-rw-r--r-- | tests/check_framework/tests.py | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/tests/check_framework/tests.py b/tests/check_framework/tests.py index e669b11c2b..9e461c5040 100644 --- a/tests/check_framework/tests.py +++ b/tests/check_framework/tests.py @@ -4,6 +4,7 @@ from io import StringIO from django.apps import apps from django.core import checks from django.core.checks import Error, Warning +from django.core.checks.messages import CheckMessage from django.core.checks.registry import CheckRegistry from django.core.management import call_command from django.core.management.base import CommandError @@ -74,6 +75,20 @@ class SystemCheckFrameworkTests(SimpleTestCase): def no_kwargs(app_configs, databases): pass + def test_register_run_checks_non_iterable(self): + registry = CheckRegistry() + + @registry.register + def return_non_iterable(**kwargs): + return Error('Message') + + msg = ( + 'The function %r did not return a list. All functions registered ' + 'with the checks registry must return a list.' % return_non_iterable + ) + with self.assertRaisesMessage(TypeError, msg): + registry.run_checks() + class MessageTests(SimpleTestCase): @@ -132,6 +147,11 @@ class MessageTests(SimpleTestCase): e = Error("Error", obj=DummyObj()) self.assertNotEqual(e, 'a string') + def test_invalid_level(self): + msg = 'The first argument should be level.' + with self.assertRaisesMessage(TypeError, msg): + CheckMessage('ERROR', 'Message') + def simple_system_check(**kwargs): simple_system_check.kwargs = kwargs |