Opened 3 years ago

Closed 3 years ago

#20974 closed Cleanup/optimization (wontfix)

Adding test for mysql-specific model validation checks

Reported by: chrismedrela Owned by: ANUBHAV JOSHI
Component: Core (System checks) Version: 1.5
Severity: Normal Keywords: checks
Cc: anubhav9042@… Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

There is some validation on model fields under MySQL backend here [1]. It checks if you do not use unique varchars longer than 255 characters. This code is not covered by any test.

I'm working at rewriting validation framework (see discussion on mailing list [2]), so please, do not work at this ticket until I finish.

[1] https://github.com/django/django/blob/master/django/db/backends/mysql/validation.py
[2] https://groups.google.com/forum/#!topic/django-developers/fEf21dtpqDE

Change History (4)

comment:1 Changed 3 years ago by Tim Graham

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Summary: Adding test for mysql-specific checksAdding test for mysql-specific model validation checks
Triage Stage: UnreviewedAccepted

comment:2 Changed 3 years ago by Tim Graham

Easy pickings: unset

comment:3 Changed 3 years ago by ANUBHAV JOSHI

Cc: anubhav9042@… added
Owner: changed from nobody to ANUBHAV JOSHI
Status: newassigned

comment:4 Changed 3 years ago by Russell Keith-Magee

Component: Database layer (models, ORM)Core (System checks)
Resolution: wontfix
Status: assignedclosed
Note: See TracTickets for help on using tickets.
Back to Top