#18990: 'manage.py loaddata' does not report if the loading file does not exist
-------------------------------------+-------------------------------------
     Reporter:  sergzach             |                    Owner:  nobody
         Type:  Bug                  |                   Status:  new
    Component:  Core (Management     |                  Version:  1.4
  commands)                          |               Resolution:
     Severity:  Normal               |             Triage Stage:  Accepted
     Keywords:  loaddata             |      Needs documentation:  0
    Has patch:  1                    |  Patch needs improvement:  1
  Needs tests:  1                    |                    UI/UX:  0
Easy pickings:  0                    |
-------------------------------------+-------------------------------------
Changes (by ptone):

 * needs_better_patch:  0 => 1


Comment:

 I think we only want to raise an error when a single fixture file was
 specified no valid fixture files were found in a dir - which I believe
 means checking at this point:

 {{{
 if os.path.isabs(fixture_name):
     fixture_dirs = [fixture_name]
 }}}

 Since we have specified a single file - and it does not exist - raising an
 actual CommandError is probably what is desired.  The tests should check
 that this is raised using assertRaises.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/18990#comment:5>
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 post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to