e_deepcopy = copy.deepcopy(e)
self.assertEqual(e, e_deepcopy)
self.assertEqual(e.as_data(), e_copy.as_data()) //HERE
Shouldn't you get the data from e_deepcopy in that line (instead the data
of e_copy)?
Thanks,
Petru-Florin Mihancea
--
Ticket URL: <https://code.djangoproject.com/ticket/30358>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
* type: Bug => Cleanup/optimization
* easy: 0 => 1
* stage: Unreviewed => Accepted
Comment:
Thanks for the report. With `e_deepcopy` assertion fails, because
`deepcopy()` creates copies of `ValidationError`. IMO we should just
remove this redundant assertion.
--
Ticket URL: <https://code.djangoproject.com/ticket/30358#comment:1>
* owner: nobody => Krisjanis Vaiders
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/30358#comment:2>
* has_patch: 0 => 1
Comment:
https://github.com/django/django/pull/11213
--
Ticket URL: <https://code.djangoproject.com/ticket/30358#comment:3>
* stage: Accepted => Ready for checkin
--
Ticket URL: <https://code.djangoproject.com/ticket/30358#comment:4>
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"79d3b905df7eee5658039cad33752e3354dad74a" 79d3b90]:
{{{
#!CommitTicketReference repository=""
revision="79d3b905df7eee5658039cad33752e3354dad74a"
Fixed #30358 -- Removed redundant assertion in
FormsUtilsTestCase.test_error_dict_copy().
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/30358#comment:5>