#36369: Missing test coverage for FORWARD_PROPERTIES entries
-------------------------------------+-------------------------------------
Reporter: Clifford Gama | Owner: Senthil
Type: | Kumar
Cleanup/optimization | Status: assigned
Component: Database layer | Version: dev
(models, ORM) |
Severity: Normal | Resolution:
Keywords: FORWARD_PROPERTIES, | Triage Stage: Accepted
Options |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Senthil Kumar):
* needs_better_patch: 1 => 0
Comment:
Thanks for the patience again. I have revised the unit tests based on the
suggested flow.
I have created a new PR as the previous PR's commit history got messed up.
I have also added the fix and all the tests are passing now.
Link to the new PR: https://github.com/django/django/pull/19645
Previous PR: https://github.com/django/django/pull/19484
I have also dropped a quick question about using apps.clear_cache() in the
PR. Hope to get some clarification on that. Thanks
--
Ticket URL: <https://code.djangoproject.com/ticket/36369#comment:11>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion visit
https://groups.google.com/d/msgid/django-updates/010701981acedab2-ab970019-97e3-4a0f-b612-6088e80290fd-000000%40eu-central-1.amazonses.com.
