1
# Copyright (C) 2005 by Canonical Ltd
2
# -*- coding: utf-8 -*-
4
# This program is free software; you can redistribute it and/or modify
5
# it under the terms of the GNU General Public License as published by
6
# the Free Software Foundation; either version 2 of the License, or
7
# (at your option) any later version.
9
# This program is distributed in the hope that it will be useful,
10
# but WITHOUT ANY WARRANTY; without even the implied warranty of
11
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12
# GNU General Public License for more details.
14
# You should have received a copy of the GNU General Public License
15
# along with this program; if not, write to the Free Software
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
19
"""Tests for bzr setting permissions.
21
Files which are created underneath .bzr/ should inherit its permissions.
22
So if the directory is group writable, the files and subdirs should be as well.
24
In the future, when we have Repository/Branch/Checkout information, the
25
permissions should be inherited individually, rather than all be the same.
28
# TODO: jam 20051215 There are no tests for ftp yet, because we have no ftp server
29
# TODO: jam 20051215 Currently the default behavior for 'bzr branch' is just
30
# defined by the local umask. This isn't terrible, is it
31
# the truly desired behavior?
36
from StringIO import StringIO
38
from bzrlib.branch import Branch
39
from bzrlib.bzrdir import BzrDir
40
from bzrlib.lockable_files import LockableFiles
41
from bzrlib.tests import TestCaseWithTransport, TestSkipped
42
from bzrlib.tests.test_sftp_transport import TestCaseWithSFTPServer
43
from bzrlib.transport import get_transport
44
from bzrlib.workingtree import WorkingTree
47
def chmod_r(base, file_mode, dir_mode):
48
"""Recursively chmod from a base directory"""
49
assert os.path.isdir(base)
50
os.chmod(base, dir_mode)
51
for root, dirs, files in os.walk(base):
53
p = os.path.join(root, d)
56
p = os.path.join(root, f)
57
os.chmod(p, file_mode)
60
def check_mode_r(test, base, file_mode, dir_mode, include_base=True):
61
"""Check that all permissions match
63
:param test: The TestCase being run
64
:param base: The path to the root directory to check
65
:param file_mode: The mode for all files
66
:param dir_mode: The mode for all directories
67
:param include_base: If false, only check the subdirectories
69
assert os.path.isdir(base)
70
t = get_transport(".")
72
test.assertTransportMode(t, base, dir_mode)
73
for root, dirs, files in os.walk(base):
75
p = os.path.join(root, d)
76
test.assertTransportMode(t, p, dir_mode)
78
p = os.path.join(root, f)
79
test.assertTransportMode(t, p, file_mode)
82
class TestPermissions(TestCaseWithTransport):
84
def test_new_files(self):
85
if sys.platform == 'win32':
86
raise TestSkipped('chmod has no effect on win32')
88
t = self.make_branch_and_tree('.')
90
open('a', 'wb').write('foo\n')
94
chmod_r('.bzr', 0644, 0755)
95
check_mode_r(self, '.bzr', 0644, 0755)
97
# although we are modifying the filesystem
98
# underneath the objects, they are not locked, and thus it must
99
# be safe for most operations. But here we want to observe a
100
# mode change in the control bits, which current do not refresh
101
# when a new lock is taken out.
102
t = WorkingTree.open('.')
104
self.assertEqualMode(0755, b.control_files._dir_mode)
105
self.assertEqualMode(0644, b.control_files._file_mode)
107
# Modifying a file shouldn't break the permissions
108
open('a', 'wb').write('foo2\n')
110
# The mode should be maintained after commit
111
check_mode_r(self, '.bzr', 0644, 0755)
113
# Adding a new file should maintain the permissions
114
open('b', 'wb').write('new b\n')
117
check_mode_r(self, '.bzr', 0644, 0755)
119
# Recursively update the modes of all files
120
chmod_r('.bzr', 0664, 0775)
121
check_mode_r(self, '.bzr', 0664, 0775)
122
t = WorkingTree.open('.')
124
self.assertEqualMode(0775, b.control_files._dir_mode)
125
self.assertEqualMode(0664, b.control_files._file_mode)
127
open('a', 'wb').write('foo3\n')
129
check_mode_r(self, '.bzr', 0664, 0775)
131
open('c', 'wb').write('new c\n')
134
check_mode_r(self, '.bzr', 0664, 0775)
136
# Test the group sticky bit
137
# Recursively update the modes of all files
138
chmod_r('.bzr', 0664, 02775)
139
check_mode_r(self, '.bzr', 0664, 02775)
140
t = WorkingTree.open('.')
142
self.assertEqualMode(02775, b.control_files._dir_mode)
143
self.assertEqualMode(0664, b.control_files._file_mode)
145
open('a', 'wb').write('foo4\n')
147
check_mode_r(self, '.bzr', 0664, 02775)
149
open('d', 'wb').write('new d\n')
152
check_mode_r(self, '.bzr', 0664, 02775)
154
def test_disable_set_mode(self):
155
# TODO: jam 20051215 Ultimately, this test should probably test that
156
# extra chmod calls aren't being made
158
transport = get_transport(self.get_url())
159
transport.put('my-lock', StringIO(''))
160
lockable = LockableFiles(transport, 'my-lock')
161
self.assertNotEqual(None, lockable._dir_mode)
162
self.assertNotEqual(None, lockable._file_mode)
164
LockableFiles._set_dir_mode = False
165
transport = get_transport('.')
166
lockable = LockableFiles(transport, 'my-lock')
167
self.assertEqual(None, lockable._dir_mode)
168
self.assertNotEqual(None, lockable._file_mode)
170
LockableFiles._set_file_mode = False
171
transport = get_transport('.')
172
lockable = LockableFiles(transport, 'my-lock')
173
self.assertEqual(None, lockable._dir_mode)
174
self.assertEqual(None, lockable._file_mode)
176
LockableFiles._set_dir_mode = True
177
transport = get_transport('.')
178
lockable = LockableFiles(transport, 'my-lock')
179
self.assertNotEqual(None, lockable._dir_mode)
180
self.assertEqual(None, lockable._file_mode)
182
LockableFiles._set_file_mode = True
183
transport = get_transport('.')
184
lockable = LockableFiles(transport, 'my-lock')
185
self.assertNotEqual(None, lockable._dir_mode)
186
self.assertNotEqual(None, lockable._file_mode)
188
LockableFiles._set_dir_mode = True
189
LockableFiles._set_file_mode = True
192
class TestSftpPermissions(TestCaseWithSFTPServer):
194
def test_new_files(self):
195
if sys.platform == 'win32':
196
raise TestSkipped('chmod has no effect on win32')
197
# Though it would be nice to test that SFTP to a server
198
# which does support chmod has the right effect
200
# bodge around for stubsftpserver not letting use connect
202
_t = get_transport(self.get_url())
205
t_local = self.make_branch_and_tree('local')
206
b_local = t_local.branch
207
open('local/a', 'wb').write('foo\n')
209
t_local.commit('foo')
211
# Delete them because we are modifying the filesystem underneath them
212
chmod_r('local/.bzr', 0644, 0755)
213
check_mode_r(self, 'local/.bzr', 0644, 0755)
215
t = WorkingTree.open('local')
217
self.assertEqualMode(0755, b_local.control_files._dir_mode)
218
self.assertEqualMode(0644, b_local.control_files._file_mode)
221
sftp_url = self.get_url('sftp')
222
b_sftp = BzrDir.create_branch_and_repo(sftp_url)
226
chmod_r('sftp/.bzr', 0644, 0755)
227
check_mode_r(self, 'sftp/.bzr', 0644, 0755)
229
b_sftp = Branch.open(sftp_url)
230
self.assertEqualMode(0755, b_sftp.control_files._dir_mode)
231
self.assertEqualMode(0644, b_sftp.control_files._file_mode)
233
open('local/a', 'wb').write('foo2\n')
234
t_local.commit('foo2')
236
# The mode should be maintained after commit
237
check_mode_r(self, 'sftp/.bzr', 0644, 0755)
239
open('local/b', 'wb').write('new b\n')
241
t_local.commit('new b')
243
check_mode_r(self, 'sftp/.bzr', 0644, 0755)
246
# Recursively update the modes of all files
247
chmod_r('sftp/.bzr', 0664, 0775)
248
check_mode_r(self, 'sftp/.bzr', 0664, 0775)
250
b_sftp = Branch.open(sftp_url)
251
self.assertEqualMode(0775, b_sftp.control_files._dir_mode)
252
self.assertEqualMode(0664, b_sftp.control_files._file_mode)
254
open('local/a', 'wb').write('foo3\n')
255
t_local.commit('foo3')
257
check_mode_r(self, 'sftp/.bzr', 0664, 0775)
259
open('local/c', 'wb').write('new c\n')
261
t_local.commit('new c')
263
check_mode_r(self, 'sftp/.bzr', 0664, 0775)
265
def test_sftp_server_modes(self):
266
if sys.platform == 'win32':
267
raise TestSkipped('chmod has no effect on win32')
270
original_umask = os.umask(umask)
273
t = get_transport(self.get_url())
274
# Direct access should be masked by umask
275
t._sftp_open_exclusive('a', mode=0666).write('foo\n')
276
self.assertTransportMode(t, 'a', 0666 &~umask)
278
# but Transport overrides umask
279
t.put('b', 'txt', mode=0666)
280
self.assertTransportMode(t, 'b', 0666)
282
t._sftp.mkdir('c', mode=0777)
283
self.assertTransportMode(t, 'c', 0777 &~umask)
285
t.mkdir('d', mode=0777)
286
self.assertTransportMode(t, 'd', 0777)
288
os.umask(original_umask)
added
removed
bzrlib/tests/test_permissions.py
