Fixed #33130 -- Restored form errors to be a dict.

Regression in 456466d932830b096d39806e291fe23ec5ed38d5.

2 files changed, 47 insertions, 0 deletions

diff --git a/tests/forms_tests/tests/test_forms.py b/tests/forms_tests/tests/test_forms.py
index 9af103dbd8..fefebde05c 100644
--- a/tests/forms_tests/tests/test_forms.py
+++ b/tests/forms_tests/tests/test_forms.py
@@ -67,6 +67,7 @@ class FormsTestCase(SimpleTestCase):
 
         self.assertTrue(p.is_bound)
         self.assertEqual(p.errors, {})
+        self.assertIsInstance(p.errors, dict)
         self.assertTrue(p.is_valid())
         self.assertHTMLEqual(p.errors.as_ul(), '')
         self.assertEqual(p.errors.as_text(), '')
diff --git a/tests/forms_tests/tests/test_utils.py b/tests/forms_tests/tests/test_utils.py
index ea0949920e..e0e9a0e22e 100644
--- a/tests/forms_tests/tests/test_utils.py
+++ b/tests/forms_tests/tests/test_utils.py
@@ -1,4 +1,5 @@
 import copy
+import json
 
 from django.core.exceptions import ValidationError
 from django.forms.utils import ErrorDict, ErrorList, flatatt
@@ -162,3 +163,48 @@ class FormsUtilsTestCase(SimpleTestCase):
         e = ErrorList(['Invalid username.'])
         self.assertTrue(hasattr(ErrorList, '__html__'))
         self.assertEqual(str(e), e.__html__())
+
+    def test_error_dict_is_dict(self):
+        self.assertIsInstance(ErrorDict(), dict)
+
+    def test_error_dict_is_json_serializable(self):
+        init_errors = ErrorDict([
+            ('__all__', ErrorList([
+                ValidationError('Sorry this form only works on leap days.')
+            ])),
+            ('name', ErrorList([ValidationError('This field is required.')])),
+        ])
+        min_value_error_list = ErrorList([
+            ValidationError('Ensure this value is greater than or equal to 0.')
+        ])
+        e = ErrorDict(
+            init_errors,
+            date=ErrorList([
+                ErrorDict({
+                    'day': min_value_error_list,
+                    'month': min_value_error_list,
+                    'year': min_value_error_list,
+                }),
+            ]),
+        )
+        e['renderer'] = ErrorList([
+            ValidationError(
+                'Select a valid choice. That choice is not one of the '
+                'available choices.'
+            ),
+        ])
+        self.assertJSONEqual(json.dumps(e), {
+            '__all__': ['Sorry this form only works on leap days.'],
+            'name': ['This field is required.'],
+            'date': [
+                {
+                    'day': ['Ensure this value is greater than or equal to 0.'],
+                    'month': ['Ensure this value is greater than or equal to 0.'],
+                    'year': ['Ensure this value is greater than or equal to 0.'],
+                },
+            ],
+            'renderer': [
+                'Select a valid choice. That choice is not one of the '
+                'available choices.'
+            ],
+        })