Opened 12 years ago
Closed 12 years ago
#19713 closed Cleanup/optimization (fixed)
Rename inspectdb tests model's field for clarity
| Reported by: | Simon Charette | Owned by: | nobody |
|---|---|---|---|
| Component: | Testing framework | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The recently added inspect db test refers to model field by numerical index.
The attached patch name those fields based on their type for clarity. This also fixes the regression on MySQL concerning blank and null ordering in NullBooleanField.
Attachments (1)
Change History (3)
by , 12 years ago
| Attachment: | 0001-Renamed-inspectdb-field-names-for-clarity.patch added |
|---|
comment:1 by , 12 years ago
comment:2 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
| Triage Stage: | Unreviewed → Accepted |
Note:
See TracTickets
for help on using tickets.
inspectdbtests pass on Python 2.7.3 and 3.2.3, SQLite, Postgresql and PostGIS.