[Django] #32953: Enable optional auto-reloading for testserver
Django
-------------------------------------+-------------------------------------
Reporter: Igor | Owner: Igor Bakutin
Bakutin |
Type: | Status: assigned
Uncategorized |
Component: Core | Version: 4.0
(Management commands) |
Severity: Normal | Keywords: testserver reloader
Triage Stage: | Has patch: 1
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
`django/core/management/commands/testserver.py:39` has the following
comment
{{{
# Run the development server. Turn off auto-reloading because it causes
# a strange error -- it causes this handle() method to be called
# multiple times.
}}}
The task is to fix the issue and add optional `--autoreload` flag to the
command
--
Ticket URL: <https://code.djangoproject.com/ticket/32953>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
-------------------------------------+-------------------------------------
| Bakutin
Type: New feature | Status: assigned
Component: Core (Management | Version: 4.0
commands) |
Severity: Normal | Resolution:
Keywords: testserver reloader | Triage Stage:
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* type: Uncategorized => New feature
--
Ticket URL: <https://code.djangoproject.com/ticket/32953#comment:1>
Django
-------------------------------------+-------------------------------------
| Bakutin
Type: New feature | Status: assigned
Component: Core (Management | Version: 4.0
commands) |
Severity: Normal | Resolution:
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* owner: Igor Bakutin => Igor Bakutin
--
Ticket URL: <https://code.djangoproject.com/ticket/32953#comment:2>
Django
-------------------------------------+-------------------------------------
| Bakutin
Type: New feature | Status: assigned
Component: Core (Management | Version: 4.0
commands) |
Severity: Normal | Resolution:
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Mariusz Felisiak):
Thanks for the ticket.
Replying to [ticket:32953 Igor Bakutin]:
> `django/core/management/commands/testserver.py:39` has the following
comment
>
> {{{
> # Run the development server. Turn off auto-reloading because it causes
> # a strange error -- it causes this handle() method to be called
> # multiple times.
> }}}
However, this comment was added in
9c55bbdef7f1717aa17f7ca17cb2396522426310 and that doesn't mean that 14
years later it's still a desirable feature. `testserver` is intended to
work with a database snapshot (loaded from fixtures) so it can be really
fragile for any changes in models or migrations. I'm not sure if we can
support it in a bulletproof way 🤔.
--
Ticket URL: <https://code.djangoproject.com/ticket/32953#comment:3>
Django
-------------------------------------+-------------------------------------
| Bakutin
Component: Core (Management | Version: 4.0
commands) |
Keywords: testserver reloader | Triage Stage:
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* status: assigned => closed
* resolution: => wontfix
Comment:
I too think this is probably a mistake to add. (If anything deprecating
the `testserver` command would be the way to go, rather than adding
surface area here.) If folks want autoreloading `runserver` should be
sufficient.
--
Ticket URL: <https://code.djangoproject.com/ticket/32953#comment:4>