~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_remote.py

(robertc) Fix lp: urls behind an https proxy.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2006-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
30
30
    branch,
31
31
    bzrdir,
32
32
    config,
33
 
    controldir,
34
33
    errors,
35
 
    graph as _mod_graph,
 
34
    graph,
36
35
    inventory,
37
36
    inventory_delta,
 
37
    pack,
38
38
    remote,
39
39
    repository,
40
40
    tests,
41
 
    transport,
42
41
    treebuilder,
 
42
    urlutils,
43
43
    versionedfile,
44
44
    )
45
45
from bzrlib.branch import Branch
46
 
from bzrlib.bzrdir import (
47
 
    BzrDir,
48
 
    BzrDirFormat,
49
 
    RemoteBzrProber,
50
 
    )
 
46
from bzrlib.bzrdir import BzrDir, BzrDirFormat
51
47
from bzrlib.remote import (
52
48
    RemoteBranch,
53
49
    RemoteBranchFormat,
56
52
    RemoteRepository,
57
53
    RemoteRepositoryFormat,
58
54
    )
59
 
from bzrlib.repofmt import groupcompress_repo, knitpack_repo
 
55
from bzrlib.repofmt import groupcompress_repo, pack_repo
60
56
from bzrlib.revision import NULL_REVISION
61
 
from bzrlib.smart import medium, request
 
57
from bzrlib.smart import medium
62
58
from bzrlib.smart.client import _SmartClient
63
 
from bzrlib.smart.repository import (
64
 
    SmartServerRepositoryGetParentMap,
65
 
    SmartServerRepositoryGetStream_1_19,
66
 
    )
 
59
from bzrlib.smart.repository import SmartServerRepositoryGetParentMap
67
60
from bzrlib.tests import (
 
61
    condition_isinstance,
 
62
    split_suite_by_condition,
 
63
    multiply_tests,
68
64
    test_server,
69
65
    )
70
 
from bzrlib.tests.scenarios import load_tests_apply_scenarios
 
66
from bzrlib.transport import get_transport
71
67
from bzrlib.transport.memory import MemoryTransport
72
68
from bzrlib.transport.remote import (
73
69
    RemoteTransport,
74
70
    RemoteSSHTransport,
75
71
    RemoteTCPTransport,
76
 
    )
77
 
 
78
 
 
79
 
load_tests = load_tests_apply_scenarios
80
 
 
81
 
 
82
 
class BasicRemoteObjectTests(tests.TestCaseWithTransport):
83
 
 
84
 
    scenarios = [
 
72
)
 
73
 
 
74
def load_tests(standard_tests, module, loader):
 
75
    to_adapt, result = split_suite_by_condition(
 
76
        standard_tests, condition_isinstance(BasicRemoteObjectTests))
 
77
    smart_server_version_scenarios = [
85
78
        ('HPSS-v2',
86
 
            {'transport_server': test_server.SmartTCPServer_for_testing_v2_only}),
 
79
         {'transport_server': test_server.SmartTCPServer_for_testing_v2_only}),
87
80
        ('HPSS-v3',
88
 
            {'transport_server': test_server.SmartTCPServer_for_testing})]
89
 
 
 
81
         {'transport_server': test_server.SmartTCPServer_for_testing})]
 
82
    return multiply_tests(to_adapt, smart_server_version_scenarios, result)
 
83
 
 
84
 
 
85
class BasicRemoteObjectTests(tests.TestCaseWithTransport):
90
86
 
91
87
    def setUp(self):
92
88
        super(BasicRemoteObjectTests, self).setUp()
93
89
        self.transport = self.get_transport()
94
90
        # make a branch that can be opened over the smart transport
95
91
        self.local_wt = BzrDir.create_standalone_workingtree('.')
96
 
        self.addCleanup(self.transport.disconnect)
 
92
 
 
93
    def tearDown(self):
 
94
        self.transport.disconnect()
 
95
        tests.TestCaseWithTransport.tearDown(self)
97
96
 
98
97
    def test_create_remote_bzrdir(self):
99
 
        b = remote.RemoteBzrDir(self.transport, RemoteBzrDirFormat())
 
98
        b = remote.RemoteBzrDir(self.transport, remote.RemoteBzrDirFormat())
100
99
        self.assertIsInstance(b, BzrDir)
101
100
 
102
101
    def test_open_remote_branch(self):
103
102
        # open a standalone branch in the working directory
104
 
        b = remote.RemoteBzrDir(self.transport, RemoteBzrDirFormat())
 
103
        b = remote.RemoteBzrDir(self.transport, remote.RemoteBzrDirFormat())
105
104
        branch = b.open_branch()
106
105
        self.assertIsInstance(branch, Branch)
107
106
 
123
122
    def test_find_correct_format(self):
124
123
        """Should open a RemoteBzrDir over a RemoteTransport"""
125
124
        fmt = BzrDirFormat.find_format(self.transport)
126
 
        self.assertTrue(bzrdir.RemoteBzrProber
127
 
                        in controldir.ControlDirFormat._server_probers)
128
 
        self.assertIsInstance(fmt, RemoteBzrDirFormat)
 
125
        self.assertTrue(RemoteBzrDirFormat
 
126
                        in BzrDirFormat._control_server_formats)
 
127
        self.assertIsInstance(fmt, remote.RemoteBzrDirFormat)
129
128
 
130
129
    def test_open_detected_smart_format(self):
131
130
        fmt = BzrDirFormat.find_format(self.transport)
360
359
        a given client_base and transport_base.
361
360
        """
362
361
        client_medium = medium.SmartClientMedium(client_base)
363
 
        t = transport.get_transport(transport_base)
364
 
        result = client_medium.remote_path_from_transport(t)
 
362
        transport = get_transport(transport_base)
 
363
        result = client_medium.remote_path_from_transport(transport)
365
364
        self.assertEqual(expected, result)
366
365
 
367
366
    def test_remote_path_from_transport(self):
378
377
        a given transport_base and relpath of that transport.  (Note that
379
378
        HttpTransportBase is a subclass of SmartClientMedium)
380
379
        """
381
 
        base_transport = transport.get_transport(transport_base)
 
380
        base_transport = get_transport(transport_base)
382
381
        client_medium = base_transport.get_smart_medium()
383
382
        cloned_transport = base_transport.clone(relpath)
384
383
        result = client_medium.remote_path_from_transport(cloned_transport)
451
450
        client.add_expected_call(
452
451
            'BzrDir.open_branchV3', ('quack/',),
453
452
            'success', ('ref', self.get_url('referenced'))),
454
 
        a_bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
453
        a_bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
455
454
            _client=client)
456
455
        result = a_bzrdir.cloning_metadir()
457
456
        # We should have got a control dir matching the referenced branch.
470
469
        client.add_expected_call(
471
470
            'BzrDir.cloning_metadir', ('quack/', 'False'),
472
471
            'success', (control_name, '', ('branch', ''))),
473
 
        a_bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
472
        a_bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
474
473
            _client=client)
475
474
        result = a_bzrdir.cloning_metadir()
476
475
        # We should have got a reference control dir with default branch and
496
495
        client.add_expected_call(
497
496
            'BzrDir.open_2.1', ('quack/',), 'success', ('no',))
498
497
        self.assertRaises(errors.NotBranchError, RemoteBzrDir, transport,
499
 
                RemoteBzrDirFormat(), _client=client, _force_probe=True)
 
498
                remote.RemoteBzrDirFormat(), _client=client, _force_probe=True)
500
499
        self.assertFinished(client)
501
500
 
502
501
    def test_present_without_workingtree(self):
503
502
        client, transport = self.make_fake_client_and_transport()
504
503
        client.add_expected_call(
505
504
            'BzrDir.open_2.1', ('quack/',), 'success', ('yes', 'no'))
506
 
        bd = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
505
        bd = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
507
506
            _client=client, _force_probe=True)
508
507
        self.assertIsInstance(bd, RemoteBzrDir)
509
508
        self.assertFalse(bd.has_workingtree())
514
513
        client, transport = self.make_fake_client_and_transport()
515
514
        client.add_expected_call(
516
515
            'BzrDir.open_2.1', ('quack/',), 'success', ('yes', 'yes'))
517
 
        bd = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
516
        bd = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
518
517
            _client=client, _force_probe=True)
519
518
        self.assertIsInstance(bd, RemoteBzrDir)
520
519
        self.assertTrue(bd.has_workingtree())
527
526
            'BzrDir.open_2.1', ('quack/',), 'unknown', ('BzrDir.open_2.1',))
528
527
        client.add_expected_call(
529
528
            'BzrDir.open', ('quack/',), 'success', ('yes',))
530
 
        bd = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
529
        bd = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
531
530
            _client=client, _force_probe=True)
532
531
        self.assertIsInstance(bd, RemoteBzrDir)
533
532
        self.assertFinished(client)
549
548
            'BzrDir.open_2.1', ('quack/',), 'unknown', ('BzrDir.open_2.1',))
550
549
        client.add_expected_call(
551
550
            'BzrDir.open', ('quack/',), 'success', ('yes',))
552
 
        bd = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
551
        bd = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
553
552
            _client=client, _force_probe=True)
554
553
        self.assertIsInstance(bd, RemoteBzrDir)
555
554
        self.assertFinished(client)
586
585
        client.add_expected_call(
587
586
            'Branch.get_stacked_on_url', ('quack/',),
588
587
            'error', ('NotStacked',))
589
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
588
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
590
589
            _client=client)
591
590
        result = bzrdir.open_branch()
592
591
        self.assertIsInstance(result, RemoteBranch)
599
598
        transport = transport.clone('quack')
600
599
        client = FakeClient(transport.base)
601
600
        client.add_error_response('nobranch')
602
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
601
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
603
602
            _client=client)
604
603
        self.assertRaises(errors.NotBranchError, bzrdir.open_branch)
605
604
        self.assertEqual(
616
615
        transport = MemoryTransport()
617
616
        # no requests on the network - catches other api calls being made.
618
617
        client = FakeClient(transport.base)
619
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
618
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
620
619
            _client=client)
621
620
        # patch the open_branch call to record that it was called.
622
621
        bzrdir.open_branch = open_branch
641
640
        client.add_expected_call(
642
641
            'Branch.get_stacked_on_url', ('~hello/',),
643
642
            'error', ('NotStacked',))
644
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
643
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
645
644
            _client=client)
646
645
        result = bzrdir.open_branch()
647
646
        self.assertFinished(client)
664
663
        client.add_success_response(
665
664
            'ok', '', rich_response, subtree_response, external_lookup,
666
665
            network_name)
667
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
666
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
668
667
            _client=client)
669
668
        result = bzrdir.open_repository()
670
669
        self.assertEqual(
687
686
        old.
688
687
        """
689
688
        self.assertRaises(errors.NotBranchError,
690
 
            RemoteBzrProber.probe_transport, OldServerTransport())
 
689
            RemoteBzrDirFormat.probe_transport, OldServerTransport())
691
690
 
692
691
 
693
692
class TestBzrDirCreateBranch(TestRemote):
716
715
            'BzrDir.create_branch', ('quack/', network_name),
717
716
            'success', ('ok', network_name, '', 'no', 'no', 'yes',
718
717
            reference_repo_name))
719
 
        a_bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
718
        a_bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
720
719
            _client=client)
721
720
        branch = a_bzrdir.create_branch()
722
721
        # We should have got a remote branch
725
724
        format = branch._format
726
725
        self.assertEqual(network_name, format.network_name())
727
726
 
728
 
    def test_already_open_repo_and_reused_medium(self):
729
 
        """Bug 726584: create_branch(..., repository=repo) should work
730
 
        regardless of what the smart medium's base URL is.
731
 
        """
732
 
        self.transport_server = test_server.SmartTCPServer_for_testing
733
 
        transport = self.get_transport('.')
734
 
        repo = self.make_repository('quack')
735
 
        # Client's medium rooted a transport root (not at the bzrdir)
736
 
        client = FakeClient(transport.base)
737
 
        transport = transport.clone('quack')
738
 
        reference_bzrdir_format = bzrdir.format_registry.get('default')()
739
 
        reference_format = reference_bzrdir_format.get_branch_format()
740
 
        network_name = reference_format.network_name()
741
 
        reference_repo_fmt = reference_bzrdir_format.repository_format
742
 
        reference_repo_name = reference_repo_fmt.network_name()
743
 
        client.add_expected_call(
744
 
            'BzrDir.create_branch', ('extra/quack/', network_name),
745
 
            'success', ('ok', network_name, '', 'no', 'no', 'yes',
746
 
            reference_repo_name))
747
 
        a_bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
748
 
            _client=client)
749
 
        branch = a_bzrdir.create_branch(repository=repo)
750
 
        # We should have got a remote branch
751
 
        self.assertIsInstance(branch, remote.RemoteBranch)
752
 
        # its format should have the settings from the response
753
 
        format = branch._format
754
 
        self.assertEqual(network_name, format.network_name())
755
 
 
756
727
 
757
728
class TestBzrDirCreateRepository(TestRemote):
758
729
 
779
750
                'Bazaar repository format 2a (needs bzr 1.16 or later)\n',
780
751
                'False'),
781
752
            'success', ('ok', 'yes', 'yes', 'yes', network_name))
782
 
        a_bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
753
        a_bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
783
754
            _client=client)
784
755
        repo = a_bzrdir.create_repository()
785
756
        # We should have got a remote repository
814
785
        client.add_success_response('stat', '0', '65535')
815
786
        remote_transport = RemoteTransport(server_url + 'quack/', medium=False,
816
787
            _client=client)
817
 
        bzrdir = RemoteBzrDir(remote_transport, RemoteBzrDirFormat(),
 
788
        bzrdir = RemoteBzrDir(remote_transport, remote.RemoteBzrDirFormat(),
818
789
            _client=client)
819
790
        repo = bzrdir.open_repository()
820
791
        self.assertEqual(
847
818
        client.add_success_response('stat', '0', '65535')
848
819
        remote_transport = RemoteTransport(server_url + 'quack/', medium=False,
849
820
            _client=client)
850
 
        bzrdir = RemoteBzrDir(remote_transport, RemoteBzrDirFormat(),
 
821
        bzrdir = RemoteBzrDir(remote_transport, remote.RemoteBzrDirFormat(),
851
822
            _client=client)
852
823
        repo = bzrdir.open_repository()
853
824
        self.assertEqual(
868
839
        transport = transport.clone('quack')
869
840
        client = FakeClient(transport.base)
870
841
        client.add_success_response('ok', '', 'no', 'no', 'no', network_name)
871
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
842
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
872
843
            _client=client)
873
844
        repo = bzrdir.open_repository()
874
845
        self.assertEqual(
881
852
 
882
853
    def test_success(self):
883
854
        """Simple test for typical successful call."""
884
 
        fmt = RemoteBzrDirFormat()
 
855
        fmt = bzrdir.RemoteBzrDirFormat()
885
856
        default_format_name = BzrDirFormat.get_default_format().network_name()
886
857
        transport = self.get_transport()
887
858
        client = FakeClient(transport.base)
903
874
        """Error responses are translated, e.g. 'PermissionDenied' raises the
904
875
        corresponding error from the client.
905
876
        """
906
 
        fmt = RemoteBzrDirFormat()
 
877
        fmt = bzrdir.RemoteBzrDirFormat()
907
878
        default_format_name = BzrDirFormat.get_default_format().network_name()
908
879
        transport = self.get_transport()
909
880
        client = FakeClient(transport.base)
927
898
        """Integration test for error translation."""
928
899
        transport = self.make_smart_server('foo')
929
900
        transport = transport.clone('no-such-path')
930
 
        fmt = RemoteBzrDirFormat()
 
901
        fmt = bzrdir.RemoteBzrDirFormat()
931
902
        err = self.assertRaises(errors.NoSuchFile,
932
903
            fmt.initialize_on_transport_ex, transport, create_prefix=False)
933
904
 
964
935
 
965
936
    def make_remote_bzrdir(self, transport, client):
966
937
        """Make a RemotebzrDir using 'client' as the _client."""
967
 
        return RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
938
        return RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
968
939
            _client=client)
969
940
 
970
941
 
1172
1143
            [('set_tags_bytes', 'tags bytes')] * 2, real_branch.calls)
1173
1144
 
1174
1145
 
1175
 
class TestBranchHeadsToFetch(RemoteBranchTestCase):
1176
 
 
1177
 
    def test_uses_last_revision_info_and_tags_by_default(self):
1178
 
        transport = MemoryTransport()
1179
 
        client = FakeClient(transport.base)
1180
 
        client.add_expected_call(
1181
 
            'Branch.get_stacked_on_url', ('quack/',),
1182
 
            'error', ('NotStacked',))
1183
 
        client.add_expected_call(
1184
 
            'Branch.last_revision_info', ('quack/',),
1185
 
            'success', ('ok', '1', 'rev-tip'))
1186
 
        client.add_expected_call(
1187
 
            'Branch.get_config_file', ('quack/',),
1188
 
            'success', ('ok',), '')
1189
 
        transport.mkdir('quack')
1190
 
        transport = transport.clone('quack')
1191
 
        branch = self.make_remote_branch(transport, client)
1192
 
        result = branch.heads_to_fetch()
1193
 
        self.assertFinished(client)
1194
 
        self.assertEqual((set(['rev-tip']), set()), result)
1195
 
 
1196
 
    def test_uses_last_revision_info_and_tags_when_set(self):
1197
 
        transport = MemoryTransport()
1198
 
        client = FakeClient(transport.base)
1199
 
        client.add_expected_call(
1200
 
            'Branch.get_stacked_on_url', ('quack/',),
1201
 
            'error', ('NotStacked',))
1202
 
        client.add_expected_call(
1203
 
            'Branch.last_revision_info', ('quack/',),
1204
 
            'success', ('ok', '1', 'rev-tip'))
1205
 
        client.add_expected_call(
1206
 
            'Branch.get_config_file', ('quack/',),
1207
 
            'success', ('ok',), 'branch.fetch_tags = True')
1208
 
        # XXX: this will break if the default format's serialization of tags
1209
 
        # changes, or if the RPC for fetching tags changes from get_tags_bytes.
1210
 
        client.add_expected_call(
1211
 
            'Branch.get_tags_bytes', ('quack/',),
1212
 
            'success', ('d5:tag-17:rev-foo5:tag-27:rev-bare',))
1213
 
        transport.mkdir('quack')
1214
 
        transport = transport.clone('quack')
1215
 
        branch = self.make_remote_branch(transport, client)
1216
 
        result = branch.heads_to_fetch()
1217
 
        self.assertFinished(client)
1218
 
        self.assertEqual(
1219
 
            (set(['rev-tip']), set(['rev-foo', 'rev-bar'])), result)
1220
 
 
1221
 
    def test_uses_rpc_for_formats_with_non_default_heads_to_fetch(self):
1222
 
        transport = MemoryTransport()
1223
 
        client = FakeClient(transport.base)
1224
 
        client.add_expected_call(
1225
 
            'Branch.get_stacked_on_url', ('quack/',),
1226
 
            'error', ('NotStacked',))
1227
 
        client.add_expected_call(
1228
 
            'Branch.heads_to_fetch', ('quack/',),
1229
 
            'success', (['tip'], ['tagged-1', 'tagged-2']))
1230
 
        transport.mkdir('quack')
1231
 
        transport = transport.clone('quack')
1232
 
        branch = self.make_remote_branch(transport, client)
1233
 
        branch._format._use_default_local_heads_to_fetch = lambda: False
1234
 
        result = branch.heads_to_fetch()
1235
 
        self.assertFinished(client)
1236
 
        self.assertEqual((set(['tip']), set(['tagged-1', 'tagged-2'])), result)
1237
 
 
1238
 
    def make_branch_with_tags(self):
1239
 
        self.setup_smart_server_with_call_log()
1240
 
        # Make a branch with a single revision.
1241
 
        builder = self.make_branch_builder('foo')
1242
 
        builder.start_series()
1243
 
        builder.build_snapshot('tip', None, [
1244
 
            ('add', ('', 'root-id', 'directory', ''))])
1245
 
        builder.finish_series()
1246
 
        branch = builder.get_branch()
1247
 
        # Add two tags to that branch
1248
 
        branch.tags.set_tag('tag-1', 'rev-1')
1249
 
        branch.tags.set_tag('tag-2', 'rev-2')
1250
 
        return branch
1251
 
 
1252
 
    def test_backwards_compatible(self):
1253
 
        branch = self.make_branch_with_tags()
1254
 
        c = branch.get_config()
1255
 
        c.set_user_option('branch.fetch_tags', 'True')
1256
 
        self.addCleanup(branch.lock_read().unlock)
1257
 
        # Disable the heads_to_fetch verb
1258
 
        verb = 'Branch.heads_to_fetch'
1259
 
        self.disable_verb(verb)
1260
 
        self.reset_smart_call_log()
1261
 
        result = branch.heads_to_fetch()
1262
 
        self.assertEqual((set(['tip']), set(['rev-1', 'rev-2'])), result)
1263
 
        self.assertEqual(
1264
 
            ['Branch.last_revision_info', 'Branch.get_config_file',
1265
 
             'Branch.get_tags_bytes'],
1266
 
            [call.call.method for call in self.hpss_calls])
1267
 
 
1268
 
    def test_backwards_compatible_no_tags(self):
1269
 
        branch = self.make_branch_with_tags()
1270
 
        c = branch.get_config()
1271
 
        c.set_user_option('branch.fetch_tags', 'False')
1272
 
        self.addCleanup(branch.lock_read().unlock)
1273
 
        # Disable the heads_to_fetch verb
1274
 
        verb = 'Branch.heads_to_fetch'
1275
 
        self.disable_verb(verb)
1276
 
        self.reset_smart_call_log()
1277
 
        result = branch.heads_to_fetch()
1278
 
        self.assertEqual((set(['tip']), set()), result)
1279
 
        self.assertEqual(
1280
 
            ['Branch.last_revision_info', 'Branch.get_config_file'],
1281
 
            [call.call.method for call in self.hpss_calls])
1282
 
 
1283
 
 
1284
1146
class TestBranchLastRevisionInfo(RemoteBranchTestCase):
1285
1147
 
1286
1148
    def test_empty_branch(self):
1341
1203
        client.add_expected_call(
1342
1204
            'Branch.get_stacked_on_url', ('stacked/',),
1343
1205
            'success', ('ok', vfs_url))
1344
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
1206
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
1345
1207
            _client=client)
1346
1208
        repo_fmt = remote.RemoteRepositoryFormat()
1347
1209
        repo_fmt._custom_format = stacked_branch.repository._format
1374
1236
        # this will also do vfs access, but that goes direct to the transport
1375
1237
        # and isn't seen by the FakeClient.
1376
1238
        bzrdir = RemoteBzrDir(self.get_transport('stacked'),
1377
 
            RemoteBzrDirFormat(), _client=client)
 
1239
            remote.RemoteBzrDirFormat(), _client=client)
1378
1240
        branch = bzrdir.open_branch()
1379
1241
        result = branch.get_stacked_on_url()
1380
1242
        self.assertEqual('../base', result)
1407
1269
            'Branch.get_stacked_on_url', ('stacked/',),
1408
1270
            'success', ('ok', '../base'))
1409
1271
        bzrdir = RemoteBzrDir(self.get_transport('stacked'),
1410
 
            RemoteBzrDirFormat(), _client=client)
 
1272
            remote.RemoteBzrDirFormat(), _client=client)
1411
1273
        branch = bzrdir.open_branch()
1412
1274
        result = branch.get_stacked_on_url()
1413
1275
        self.assertEqual('../base', result)
1421
1283
class TestBranchSetLastRevision(RemoteBranchTestCase):
1422
1284
 
1423
1285
    def test_set_empty(self):
1424
 
        # _set_last_revision_info('null:') is translated to calling
 
1286
        # set_revision_history([]) is translated to calling
1425
1287
        # Branch.set_last_revision(path, '') on the wire.
1426
1288
        transport = MemoryTransport()
1427
1289
        transport.mkdir('branch')
1449
1311
        # unnecessarily invokes _ensure_real upon a call to lock_write.
1450
1312
        branch._ensure_real = lambda: None
1451
1313
        branch.lock_write()
1452
 
        result = branch._set_last_revision(NULL_REVISION)
 
1314
        result = branch.set_revision_history([])
1453
1315
        branch.unlock()
1454
1316
        self.assertEqual(None, result)
1455
1317
        self.assertFinished(client)
1456
1318
 
1457
1319
    def test_set_nonempty(self):
1458
 
        # set_last_revision_info(N, rev-idN) is translated to calling
 
1320
        # set_revision_history([rev-id1, ..., rev-idN]) is translated to calling
1459
1321
        # Branch.set_last_revision(path, rev-idN) on the wire.
1460
1322
        transport = MemoryTransport()
1461
1323
        transport.mkdir('branch')
1487
1349
        branch._ensure_real = lambda: None
1488
1350
        # Lock the branch, reset the record of remote calls.
1489
1351
        branch.lock_write()
1490
 
        result = branch._set_last_revision('rev-id2')
 
1352
        result = branch.set_revision_history(['rev-id1', 'rev-id2'])
1491
1353
        branch.unlock()
1492
1354
        self.assertEqual(None, result)
1493
1355
        self.assertFinished(client)
1523
1385
        branch = self.make_remote_branch(transport, client)
1524
1386
        branch.lock_write()
1525
1387
        self.assertRaises(
1526
 
            errors.NoSuchRevision, branch._set_last_revision, 'rev-id')
 
1388
            errors.NoSuchRevision, branch.set_revision_history, ['rev-id'])
1527
1389
        branch.unlock()
1528
1390
        self.assertFinished(client)
1529
1391
 
1560
1422
        branch._ensure_real = lambda: None
1561
1423
        branch.lock_write()
1562
1424
        # The 'TipChangeRejected' error response triggered by calling
1563
 
        # set_last_revision_info causes a TipChangeRejected exception.
 
1425
        # set_revision_history causes a TipChangeRejected exception.
1564
1426
        err = self.assertRaises(
1565
 
            errors.TipChangeRejected,
1566
 
            branch._set_last_revision, 'rev-id')
 
1427
            errors.TipChangeRejected, branch.set_revision_history, ['rev-id'])
1567
1428
        # The UTF-8 message from the response has been decoded into a unicode
1568
1429
        # object.
1569
1430
        self.assertIsInstance(err.msg, unicode)
1757
1618
    def test_get_multi_line_branch_conf(self):
1758
1619
        # Make sure that multiple-line branch.conf files are supported
1759
1620
        #
1760
 
        # https://bugs.launchpad.net/bzr/+bug/354075
 
1621
        # https://bugs.edge.launchpad.net/bzr/+bug/354075
1761
1622
        client = FakeClient()
1762
1623
        client.add_expected_call(
1763
1624
            'Branch.get_stacked_on_url', ('memory:///',),
1791
1652
        branch.unlock()
1792
1653
        self.assertFinished(client)
1793
1654
 
1794
 
    def test_set_option_with_dict(self):
1795
 
        client = FakeClient()
1796
 
        client.add_expected_call(
1797
 
            'Branch.get_stacked_on_url', ('memory:///',),
1798
 
            'error', ('NotStacked',),)
1799
 
        client.add_expected_call(
1800
 
            'Branch.lock_write', ('memory:///', '', ''),
1801
 
            'success', ('ok', 'branch token', 'repo token'))
1802
 
        encoded_dict_value = 'd5:ascii1:a11:unicode \xe2\x8c\x9a3:\xe2\x80\xbde'
1803
 
        client.add_expected_call(
1804
 
            'Branch.set_config_option_dict', ('memory:///', 'branch token',
1805
 
            'repo token', encoded_dict_value, 'foo', ''),
1806
 
            'success', ())
1807
 
        client.add_expected_call(
1808
 
            'Branch.unlock', ('memory:///', 'branch token', 'repo token'),
1809
 
            'success', ('ok',))
1810
 
        transport = MemoryTransport()
1811
 
        branch = self.make_remote_branch(transport, client)
1812
 
        branch.lock_write()
1813
 
        config = branch._get_config()
1814
 
        config.set_option(
1815
 
            {'ascii': 'a', u'unicode \N{WATCH}': u'\N{INTERROBANG}'},
1816
 
            'foo')
1817
 
        branch.unlock()
1818
 
        self.assertFinished(client)
1819
 
 
1820
1655
    def test_backwards_compat_set_option(self):
1821
1656
        self.setup_smart_server_with_call_log()
1822
1657
        branch = self.make_branch('.')
1829
1664
        self.assertLength(10, self.hpss_calls)
1830
1665
        self.assertEqual('value', branch._get_config().get_option('name'))
1831
1666
 
1832
 
    def test_backwards_compat_set_option_with_dict(self):
1833
 
        self.setup_smart_server_with_call_log()
1834
 
        branch = self.make_branch('.')
1835
 
        verb = 'Branch.set_config_option_dict'
1836
 
        self.disable_verb(verb)
1837
 
        branch.lock_write()
1838
 
        self.addCleanup(branch.unlock)
1839
 
        self.reset_smart_call_log()
1840
 
        config = branch._get_config()
1841
 
        value_dict = {'ascii': 'a', u'unicode \N{WATCH}': u'\N{INTERROBANG}'}
1842
 
        config.set_option(value_dict, 'name')
1843
 
        self.assertLength(10, self.hpss_calls)
1844
 
        self.assertEqual(value_dict, branch._get_config().get_option('name'))
1845
 
 
1846
1667
 
1847
1668
class TestBranchLockWrite(RemoteBranchTestCase):
1848
1669
 
1984
1805
        client = FakeClient(transport.base)
1985
1806
        transport = transport.clone(transport_path)
1986
1807
        # we do not want bzrdir to make any remote calls
1987
 
        bzrdir = RemoteBzrDir(transport, RemoteBzrDirFormat(),
 
1808
        bzrdir = RemoteBzrDir(transport, remote.RemoteBzrDirFormat(),
1988
1809
            _client=False)
1989
1810
        repo = RemoteRepository(bzrdir, None, _client=client)
1990
1811
        return repo, client
1998
1819
 
1999
1820
    def test_get_format_description(self):
2000
1821
        remote_format = RemoteBranchFormat()
2001
 
        real_format = branch.format_registry.get_default()
 
1822
        real_format = branch.BranchFormat.get_default_format()
2002
1823
        remote_format._network_name = real_format.network_name()
2003
1824
        self.assertEqual(remoted_description(real_format),
2004
1825
            remote_format.get_format_description())
2007
1828
class TestRepositoryFormat(TestRemoteRepository):
2008
1829
 
2009
1830
    def test_fast_delta(self):
2010
 
        true_name = groupcompress_repo.RepositoryFormat2a().network_name()
 
1831
        true_name = groupcompress_repo.RepositoryFormatCHK1().network_name()
2011
1832
        true_format = RemoteRepositoryFormat()
2012
1833
        true_format._network_name = true_name
2013
1834
        self.assertEqual(True, true_format.fast_deltas)
2014
 
        false_name = knitpack_repo.RepositoryFormatKnitPack1().network_name()
 
1835
        false_name = pack_repo.RepositoryFormatKnitPack1().network_name()
2015
1836
        false_format = RemoteRepositoryFormat()
2016
1837
        false_format._network_name = false_name
2017
1838
        self.assertEqual(False, false_format.fast_deltas)
2018
1839
 
2019
1840
    def test_get_format_description(self):
2020
1841
        remote_repo_format = RemoteRepositoryFormat()
2021
 
        real_format = repository.format_registry.get_default()
 
1842
        real_format = repository.RepositoryFormat.get_default_format()
2022
1843
        remote_repo_format._network_name = real_format.network_name()
2023
1844
        self.assertEqual(remoted_description(real_format),
2024
1845
            remote_repo_format.get_format_description())
2147
1968
        parents = repo.get_parent_map([rev_id])
2148
1969
        self.assertEqual(
2149
1970
            [('call_with_body_bytes_expecting_body',
2150
 
              'Repository.get_parent_map',
2151
 
              ('quack/', 'include-missing:', rev_id), '\n\n0'),
 
1971
              'Repository.get_parent_map', ('quack/', 'include-missing:',
 
1972
              rev_id), '\n\n0'),
2152
1973
             ('disconnect medium',),
2153
1974
             ('call_expecting_body', 'Repository.get_revision_graph',
2154
1975
              ('quack/', ''))],
2274
2095
        self.assertEqual({}, repo.get_parent_map(['non-existant']))
2275
2096
        self.assertLength(0, self.hpss_calls)
2276
2097
 
2277
 
    def test_exposes_get_cached_parent_map(self):
2278
 
        """RemoteRepository exposes get_cached_parent_map from
2279
 
        _unstacked_provider
2280
 
        """
2281
 
        r1 = u'\u0e33'.encode('utf8')
2282
 
        r2 = u'\u0dab'.encode('utf8')
2283
 
        lines = [' '.join([r2, r1]), r1]
2284
 
        encoded_body = bz2.compress('\n'.join(lines))
2285
 
 
2286
 
        transport_path = 'quack'
2287
 
        repo, client = self.setup_fake_client_and_repository(transport_path)
2288
 
        client.add_success_response_with_body(encoded_body, 'ok')
2289
 
        repo.lock_read()
2290
 
        # get_cached_parent_map should *not* trigger an RPC
2291
 
        self.assertEqual({}, repo.get_cached_parent_map([r1]))
2292
 
        self.assertEqual([], client._calls)
2293
 
        self.assertEqual({r2: (r1,)}, repo.get_parent_map([r2]))
2294
 
        self.assertEqual({r1: (NULL_REVISION,)},
2295
 
            repo.get_cached_parent_map([r1]))
2296
 
        self.assertEqual(
2297
 
            [('call_with_body_bytes_expecting_body',
2298
 
              'Repository.get_parent_map', ('quack/', 'include-missing:', r2),
2299
 
              '\n\n0')],
2300
 
            client._calls)
2301
 
        repo.unlock()
2302
 
 
2303
2098
 
2304
2099
class TestGetParentMapAllowsNew(tests.TestCaseWithTransport):
2305
2100
 
2462
2257
        self.setup_smart_server_with_call_log()
2463
2258
        tree = self.make_branch_and_memory_tree('.')
2464
2259
        tree.lock_write()
2465
 
        tree.add('')
2466
2260
        rev1 = tree.commit('First')
2467
2261
        rev2 = tree.commit('Second')
2468
2262
        tree.unlock()
2507
2301
        transport_path = 'quack'
2508
2302
        repo, client = self.setup_fake_client_and_repository(transport_path)
2509
2303
        client.add_success_response('ok', 'a token')
2510
 
        token = repo.lock_write().repository_token
 
2304
        result = repo.lock_write()
2511
2305
        self.assertEqual(
2512
2306
            [('call', 'Repository.lock_write', ('quack/', ''))],
2513
2307
            client._calls)
2514
 
        self.assertEqual('a token', token)
 
2308
        self.assertEqual('a token', result)
2515
2309
 
2516
2310
    def test_lock_write_already_locked(self):
2517
2311
        transport_path = 'quack'
2608
2402
        the client is finished.
2609
2403
        """
2610
2404
        sink = repo._get_sink()
2611
 
        fmt = repository.format_registry.get_default()
 
2405
        fmt = repository.RepositoryFormat.get_default_format()
2612
2406
        resume_tokens, missing_keys = sink.insert_stream([], fmt, [])
2613
2407
        self.assertEqual([], resume_tokens)
2614
2408
        self.assertEqual(set(), missing_keys)
2714
2508
                return True
2715
2509
        repo._real_repository = FakeRealRepository()
2716
2510
        sink = repo._get_sink()
2717
 
        fmt = repository.format_registry.get_default()
 
2511
        fmt = repository.RepositoryFormat.get_default_format()
2718
2512
        stream = self.make_stream_with_inv_deltas(fmt)
2719
2513
        resume_tokens, missing_keys = sink.insert_stream(stream, fmt, [])
2720
2514
        # Every record from the first inventory delta should have been sent to
2940
2734
             ('pack collection autopack',)],
2941
2735
            client._calls)
2942
2736
 
2943
 
    def test_oom_error_reporting(self):
2944
 
        """An out-of-memory condition on the server is reported clearly"""
2945
 
        transport_path = 'quack'
2946
 
        repo, client = self.setup_fake_client_and_repository(transport_path)
2947
 
        client.add_expected_call(
2948
 
            'PackRepository.autopack', ('quack/',),
2949
 
            'error', ('MemoryError',))
2950
 
        err = self.assertRaises(errors.BzrError, repo.autopack)
2951
 
        self.assertContainsRe(str(err), "^remote server out of mem")
2952
 
 
2953
2737
 
2954
2738
class TestErrorTranslationBase(tests.TestCaseWithMemoryTransport):
2955
2739
    """Base class for unit tests for bzrlib.remote._translate_error."""
3028
2812
            detail='extra detail')
3029
2813
        self.assertEqual(expected_error, translated_error)
3030
2814
 
3031
 
    def test_norepository(self):
3032
 
        bzrdir = self.make_bzrdir('')
3033
 
        translated_error = self.translateTuple(('norepository',),
3034
 
            bzrdir=bzrdir)
3035
 
        expected_error = errors.NoRepositoryPresent(bzrdir)
3036
 
        self.assertEqual(expected_error, translated_error)
3037
 
 
3038
2815
    def test_LockContention(self):
3039
2816
        translated_error = self.translateTuple(('LockContention',))
3040
2817
        expected_error = errors.LockContention('(remote lock)')
3068
2845
        expected_error = errors.DivergedBranches(branch, other_branch)
3069
2846
        self.assertEqual(expected_error, translated_error)
3070
2847
 
3071
 
    def test_NotStacked(self):
3072
 
        branch = self.make_branch('')
3073
 
        translated_error = self.translateTuple(('NotStacked',), branch=branch)
3074
 
        expected_error = errors.NotStacked(branch)
3075
 
        self.assertEqual(expected_error, translated_error)
3076
 
 
3077
2848
    def test_ReadError_no_args(self):
3078
2849
        path = 'a path'
3079
2850
        translated_error = self.translateTuple(('ReadError',), path=path)
3095
2866
 
3096
2867
    def test_PermissionDenied_no_args(self):
3097
2868
        path = 'a path'
3098
 
        translated_error = self.translateTuple(('PermissionDenied',),
3099
 
            path=path)
 
2869
        translated_error = self.translateTuple(('PermissionDenied',), path=path)
3100
2870
        expected_error = errors.PermissionDenied(path)
3101
2871
        self.assertEqual(expected_error, translated_error)
3102
2872
 
3125
2895
        expected_error = errors.PermissionDenied(path, extra)
3126
2896
        self.assertEqual(expected_error, translated_error)
3127
2897
 
3128
 
    # GZ 2011-03-02: TODO test for PermissionDenied with non-ascii 'extra'
3129
 
 
3130
 
    def test_NoSuchFile_context_path(self):
3131
 
        local_path = "local path"
3132
 
        translated_error = self.translateTuple(('ReadError', "remote path"),
3133
 
            path=local_path)
3134
 
        expected_error = errors.ReadError(local_path)
3135
 
        self.assertEqual(expected_error, translated_error)
3136
 
 
3137
 
    def test_NoSuchFile_without_context(self):
3138
 
        remote_path = "remote path"
3139
 
        translated_error = self.translateTuple(('ReadError', remote_path))
3140
 
        expected_error = errors.ReadError(remote_path)
3141
 
        self.assertEqual(expected_error, translated_error)
3142
 
 
3143
 
    def test_ReadOnlyError(self):
3144
 
        translated_error = self.translateTuple(('ReadOnlyError',))
3145
 
        expected_error = errors.TransportNotPossible("readonly transport")
3146
 
        self.assertEqual(expected_error, translated_error)
3147
 
 
3148
 
    def test_MemoryError(self):
3149
 
        translated_error = self.translateTuple(('MemoryError',))
3150
 
        self.assertStartsWith(str(translated_error),
3151
 
            "remote server out of memory")
3152
 
 
3153
 
    def test_generic_IndexError_no_classname(self):
3154
 
        err = errors.ErrorFromSmartServer(('error', "list index out of range"))
3155
 
        translated_error = self.translateErrorFromSmartServer(err)
3156
 
        expected_error = errors.UnknownErrorFromSmartServer(err)
3157
 
        self.assertEqual(expected_error, translated_error)
3158
 
 
3159
 
    # GZ 2011-03-02: TODO test generic non-ascii error string
3160
 
 
3161
 
    def test_generic_KeyError(self):
3162
 
        err = errors.ErrorFromSmartServer(('error', 'KeyError', "1"))
3163
 
        translated_error = self.translateErrorFromSmartServer(err)
3164
 
        expected_error = errors.UnknownErrorFromSmartServer(err)
3165
 
        self.assertEqual(expected_error, translated_error)
3166
 
 
3167
2898
 
3168
2899
class TestErrorTranslationRobustness(TestErrorTranslationBase):
3169
2900
    """Unit tests for bzrlib.remote._translate_error's robustness.
3314
3045
        _, stacked = branch_factory()
3315
3046
        source = stacked.repository._get_source(target_repository_format)
3316
3047
        tip = stacked.last_revision()
3317
 
        stacked.repository._ensure_real()
3318
 
        graph = stacked.repository.get_graph()
3319
 
        revs = [r for (r,ps) in graph.iter_ancestry([tip])
3320
 
                if r != NULL_REVISION]
3321
 
        revs.reverse()
3322
 
        search = _mod_graph.PendingAncestryResult([tip], stacked.repository)
 
3048
        revs = stacked.repository.get_ancestry(tip)
 
3049
        search = graph.PendingAncestryResult([tip], stacked.repository)
3323
3050
        self.reset_smart_call_log()
3324
3051
        stream = source.get_stream(search)
 
3052
        if None in revs:
 
3053
            revs.remove(None)
3325
3054
        # We trust that if a revision is in the stream the rest of the new
3326
3055
        # content for it is too, as per our main fetch tests; here we are
3327
3056
        # checking that the revisions are actually included at all, and their
3366
3095
        self.assertEqual(expected_revs, rev_ord)
3367
3096
        # Getting topological sort requires VFS calls still - one of which is
3368
3097
        # pushing up from the bound branch.
3369
 
        self.assertLength(14, self.hpss_calls)
 
3098
        self.assertLength(13, self.hpss_calls)
3370
3099
 
3371
3100
    def test_stacked_get_stream_groupcompress(self):
3372
3101
        # Repository._get_source.get_stream() from a stacked repository with
3413
3142
 
3414
3143
    def test_copy_content_into_avoids_revision_history(self):
3415
3144
        local = self.make_branch('local')
3416
 
        builder = self.make_branch_builder('remote')
3417
 
        builder.build_commit(message="Commit.")
 
3145
        remote_backing_tree = self.make_branch_and_tree('remote')
 
3146
        remote_backing_tree.commit("Commit.")
3418
3147
        remote_branch_url = self.smart_server.get_url() + 'remote'
3419
3148
        remote_branch = bzrdir.BzrDir.open(remote_branch_url).open_branch()
3420
3149
        local.repository.fetch(remote_branch.repository)
3421
3150
        self.hpss_calls = []
3422
3151
        remote_branch.copy_content_into(local)
3423
3152
        self.assertFalse('Branch.revision_history' in self.hpss_calls)
3424
 
 
3425
 
    def test_fetch_everything_needs_just_one_call(self):
3426
 
        local = self.make_branch('local')
3427
 
        builder = self.make_branch_builder('remote')
3428
 
        builder.build_commit(message="Commit.")
3429
 
        remote_branch_url = self.smart_server.get_url() + 'remote'
3430
 
        remote_branch = bzrdir.BzrDir.open(remote_branch_url).open_branch()
3431
 
        self.hpss_calls = []
3432
 
        local.repository.fetch(
3433
 
            remote_branch.repository,
3434
 
            fetch_spec=_mod_graph.EverythingResult(remote_branch.repository))
3435
 
        self.assertEqual(['Repository.get_stream_1.19'], self.hpss_calls)
3436
 
 
3437
 
    def override_verb(self, verb_name, verb):
3438
 
        request_handlers = request.request_handlers
3439
 
        orig_verb = request_handlers.get(verb_name)
3440
 
        request_handlers.register(verb_name, verb, override_existing=True)
3441
 
        self.addCleanup(request_handlers.register, verb_name, orig_verb,
3442
 
                override_existing=True)
3443
 
 
3444
 
    def test_fetch_everything_backwards_compat(self):
3445
 
        """Can fetch with EverythingResult even with pre 2.4 servers.
3446
 
        
3447
 
        Pre-2.4 do not support 'everything' searches with the
3448
 
        Repository.get_stream_1.19 verb.
3449
 
        """
3450
 
        verb_log = []
3451
 
        class OldGetStreamVerb(SmartServerRepositoryGetStream_1_19):
3452
 
            """A version of the Repository.get_stream_1.19 verb patched to
3453
 
            reject 'everything' searches the way 2.3 and earlier do.
3454
 
            """
3455
 
            def recreate_search(self, repository, search_bytes,
3456
 
                                discard_excess=False):
3457
 
                verb_log.append(search_bytes.split('\n', 1)[0])
3458
 
                if search_bytes == 'everything':
3459
 
                    return (None,
3460
 
                            request.FailedSmartServerResponse(('BadSearch',)))
3461
 
                return super(OldGetStreamVerb,
3462
 
                        self).recreate_search(repository, search_bytes,
3463
 
                            discard_excess=discard_excess)
3464
 
        self.override_verb('Repository.get_stream_1.19', OldGetStreamVerb)
3465
 
        local = self.make_branch('local')
3466
 
        builder = self.make_branch_builder('remote')
3467
 
        builder.build_commit(message="Commit.")
3468
 
        remote_branch_url = self.smart_server.get_url() + 'remote'
3469
 
        remote_branch = bzrdir.BzrDir.open(remote_branch_url).open_branch()
3470
 
        self.hpss_calls = []
3471
 
        local.repository.fetch(
3472
 
            remote_branch.repository,
3473
 
            fetch_spec=_mod_graph.EverythingResult(remote_branch.repository))
3474
 
        # make sure the overridden verb was used
3475
 
        self.assertLength(1, verb_log)
3476
 
        # more than one HPSS call is needed, but because it's a VFS callback
3477
 
        # its hard to predict exactly how many.
3478
 
        self.assertTrue(len(self.hpss_calls) > 1)
3479
 
 
3480
 
 
3481
 
class TestUpdateBoundBranchWithModifiedBoundLocation(
3482
 
    tests.TestCaseWithTransport):
3483
 
    """Ensure correct handling of bound_location modifications.
3484
 
 
3485
 
    This is tested against a smart server as http://pad.lv/786980 was about a
3486
 
    ReadOnlyError (write attempt during a read-only transaction) which can only
3487
 
    happen in this context.
3488
 
    """
3489
 
 
3490
 
    def setUp(self):
3491
 
        super(TestUpdateBoundBranchWithModifiedBoundLocation, self).setUp()
3492
 
        self.transport_server = test_server.SmartTCPServer_for_testing
3493
 
 
3494
 
    def make_master_and_checkout(self, master_name, checkout_name):
3495
 
        # Create the master branch and its associated checkout
3496
 
        self.master = self.make_branch_and_tree(master_name)
3497
 
        self.checkout = self.master.branch.create_checkout(checkout_name)
3498
 
        # Modify the master branch so there is something to update
3499
 
        self.master.commit('add stuff')
3500
 
        self.last_revid = self.master.commit('even more stuff')
3501
 
        self.bound_location = self.checkout.branch.get_bound_location()
3502
 
 
3503
 
    def assertUpdateSucceeds(self, new_location):
3504
 
        self.checkout.branch.set_bound_location(new_location)
3505
 
        self.checkout.update()
3506
 
        self.assertEquals(self.last_revid, self.checkout.last_revision())
3507
 
 
3508
 
    def test_without_final_slash(self):
3509
 
        self.make_master_and_checkout('master', 'checkout')
3510
 
        # For unclear reasons some users have a bound_location without a final
3511
 
        # '/', simulate that by forcing such a value
3512
 
        self.assertEndsWith(self.bound_location, '/')
3513
 
        self.assertUpdateSucceeds(self.bound_location.rstrip('/'))
3514
 
 
3515
 
    def test_plus_sign(self):
3516
 
        self.make_master_and_checkout('+master', 'checkout')
3517
 
        self.assertUpdateSucceeds(self.bound_location.replace('%2B', '+', 1))
3518
 
 
3519
 
    def test_tilda(self):
3520
 
        # Embed ~ in the middle of the path just to avoid any $HOME
3521
 
        # interpretation
3522
 
        self.make_master_and_checkout('mas~ter', 'checkout')
3523
 
        self.assertUpdateSucceeds(self.bound_location.replace('%2E', '~', 1))