~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_fetch.py

  • Committer: Robert Collins
  • Date: 2005-08-23 06:52:09 UTC
  • mto: (974.1.50) (1185.1.10) (1092.3.1)
  • mto: This revision was merged to the branch mainline in revision 1139.
  • Revision ID: robertc@robertcollins.net-20050823065209-81cd5962c401751b
move io redirection into each test case from the global runner

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2007 Canonical Ltd
2
 
#
3
 
# This program is free software; you can redistribute it and/or modify
4
 
# it under the terms of the GNU General Public License as published by
5
 
# the Free Software Foundation; either version 2 of the License, or
6
 
# (at your option) any later version.
7
 
#
8
 
# This program is distributed in the hope that it will be useful,
9
 
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
 
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
 
# GNU General Public License for more details.
12
 
#
13
 
# You should have received a copy of the GNU General Public License
14
 
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
import os
18
 
import re
19
 
import sys
20
 
 
21
 
import bzrlib
22
 
from bzrlib import (
23
 
    bzrdir,
24
 
    errors,
25
 
    merge,
26
 
    repository,
27
 
    versionedfile,
28
 
    )
29
 
from bzrlib.branch import Branch
30
 
from bzrlib.bzrdir import BzrDir
31
 
from bzrlib.repofmt import knitrepo
32
 
from bzrlib.tests import TestCaseWithTransport
33
 
from bzrlib.tests.http_utils import TestCaseWithWebserver
34
 
from bzrlib.tests.test_revision import make_branches
35
 
from bzrlib.trace import mutter
36
 
from bzrlib.upgrade import Convert
37
 
from bzrlib.workingtree import WorkingTree
38
 
 
39
 
# These tests are a bit old; please instead add new tests into
40
 
# interrepository_implementations/ so they'll run on all relevant
41
 
# combinations.
42
 
 
43
 
 
44
 
def has_revision(branch, revision_id):
45
 
    return branch.repository.has_revision(revision_id)
46
 
 
47
 
def fetch_steps(self, br_a, br_b, writable_a):
48
 
    """A foreign test method for testing fetch locally and remotely."""
49
 
     
50
 
    # TODO RBC 20060201 make this a repository test.
51
 
    repo_b = br_b.repository
52
 
    self.assertFalse(repo_b.has_revision(br_a.revision_history()[3]))
53
 
    self.assertTrue(repo_b.has_revision(br_a.revision_history()[2]))
54
 
    self.assertEquals(len(br_b.revision_history()), 7)
55
 
    self.assertEquals(br_b.fetch(br_a, br_a.revision_history()[2])[0], 0)
56
 
    # branch.fetch is not supposed to alter the revision history
57
 
    self.assertEquals(len(br_b.revision_history()), 7)
58
 
    self.assertFalse(repo_b.has_revision(br_a.revision_history()[3]))
59
 
 
60
 
    # fetching the next revision up in sample data copies one revision
61
 
    self.assertEquals(br_b.fetch(br_a, br_a.revision_history()[3])[0], 1)
62
 
    self.assertTrue(repo_b.has_revision(br_a.revision_history()[3]))
63
 
    self.assertFalse(has_revision(br_a, br_b.revision_history()[6]))
64
 
    self.assertTrue(br_a.repository.has_revision(br_b.revision_history()[5]))
65
 
 
66
 
    # When a non-branch ancestor is missing, it should be unlisted...
67
 
    # as its not reference from the inventory weave.
68
 
    br_b4 = self.make_branch('br_4')
69
 
    count, failures = br_b4.fetch(br_b)
70
 
    self.assertEqual(count, 7)
71
 
    self.assertEqual(failures, [])
72
 
 
73
 
    self.assertEqual(writable_a.fetch(br_b)[0], 1)
74
 
    self.assertTrue(has_revision(br_a, br_b.revision_history()[3]))
75
 
    self.assertTrue(has_revision(br_a, br_b.revision_history()[4]))
76
 
        
77
 
    br_b2 = self.make_branch('br_b2')
78
 
    self.assertEquals(br_b2.fetch(br_b)[0], 7)
79
 
    self.assertTrue(has_revision(br_b2, br_b.revision_history()[4]))
80
 
    self.assertTrue(has_revision(br_b2, br_a.revision_history()[2]))
81
 
    self.assertFalse(has_revision(br_b2, br_a.revision_history()[3]))
82
 
 
83
 
    br_a2 = self.make_branch('br_a2')
84
 
    self.assertEquals(br_a2.fetch(br_a)[0], 9)
85
 
    self.assertTrue(has_revision(br_a2, br_b.revision_history()[4]))
86
 
    self.assertTrue(has_revision(br_a2, br_a.revision_history()[3]))
87
 
    self.assertTrue(has_revision(br_a2, br_a.revision_history()[2]))
88
 
 
89
 
    br_a3 = self.make_branch('br_a3')
90
 
    # pulling a branch with no revisions grabs nothing, regardless of 
91
 
    # whats in the inventory.
92
 
    self.assertEquals(br_a3.fetch(br_a2)[0], 0)
93
 
    for revno in range(4):
94
 
        self.assertFalse(
95
 
            br_a3.repository.has_revision(br_a.revision_history()[revno]))
96
 
    self.assertEqual(br_a3.fetch(br_a2, br_a.revision_history()[2])[0], 3)
97
 
    # pull the 3 revisions introduced by a@u-0-3
98
 
    fetched = br_a3.fetch(br_a2, br_a.revision_history()[3])[0]
99
 
    self.assertEquals(fetched, 3, "fetched %d instead of 3" % fetched)
100
 
    # InstallFailed should be raised if the branch is missing the revision
101
 
    # that was requested.
102
 
    self.assertRaises(errors.InstallFailed, br_a3.fetch, br_a2, 'pizza')
103
 
 
104
 
    # TODO: Test trying to fetch from a branch that points to a revision not
105
 
    # actually present in its repository.  Not every branch format allows you
106
 
    # to directly point to such revisions, so it's a bit complicated to
107
 
    # construct.  One way would be to uncommit and gc the revision, but not
108
 
    # every branch supports that.  -- mbp 20070814
109
 
 
110
 
    #TODO: test that fetch correctly does reweaving when needed. RBC 20051008
111
 
    # Note that this means - updating the weave when ghosts are filled in to 
112
 
    # add the right parents.
113
 
 
114
 
 
115
 
class TestFetch(TestCaseWithTransport):
116
 
 
117
 
    def test_fetch(self):
118
 
        #highest indices a: 5, b: 7
119
 
        br_a, br_b = make_branches(self, format='dirstate-tags')
120
 
        fetch_steps(self, br_a, br_b, br_a)
121
 
 
122
 
    def test_fetch_self(self):
123
 
        wt = self.make_branch_and_tree('br')
124
 
        self.assertEqual(wt.branch.fetch(wt.branch), (0, []))
125
 
 
126
 
    def test_fetch_root_knit(self):
127
 
        """Ensure that knit2.fetch() updates the root knit
128
 
        
129
 
        This tests the case where the root has a new revision, but there are no
130
 
        corresponding filename, parent, contents or other changes.
131
 
        """
132
 
        knit1_format = bzrdir.BzrDirMetaFormat1()
133
 
        knit1_format.repository_format = knitrepo.RepositoryFormatKnit1()
134
 
        knit2_format = bzrdir.BzrDirMetaFormat1()
135
 
        knit2_format.repository_format = knitrepo.RepositoryFormatKnit3()
136
 
        # we start with a knit1 repository because that causes the
137
 
        # root revision to change for each commit, even though the content,
138
 
        # parent, name, and other attributes are unchanged.
139
 
        tree = self.make_branch_and_tree('tree', knit1_format)
140
 
        tree.set_root_id('tree-root')
141
 
        tree.commit('rev1', rev_id='rev1')
142
 
        tree.commit('rev2', rev_id='rev2')
143
 
 
144
 
        # Now we convert it to a knit2 repository so that it has a root knit
145
 
        Convert(tree.basedir, knit2_format)
146
 
        tree = WorkingTree.open(tree.basedir)
147
 
        branch = self.make_branch('branch', format=knit2_format)
148
 
        branch.pull(tree.branch, stop_revision='rev1')
149
 
        repo = branch.repository
150
 
        repo.lock_read()
151
 
        try:
152
 
            # Make sure fetch retrieved only what we requested
153
 
            self.assertEqual({('tree-root', 'rev1'):()},
154
 
                repo.texts.get_parent_map(
155
 
                    [('tree-root', 'rev1'), ('tree-root', 'rev2')]))
156
 
        finally:
157
 
            repo.unlock()
158
 
        branch.pull(tree.branch)
159
 
        # Make sure that the next revision in the root knit was retrieved,
160
 
        # even though the text, name, parent_id, etc., were unchanged.
161
 
        repo.lock_read()
162
 
        try:
163
 
            # Make sure fetch retrieved only what we requested
164
 
            self.assertEqual({('tree-root', 'rev2'):(('tree-root', 'rev1'),)},
165
 
                repo.texts.get_parent_map([('tree-root', 'rev2')]))
166
 
        finally:
167
 
            repo.unlock()
168
 
 
169
 
    def test_fetch_incompatible(self):
170
 
        knit_tree = self.make_branch_and_tree('knit', format='knit')
171
 
        knit3_tree = self.make_branch_and_tree('knit3',
172
 
            format='dirstate-with-subtree')
173
 
        knit3_tree.commit('blah')
174
 
        e = self.assertRaises(errors.IncompatibleRepositories,
175
 
                              knit_tree.branch.fetch, knit3_tree.branch)
176
 
        self.assertContainsRe(str(e),
177
 
            r"(?m).*/knit.*\nis not compatible with\n.*/knit3/.*\n"
178
 
            r"different rich-root support")
179
 
 
180
 
 
181
 
class TestMergeFetch(TestCaseWithTransport):
182
 
 
183
 
    def test_merge_fetches_unrelated(self):
184
 
        """Merge brings across history from unrelated source"""
185
 
        wt1 = self.make_branch_and_tree('br1')
186
 
        br1 = wt1.branch
187
 
        wt1.commit(message='rev 1-1', rev_id='1-1')
188
 
        wt1.commit(message='rev 1-2', rev_id='1-2')
189
 
        wt2 = self.make_branch_and_tree('br2')
190
 
        br2 = wt2.branch
191
 
        wt2.commit(message='rev 2-1', rev_id='2-1')
192
 
        wt2.merge_from_branch(br1, from_revision='null:')
193
 
        self._check_revs_present(br2)
194
 
 
195
 
    def test_merge_fetches(self):
196
 
        """Merge brings across history from source"""
197
 
        wt1 = self.make_branch_and_tree('br1')
198
 
        br1 = wt1.branch
199
 
        wt1.commit(message='rev 1-1', rev_id='1-1')
200
 
        dir_2 = br1.bzrdir.sprout('br2')
201
 
        br2 = dir_2.open_branch()
202
 
        wt1.commit(message='rev 1-2', rev_id='1-2')
203
 
        wt2 = dir_2.open_workingtree()
204
 
        wt2.commit(message='rev 2-1', rev_id='2-1')
205
 
        wt2.merge_from_branch(br1)
206
 
        self._check_revs_present(br2)
207
 
 
208
 
    def _check_revs_present(self, br2):
209
 
        for rev_id in '1-1', '1-2', '2-1':
210
 
            self.assertTrue(br2.repository.has_revision(rev_id))
211
 
            rev = br2.repository.get_revision(rev_id)
212
 
            self.assertEqual(rev.revision_id, rev_id)
213
 
            self.assertTrue(br2.repository.get_inventory(rev_id))
214
 
 
215
 
 
216
 
class TestMergeFileHistory(TestCaseWithTransport):
217
 
 
218
 
    def setUp(self):
219
 
        super(TestMergeFileHistory, self).setUp()
220
 
        wt1 = self.make_branch_and_tree('br1')
221
 
        br1 = wt1.branch
222
 
        self.build_tree_contents([('br1/file', 'original contents\n')])
223
 
        wt1.add('file', 'this-file-id')
224
 
        wt1.commit(message='rev 1-1', rev_id='1-1')
225
 
        dir_2 = br1.bzrdir.sprout('br2')
226
 
        br2 = dir_2.open_branch()
227
 
        wt2 = dir_2.open_workingtree()
228
 
        self.build_tree_contents([('br1/file', 'original from 1\n')])
229
 
        wt1.commit(message='rev 1-2', rev_id='1-2')
230
 
        self.build_tree_contents([('br1/file', 'agreement\n')])
231
 
        wt1.commit(message='rev 1-3', rev_id='1-3')
232
 
        self.build_tree_contents([('br2/file', 'contents in 2\n')])
233
 
        wt2.commit(message='rev 2-1', rev_id='2-1')
234
 
        self.build_tree_contents([('br2/file', 'agreement\n')])
235
 
        wt2.commit(message='rev 2-2', rev_id='2-2')
236
 
 
237
 
    def test_merge_fetches_file_history(self):
238
 
        """Merge brings across file histories"""
239
 
        br2 = Branch.open('br2')
240
 
        br1 = Branch.open('br1')
241
 
        wt2 = WorkingTree.open('br2').merge_from_branch(br1)
242
 
        br2.lock_read()
243
 
        self.addCleanup(br2.unlock)
244
 
        for rev_id, text in [('1-2', 'original from 1\n'),
245
 
                             ('1-3', 'agreement\n'),
246
 
                             ('2-1', 'contents in 2\n'),
247
 
                             ('2-2', 'agreement\n')]:
248
 
            self.assertEqualDiff(
249
 
                br2.repository.revision_tree(
250
 
                    rev_id).get_file_text('this-file-id'), text)
251
 
 
252
 
 
253
 
class TestHttpFetch(TestCaseWithWebserver):
254
 
    # FIXME RBC 20060124 this really isn't web specific, perhaps an
255
 
    # instrumented readonly transport? Can we do an instrumented
256
 
    # adapter and use self.get_readonly_url ?
257
 
 
258
 
    def test_fetch(self):
259
 
        #highest indices a: 5, b: 7
260
 
        br_a, br_b = make_branches(self)
261
 
        br_rem_a = Branch.open(self.get_readonly_url('branch1'))
262
 
        fetch_steps(self, br_rem_a, br_b, br_a)
263
 
 
264
 
    def _count_log_matches(self, target, logs):
265
 
        """Count the number of times the target file pattern was fetched in an http log"""
266
 
        get_succeeds_re = re.compile(
267
 
            '.*"GET .*%s HTTP/1.1" 20[06] - "-" "bzr/%s' %
268
 
            (     target,                    bzrlib.__version__))
269
 
        c = 0
270
 
        for line in logs:
271
 
            if get_succeeds_re.match(line):
272
 
                c += 1
273
 
        return c
274
 
 
275
 
    def test_weaves_are_retrieved_once(self):
276
 
        self.build_tree(("source/", "source/file", "target/"))
277
 
        # This test depends on knit dasta storage.
278
 
        wt = self.make_branch_and_tree('source', format='dirstate-tags')
279
 
        branch = wt.branch
280
 
        wt.add(["file"], ["id"])
281
 
        wt.commit("added file")
282
 
        open("source/file", 'w').write("blah\n")
283
 
        wt.commit("changed file")
284
 
        target = BzrDir.create_branch_and_repo("target/")
285
 
        source = Branch.open(self.get_readonly_url("source/"))
286
 
        self.assertEqual(target.fetch(source), (2, []))
287
 
        # this is the path to the literal file. As format changes 
288
 
        # occur it needs to be updated. FIXME: ask the store for the
289
 
        # path.
290
 
        self.log("web server logs are:")
291
 
        http_logs = self.get_readonly_server().logs
292
 
        self.log('\n'.join(http_logs))
293
 
        # unfortunately this log entry is branch format specific. We could 
294
 
        # factor out the 'what files does this format use' to a method on the 
295
 
        # repository, which would let us to this generically. RBC 20060419
296
 
        # RBC 20080408: Or perhaps we can assert that no files are fully read
297
 
        # twice?
298
 
        self.assertEqual(1, self._count_log_matches('/ce/id.kndx', http_logs))
299
 
        self.assertEqual(1, self._count_log_matches('/ce/id.knit', http_logs))
300
 
        self.assertEqual(1, self._count_log_matches('inventory.kndx', http_logs))
301
 
        # this r-h check test will prevent regressions, but it currently already 
302
 
        # passes, before the patch to cache-rh is applied :[
303
 
        self.assertTrue(1 >= self._count_log_matches('revision-history',
304
 
                                                     http_logs))
305
 
        self.assertTrue(1 >= self._count_log_matches('last-revision',
306
 
                                                     http_logs))
307
 
        # FIXME naughty poking in there.
308
 
        self.get_readonly_server().logs = []
309
 
        # check there is nothing more to fetch.  We take care to re-use the
310
 
        # existing transport so that the request logs we're about to examine
311
 
        # aren't cluttered with redundant probes for a smart server.
312
 
        # XXX: Perhaps this further parameterisation: test http with smart
313
 
        # server, and test http without smart server?
314
 
        source = Branch.open(
315
 
            self.get_readonly_url("source/"),
316
 
            possible_transports=[source.bzrdir.root_transport])
317
 
        self.assertEqual(target.fetch(source), (0, []))
318
 
        # should make just two requests
319
 
        http_logs = self.get_readonly_server().logs
320
 
        self.log("web server logs are:")
321
 
        self.log('\n'.join(http_logs))
322
 
        self.assertEqual(1, self._count_log_matches('branch-format', http_logs))
323
 
        self.assertEqual(1, self._count_log_matches('branch/format', http_logs))
324
 
        self.assertEqual(1, self._count_log_matches('repository/format',
325
 
            http_logs))
326
 
        self.assertTrue(1 >= self._count_log_matches('revision-history',
327
 
                                                     http_logs))
328
 
        self.assertTrue(1 >= self._count_log_matches('last-revision',
329
 
                                                     http_logs))
330
 
        self.assertEqual(4, len(http_logs))
331
 
 
332
 
 
333
 
class TestKnitToPackFetch(TestCaseWithTransport):
334
 
 
335
 
    def find_get_record_stream(self, calls):
336
 
        """In a list of calls, find 'get_record_stream' calls.
337
 
 
338
 
        This also ensures that there is only one get_record_stream call.
339
 
        """
340
 
        get_record_call = None
341
 
        for call in calls:
342
 
            if call[0] == 'get_record_stream':
343
 
                self.assertIs(None, get_record_call,
344
 
                              "there should only be one call to"
345
 
                              " get_record_stream")
346
 
                get_record_call = call
347
 
        self.assertIsNot(None, get_record_call,
348
 
                         "there should be exactly one call to "
349
 
                         " get_record_stream")
350
 
        return get_record_call
351
 
 
352
 
    def test_fetch_with_deltas_no_delta_closure(self):
353
 
        tree = self.make_branch_and_tree('source', format='dirstate')
354
 
        target = self.make_repository('target', format='pack-0.92')
355
 
        self.build_tree(['source/file'])
356
 
        tree.set_root_id('root-id')
357
 
        tree.add('file', 'file-id')
358
 
        tree.commit('one', rev_id='rev-one')
359
 
        source = tree.branch.repository
360
 
        source.texts = versionedfile.RecordingVersionedFilesDecorator(
361
 
                        source.texts)
362
 
        source.signatures = versionedfile.RecordingVersionedFilesDecorator(
363
 
                        source.signatures)
364
 
        source.revisions = versionedfile.RecordingVersionedFilesDecorator(
365
 
                        source.revisions)
366
 
        source.inventories = versionedfile.RecordingVersionedFilesDecorator(
367
 
                        source.inventories)
368
 
        # precondition
369
 
        self.assertTrue(target._fetch_uses_deltas)
370
 
        target.fetch(source, revision_id='rev-one')
371
 
        self.assertEqual(('get_record_stream', [('file-id', 'rev-one')],
372
 
                          target._fetch_order, False),
373
 
                         self.find_get_record_stream(source.texts.calls))
374
 
        self.assertEqual(('get_record_stream', [('rev-one',)],
375
 
                          target._fetch_order, False),
376
 
                         self.find_get_record_stream(source.inventories.calls))
377
 
        # Because of bugs in the old fetch code, revisions could accidentally
378
 
        # have deltas present in knits. However, it was never intended, so we
379
 
        # always for include_delta_closure=True, to make sure we get fulltexts.
380
 
        # bug #261339
381
 
        self.assertEqual(('get_record_stream', [('rev-one',)],
382
 
                          target._fetch_order, True),
383
 
                         self.find_get_record_stream(source.revisions.calls))
384
 
        # XXX: Signatures is special, and slightly broken. The
385
 
        # standard item_keys_introduced_by actually does a lookup for every
386
 
        # signature to see if it exists, rather than waiting to do them all at
387
 
        # once at the end. The fetch code then does an all-at-once and just
388
 
        # allows for some of them to be missing.
389
 
        # So we know there will be extra calls, but the *last* one is the one
390
 
        # we care about.
391
 
        signature_calls = source.signatures.calls[-1:]
392
 
        self.assertEqual(('get_record_stream', [('rev-one',)],
393
 
                          target._fetch_order, True),
394
 
                         self.find_get_record_stream(signature_calls))
395
 
 
396
 
    def test_fetch_no_deltas_with_delta_closure(self):
397
 
        tree = self.make_branch_and_tree('source', format='dirstate')
398
 
        target = self.make_repository('target', format='pack-0.92')
399
 
        self.build_tree(['source/file'])
400
 
        tree.set_root_id('root-id')
401
 
        tree.add('file', 'file-id')
402
 
        tree.commit('one', rev_id='rev-one')
403
 
        source = tree.branch.repository
404
 
        source.texts = versionedfile.RecordingVersionedFilesDecorator(
405
 
                        source.texts)
406
 
        source.signatures = versionedfile.RecordingVersionedFilesDecorator(
407
 
                        source.signatures)
408
 
        source.revisions = versionedfile.RecordingVersionedFilesDecorator(
409
 
                        source.revisions)
410
 
        source.inventories = versionedfile.RecordingVersionedFilesDecorator(
411
 
                        source.inventories)
412
 
        target._fetch_uses_deltas = False
413
 
        target.fetch(source, revision_id='rev-one')
414
 
        self.assertEqual(('get_record_stream', [('file-id', 'rev-one')],
415
 
                          target._fetch_order, True),
416
 
                         self.find_get_record_stream(source.texts.calls))
417
 
        self.assertEqual(('get_record_stream', [('rev-one',)],
418
 
                          target._fetch_order, True),
419
 
                         self.find_get_record_stream(source.inventories.calls))
420
 
        self.assertEqual(('get_record_stream', [('rev-one',)],
421
 
                          target._fetch_order, True),
422
 
                         self.find_get_record_stream(source.revisions.calls))
423
 
        # XXX: Signatures is special, and slightly broken. The
424
 
        # standard item_keys_introduced_by actually does a lookup for every
425
 
        # signature to see if it exists, rather than waiting to do them all at
426
 
        # once at the end. The fetch code then does an all-at-once and just
427
 
        # allows for some of them to be missing.
428
 
        # So we know there will be extra calls, but the *last* one is the one
429
 
        # we care about.
430
 
        signature_calls = source.signatures.calls[-1:]
431
 
        self.assertEqual(('get_record_stream', [('rev-one',)],
432
 
                          target._fetch_order, True),
433
 
                         self.find_get_record_stream(signature_calls))
434
 
 
435
 
 
436
 
class Test1To2Fetch(TestCaseWithTransport):
437
 
    """Tests for Model1To2 failure modes"""
438
 
 
439
 
    def make_tree_and_repo(self):
440
 
        self.tree = self.make_branch_and_tree('tree', format='pack-0.92')
441
 
        self.repo = self.make_repository('rich-repo', format='rich-root-pack')
442
 
        self.repo.lock_write()
443
 
        self.addCleanup(self.repo.unlock)
444
 
 
445
 
    def do_fetch_order_test(self, first, second):
446
 
        """Test that fetch works no matter what the set order of revision is.
447
 
 
448
 
        This test depends on the order of items in a set, which is
449
 
        implementation-dependant, so we test A, B and then B, A.
450
 
        """
451
 
        self.make_tree_and_repo()
452
 
        self.tree.commit('Commit 1', rev_id=first)
453
 
        self.tree.commit('Commit 2', rev_id=second)
454
 
        self.repo.fetch(self.tree.branch.repository, second)
455
 
 
456
 
    def test_fetch_order_AB(self):
457
 
        """See do_fetch_order_test"""
458
 
        self.do_fetch_order_test('A', 'B')
459
 
 
460
 
    def test_fetch_order_BA(self):
461
 
        """See do_fetch_order_test"""
462
 
        self.do_fetch_order_test('B', 'A')
463
 
 
464
 
    def get_parents(self, file_id, revision_id):
465
 
        self.repo.lock_read()
466
 
        try:
467
 
            parent_map = self.repo.texts.get_parent_map([(file_id, revision_id)])
468
 
            return parent_map[(file_id, revision_id)]
469
 
        finally:
470
 
            self.repo.unlock()
471
 
 
472
 
    def test_fetch_ghosts(self):
473
 
        self.make_tree_and_repo()
474
 
        self.tree.commit('first commit', rev_id='left-parent')
475
 
        self.tree.add_parent_tree_id('ghost-parent')
476
 
        fork = self.tree.bzrdir.sprout('fork', 'null:').open_workingtree()
477
 
        fork.commit('not a ghost', rev_id='not-ghost-parent')
478
 
        self.tree.branch.repository.fetch(fork.branch.repository,
479
 
                                     'not-ghost-parent')
480
 
        self.tree.add_parent_tree_id('not-ghost-parent')
481
 
        self.tree.commit('second commit', rev_id='second-id')
482
 
        self.repo.fetch(self.tree.branch.repository, 'second-id')
483
 
        root_id = self.tree.get_root_id()
484
 
        self.assertEqual(
485
 
            ((root_id, 'left-parent'), (root_id, 'ghost-parent'),
486
 
             (root_id, 'not-ghost-parent')),
487
 
            self.get_parents(root_id, 'second-id'))
488
 
 
489
 
    def make_two_commits(self, change_root, fetch_twice):
490
 
        self.make_tree_and_repo()
491
 
        self.tree.commit('first commit', rev_id='first-id')
492
 
        if change_root:
493
 
            self.tree.set_root_id('unique-id')
494
 
        self.tree.commit('second commit', rev_id='second-id')
495
 
        if fetch_twice:
496
 
            self.repo.fetch(self.tree.branch.repository, 'first-id')
497
 
        self.repo.fetch(self.tree.branch.repository, 'second-id')
498
 
 
499
 
    def test_fetch_changed_root(self):
500
 
        self.make_two_commits(change_root=True, fetch_twice=False)
501
 
        self.assertEqual((), self.get_parents('unique-id', 'second-id'))
502
 
 
503
 
    def test_two_fetch_changed_root(self):
504
 
        self.make_two_commits(change_root=True, fetch_twice=True)
505
 
        self.assertEqual((), self.get_parents('unique-id', 'second-id'))
506
 
 
507
 
    def test_two_fetches(self):
508
 
        self.make_two_commits(change_root=False, fetch_twice=True)
509
 
        self.assertEqual((('TREE_ROOT', 'first-id'),),
510
 
            self.get_parents('TREE_ROOT', 'second-id'))