~bzr-pqm/bzr/bzr.dev

3416.2.2 by Martin Pool
Change some callers to get file and directory permissions from bzrdir not LockableFiles
1
# Copyright (C) 2005, 2008 Canonical Ltd
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
2
#
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
7
#
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
12
#
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
16
17
18
"""Tests for bzr setting permissions.
19
20
Files in the branch control directory (.bzr or .bzr/branch) should inherit
21
the .bzr directory permissions.
22
So if the directory is group writable, the files and subdirs should be as well.
23
"""
24
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
25
# TODO: jam 20051215 Currently the default behavior for 'bzr branch' is just
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
26
#                    defined by the local umask. This isn't terrible, is it
27
#                    the truly desired behavior?
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
28
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
29
import os
30
import sys
31
import stat
32
from StringIO import StringIO
33
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
34
from bzrlib import tests
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
35
from bzrlib.branch import Branch
36
from bzrlib.bzrdir import BzrDir
37
from bzrlib.lockable_files import LockableFiles
2018.5.99 by Andrew Bennetts
Don't test mode setting for remote branches because they don't do mode setting.
38
from bzrlib.remote import RemoteBranchFormat
1685.1.17 by John Arbash Meinel
test_permissions.check_mode_r needs to pass a path that the '.' transport can handle
39
from bzrlib.tests.test_permissions import chmod_r, check_mode_r
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
40
from bzrlib.tests.test_sftp_transport import TestCaseWithSFTPServer
41
from bzrlib.transport import get_transport
42
from bzrlib.workingtree import WorkingTree
43
44
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
45
class _NullPermsStat(object):
46
    """A class that proxy's a stat result and strips permissions."""
47
48
    def __init__(self, orig_stat):
49
        self._orig_stat = orig_stat
50
        # We strip all permission bits from st_mode
51
        self.st_mode = orig_stat.st_mode & ~07777
52
53
    def __getattr__(self, name):
54
        return getattr(self._orig_stat, name)
55
56
57
class TestPermissions(tests.TestCaseWithTransport):
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
58
59
    def test_new_branch(self):
2018.5.99 by Andrew Bennetts
Don't test mode setting for remote branches because they don't do mode setting.
60
        if isinstance(self.branch_format, RemoteBranchFormat):
61
            # Remote branch format have no permission logic in them; there's
62
            # nothing to test here.
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
63
            raise tests.TestNotApplicable('Remote branches have no'
64
                                          ' permission logic')
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
65
        if sys.platform == 'win32':
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
66
            raise tests.TestNotApplicable('chmod has no effect on win32')
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
67
        # also, these are BzrBranch format specific things..
68
        os.mkdir('a')
69
        mode = stat.S_IMODE(os.stat('a').st_mode)
70
        t = self.make_branch_and_tree('.')
71
        b = t.branch
3416.2.2 by Martin Pool
Change some callers to get file and directory permissions from bzrdir not LockableFiles
72
        self.assertEqualMode(mode, b.bzrdir._get_dir_mode())
73
        self.assertEqualMode(mode & ~07111, b.bzrdir._get_file_mode())
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
74
        self.assertEqualMode(mode, b.control_files._dir_mode)
75
        self.assertEqualMode(mode & ~07111, b.control_files._file_mode)
76
3638.3.6 by Vincent Ladeuil
Isolate group sticky bit related tests.
77
        os.mkdir('d')
78
        os.chmod('d', 0700)
79
        b = self.make_branch('d')
80
        self.assertEqualMode(0700, b.bzrdir._get_dir_mode())
81
        self.assertEqualMode(0600, b.bzrdir._get_file_mode())
82
        self.assertEqualMode(0700, b.control_files._dir_mode)
83
        self.assertEqualMode(0600, b.control_files._file_mode)
84
        check_mode_r(self, 'd/.bzr', 00600, 00700)
85
86
    def test_new_branch_group_sticky_bit(self):
87
        if isinstance(self.branch_format, RemoteBranchFormat):
88
            # Remote branch format have no permission logic in them; there's
89
            # nothing to test here.
90
            raise tests.TestNotApplicable('Remote branches have no'
91
                                          ' permission logic')
92
        if sys.platform == 'win32':
93
            raise tests.TestNotApplicable('chmod has no effect on win32')
94
        elif sys.platform == 'darwin':
95
            # OS X creates temp dirs with the 'wheel' group, which users are
96
            # not likely to be in, and this prevents us from setting the sgid
97
            # bit
98
            os.chown(self.test_dir, os.getuid(), os.getgid())
99
        # also, these are BzrBranch format specific things..
100
        t = self.make_branch_and_tree('.')
101
        b = t.branch
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
102
        os.mkdir('b')
103
        os.chmod('b', 02777)
1666.1.4 by Robert Collins
* 'Metadir' is now the default disk format. This improves behaviour in
104
        b = self.make_branch('b')
3416.2.2 by Martin Pool
Change some callers to get file and directory permissions from bzrdir not LockableFiles
105
        self.assertEqualMode(02777, b.bzrdir._get_dir_mode())
106
        self.assertEqualMode(00666, b.bzrdir._get_file_mode())
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
107
        self.assertEqualMode(02777, b.control_files._dir_mode)
108
        self.assertEqualMode(00666, b.control_files._file_mode)
1685.1.17 by John Arbash Meinel
test_permissions.check_mode_r needs to pass a path that the '.' transport can handle
109
        check_mode_r(self, 'b/.bzr', 00666, 02777)
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
110
111
        os.mkdir('c')
112
        os.chmod('c', 02750)
1666.1.4 by Robert Collins
* 'Metadir' is now the default disk format. This improves behaviour in
113
        b = self.make_branch('c')
3416.2.2 by Martin Pool
Change some callers to get file and directory permissions from bzrdir not LockableFiles
114
        self.assertEqualMode(02750, b.bzrdir._get_dir_mode())
3416.2.5 by Martin Pool
Correction to branch permissions test
115
        self.assertEqualMode(00640, b.bzrdir._get_file_mode())
116
        self.assertEqualMode(02750, b.control_files._dir_mode)
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
117
        self.assertEqualMode(00640, b.control_files._file_mode)
1685.1.17 by John Arbash Meinel
test_permissions.check_mode_r needs to pass a path that the '.' transport can handle
118
        check_mode_r(self, 'c/.bzr', 00640, 02750)
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
119
3641.2.1 by John Arbash Meinel
Fix bug #259855, if a Transport returns 0 for permission bits, ignore it
120
    def test_mode_0(self):
121
        """Test when a transport returns null permissions for .bzr"""
122
        if isinstance(self.branch_format, RemoteBranchFormat):
123
            # Remote branch format have no permission logic in them; there's
124
            # nothing to test here.
125
            raise tests.TestNotApplicable('Remote branches have no'
126
                                          ' permission logic')
127
        self.make_branch_and_tree('.')
128
        bzrdir = BzrDir.open('.')
129
        # Monkey patch the transport
130
        _orig_stat = bzrdir.transport.stat
131
        def null_perms_stat(*args, **kwargs):
132
            result = _orig_stat(*args, **kwargs)
133
            return _NullPermsStat(result)
134
        bzrdir.transport.stat = null_perms_stat
135
        self.assertIs(None, bzrdir._get_dir_mode())
136
        self.assertIs(None, bzrdir._get_file_mode())