[rbdemo] Add some checks to prevent us from breaking the demo site.

Review Request #6338 — Created Sept. 19, 2014 and submitted

Information

rb-extension-pack
2bae2dc...

Reviewers

This adds some pre-reset checks to rbdemo's reset-demo command that
ensures we have all the proper permissions and abilities we need prior
to resetting the demo database state. This includes read-access to the
fixtures and media directories, write access to the destination media
directory, and the ability to change ownership. It also validates that
the specified user and group are valid.

Tested each possible validation failure and saw that I got appropriate
errors.

Tested with a valid setup and saw it reset the demo database.

reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        rbdemo/rbdemo/management/commands/reset-demo.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        rbdemo/rbdemo/management/commands/reset-demo.py
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (3a8693e)
Loading...