~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_smart.py

  • Committer: Vincent Ladeuil
  • Date: 2007-06-06 14:26:08 UTC
  • mto: (2485.8.44 bzr.connection.sharing)
  • mto: This revision was merged to the branch mainline in revision 2646.
  • Revision ID: v.ladeuil+lp@free.fr-20070606142608-i9ufaqewadslf1cn
Finish sftp refactoring. Test suite passing.

* bzrlib/transport/sftp.py:
(clear_connection_cache): Deprecated.
(_sftp_connect, _sftp_connect_uncached): Deleted.
(SFTPTransport.__init__): Simplified.
(SFTPTransport._create_connection): New method. Copied from
_sftp_connect_uncached
(SFTPTransport._get_sftp): New method. Ensures that the connection
is established.
(SFTPTransport.clone): Deleted.
(SFTPTransport.has, SFTPTransport.get, SFTPTransport.readv,
SFTPTransport._put,
SFTPTransport._put_non_atomic_helper._open_and_write_file,
SFTPTransport._mkdir, SFTPTransport.append_file,
SFTPTransport.rename, SFTPTransport._rename_and_overwrite,
SFTPTransport.delete, SFTPTransport.rmdir, SFTPTransport.stat):
Use _get_sftp.

* bzrlib/tests/test_transport_implementations.py:
(TransportTests.test_connection_error): Simplified now that sftp
does not connection on construction.

* bzrlib/tests/test_sftp_transport.py:
(SFTPLockTests.test_sftp_locks): Delete test_multiple_connections.
(FakeSFTPTransport): Deleted.
(SFTPNonServerTest.test_parse_url_with_home_dir,
SFTPNonServerTest.test_relpath,
SSHVendorBadConnection.test_bad_connection_paramiko): Delete the
from_transport parameter as it's not needed anymore.
(SFTPLatencyKnob.test_latency_knob_slows_transport,
SFTPLatencyKnob.test_default): Force connection by issuing a
request.

Show diffs side-by-side

added added

removed removed

Lines of Context:
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
 
 
17
 
"""Tests for the smart wire/domain protocol.
18
 
 
19
 
This module contains tests for the domain-level smart requests and responses,
20
 
such as the 'Branch.lock_write' request. Many of these use specific disk
21
 
formats to exercise calls that only make sense for formats with specific
22
 
properties.
23
 
 
24
 
Tests for low-level protocol encoding are found in test_smart_transport.
25
 
"""
26
 
 
27
 
import bz2
28
 
from cStringIO import StringIO
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Tests for the smart wire/domain protococl."""
 
18
 
 
19
from StringIO import StringIO
 
20
import tempfile
29
21
import tarfile
30
22
 
31
 
from bzrlib import (
32
 
    bzrdir,
33
 
    errors,
34
 
    pack,
35
 
    smart,
36
 
    tests,
37
 
    urlutils,
38
 
    )
39
 
from bzrlib.branch import Branch, BranchReferenceFormat
 
23
from bzrlib import bzrdir, errors, smart, tests
 
24
from bzrlib.smart.request import SmartServerResponse
 
25
import bzrlib.smart.bzrdir
40
26
import bzrlib.smart.branch
41
 
import bzrlib.smart.bzrdir, bzrlib.smart.bzrdir as smart_dir
42
 
import bzrlib.smart.packrepository
43
27
import bzrlib.smart.repository
44
 
from bzrlib.smart.request import (
45
 
    FailedSmartServerResponse,
46
 
    SmartServerRequest,
47
 
    SmartServerResponse,
48
 
    SuccessfulSmartServerResponse,
49
 
    )
50
 
from bzrlib.tests import (
51
 
    split_suite_by_re,
52
 
    )
53
 
from bzrlib.transport import chroot, get_transport
54
 
from bzrlib.util import bencode
55
 
 
56
 
 
57
 
def load_tests(standard_tests, module, loader):
58
 
    """Multiply tests version and protocol consistency."""
59
 
    # FindRepository tests.
60
 
    bzrdir_mod = bzrlib.smart.bzrdir
61
 
    scenarios = [
62
 
        ("find_repository", {
63
 
            "_request_class":bzrdir_mod.SmartServerRequestFindRepositoryV1}),
64
 
        ("find_repositoryV2", {
65
 
            "_request_class":bzrdir_mod.SmartServerRequestFindRepositoryV2}),
66
 
        ("find_repositoryV3", {
67
 
            "_request_class":bzrdir_mod.SmartServerRequestFindRepositoryV3}),
68
 
        ]
69
 
    to_adapt, result = split_suite_by_re(standard_tests,
70
 
        "TestSmartServerRequestFindRepository")
71
 
    v2_only, v1_and_2 = split_suite_by_re(to_adapt,
72
 
        "_v2")
73
 
    tests.multiply_tests(v1_and_2, scenarios, result)
74
 
    # The first scenario is only applicable to v1 protocols, it is deleted
75
 
    # since.
76
 
    tests.multiply_tests(v2_only, scenarios[1:], result)
77
 
    return result
78
 
 
79
 
 
80
 
class TestCaseWithChrootedTransport(tests.TestCaseWithTransport):
81
 
 
82
 
    def setUp(self):
83
 
        tests.TestCaseWithTransport.setUp(self)
84
 
        self._chroot_server = None
85
 
 
86
 
    def get_transport(self, relpath=None):
87
 
        if self._chroot_server is None:
88
 
            backing_transport = tests.TestCaseWithTransport.get_transport(self)
89
 
            self._chroot_server = chroot.ChrootServer(backing_transport)
90
 
            self._chroot_server.setUp()
91
 
            self.addCleanup(self._chroot_server.tearDown)
92
 
        t = get_transport(self._chroot_server.get_url())
93
 
        if relpath is not None:
94
 
            t = t.clone(relpath)
95
 
        return t
96
28
 
97
29
 
98
30
class TestCaseWithSmartMedium(tests.TestCaseWithTransport):
103
35
        # the default or a parameterized class, but rather use the
104
36
        # TestCaseWithTransport infrastructure to set up a smart server and
105
37
        # transport.
106
 
        self.transport_server = self.make_transport_server
107
 
 
108
 
    def make_transport_server(self):
109
 
        return smart.server.SmartTCPServer_for_testing('-' + self.id())
 
38
        self.transport_server = smart.server.SmartTCPServer_for_testing
110
39
 
111
40
    def get_smart_medium(self):
112
41
        """Get a smart medium to use in tests."""
127
56
        self.assertNotEqual(None,
128
57
            SmartServerResponse(('ok', )))
129
58
 
130
 
    def test__str__(self):
131
 
        """SmartServerResponses can be stringified."""
132
 
        self.assertEqual(
133
 
            "<SuccessfulSmartServerResponse args=('args',) body='body'>",
134
 
            str(SuccessfulSmartServerResponse(('args',), 'body')))
135
 
        self.assertEqual(
136
 
            "<FailedSmartServerResponse args=('args',) body='body'>",
137
 
            str(FailedSmartServerResponse(('args',), 'body')))
138
 
 
139
 
 
140
 
class TestSmartServerRequest(tests.TestCaseWithMemoryTransport):
141
 
 
142
 
    def test_translate_client_path(self):
143
 
        transport = self.get_transport()
144
 
        request = SmartServerRequest(transport, 'foo/')
145
 
        self.assertEqual('./', request.translate_client_path('foo/'))
146
 
        self.assertRaises(
147
 
            errors.InvalidURLJoin, request.translate_client_path, 'foo/..')
148
 
        self.assertRaises(
149
 
            errors.PathNotChild, request.translate_client_path, '/')
150
 
        self.assertRaises(
151
 
            errors.PathNotChild, request.translate_client_path, 'bar/')
152
 
        self.assertEqual('./baz', request.translate_client_path('foo/baz'))
153
 
 
154
 
    def test_transport_from_client_path(self):
155
 
        transport = self.get_transport()
156
 
        request = SmartServerRequest(transport, 'foo/')
157
 
        self.assertEqual(
158
 
            transport.base,
159
 
            request.transport_from_client_path('foo/').base)
160
 
 
161
 
 
162
 
class TestSmartServerBzrDirRequestCloningMetaDir(
163
 
    tests.TestCaseWithMemoryTransport):
164
 
    """Tests for BzrDir.cloning_metadir."""
165
 
 
166
 
    def test_cloning_metadir(self):
167
 
        """When there is a bzrdir present, the call succeeds."""
168
 
        backing = self.get_transport()
169
 
        dir = self.make_bzrdir('.')
170
 
        local_result = dir.cloning_metadir()
171
 
        request_class = smart_dir.SmartServerBzrDirRequestCloningMetaDir
172
 
        request = request_class(backing)
173
 
        expected = SuccessfulSmartServerResponse(
174
 
            (local_result.network_name(),
175
 
            local_result.repository_format.network_name(),
176
 
            ('branch', local_result.get_branch_format().network_name())))
177
 
        self.assertEqual(expected, request.execute('', 'False'))
178
 
 
179
 
    def test_cloning_metadir_reference(self):
180
 
        """The request fails when bzrdir contains a branch reference."""
181
 
        backing = self.get_transport()
182
 
        referenced_branch = self.make_branch('referenced')
183
 
        dir = self.make_bzrdir('.')
184
 
        local_result = dir.cloning_metadir()
185
 
        reference = BranchReferenceFormat().initialize(dir, referenced_branch)
186
 
        reference_url = BranchReferenceFormat().get_reference(dir)
187
 
        # The server shouldn't try to follow the branch reference, so it's fine
188
 
        # if the referenced branch isn't reachable.
189
 
        backing.rename('referenced', 'moved')
190
 
        request_class = smart_dir.SmartServerBzrDirRequestCloningMetaDir
191
 
        request = request_class(backing)
192
 
        expected = FailedSmartServerResponse(('BranchReference',))
193
 
        self.assertEqual(expected, request.execute('', 'False'))
194
 
 
195
 
 
196
 
class TestSmartServerRequestCreateRepository(tests.TestCaseWithMemoryTransport):
197
 
    """Tests for BzrDir.create_repository."""
198
 
 
199
 
    def test_makes_repository(self):
200
 
        """When there is a bzrdir present, the call succeeds."""
201
 
        backing = self.get_transport()
202
 
        self.make_bzrdir('.')
203
 
        request_class = bzrlib.smart.bzrdir.SmartServerRequestCreateRepository
204
 
        request = request_class(backing)
205
 
        reference_bzrdir_format = bzrdir.format_registry.get('default')()
206
 
        reference_format = reference_bzrdir_format.repository_format
207
 
        network_name = reference_format.network_name()
208
 
        expected = SuccessfulSmartServerResponse(
209
 
            ('ok', 'no', 'no', 'no', network_name))
210
 
        self.assertEqual(expected, request.execute('', network_name, 'True'))
211
 
 
212
 
 
213
 
class TestSmartServerRequestFindRepository(tests.TestCaseWithMemoryTransport):
 
59
 
 
60
class TestSmartServerRequestFindRepository(tests.TestCaseWithTransport):
214
61
    """Tests for BzrDir.find_repository."""
215
62
 
216
63
    def test_no_repository(self):
217
64
        """When there is no repository to be found, ('norepository', ) is returned."""
218
65
        backing = self.get_transport()
219
 
        request = self._request_class(backing)
 
66
        request = smart.bzrdir.SmartServerRequestFindRepository(backing)
220
67
        self.make_bzrdir('.')
221
68
        self.assertEqual(SmartServerResponse(('norepository', )),
222
 
            request.execute(''))
 
69
            request.execute(backing.local_abspath('')))
223
70
 
224
71
    def test_nonshared_repository(self):
225
 
        # nonshared repositorys only allow 'find' to return a handle when the
226
 
        # path the repository is being searched on is the same as that that
 
72
        # nonshared repositorys only allow 'find' to return a handle when the 
 
73
        # path the repository is being searched on is the same as that that 
227
74
        # the repository is at.
228
75
        backing = self.get_transport()
229
 
        request = self._request_class(backing)
 
76
        request = smart.bzrdir.SmartServerRequestFindRepository(backing)
230
77
        result = self._make_repository_and_result()
231
 
        self.assertEqual(result, request.execute(''))
 
78
        self.assertEqual(result, request.execute(backing.local_abspath('')))
232
79
        self.make_bzrdir('subdir')
233
80
        self.assertEqual(SmartServerResponse(('norepository', )),
234
 
            request.execute('subdir'))
 
81
            request.execute(backing.local_abspath('subdir')))
235
82
 
236
83
    def _make_repository_and_result(self, shared=False, format=None):
237
84
        """Convenience function to setup a repository.
247
94
            subtrees = 'yes'
248
95
        else:
249
96
            subtrees = 'no'
250
 
        if (smart.bzrdir.SmartServerRequestFindRepositoryV3 ==
251
 
            self._request_class):
252
 
            return SuccessfulSmartServerResponse(
253
 
                ('ok', '', rich_root, subtrees, 'no',
254
 
                 repo._format.network_name()))
255
 
        elif (smart.bzrdir.SmartServerRequestFindRepositoryV2 ==
256
 
            self._request_class):
257
 
            # All tests so far are on formats, and for non-external
258
 
            # repositories.
259
 
            return SuccessfulSmartServerResponse(
260
 
                ('ok', '', rich_root, subtrees, 'no'))
261
 
        else:
262
 
            return SuccessfulSmartServerResponse(('ok', '', rich_root, subtrees))
 
97
        return SmartServerResponse(('ok', '', rich_root, subtrees))
263
98
 
264
99
    def test_shared_repository(self):
265
100
        """When there is a shared repository, we get 'ok', 'relpath-to-repo'."""
266
101
        backing = self.get_transport()
267
 
        request = self._request_class(backing)
 
102
        request = smart.bzrdir.SmartServerRequestFindRepository(backing)
268
103
        result = self._make_repository_and_result(shared=True)
269
 
        self.assertEqual(result, request.execute(''))
 
104
        self.assertEqual(result, request.execute(backing.local_abspath('')))
270
105
        self.make_bzrdir('subdir')
271
106
        result2 = SmartServerResponse(result.args[0:1] + ('..', ) + result.args[2:])
272
107
        self.assertEqual(result2,
273
 
            request.execute('subdir'))
 
108
            request.execute(backing.local_abspath('subdir')))
274
109
        self.make_bzrdir('subdir/deeper')
275
110
        result3 = SmartServerResponse(result.args[0:1] + ('../..', ) + result.args[2:])
276
111
        self.assertEqual(result3,
277
 
            request.execute('subdir/deeper'))
 
112
            request.execute(backing.local_abspath('subdir/deeper')))
278
113
 
279
114
    def test_rich_root_and_subtree_encoding(self):
280
115
        """Test for the format attributes for rich root and subtree support."""
281
116
        backing = self.get_transport()
282
 
        request = self._request_class(backing)
 
117
        request = smart.bzrdir.SmartServerRequestFindRepository(backing)
283
118
        result = self._make_repository_and_result(format='dirstate-with-subtree')
284
119
        # check the test will be valid
285
120
        self.assertEqual('yes', result.args[2])
286
121
        self.assertEqual('yes', result.args[3])
287
 
        self.assertEqual(result, request.execute(''))
288
 
 
289
 
    def test_supports_external_lookups_no_v2(self):
290
 
        """Test for the supports_external_lookups attribute."""
291
 
        backing = self.get_transport()
292
 
        request = self._request_class(backing)
293
 
        result = self._make_repository_and_result(format='dirstate-with-subtree')
294
 
        # check the test will be valid
295
 
        self.assertEqual('no', result.args[4])
296
 
        self.assertEqual(result, request.execute(''))
297
 
 
298
 
 
299
 
class TestSmartServerRequestInitializeBzrDir(tests.TestCaseWithMemoryTransport):
 
122
        self.assertEqual(result, request.execute(backing.local_abspath('')))
 
123
 
 
124
 
 
125
class TestSmartServerRequestInitializeBzrDir(tests.TestCaseWithTransport):
300
126
 
301
127
    def test_empty_dir(self):
302
128
        """Initializing an empty dir should succeed and do it."""
303
129
        backing = self.get_transport()
304
130
        request = smart.bzrdir.SmartServerRequestInitializeBzrDir(backing)
305
131
        self.assertEqual(SmartServerResponse(('ok', )),
306
 
            request.execute(''))
 
132
            request.execute(backing.local_abspath('.')))
307
133
        made_dir = bzrdir.BzrDir.open_from_transport(backing)
308
 
        # no branch, tree or repository is expected with the current
 
134
        # no branch, tree or repository is expected with the current 
309
135
        # default formart.
310
136
        self.assertRaises(errors.NoWorkingTree, made_dir.open_workingtree)
311
137
        self.assertRaises(errors.NotBranchError, made_dir.open_branch)
316
142
        backing = self.get_transport()
317
143
        request = smart.bzrdir.SmartServerRequestInitializeBzrDir(backing)
318
144
        self.assertRaises(errors.NoSuchFile,
319
 
            request.execute, 'subdir')
 
145
            request.execute, backing.local_abspath('subdir'))
320
146
 
321
147
    def test_initialized_dir(self):
322
148
        """Initializing an extant bzrdir should fail like the bzrdir api."""
324
150
        request = smart.bzrdir.SmartServerRequestInitializeBzrDir(backing)
325
151
        self.make_bzrdir('subdir')
326
152
        self.assertRaises(errors.FileExists,
327
 
            request.execute, 'subdir')
328
 
 
329
 
 
330
 
class TestSmartServerRequestOpenBranch(TestCaseWithChrootedTransport):
 
153
            request.execute, backing.local_abspath('subdir'))
 
154
 
 
155
 
 
156
class TestSmartServerRequestOpenBranch(tests.TestCaseWithTransport):
331
157
 
332
158
    def test_no_branch(self):
333
159
        """When there is no branch, ('nobranch', ) is returned."""
335
161
        request = smart.bzrdir.SmartServerRequestOpenBranch(backing)
336
162
        self.make_bzrdir('.')
337
163
        self.assertEqual(SmartServerResponse(('nobranch', )),
338
 
            request.execute(''))
 
164
            request.execute(backing.local_abspath('')))
339
165
 
340
166
    def test_branch(self):
341
167
        """When there is a branch, 'ok' is returned."""
343
169
        request = smart.bzrdir.SmartServerRequestOpenBranch(backing)
344
170
        self.make_branch('.')
345
171
        self.assertEqual(SmartServerResponse(('ok', '')),
346
 
            request.execute(''))
 
172
            request.execute(backing.local_abspath('')))
347
173
 
348
174
    def test_branch_reference(self):
349
175
        """When there is a branch reference, the reference URL is returned."""
351
177
        request = smart.bzrdir.SmartServerRequestOpenBranch(backing)
352
178
        branch = self.make_branch('branch')
353
179
        checkout = branch.create_checkout('reference',lightweight=True)
354
 
        reference_url = BranchReferenceFormat().get_reference(checkout.bzrdir)
 
180
        # TODO: once we have an API to probe for references of any sort, we
 
181
        # can use it here.
 
182
        reference_url = backing.abspath('branch') + '/'
355
183
        self.assertFileEqual(reference_url, 'reference/.bzr/branch/location')
356
184
        self.assertEqual(SmartServerResponse(('ok', reference_url)),
357
 
            request.execute('reference'))
358
 
 
359
 
 
360
 
class TestSmartServerRequestOpenBranchV2(TestCaseWithChrootedTransport):
361
 
 
362
 
    def test_no_branch(self):
363
 
        """When there is no branch, ('nobranch', ) is returned."""
364
 
        backing = self.get_transport()
365
 
        self.make_bzrdir('.')
366
 
        request = smart.bzrdir.SmartServerRequestOpenBranchV2(backing)
367
 
        self.assertEqual(SmartServerResponse(('nobranch', )),
368
 
            request.execute(''))
369
 
 
370
 
    def test_branch(self):
371
 
        """When there is a branch, 'ok' is returned."""
372
 
        backing = self.get_transport()
373
 
        expected = self.make_branch('.')._format.network_name()
374
 
        request = smart.bzrdir.SmartServerRequestOpenBranchV2(backing)
375
 
        self.assertEqual(SuccessfulSmartServerResponse(('branch', expected)),
376
 
            request.execute(''))
377
 
 
378
 
    def test_branch_reference(self):
379
 
        """When there is a branch reference, the reference URL is returned."""
380
 
        backing = self.get_transport()
381
 
        request = smart.bzrdir.SmartServerRequestOpenBranchV2(backing)
382
 
        branch = self.make_branch('branch')
383
 
        checkout = branch.create_checkout('reference',lightweight=True)
384
 
        reference_url = BranchReferenceFormat().get_reference(checkout.bzrdir)
385
 
        self.assertFileEqual(reference_url, 'reference/.bzr/branch/location')
386
 
        self.assertEqual(SuccessfulSmartServerResponse(('ref', reference_url)),
387
 
            request.execute('reference'))
388
 
 
389
 
    def test_stacked_branch(self):
390
 
        """Opening a stacked branch does not open the stacked-on branch."""
391
 
        trunk = self.make_branch('trunk')
392
 
        feature = self.make_branch('feature', format='1.9')
393
 
        feature.set_stacked_on_url(trunk.base)
394
 
        opened_branches = []
395
 
        Branch.hooks.install_named_hook('open', opened_branches.append, None)
396
 
        backing = self.get_transport()
397
 
        request = smart.bzrdir.SmartServerRequestOpenBranchV2(backing)
398
 
        request.setup_jail()
399
 
        try:
400
 
            response = request.execute('feature')
401
 
        finally:
402
 
            request.teardown_jail()
403
 
        expected_format = feature._format.network_name()
404
 
        self.assertEqual(
405
 
            SuccessfulSmartServerResponse(('branch', expected_format)),
406
 
            response)
407
 
        self.assertLength(1, opened_branches)
408
 
 
409
 
 
410
 
class TestSmartServerRequestRevisionHistory(tests.TestCaseWithMemoryTransport):
 
185
            request.execute(backing.local_abspath('reference')))
 
186
 
 
187
 
 
188
class TestSmartServerRequestRevisionHistory(tests.TestCaseWithTransport):
411
189
 
412
190
    def test_empty(self):
413
191
        """For an empty branch, the body is empty."""
415
193
        request = smart.branch.SmartServerRequestRevisionHistory(backing)
416
194
        self.make_branch('.')
417
195
        self.assertEqual(SmartServerResponse(('ok', ), ''),
418
 
            request.execute(''))
 
196
            request.execute(backing.local_abspath('')))
419
197
 
420
198
    def test_not_empty(self):
421
199
        """For a non-empty branch, the body is empty."""
429
207
        tree.unlock()
430
208
        self.assertEqual(
431
209
            SmartServerResponse(('ok', ), ('\x00'.join([r1, r2]))),
432
 
            request.execute(''))
433
 
 
434
 
 
435
 
class TestSmartServerBranchRequest(tests.TestCaseWithMemoryTransport):
 
210
            request.execute(backing.local_abspath('')))
 
211
 
 
212
 
 
213
class TestSmartServerBranchRequest(tests.TestCaseWithTransport):
436
214
 
437
215
    def test_no_branch(self):
438
216
        """When there is a bzrdir and no branch, NotBranchError is raised."""
440
218
        request = smart.branch.SmartServerBranchRequest(backing)
441
219
        self.make_bzrdir('.')
442
220
        self.assertRaises(errors.NotBranchError,
443
 
            request.execute, '')
 
221
            request.execute, backing.local_abspath(''))
444
222
 
445
223
    def test_branch_reference(self):
446
224
        """When there is a branch reference, NotBranchError is raised."""
449
227
        branch = self.make_branch('branch')
450
228
        checkout = branch.create_checkout('reference',lightweight=True)
451
229
        self.assertRaises(errors.NotBranchError,
452
 
            request.execute, 'checkout')
453
 
 
454
 
 
455
 
class TestSmartServerBranchRequestLastRevisionInfo(tests.TestCaseWithMemoryTransport):
 
230
            request.execute, backing.local_abspath('checkout'))
 
231
 
 
232
 
 
233
class TestSmartServerBranchRequestLastRevisionInfo(tests.TestCaseWithTransport):
456
234
 
457
235
    def test_empty(self):
458
236
        """For an empty branch, the result is ('ok', '0', 'null:')."""
460
238
        request = smart.branch.SmartServerBranchRequestLastRevisionInfo(backing)
461
239
        self.make_branch('.')
462
240
        self.assertEqual(SmartServerResponse(('ok', '0', 'null:')),
463
 
            request.execute(''))
 
241
            request.execute(backing.local_abspath('')))
464
242
 
465
243
    def test_not_empty(self):
466
244
        """For a non-empty branch, the result is ('ok', 'revno', 'revid')."""
475
253
        tree.unlock()
476
254
        self.assertEqual(
477
255
            SmartServerResponse(('ok', '2', rev_id_utf8)),
478
 
            request.execute(''))
479
 
 
480
 
 
481
 
class TestSmartServerBranchRequestGetConfigFile(tests.TestCaseWithMemoryTransport):
 
256
            request.execute(backing.local_abspath('')))
 
257
 
 
258
 
 
259
class TestSmartServerBranchRequestGetConfigFile(tests.TestCaseWithTransport):
482
260
 
483
261
    def test_default(self):
484
262
        """With no file, we get empty content."""
488
266
        # there should be no file by default
489
267
        content = ''
490
268
        self.assertEqual(SmartServerResponse(('ok', ), content),
491
 
            request.execute(''))
 
269
            request.execute(backing.local_abspath('')))
492
270
 
493
271
    def test_with_content(self):
494
272
        # SmartServerBranchGetConfigFile should return the content from
495
273
        # branch.control_files.get('branch.conf') for now - in the future it may
496
 
        # perform more complex processing.
 
274
        # perform more complex processing. 
497
275
        backing = self.get_transport()
498
276
        request = smart.branch.SmartServerBranchGetConfigFile(backing)
499
277
        branch = self.make_branch('.')
500
 
        branch._transport.put_bytes('branch.conf', 'foo bar baz')
 
278
        branch.control_files.put_utf8('branch.conf', 'foo bar baz')
501
279
        self.assertEqual(SmartServerResponse(('ok', ), 'foo bar baz'),
502
 
            request.execute(''))
503
 
 
504
 
 
505
 
class TestLockedBranch(tests.TestCaseWithMemoryTransport):
506
 
 
507
 
    def get_lock_tokens(self, branch):
508
 
        branch_token = branch.lock_write()
509
 
        repo_token = branch.repository.lock_write()
510
 
        branch.repository.unlock()
511
 
        return branch_token, repo_token
512
 
 
513
 
 
514
 
class TestSmartServerBranchRequestSetConfigOption(TestLockedBranch):
515
 
 
516
 
    def test_value_name(self):
517
 
        branch = self.make_branch('.')
518
 
        request = smart.branch.SmartServerBranchRequestSetConfigOption(
519
 
            branch.bzrdir.root_transport)
520
 
        branch_token, repo_token = self.get_lock_tokens(branch)
521
 
        config = branch._get_config()
522
 
        result = request.execute('', branch_token, repo_token, 'bar', 'foo',
523
 
            '')
524
 
        self.assertEqual(SuccessfulSmartServerResponse(()), result)
525
 
        self.assertEqual('bar', config.get_option('foo'))
526
 
 
527
 
    def test_value_name_section(self):
528
 
        branch = self.make_branch('.')
529
 
        request = smart.branch.SmartServerBranchRequestSetConfigOption(
530
 
            branch.bzrdir.root_transport)
531
 
        branch_token, repo_token = self.get_lock_tokens(branch)
532
 
        config = branch._get_config()
533
 
        result = request.execute('', branch_token, repo_token, 'bar', 'foo',
534
 
            'gam')
535
 
        self.assertEqual(SuccessfulSmartServerResponse(()), result)
536
 
        self.assertEqual('bar', config.get_option('foo', 'gam'))
537
 
 
538
 
 
539
 
class SetLastRevisionTestBase(TestLockedBranch):
540
 
    """Base test case for verbs that implement set_last_revision."""
541
 
 
542
 
    def setUp(self):
543
 
        tests.TestCaseWithMemoryTransport.setUp(self)
544
 
        backing_transport = self.get_transport()
545
 
        self.request = self.request_class(backing_transport)
546
 
        self.tree = self.make_branch_and_memory_tree('.')
547
 
 
548
 
    def lock_branch(self):
549
 
        return self.get_lock_tokens(self.tree.branch)
550
 
 
551
 
    def unlock_branch(self):
552
 
        self.tree.branch.unlock()
553
 
 
554
 
    def set_last_revision(self, revision_id, revno):
555
 
        branch_token, repo_token = self.lock_branch()
556
 
        response = self._set_last_revision(
557
 
            revision_id, revno, branch_token, repo_token)
558
 
        self.unlock_branch()
559
 
        return response
560
 
 
561
 
    def assertRequestSucceeds(self, revision_id, revno):
562
 
        response = self.set_last_revision(revision_id, revno)
563
 
        self.assertEqual(SuccessfulSmartServerResponse(('ok',)), response)
564
 
 
565
 
 
566
 
class TestSetLastRevisionVerbMixin(object):
567
 
    """Mixin test case for verbs that implement set_last_revision."""
568
 
 
569
 
    def test_set_null_to_null(self):
570
 
        """An empty branch can have its last revision set to 'null:'."""
571
 
        self.assertRequestSucceeds('null:', 0)
572
 
 
573
 
    def test_NoSuchRevision(self):
574
 
        """If the revision_id is not present, the verb returns NoSuchRevision.
575
 
        """
576
 
        revision_id = 'non-existent revision'
577
 
        self.assertEqual(
578
 
            FailedSmartServerResponse(('NoSuchRevision', revision_id)),
579
 
            self.set_last_revision(revision_id, 1))
580
 
 
581
 
    def make_tree_with_two_commits(self):
582
 
        self.tree.lock_write()
583
 
        self.tree.add('')
584
 
        rev_id_utf8 = u'\xc8'.encode('utf-8')
585
 
        r1 = self.tree.commit('1st commit', rev_id=rev_id_utf8)
586
 
        r2 = self.tree.commit('2nd commit', rev_id='rev-2')
587
 
        self.tree.unlock()
588
 
 
589
 
    def test_branch_last_revision_info_is_updated(self):
590
 
        """A branch's tip can be set to a revision that is present in its
591
 
        repository.
592
 
        """
593
 
        # Make a branch with an empty revision history, but two revisions in
594
 
        # its repository.
595
 
        self.make_tree_with_two_commits()
596
 
        rev_id_utf8 = u'\xc8'.encode('utf-8')
597
 
        self.tree.branch.set_revision_history([])
598
 
        self.assertEqual(
599
 
            (0, 'null:'), self.tree.branch.last_revision_info())
600
 
        # We can update the branch to a revision that is present in the
601
 
        # repository.
602
 
        self.assertRequestSucceeds(rev_id_utf8, 1)
603
 
        self.assertEqual(
604
 
            (1, rev_id_utf8), self.tree.branch.last_revision_info())
605
 
 
606
 
    def test_branch_last_revision_info_rewind(self):
607
 
        """A branch's tip can be set to a revision that is an ancestor of the
608
 
        current tip.
609
 
        """
610
 
        self.make_tree_with_two_commits()
611
 
        rev_id_utf8 = u'\xc8'.encode('utf-8')
612
 
        self.assertEqual(
613
 
            (2, 'rev-2'), self.tree.branch.last_revision_info())
614
 
        self.assertRequestSucceeds(rev_id_utf8, 1)
615
 
        self.assertEqual(
616
 
            (1, rev_id_utf8), self.tree.branch.last_revision_info())
617
 
 
618
 
    def test_TipChangeRejected(self):
619
 
        """If a pre_change_branch_tip hook raises TipChangeRejected, the verb
620
 
        returns TipChangeRejected.
621
 
        """
622
 
        rejection_message = u'rejection message\N{INTERROBANG}'
623
 
        def hook_that_rejects(params):
624
 
            raise errors.TipChangeRejected(rejection_message)
625
 
        Branch.hooks.install_named_hook(
626
 
            'pre_change_branch_tip', hook_that_rejects, None)
627
 
        self.assertEqual(
628
 
            FailedSmartServerResponse(
629
 
                ('TipChangeRejected', rejection_message.encode('utf-8'))),
630
 
            self.set_last_revision('null:', 0))
631
 
 
632
 
 
633
 
class TestSmartServerBranchRequestSetLastRevision(
634
 
        SetLastRevisionTestBase, TestSetLastRevisionVerbMixin):
635
 
    """Tests for Branch.set_last_revision verb."""
636
 
 
637
 
    request_class = smart.branch.SmartServerBranchRequestSetLastRevision
638
 
 
639
 
    def _set_last_revision(self, revision_id, revno, branch_token, repo_token):
640
 
        return self.request.execute(
641
 
            '', branch_token, repo_token, revision_id)
642
 
 
643
 
 
644
 
class TestSmartServerBranchRequestSetLastRevisionInfo(
645
 
        SetLastRevisionTestBase, TestSetLastRevisionVerbMixin):
646
 
    """Tests for Branch.set_last_revision_info verb."""
647
 
 
648
 
    request_class = smart.branch.SmartServerBranchRequestSetLastRevisionInfo
649
 
 
650
 
    def _set_last_revision(self, revision_id, revno, branch_token, repo_token):
651
 
        return self.request.execute(
652
 
            '', branch_token, repo_token, revno, revision_id)
653
 
 
654
 
    def test_NoSuchRevision(self):
655
 
        """Branch.set_last_revision_info does not have to return
656
 
        NoSuchRevision if the revision_id is absent.
657
 
        """
658
 
        raise tests.TestNotApplicable()
659
 
 
660
 
 
661
 
class TestSmartServerBranchRequestSetLastRevisionEx(
662
 
        SetLastRevisionTestBase, TestSetLastRevisionVerbMixin):
663
 
    """Tests for Branch.set_last_revision_ex verb."""
664
 
 
665
 
    request_class = smart.branch.SmartServerBranchRequestSetLastRevisionEx
666
 
 
667
 
    def _set_last_revision(self, revision_id, revno, branch_token, repo_token):
668
 
        return self.request.execute(
669
 
            '', branch_token, repo_token, revision_id, 0, 0)
670
 
 
671
 
    def assertRequestSucceeds(self, revision_id, revno):
672
 
        response = self.set_last_revision(revision_id, revno)
673
 
        self.assertEqual(
674
 
            SuccessfulSmartServerResponse(('ok', revno, revision_id)),
675
 
            response)
676
 
 
677
 
    def test_branch_last_revision_info_rewind(self):
678
 
        """A branch's tip can be set to a revision that is an ancestor of the
679
 
        current tip, but only if allow_overwrite_descendant is passed.
680
 
        """
681
 
        self.make_tree_with_two_commits()
682
 
        rev_id_utf8 = u'\xc8'.encode('utf-8')
683
 
        self.assertEqual(
684
 
            (2, 'rev-2'), self.tree.branch.last_revision_info())
685
 
        # If allow_overwrite_descendant flag is 0, then trying to set the tip
686
 
        # to an older revision ID has no effect.
687
 
        branch_token, repo_token = self.lock_branch()
688
 
        response = self.request.execute(
689
 
            '', branch_token, repo_token, rev_id_utf8, 0, 0)
690
 
        self.assertEqual(
691
 
            SuccessfulSmartServerResponse(('ok', 2, 'rev-2')),
692
 
            response)
693
 
        self.assertEqual(
694
 
            (2, 'rev-2'), self.tree.branch.last_revision_info())
695
 
 
696
 
        # If allow_overwrite_descendant flag is 1, then setting the tip to an
697
 
        # ancestor works.
698
 
        response = self.request.execute(
699
 
            '', branch_token, repo_token, rev_id_utf8, 0, 1)
700
 
        self.assertEqual(
701
 
            SuccessfulSmartServerResponse(('ok', 1, rev_id_utf8)),
702
 
            response)
703
 
        self.unlock_branch()
704
 
        self.assertEqual(
705
 
            (1, rev_id_utf8), self.tree.branch.last_revision_info())
706
 
 
707
 
    def make_branch_with_divergent_history(self):
708
 
        """Make a branch with divergent history in its repo.
709
 
 
710
 
        The branch's tip will be 'child-2', and the repo will also contain
711
 
        'child-1', which diverges from a common base revision.
712
 
        """
713
 
        self.tree.lock_write()
714
 
        self.tree.add('')
715
 
        r1 = self.tree.commit('1st commit')
716
 
        revno_1, revid_1 = self.tree.branch.last_revision_info()
717
 
        r2 = self.tree.commit('2nd commit', rev_id='child-1')
718
 
        # Undo the second commit
719
 
        self.tree.branch.set_last_revision_info(revno_1, revid_1)
720
 
        self.tree.set_parent_ids([revid_1])
721
 
        # Make a new second commit, child-2.  child-2 has diverged from
722
 
        # child-1.
723
 
        new_r2 = self.tree.commit('2nd commit', rev_id='child-2')
724
 
        self.tree.unlock()
725
 
 
726
 
    def test_not_allow_diverged(self):
727
 
        """If allow_diverged is not passed, then setting a divergent history
728
 
        returns a Diverged error.
729
 
        """
730
 
        self.make_branch_with_divergent_history()
731
 
        self.assertEqual(
732
 
            FailedSmartServerResponse(('Diverged',)),
733
 
            self.set_last_revision('child-1', 2))
734
 
        # The branch tip was not changed.
735
 
        self.assertEqual('child-2', self.tree.branch.last_revision())
736
 
 
737
 
    def test_allow_diverged(self):
738
 
        """If allow_diverged is passed, then setting a divergent history
739
 
        succeeds.
740
 
        """
741
 
        self.make_branch_with_divergent_history()
742
 
        branch_token, repo_token = self.lock_branch()
743
 
        response = self.request.execute(
744
 
            '', branch_token, repo_token, 'child-1', 1, 0)
745
 
        self.assertEqual(
746
 
            SuccessfulSmartServerResponse(('ok', 2, 'child-1')),
747
 
            response)
748
 
        self.unlock_branch()
749
 
        # The branch tip was changed.
750
 
        self.assertEqual('child-1', self.tree.branch.last_revision())
751
 
 
752
 
 
753
 
class TestSmartServerBranchRequestGetParent(tests.TestCaseWithMemoryTransport):
754
 
 
755
 
    def test_get_parent_none(self):
756
 
        base_branch = self.make_branch('base')
757
 
        request = smart.branch.SmartServerBranchGetParent(self.get_transport())
758
 
        response = request.execute('base')
759
 
        self.assertEquals(
760
 
            SuccessfulSmartServerResponse(('',)), response)
761
 
 
762
 
    def test_get_parent_something(self):
763
 
        base_branch = self.make_branch('base')
764
 
        base_branch.set_parent(self.get_url('foo'))
765
 
        request = smart.branch.SmartServerBranchGetParent(self.get_transport())
766
 
        response = request.execute('base')
767
 
        self.assertEquals(
768
 
            SuccessfulSmartServerResponse(("../foo",)),
769
 
            response)
770
 
 
771
 
 
772
 
class TestSmartServerBranchRequestGetTagsBytes(tests.TestCaseWithMemoryTransport):
773
 
# Only called when the branch format and tags match [yay factory
774
 
# methods] so only need to test straight forward cases.
775
 
 
776
 
    def test_get_bytes(self):
777
 
        base_branch = self.make_branch('base')
778
 
        request = smart.branch.SmartServerBranchGetTagsBytes(
779
 
            self.get_transport())
780
 
        response = request.execute('base')
781
 
        self.assertEquals(
782
 
            SuccessfulSmartServerResponse(('',)), response)
783
 
 
784
 
 
785
 
class TestSmartServerBranchRequestGetStackedOnURL(tests.TestCaseWithMemoryTransport):
786
 
 
787
 
    def test_get_stacked_on_url(self):
788
 
        base_branch = self.make_branch('base', format='1.6')
789
 
        stacked_branch = self.make_branch('stacked', format='1.6')
790
 
        # typically should be relative
791
 
        stacked_branch.set_stacked_on_url('../base')
792
 
        request = smart.branch.SmartServerBranchRequestGetStackedOnURL(
793
 
            self.get_transport())
794
 
        response = request.execute('stacked')
795
 
        self.assertEquals(
796
 
            SmartServerResponse(('ok', '../base')),
797
 
            response)
798
 
 
799
 
 
800
 
class TestSmartServerBranchRequestLockWrite(tests.TestCaseWithMemoryTransport):
801
 
 
802
 
    def setUp(self):
803
 
        tests.TestCaseWithMemoryTransport.setUp(self)
 
280
            request.execute(backing.local_abspath('')))
 
281
 
 
282
 
 
283
class TestSmartServerBranchRequestSetLastRevision(tests.TestCaseWithTransport):
 
284
 
 
285
    def test_empty(self):
 
286
        backing = self.get_transport()
 
287
        request = smart.branch.SmartServerBranchRequestSetLastRevision(backing)
 
288
        b = self.make_branch('.')
 
289
        branch_token = b.lock_write()
 
290
        repo_token = b.repository.lock_write()
 
291
        b.repository.unlock()
 
292
        try:
 
293
            self.assertEqual(SmartServerResponse(('ok',)),
 
294
                request.execute(
 
295
                    backing.local_abspath(''), branch_token, repo_token,
 
296
                    'null:'))
 
297
        finally:
 
298
            b.unlock()
 
299
 
 
300
    def test_not_present_revision_id(self):
 
301
        backing = self.get_transport()
 
302
        request = smart.branch.SmartServerBranchRequestSetLastRevision(backing)
 
303
        b = self.make_branch('.')
 
304
        branch_token = b.lock_write()
 
305
        repo_token = b.repository.lock_write()
 
306
        b.repository.unlock()
 
307
        try:
 
308
            revision_id = 'non-existent revision'
 
309
            self.assertEqual(
 
310
                SmartServerResponse(('NoSuchRevision', revision_id)),
 
311
                request.execute(
 
312
                    backing.local_abspath(''), branch_token, repo_token,
 
313
                    revision_id))
 
314
        finally:
 
315
            b.unlock()
 
316
 
 
317
    def test_revision_id_present(self):
 
318
        backing = self.get_transport()
 
319
        request = smart.branch.SmartServerBranchRequestSetLastRevision(backing)
 
320
        tree = self.make_branch_and_memory_tree('.')
 
321
        tree.lock_write()
 
322
        tree.add('')
 
323
        rev_id_utf8 = u'\xc8'.encode('utf-8')
 
324
        r1 = tree.commit('1st commit', rev_id=rev_id_utf8)
 
325
        r2 = tree.commit('2nd commit')
 
326
        tree.unlock()
 
327
        branch_token = tree.branch.lock_write()
 
328
        repo_token = tree.branch.repository.lock_write()
 
329
        tree.branch.repository.unlock()
 
330
        try:
 
331
            self.assertEqual(
 
332
                SmartServerResponse(('ok',)),
 
333
                request.execute(
 
334
                    backing.local_abspath(''), branch_token, repo_token,
 
335
                    rev_id_utf8))
 
336
            self.assertEqual([rev_id_utf8], tree.branch.revision_history())
 
337
        finally:
 
338
            tree.branch.unlock()
 
339
 
 
340
    def test_revision_id_present2(self):
 
341
        backing = self.get_transport()
 
342
        request = smart.branch.SmartServerBranchRequestSetLastRevision(backing)
 
343
        tree = self.make_branch_and_memory_tree('.')
 
344
        tree.lock_write()
 
345
        tree.add('')
 
346
        rev_id_utf8 = u'\xc8'.encode('utf-8')
 
347
        r1 = tree.commit('1st commit', rev_id=rev_id_utf8)
 
348
        r2 = tree.commit('2nd commit')
 
349
        tree.unlock()
 
350
        tree.branch.set_revision_history([])
 
351
        branch_token = tree.branch.lock_write()
 
352
        repo_token = tree.branch.repository.lock_write()
 
353
        tree.branch.repository.unlock()
 
354
        try:
 
355
            self.assertEqual(
 
356
                SmartServerResponse(('ok',)),
 
357
                request.execute(
 
358
                    backing.local_abspath(''), branch_token, repo_token,
 
359
                    rev_id_utf8))
 
360
            self.assertEqual([rev_id_utf8], tree.branch.revision_history())
 
361
        finally:
 
362
            tree.branch.unlock()
 
363
 
 
364
 
 
365
class TestSmartServerBranchRequestLockWrite(tests.TestCaseWithTransport):
 
366
 
 
367
    def setUp(self):
 
368
        tests.TestCaseWithTransport.setUp(self)
 
369
        self.reduceLockdirTimeout()
804
370
 
805
371
    def test_lock_write_on_unlocked_branch(self):
806
372
        backing = self.get_transport()
807
373
        request = smart.branch.SmartServerBranchRequestLockWrite(backing)
808
 
        branch = self.make_branch('.', format='knit')
 
374
        branch = self.make_branch('.')
809
375
        repository = branch.repository
810
 
        response = request.execute('')
 
376
        response = request.execute(backing.local_abspath(''))
811
377
        branch_nonce = branch.control_files._lock.peek().get('nonce')
812
378
        repository_nonce = repository.control_files._lock.peek().get('nonce')
813
379
        self.assertEqual(
825
391
        branch.lock_write()
826
392
        branch.leave_lock_in_place()
827
393
        branch.unlock()
828
 
        response = request.execute('')
 
394
        response = request.execute(backing.local_abspath(''))
829
395
        self.assertEqual(
830
396
            SmartServerResponse(('LockContention',)), response)
831
397
 
832
398
    def test_lock_write_with_tokens_on_locked_branch(self):
833
399
        backing = self.get_transport()
834
400
        request = smart.branch.SmartServerBranchRequestLockWrite(backing)
835
 
        branch = self.make_branch('.', format='knit')
 
401
        branch = self.make_branch('.')
836
402
        branch_token = branch.lock_write()
837
403
        repo_token = branch.repository.lock_write()
838
404
        branch.repository.unlock()
839
405
        branch.leave_lock_in_place()
840
406
        branch.repository.leave_lock_in_place()
841
407
        branch.unlock()
842
 
        response = request.execute('',
 
408
        response = request.execute(backing.local_abspath(''),
843
409
                                   branch_token, repo_token)
844
410
        self.assertEqual(
845
411
            SmartServerResponse(('ok', branch_token, repo_token)), response)
847
413
    def test_lock_write_with_mismatched_tokens_on_locked_branch(self):
848
414
        backing = self.get_transport()
849
415
        request = smart.branch.SmartServerBranchRequestLockWrite(backing)
850
 
        branch = self.make_branch('.', format='knit')
 
416
        branch = self.make_branch('.')
851
417
        branch_token = branch.lock_write()
852
418
        repo_token = branch.repository.lock_write()
853
419
        branch.repository.unlock()
854
420
        branch.leave_lock_in_place()
855
421
        branch.repository.leave_lock_in_place()
856
422
        branch.unlock()
857
 
        response = request.execute('',
 
423
        response = request.execute(backing.local_abspath(''),
858
424
                                   branch_token+'xxx', repo_token)
859
425
        self.assertEqual(
860
426
            SmartServerResponse(('TokenMismatch',)), response)
862
428
    def test_lock_write_on_locked_repo(self):
863
429
        backing = self.get_transport()
864
430
        request = smart.branch.SmartServerBranchRequestLockWrite(backing)
865
 
        branch = self.make_branch('.', format='knit')
 
431
        branch = self.make_branch('.')
866
432
        branch.repository.lock_write()
867
433
        branch.repository.leave_lock_in_place()
868
434
        branch.repository.unlock()
869
 
        response = request.execute('')
 
435
        response = request.execute(backing.local_abspath(''))
870
436
        self.assertEqual(
871
437
            SmartServerResponse(('LockContention',)), response)
872
438
 
874
440
        backing = self.get_readonly_transport()
875
441
        request = smart.branch.SmartServerBranchRequestLockWrite(backing)
876
442
        branch = self.make_branch('.')
877
 
        root = self.get_transport().clone('/')
878
 
        path = urlutils.relative_url(root.base, self.get_transport().base)
879
 
        response = request.execute(path)
880
 
        error_name, lock_str, why_str = response.args
881
 
        self.assertFalse(response.is_successful())
882
 
        self.assertEqual('LockFailed', error_name)
883
 
 
884
 
 
885
 
class TestSmartServerBranchRequestUnlock(tests.TestCaseWithMemoryTransport):
 
443
        response = request.execute('')
 
444
        self.assertEqual(
 
445
            SmartServerResponse(('UnlockableTransport',)), response)
 
446
 
 
447
 
 
448
class TestSmartServerBranchRequestUnlock(tests.TestCaseWithTransport):
886
449
 
887
450
    def setUp(self):
888
 
        tests.TestCaseWithMemoryTransport.setUp(self)
 
451
        tests.TestCaseWithTransport.setUp(self)
 
452
        self.reduceLockdirTimeout()
889
453
 
890
454
    def test_unlock_on_locked_branch_and_repo(self):
891
455
        backing = self.get_transport()
892
456
        request = smart.branch.SmartServerBranchRequestUnlock(backing)
893
 
        branch = self.make_branch('.', format='knit')
 
457
        branch = self.make_branch('.')
894
458
        # Lock the branch
895
459
        branch_token = branch.lock_write()
896
460
        repo_token = branch.repository.lock_write()
900
464
        branch.leave_lock_in_place()
901
465
        branch.repository.leave_lock_in_place()
902
466
        branch.unlock()
903
 
        response = request.execute('',
 
467
        response = request.execute(backing.local_abspath(''),
904
468
                                   branch_token, repo_token)
905
469
        self.assertEqual(
906
470
            SmartServerResponse(('ok',)), response)
913
477
    def test_unlock_on_unlocked_branch_unlocked_repo(self):
914
478
        backing = self.get_transport()
915
479
        request = smart.branch.SmartServerBranchRequestUnlock(backing)
916
 
        branch = self.make_branch('.', format='knit')
 
480
        branch = self.make_branch('.')
917
481
        response = request.execute(
918
 
            '', 'branch token', 'repo token')
 
482
            backing.local_abspath(''), 'branch token', 'repo token')
919
483
        self.assertEqual(
920
484
            SmartServerResponse(('TokenMismatch',)), response)
921
485
 
922
486
    def test_unlock_on_unlocked_branch_locked_repo(self):
923
487
        backing = self.get_transport()
924
488
        request = smart.branch.SmartServerBranchRequestUnlock(backing)
925
 
        branch = self.make_branch('.', format='knit')
 
489
        branch = self.make_branch('.')
926
490
        # Lock the repository.
927
491
        repo_token = branch.repository.lock_write()
928
492
        branch.repository.leave_lock_in_place()
930
494
        # Issue branch lock_write request on the unlocked branch (with locked
931
495
        # repo).
932
496
        response = request.execute(
933
 
            '', 'branch token', repo_token)
 
497
            backing.local_abspath(''), 'branch token', repo_token)
934
498
        self.assertEqual(
935
499
            SmartServerResponse(('TokenMismatch',)), response)
936
500
 
937
501
 
938
 
class TestSmartServerRepositoryRequest(tests.TestCaseWithMemoryTransport):
 
502
class TestSmartServerRepositoryRequest(tests.TestCaseWithTransport):
939
503
 
940
504
    def test_no_repository(self):
941
505
        """Raise NoRepositoryPresent when there is a bzrdir and no repo."""
948
512
        self.make_repository('.', shared=True)
949
513
        self.make_bzrdir('subdir')
950
514
        self.assertRaises(errors.NoRepositoryPresent,
951
 
            request.execute, 'subdir')
952
 
 
953
 
 
954
 
class TestSmartServerRepositoryGetParentMap(tests.TestCaseWithMemoryTransport):
955
 
 
956
 
    def test_trivial_bzipped(self):
957
 
        # This tests that the wire encoding is actually bzipped
958
 
        backing = self.get_transport()
959
 
        request = smart.repository.SmartServerRepositoryGetParentMap(backing)
960
 
        tree = self.make_branch_and_memory_tree('.')
961
 
 
962
 
        self.assertEqual(None,
963
 
            request.execute('', 'missing-id'))
964
 
        # Note that it returns a body that is bzipped.
965
 
        self.assertEqual(
966
 
            SuccessfulSmartServerResponse(('ok', ), bz2.compress('')),
967
 
            request.do_body('\n\n0\n'))
968
 
 
969
 
    def test_trivial_include_missing(self):
970
 
        backing = self.get_transport()
971
 
        request = smart.repository.SmartServerRepositoryGetParentMap(backing)
972
 
        tree = self.make_branch_and_memory_tree('.')
973
 
 
974
 
        self.assertEqual(None,
975
 
            request.execute('', 'missing-id', 'include-missing:'))
976
 
        self.assertEqual(
977
 
            SuccessfulSmartServerResponse(('ok', ),
978
 
                bz2.compress('missing:missing-id')),
979
 
            request.do_body('\n\n0\n'))
980
 
 
981
 
 
982
 
class TestSmartServerRepositoryGetRevisionGraph(tests.TestCaseWithMemoryTransport):
 
515
            request.execute, backing.local_abspath('subdir'))
 
516
 
 
517
 
 
518
class TestSmartServerRepositoryGetRevisionGraph(tests.TestCaseWithTransport):
983
519
 
984
520
    def test_none_argument(self):
985
521
        backing = self.get_transport()
994
530
        # the lines of revision_id->revision_parent_list has no guaranteed
995
531
        # order coming out of a dict, so sort both our test and response
996
532
        lines = sorted([' '.join([r2, r1]), r1])
997
 
        response = request.execute('', '')
 
533
        response = request.execute(backing.local_abspath(''), '')
998
534
        response.body = '\n'.join(sorted(response.body.split('\n')))
999
535
 
1000
536
        self.assertEqual(
1012
548
        tree.unlock()
1013
549
 
1014
550
        self.assertEqual(SmartServerResponse(('ok', ), rev_id_utf8),
1015
 
            request.execute('', rev_id_utf8))
1016
 
 
 
551
            request.execute(backing.local_abspath(''), rev_id_utf8))
 
552
    
1017
553
    def test_no_such_revision(self):
1018
554
        backing = self.get_transport()
1019
555
        request = smart.repository.SmartServerRepositoryGetRevisionGraph(backing)
1026
562
        # Note that it still returns body (of zero bytes).
1027
563
        self.assertEqual(
1028
564
            SmartServerResponse(('nosuchrevision', 'missingrevision', ), ''),
1029
 
            request.execute('', 'missingrevision'))
1030
 
 
1031
 
 
1032
 
class TestSmartServerRepositoryGetStream(tests.TestCaseWithMemoryTransport):
1033
 
 
1034
 
    def make_two_commit_repo(self):
1035
 
        tree = self.make_branch_and_memory_tree('.')
1036
 
        tree.lock_write()
1037
 
        tree.add('')
1038
 
        r1 = tree.commit('1st commit')
1039
 
        r2 = tree.commit('2nd commit', rev_id=u'\xc8'.encode('utf-8'))
1040
 
        tree.unlock()
1041
 
        repo = tree.branch.repository
1042
 
        return repo, r1, r2
1043
 
 
1044
 
    def test_ancestry_of(self):
1045
 
        """The search argument may be a 'ancestry-of' some heads'."""
1046
 
        backing = self.get_transport()
1047
 
        request = smart.repository.SmartServerRepositoryGetStream(backing)
1048
 
        repo, r1, r2 = self.make_two_commit_repo()
1049
 
        fetch_spec = ['ancestry-of', r2]
1050
 
        lines = '\n'.join(fetch_spec)
1051
 
        request.execute('', repo._format.network_name())
1052
 
        response = request.do_body(lines)
1053
 
        self.assertEqual(('ok',), response.args)
1054
 
        stream_bytes = ''.join(response.body_stream)
1055
 
        self.assertStartsWith(stream_bytes, 'Bazaar pack format 1')
1056
 
 
1057
 
    def test_search(self):
1058
 
        """The search argument may be a 'search' of some explicit keys."""
1059
 
        backing = self.get_transport()
1060
 
        request = smart.repository.SmartServerRepositoryGetStream(backing)
1061
 
        repo, r1, r2 = self.make_two_commit_repo()
1062
 
        fetch_spec = ['search', '%s %s' % (r1, r2), 'null:', '2']
1063
 
        lines = '\n'.join(fetch_spec)
1064
 
        request.execute('', repo._format.network_name())
1065
 
        response = request.do_body(lines)
1066
 
        self.assertEqual(('ok',), response.args)
1067
 
        stream_bytes = ''.join(response.body_stream)
1068
 
        self.assertStartsWith(stream_bytes, 'Bazaar pack format 1')
1069
 
 
1070
 
 
1071
 
class TestSmartServerRequestHasRevision(tests.TestCaseWithMemoryTransport):
 
565
            request.execute(backing.local_abspath(''), 'missingrevision'))
 
566
 
 
567
 
 
568
class TestSmartServerRequestHasRevision(tests.TestCaseWithTransport):
1072
569
 
1073
570
    def test_missing_revision(self):
1074
571
        """For a missing revision, ('no', ) is returned."""
1076
573
        request = smart.repository.SmartServerRequestHasRevision(backing)
1077
574
        self.make_repository('.')
1078
575
        self.assertEqual(SmartServerResponse(('no', )),
1079
 
            request.execute('', 'revid'))
 
576
            request.execute(backing.local_abspath(''), 'revid'))
1080
577
 
1081
578
    def test_present_revision(self):
1082
579
        """For a present revision, ('yes', ) is returned."""
1090
587
        tree.unlock()
1091
588
        self.assertTrue(tree.branch.repository.has_revision(rev_id_utf8))
1092
589
        self.assertEqual(SmartServerResponse(('yes', )),
1093
 
            request.execute('', rev_id_utf8))
1094
 
 
1095
 
 
1096
 
class TestSmartServerRepositoryGatherStats(tests.TestCaseWithMemoryTransport):
 
590
            request.execute(backing.local_abspath(''), rev_id_utf8))
 
591
 
 
592
 
 
593
class TestSmartServerRepositoryGatherStats(tests.TestCaseWithTransport):
1097
594
 
1098
595
    def test_empty_revid(self):
1099
596
        """With an empty revid, we get only size an number and revisions"""
1101
598
        request = smart.repository.SmartServerRepositoryGatherStats(backing)
1102
599
        repository = self.make_repository('.')
1103
600
        stats = repository.gather_stats()
1104
 
        expected_body = 'revisions: 0\n'
 
601
        size = stats['size']
 
602
        expected_body = 'revisions: 0\nsize: %d\n' % size
1105
603
        self.assertEqual(SmartServerResponse(('ok', ), expected_body),
1106
 
                         request.execute('', '', 'no'))
 
604
                         request.execute(backing.local_abspath(''), '', 'no'))
1107
605
 
1108
606
    def test_revid_with_committers(self):
1109
607
        """For a revid we get more infos."""
1120
618
        tree.unlock()
1121
619
 
1122
620
        stats = tree.branch.repository.gather_stats()
 
621
        size = stats['size']
1123
622
        expected_body = ('firstrev: 123456.200 3600\n'
1124
623
                         'latestrev: 654321.400 0\n'
1125
 
                         'revisions: 2\n')
 
624
                         'revisions: 2\n'
 
625
                         'size: %d\n' % size)
1126
626
        self.assertEqual(SmartServerResponse(('ok', ), expected_body),
1127
 
                         request.execute('',
 
627
                         request.execute(backing.local_abspath(''),
1128
628
                                         rev_id_utf8, 'no'))
1129
629
 
1130
630
    def test_not_empty_repository_with_committers(self):
1143
643
        tree.unlock()
1144
644
        stats = tree.branch.repository.gather_stats()
1145
645
 
 
646
        size = stats['size']
1146
647
        expected_body = ('committers: 2\n'
1147
648
                         'firstrev: 123456.200 3600\n'
1148
649
                         'latestrev: 654321.400 0\n'
1149
 
                         'revisions: 2\n')
 
650
                         'revisions: 2\n'
 
651
                         'size: %d\n' % size)
1150
652
        self.assertEqual(SmartServerResponse(('ok', ), expected_body),
1151
 
                         request.execute('',
 
653
                         request.execute(backing.local_abspath(''),
1152
654
                                         rev_id_utf8, 'yes'))
1153
655
 
1154
656
 
1155
 
class TestSmartServerRepositoryIsShared(tests.TestCaseWithMemoryTransport):
 
657
class TestSmartServerRepositoryIsShared(tests.TestCaseWithTransport):
1156
658
 
1157
659
    def test_is_shared(self):
1158
660
        """For a shared repository, ('yes', ) is returned."""
1160
662
        request = smart.repository.SmartServerRepositoryIsShared(backing)
1161
663
        self.make_repository('.', shared=True)
1162
664
        self.assertEqual(SmartServerResponse(('yes', )),
1163
 
            request.execute('', ))
 
665
            request.execute(backing.local_abspath(''), ))
1164
666
 
1165
667
    def test_is_not_shared(self):
1166
668
        """For a shared repository, ('no', ) is returned."""
1168
670
        request = smart.repository.SmartServerRepositoryIsShared(backing)
1169
671
        self.make_repository('.', shared=False)
1170
672
        self.assertEqual(SmartServerResponse(('no', )),
1171
 
            request.execute('', ))
1172
 
 
1173
 
 
1174
 
class TestSmartServerRepositoryLockWrite(tests.TestCaseWithMemoryTransport):
 
673
            request.execute(backing.local_abspath(''), ))
 
674
 
 
675
 
 
676
class TestSmartServerRepositoryLockWrite(tests.TestCaseWithTransport):
 
677
 
 
678
    def setUp(self):
 
679
        tests.TestCaseWithTransport.setUp(self)
 
680
        self.reduceLockdirTimeout()
1175
681
 
1176
682
    def test_lock_write_on_unlocked_repo(self):
1177
683
        backing = self.get_transport()
1178
684
        request = smart.repository.SmartServerRepositoryLockWrite(backing)
1179
 
        repository = self.make_repository('.', format='knit')
1180
 
        response = request.execute('')
 
685
        repository = self.make_repository('.')
 
686
        response = request.execute(backing.local_abspath(''))
1181
687
        nonce = repository.control_files._lock.peek().get('nonce')
1182
688
        self.assertEqual(SmartServerResponse(('ok', nonce)), response)
1183
689
        # The repository is now locked.  Verify that with a new repository
1188
694
    def test_lock_write_on_locked_repo(self):
1189
695
        backing = self.get_transport()
1190
696
        request = smart.repository.SmartServerRepositoryLockWrite(backing)
1191
 
        repository = self.make_repository('.', format='knit')
 
697
        repository = self.make_repository('.')
1192
698
        repository.lock_write()
1193
699
        repository.leave_lock_in_place()
1194
700
        repository.unlock()
1195
 
        response = request.execute('')
 
701
        response = request.execute(backing.local_abspath(''))
1196
702
        self.assertEqual(
1197
703
            SmartServerResponse(('LockContention',)), response)
1198
704
 
1199
705
    def test_lock_write_on_readonly_transport(self):
1200
706
        backing = self.get_readonly_transport()
1201
707
        request = smart.repository.SmartServerRepositoryLockWrite(backing)
1202
 
        repository = self.make_repository('.', format='knit')
1203
 
        response = request.execute('')
1204
 
        self.assertFalse(response.is_successful())
1205
 
        self.assertEqual('LockFailed', response.args[0])
1206
 
 
1207
 
 
1208
 
class TestInsertStreamBase(tests.TestCaseWithMemoryTransport):
1209
 
 
1210
 
    def make_empty_byte_stream(self, repo):
1211
 
        byte_stream = smart.repository._stream_to_byte_stream([], repo._format)
1212
 
        return ''.join(byte_stream)
1213
 
 
1214
 
 
1215
 
class TestSmartServerRepositoryInsertStream(TestInsertStreamBase):
1216
 
 
1217
 
    def test_insert_stream_empty(self):
1218
 
        backing = self.get_transport()
1219
 
        request = smart.repository.SmartServerRepositoryInsertStream(backing)
1220
708
        repository = self.make_repository('.')
1221
 
        response = request.execute('', '')
1222
 
        self.assertEqual(None, response)
1223
 
        response = request.do_chunk(self.make_empty_byte_stream(repository))
1224
 
        self.assertEqual(None, response)
1225
 
        response = request.do_end()
1226
 
        self.assertEqual(SmartServerResponse(('ok', )), response)
1227
 
        
1228
 
 
1229
 
class TestSmartServerRepositoryInsertStreamLocked(TestInsertStreamBase):
1230
 
 
1231
 
    def test_insert_stream_empty(self):
1232
 
        backing = self.get_transport()
1233
 
        request = smart.repository.SmartServerRepositoryInsertStreamLocked(
1234
 
            backing)
1235
 
        repository = self.make_repository('.', format='knit')
1236
 
        lock_token = repository.lock_write()
1237
 
        response = request.execute('', '', lock_token)
1238
 
        self.assertEqual(None, response)
1239
 
        response = request.do_chunk(self.make_empty_byte_stream(repository))
1240
 
        self.assertEqual(None, response)
1241
 
        response = request.do_end()
1242
 
        self.assertEqual(SmartServerResponse(('ok', )), response)
1243
 
        repository.unlock()
1244
 
 
1245
 
    def test_insert_stream_with_wrong_lock_token(self):
1246
 
        backing = self.get_transport()
1247
 
        request = smart.repository.SmartServerRepositoryInsertStreamLocked(
1248
 
            backing)
1249
 
        repository = self.make_repository('.', format='knit')
1250
 
        lock_token = repository.lock_write()
1251
 
        self.assertRaises(
1252
 
            errors.TokenMismatch, request.execute, '', '', 'wrong-token')
1253
 
        repository.unlock()
1254
 
 
1255
 
 
1256
 
class TestSmartServerRepositoryUnlock(tests.TestCaseWithMemoryTransport):
 
709
        response = request.execute('')
 
710
        self.assertEqual(
 
711
            SmartServerResponse(('UnlockableTransport',)), response)
 
712
 
 
713
 
 
714
class TestSmartServerRepositoryUnlock(tests.TestCaseWithTransport):
1257
715
 
1258
716
    def setUp(self):
1259
 
        tests.TestCaseWithMemoryTransport.setUp(self)
 
717
        tests.TestCaseWithTransport.setUp(self)
 
718
        self.reduceLockdirTimeout()
1260
719
 
1261
720
    def test_unlock_on_locked_repo(self):
1262
721
        backing = self.get_transport()
1263
722
        request = smart.repository.SmartServerRepositoryUnlock(backing)
1264
 
        repository = self.make_repository('.', format='knit')
 
723
        repository = self.make_repository('.')
1265
724
        token = repository.lock_write()
1266
725
        repository.leave_lock_in_place()
1267
726
        repository.unlock()
1268
 
        response = request.execute('', token)
 
727
        response = request.execute(backing.local_abspath(''), token)
1269
728
        self.assertEqual(
1270
729
            SmartServerResponse(('ok',)), response)
1271
730
        # The repository is now unlocked.  Verify that with a new repository
1277
736
    def test_unlock_on_unlocked_repo(self):
1278
737
        backing = self.get_transport()
1279
738
        request = smart.repository.SmartServerRepositoryUnlock(backing)
1280
 
        repository = self.make_repository('.', format='knit')
1281
 
        response = request.execute('', 'some token')
 
739
        repository = self.make_repository('.')
 
740
        response = request.execute(backing.local_abspath(''), 'some token')
1282
741
        self.assertEqual(
1283
742
            SmartServerResponse(('TokenMismatch',)), response)
1284
743
 
1285
744
 
1286
 
class TestSmartServerIsReadonly(tests.TestCaseWithMemoryTransport):
 
745
class TestSmartServerRepositoryTarball(tests.TestCaseWithTransport):
 
746
 
 
747
    def test_repository_tarball(self):
 
748
        backing = self.get_transport()
 
749
        request = smart.repository.SmartServerRepositoryTarball(backing)
 
750
        repository = self.make_repository('.')
 
751
        # make some extraneous junk in the repository directory which should
 
752
        # not be copied
 
753
        self.build_tree(['.bzr/repository/extra-junk'])
 
754
        response = request.execute(backing.local_abspath(''), 'bz2')
 
755
        self.assertEqual(('ok',), response.args)
 
756
        # body should be a tbz2
 
757
        body_file = StringIO(response.body)
 
758
        body_tar = tarfile.open('body_tar.tbz2', fileobj=body_file,
 
759
            mode='r|bz2')
 
760
        # let's make sure there are some key repository components inside it.
 
761
        # the tarfile returns directories with trailing slashes...
 
762
        names = set([n.rstrip('/') for n in body_tar.getnames()])
 
763
        self.assertTrue('.bzr/repository/lock' in names)
 
764
        self.assertTrue('.bzr/repository/format' in names)
 
765
        self.assertTrue('.bzr/repository/extra-junk' not in names,
 
766
            "extraneous file present in tar file")
 
767
 
 
768
 
 
769
class TestSmartServerIsReadonly(tests.TestCaseWithTransport):
1287
770
 
1288
771
    def test_is_readonly_no(self):
1289
772
        backing = self.get_transport()
1300
783
            SmartServerResponse(('yes',)), response)
1301
784
 
1302
785
 
1303
 
class TestSmartServerRepositorySetMakeWorkingTrees(tests.TestCaseWithMemoryTransport):
1304
 
 
1305
 
    def test_set_false(self):
1306
 
        backing = self.get_transport()
1307
 
        repo = self.make_repository('.', shared=True)
1308
 
        repo.set_make_working_trees(True)
1309
 
        request_class = smart.repository.SmartServerRepositorySetMakeWorkingTrees
1310
 
        request = request_class(backing)
1311
 
        self.assertEqual(SuccessfulSmartServerResponse(('ok',)),
1312
 
            request.execute('', 'False'))
1313
 
        repo = repo.bzrdir.open_repository()
1314
 
        self.assertFalse(repo.make_working_trees())
1315
 
 
1316
 
    def test_set_true(self):
1317
 
        backing = self.get_transport()
1318
 
        repo = self.make_repository('.', shared=True)
1319
 
        repo.set_make_working_trees(False)
1320
 
        request_class = smart.repository.SmartServerRepositorySetMakeWorkingTrees
1321
 
        request = request_class(backing)
1322
 
        self.assertEqual(SuccessfulSmartServerResponse(('ok',)),
1323
 
            request.execute('', 'True'))
1324
 
        repo = repo.bzrdir.open_repository()
1325
 
        self.assertTrue(repo.make_working_trees())
1326
 
 
1327
 
 
1328
 
class TestSmartServerPackRepositoryAutopack(tests.TestCaseWithTransport):
1329
 
 
1330
 
    def make_repo_needing_autopacking(self, path='.'):
1331
 
        # Make a repo in need of autopacking.
1332
 
        tree = self.make_branch_and_tree('.', format='pack-0.92')
1333
 
        repo = tree.branch.repository
1334
 
        # monkey-patch the pack collection to disable autopacking
1335
 
        repo._pack_collection._max_pack_count = lambda count: count
1336
 
        for x in range(10):
1337
 
            tree.commit('commit %s' % x)
1338
 
        self.assertEqual(10, len(repo._pack_collection.names()))
1339
 
        del repo._pack_collection._max_pack_count
1340
 
        return repo
1341
 
 
1342
 
    def test_autopack_needed(self):
1343
 
        repo = self.make_repo_needing_autopacking()
1344
 
        repo.lock_write()
1345
 
        self.addCleanup(repo.unlock)
1346
 
        backing = self.get_transport()
1347
 
        request = smart.packrepository.SmartServerPackRepositoryAutopack(
1348
 
            backing)
1349
 
        response = request.execute('')
1350
 
        self.assertEqual(SmartServerResponse(('ok',)), response)
1351
 
        repo._pack_collection.reload_pack_names()
1352
 
        self.assertEqual(1, len(repo._pack_collection.names()))
1353
 
 
1354
 
    def test_autopack_not_needed(self):
1355
 
        tree = self.make_branch_and_tree('.', format='pack-0.92')
1356
 
        repo = tree.branch.repository
1357
 
        repo.lock_write()
1358
 
        self.addCleanup(repo.unlock)
1359
 
        for x in range(9):
1360
 
            tree.commit('commit %s' % x)
1361
 
        backing = self.get_transport()
1362
 
        request = smart.packrepository.SmartServerPackRepositoryAutopack(
1363
 
            backing)
1364
 
        response = request.execute('')
1365
 
        self.assertEqual(SmartServerResponse(('ok',)), response)
1366
 
        repo._pack_collection.reload_pack_names()
1367
 
        self.assertEqual(9, len(repo._pack_collection.names()))
1368
 
 
1369
 
    def test_autopack_on_nonpack_format(self):
1370
 
        """A request to autopack a non-pack repo is a no-op."""
1371
 
        repo = self.make_repository('.', format='knit')
1372
 
        backing = self.get_transport()
1373
 
        request = smart.packrepository.SmartServerPackRepositoryAutopack(
1374
 
            backing)
1375
 
        response = request.execute('')
1376
 
        self.assertEqual(SmartServerResponse(('ok',)), response)
1377
 
 
1378
 
 
1379
786
class TestHandlers(tests.TestCase):
1380
787
    """Tests for the request.request_handlers object."""
1381
788
 
1382
 
    def test_all_registrations_exist(self):
1383
 
        """All registered request_handlers can be found."""
1384
 
        # If there's a typo in a register_lazy call, this loop will fail with
1385
 
        # an AttributeError.
1386
 
        for key, item in smart.request.request_handlers.iteritems():
1387
 
            pass
1388
 
 
1389
789
    def test_registered_methods(self):
1390
790
        """Test that known methods are registered to the correct object."""
1391
791
        self.assertEqual(
1392
792
            smart.request.request_handlers.get('Branch.get_config_file'),
1393
793
            smart.branch.SmartServerBranchGetConfigFile)
1394
794
        self.assertEqual(
1395
 
            smart.request.request_handlers.get('Branch.get_parent'),
1396
 
            smart.branch.SmartServerBranchGetParent)
1397
 
        self.assertEqual(
1398
 
            smart.request.request_handlers.get('Branch.get_tags_bytes'),
1399
 
            smart.branch.SmartServerBranchGetTagsBytes)
1400
 
        self.assertEqual(
1401
795
            smart.request.request_handlers.get('Branch.lock_write'),
1402
796
            smart.branch.SmartServerBranchRequestLockWrite)
1403
797
        self.assertEqual(
1407
801
            smart.request.request_handlers.get('Branch.revision_history'),
1408
802
            smart.branch.SmartServerRequestRevisionHistory)
1409
803
        self.assertEqual(
1410
 
            smart.request.request_handlers.get('Branch.set_config_option'),
1411
 
            smart.branch.SmartServerBranchRequestSetConfigOption)
1412
 
        self.assertEqual(
1413
804
            smart.request.request_handlers.get('Branch.set_last_revision'),
1414
805
            smart.branch.SmartServerBranchRequestSetLastRevision)
1415
806
        self.assertEqual(
1416
 
            smart.request.request_handlers.get('Branch.set_last_revision_info'),
1417
 
            smart.branch.SmartServerBranchRequestSetLastRevisionInfo)
1418
 
        self.assertEqual(
1419
807
            smart.request.request_handlers.get('Branch.unlock'),
1420
808
            smart.branch.SmartServerBranchRequestUnlock)
1421
809
        self.assertEqual(
1422
810
            smart.request.request_handlers.get('BzrDir.find_repository'),
1423
 
            smart.bzrdir.SmartServerRequestFindRepositoryV1)
1424
 
        self.assertEqual(
1425
 
            smart.request.request_handlers.get('BzrDir.find_repositoryV2'),
1426
 
            smart.bzrdir.SmartServerRequestFindRepositoryV2)
 
811
            smart.bzrdir.SmartServerRequestFindRepository)
1427
812
        self.assertEqual(
1428
813
            smart.request.request_handlers.get('BzrDirFormat.initialize'),
1429
814
            smart.bzrdir.SmartServerRequestInitializeBzrDir)
1430
815
        self.assertEqual(
1431
 
            smart.request.request_handlers.get('BzrDir.cloning_metadir'),
1432
 
            smart.bzrdir.SmartServerBzrDirRequestCloningMetaDir)
1433
 
        self.assertEqual(
1434
816
            smart.request.request_handlers.get('BzrDir.open_branch'),
1435
817
            smart.bzrdir.SmartServerRequestOpenBranch)
1436
818
        self.assertEqual(
1437
 
            smart.request.request_handlers.get('BzrDir.open_branchV2'),
1438
 
            smart.bzrdir.SmartServerRequestOpenBranchV2)
1439
 
        self.assertEqual(
1440
 
            smart.request.request_handlers.get('PackRepository.autopack'),
1441
 
            smart.packrepository.SmartServerPackRepositoryAutopack)
1442
 
        self.assertEqual(
1443
819
            smart.request.request_handlers.get('Repository.gather_stats'),
1444
820
            smart.repository.SmartServerRepositoryGatherStats)
1445
821
        self.assertEqual(
1446
 
            smart.request.request_handlers.get('Repository.get_parent_map'),
1447
 
            smart.repository.SmartServerRepositoryGetParentMap)
1448
 
        self.assertEqual(
1449
 
            smart.request.request_handlers.get(
1450
 
                'Repository.get_revision_graph'),
 
822
            smart.request.request_handlers.get('Repository.get_revision_graph'),
1451
823
            smart.repository.SmartServerRepositoryGetRevisionGraph)
1452
824
        self.assertEqual(
1453
 
            smart.request.request_handlers.get('Repository.get_stream'),
1454
 
            smart.repository.SmartServerRepositoryGetStream)
1455
 
        self.assertEqual(
1456
825
            smart.request.request_handlers.get('Repository.has_revision'),
1457
826
            smart.repository.SmartServerRequestHasRevision)
1458
827
        self.assertEqual(
1459
 
            smart.request.request_handlers.get('Repository.insert_stream'),
1460
 
            smart.repository.SmartServerRepositoryInsertStream)
1461
 
        self.assertEqual(
1462
 
            smart.request.request_handlers.get('Repository.insert_stream_locked'),
1463
 
            smart.repository.SmartServerRepositoryInsertStreamLocked)
1464
 
        self.assertEqual(
1465
828
            smart.request.request_handlers.get('Repository.is_shared'),
1466
829
            smart.repository.SmartServerRepositoryIsShared)
1467
830
        self.assertEqual(
1468
831
            smart.request.request_handlers.get('Repository.lock_write'),
1469
832
            smart.repository.SmartServerRepositoryLockWrite)
1470
833
        self.assertEqual(
 
834
            smart.request.request_handlers.get('Repository.unlock'),
 
835
            smart.repository.SmartServerRepositoryUnlock)
 
836
        self.assertEqual(
1471
837
            smart.request.request_handlers.get('Repository.tarball'),
1472
838
            smart.repository.SmartServerRepositoryTarball)
1473
839
        self.assertEqual(
1474
 
            smart.request.request_handlers.get('Repository.unlock'),
1475
 
            smart.repository.SmartServerRepositoryUnlock)
1476
 
        self.assertEqual(
1477
840
            smart.request.request_handlers.get('Transport.is_readonly'),
1478
841
            smart.request.SmartServerIsReadonly)