~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_permissions.py

  • Committer: Martin Pool
  • Date: 2008-04-24 07:22:53 UTC
  • mto: This revision was merged to the branch mainline in revision 3415.
  • Revision ID: mbp@sourcefrog.net-20080424072253-opmjij7xfy38w27f
Remove every assert statement from bzrlib!

Depending on the context they are:

 * turned into an explicit if/raise of either AssertionError 
   or something more specific -- particularly where they protect
   programming interfaces, complex invariants, or data file integrity
 * removed, if they're redundant with a later check, not protecting
   a meaningful invariant
 * turned into a selftest method on tests

Show diffs side-by-side

added added

removed removed

Lines of Context:
47
47
 
48
48
def chmod_r(base, file_mode, dir_mode):
49
49
    """Recursively chmod from a base directory"""
50
 
    assert os.path.isdir(base)
51
50
    os.chmod(base, dir_mode)
52
51
    for root, dirs, files in os.walk(base):
53
52
        for d in dirs:
67
66
    :param dir_mode: The mode for all directories
68
67
    :param include_base: If false, only check the subdirectories
69
68
    """
70
 
    assert os.path.isdir(base)
71
69
    t = get_transport(".")
72
70
    if include_base:
73
71
        test.assertTransportMode(t, base, dir_mode)