#21883: Use of xrange in custom lookups code not valid in Python 3; "__in list 
too
long" case not tested
-------------------------------------+-------------------------------------
               Reporter:  carljm     |          Owner:  nobody
                   Type:  Bug        |         Status:  new
              Component:  Database   |        Version:  master
  layer (models, ORM)                |       Keywords:
               Severity:  Normal     |      Has patch:  0
           Triage Stage:  Accepted   |    Needs tests:  0
    Needs documentation:  0          |  Easy pickings:  0
Patch needs improvement:  0          |
                  UI/UX:  0          |
-------------------------------------+-------------------------------------
 There is a use of 'xrange' in the `In` lookup class in
 `django/db/models/lookups.py`, within the "list too long for db backend"
 case. "xrange" is not present on Python 3, so this code will fail there.

 The fact that tests are passing on both Python 2 and 3 suggests that this
 code branch is untested, and I confirmed by inserting a `pdb.set_trace()`
 at that point and running the full test suite. That's a sizable code
 branch; it should be tested.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/21883>
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 [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/049.1bfea4c4b5c3a179952c8af6a79f187a%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to