Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#23986 closed Bug (fixed)

Collectstatic --clear fails if the static dir doesn't exist yet

Reported by: Michael Owned by: Sztrovacsek
Component: contrib.staticfiles Version: 1.7
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


The --clear flag on collectstatic fails if the target directory doesn't exist yet.

Traceback (most recent call last):
  File "redacted/bin/", line 16, in <module>
  File "redacted/local/lib/python2.7/site-packages/django/core/management/", line 385, in execute_from_command_line
  File "redacted/local/lib/python2.7/site-packages/django/core/management/", line 377, in execute
  File "redacted/local/lib/python2.7/site-packages/django/core/management/", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "redacted/local/lib/python2.7/site-packages/django/core/management/", line 338, in execute
    output = self.handle(*args, **options)
  File "redacted/local/lib/python2.7/site-packages/django/core/management/", line 533, in handle
    return self.handle_noargs(**options)
  File "redacted/local/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/", line 168, in handle_noargs
    collected = self.collect()
  File "redacted/local/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/", line 89, in collect
  File "redacted/local/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/", line 202, in clear_dir
    dirs, files =
  File "redacted/local/lib/python2.7/site-packages/django/core/files/", line 270, in listdir
    for entry in os.listdir(path):
OSError: [Errno 2] No such file or directory: '/redacted/public/static/2.1.0-dev'


        if not os.path.exists(

In Command.clear_dir() fixes the issue

Change History (12)

comment:1 by Tim Graham, 9 years ago

Triage Stage: UnreviewedAccepted
Type: UncategorizedBug

comment:2 by roebk, 9 years ago

Owner: changed from nobody to roebk
Status: newassigned

comment:3 by roebk, 9 years ago

I've created a pull request for this ticket which can be viewed at I hope I've done everything to the correctly and to the required standard *fingers crossed*.

comment:4 by Berker Peksag, 9 years ago

Has patch: set
Needs tests: set

comment:5 by Berker Peksag, 9 years ago

Patch needs improvement: set

comment:6 by roebk, 9 years ago

For a little guidance and to ensure I implement the tests in the best possible way, what would be the best way in adding the test for a missing target directory? Should I be looking at adding a new class something along the lines of TestCollectionClearNoTargetDirectory?

For reference the current test of the --clear option is as follows:

class TestCollectionClear(CollectionTestCase):
    Test the ``--clear`` option of the ``collectstatic`` management command.
    def run_collectstatic(self, **kwargs):
        clear_filepath = os.path.join(settings.STATIC_ROOT, 'cleared.txt')
        with open(clear_filepath, 'w') as f:
            f.write('should be cleared')
        super(TestCollectionClear, self).run_collectstatic(clear=True)

    def test_cleared_not_found(self):

Thanks in advance :)

comment:7 by Berker Peksag, 9 years ago

You can just override the STATIC_ROOT setting and call self.run_collectstatic():

@override_settings(STATIC_ROOT=os.path.join(TEST_ROOT, 'project', 'site_media', 'static-does-not-exist'))
def test_empty_directory(self):

but, I think this is not the cleanest solution. You also don't need to call BaseCollectionTestCase.setUp() because it tries to create a directory if it doesn't exist:

I'd go with a new test case (without using BaseCollectionTestCase mixin).

comment:8 by Sztrovacsek, 9 years ago

Owner: changed from roebk to Sztrovacsek

comment:10 by Baptiste Mispelon, 9 years ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

Looks good to me, thanks!

comment:11 by Tim Graham <timograham@…>, 9 years ago

Resolution: fixed
Status: assignedclosed

In 87d78241a2fc85e5715fb51c554fe06e91deee58:

Fixed #23986 -- Fixed collectstatic --clear failure if STATIC_ROOT dir doesn't exist.

comment:12 by Tim Graham <timograham@…>, 9 years ago

In 389bacca:

Fixed staticfiles test on Windows; refs #23986.

Note: See TracTickets for help on using tickets.
Back to Top