← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_repository.py

  • Committer: Andrew Bennetts
  • Date: 2011-03-24 00:13:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5739.
  • Revision ID: andrew.bennetts@canonical.com-20110324001310-5w56qjom53j091r3
Minor English tweaks.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_repository.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006-2011 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
23
23
"""
24
24
 
25
25
from stat import S_ISDIR
26
 
import sys
27
26
 
28
27
import bzrlib
29
 
from bzrlib.errors import (NoSuchFile,
30
 
                           UnknownFormatError,
31
 
                           UnsupportedFormatError,
32
 
                           )
 
28
from bzrlib.errors import (
 
29
    UnknownFormatError,
 
30
    UnsupportedFormatError,
 
31
    )
33
32
from bzrlib import (
 
33
    btree_index,
34
34
    graph,
 
35
    symbol_versioning,
35
36
    tests,
 
37
    transport,
36
38
    )
37
39
from bzrlib.btree_index import BTreeBuilder, BTreeGraphIndex
38
40
from bzrlib.index import GraphIndex
41
43
    TestCase,
42
44
    TestCaseWithTransport,
43
45
    )
44
 
from bzrlib.transport import (
45
 
    get_transport,
46
 
    )
47
46
from bzrlib import (
48
47
    bzrdir,
49
48
    errors,
59
58
    groupcompress_repo,
60
59
    knitrepo,
61
60
    pack_repo,
62
 
    weaverepo,
63
61
    )
64
62
 
65
63
 
68
66
    def test_get_set_default_format(self):
69
67
        old_default = bzrdir.format_registry.get('default')
70
68
        private_default = old_default().repository_format.__class__
71
 
        old_format = repository.RepositoryFormat.get_default_format()
 
69
        old_format = repository.format_registry.get_default()
72
70
        self.assertTrue(isinstance(old_format, private_default))
73
71
        def make_sample_bzrdir():
74
72
            my_bzrdir = bzrdir.BzrDirMetaFormat1()
88
86
            bzrdir.format_registry.remove('default')
89
87
            bzrdir.format_registry.remove('sample')
90
88
            bzrdir.format_registry.register('default', old_default, '')
91
 
        self.assertIsInstance(repository.RepositoryFormat.get_default_format(),
 
89
        self.assertIsInstance(repository.format_registry.get_default(),
92
90
                              old_format.__class__)
93
91
 
94
92
 
116
114
        return "opened repository."
117
115
 
118
116
 
 
117
class SampleExtraRepositoryFormat(repository.RepositoryFormat):
 
118
    """A sample format that can not be used in a metadir
 
119
 
 
120
    """
 
121
 
 
122
    def get_format_string(self):
 
123
        raise NotImplementedError
 
124
 
 
125
 
119
126
class TestRepositoryFormat(TestCaseWithTransport):
120
127
    """Tests for the Repository format detection used by the bzr meta dir facility.BzrBranchFormat facility."""
121
128
 
127
134
        def check_format(format, url):
128
135
            dir = format._matchingbzrdir.initialize(url)
129
136
            format.initialize(dir)
130
 
            t = get_transport(url)
 
137
            t = transport.get_transport(url)
131
138
            found_format = repository.RepositoryFormat.find_format(dir)
132
139
            self.failUnless(isinstance(found_format, format.__class__))
133
 
        check_format(weaverepo.RepositoryFormat7(), "bar")
 
140
        check_format(repository.format_registry.get_default(), "bar")
134
141
 
135
142
    def test_find_format_no_repository(self):
136
143
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
146
153
                          dir)
147
154
 
148
155
    def test_register_unregister_format(self):
 
156
        # Test deprecated format registration functions
149
157
        format = SampleRepositoryFormat()
150
158
        # make a control dir
151
159
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
152
160
        # make a repo
153
161
        format.initialize(dir)
154
162
        # register a format for it.
155
 
        repository.RepositoryFormat.register_format(format)
 
163
        self.applyDeprecated(symbol_versioning.deprecated_in((2, 4, 0)),
 
164
            repository.RepositoryFormat.register_format, format)
156
165
        # which repository.Open will refuse (not supported)
157
 
        self.assertRaises(UnsupportedFormatError, repository.Repository.open, self.get_url())
 
166
        self.assertRaises(UnsupportedFormatError, repository.Repository.open,
 
167
            self.get_url())
158
168
        # but open(unsupported) will work
159
169
        self.assertEqual(format.open(dir), "opened repository.")
160
170
        # unregister the format
161
 
        repository.RepositoryFormat.unregister_format(format)
162
 
 
163
 
 
164
 
class TestFormat6(TestCaseWithTransport):
165
 
 
166
 
    def test_attribute__fetch_order(self):
167
 
        """Weaves need topological data insertion."""
168
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
169
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
170
 
        self.assertEqual('topological', repo._format._fetch_order)
171
 
 
172
 
    def test_attribute__fetch_uses_deltas(self):
173
 
        """Weaves do not reuse deltas."""
174
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
175
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
176
 
        self.assertEqual(False, repo._format._fetch_uses_deltas)
177
 
 
178
 
    def test_attribute__fetch_reconcile(self):
179
 
        """Weave repositories need a reconcile after fetch."""
180
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
181
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
182
 
        self.assertEqual(True, repo._format._fetch_reconcile)
183
 
 
184
 
    def test_no_ancestry_weave(self):
185
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
186
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
187
 
        # We no longer need to create the ancestry.weave file
188
 
        # since it is *never* used.
189
 
        self.assertRaises(NoSuchFile,
190
 
                          control.transport.get,
191
 
                          'ancestry.weave')
192
 
 
193
 
    def test_supports_external_lookups(self):
194
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
195
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
196
 
        self.assertFalse(repo._format.supports_external_lookups)
197
 
 
198
 
 
199
 
class TestFormat7(TestCaseWithTransport):
200
 
 
201
 
    def test_attribute__fetch_order(self):
202
 
        """Weaves need topological data insertion."""
203
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
204
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
205
 
        self.assertEqual('topological', repo._format._fetch_order)
206
 
 
207
 
    def test_attribute__fetch_uses_deltas(self):
208
 
        """Weaves do not reuse deltas."""
209
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
210
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
211
 
        self.assertEqual(False, repo._format._fetch_uses_deltas)
212
 
 
213
 
    def test_attribute__fetch_reconcile(self):
214
 
        """Weave repositories need a reconcile after fetch."""
215
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
216
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
217
 
        self.assertEqual(True, repo._format._fetch_reconcile)
218
 
 
219
 
    def test_disk_layout(self):
220
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
221
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
222
 
        # in case of side effects of locking.
223
 
        repo.lock_write()
224
 
        repo.unlock()
225
 
        # we want:
226
 
        # format 'Bazaar-NG Repository format 7'
227
 
        # lock ''
228
 
        # inventory.weave == empty_weave
229
 
        # empty revision-store directory
230
 
        # empty weaves directory
231
 
        t = control.get_repository_transport(None)
232
 
        self.assertEqualDiff('Bazaar-NG Repository format 7',
233
 
                             t.get('format').read())
234
 
        self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
235
 
        self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
236
 
        self.assertEqualDiff('# bzr weave file v5\n'
237
 
                             'w\n'
238
 
                             'W\n',
239
 
                             t.get('inventory.weave').read())
240
 
        # Creating a file with id Foo:Bar results in a non-escaped file name on
241
 
        # disk.
242
 
        control.create_branch()
243
 
        tree = control.create_workingtree()
244
 
        tree.add(['foo'], ['Foo:Bar'], ['file'])
245
 
        tree.put_file_bytes_non_atomic('Foo:Bar', 'content\n')
246
 
        try:
247
 
            tree.commit('first post', rev_id='first')
248
 
        except errors.IllegalPath:
249
 
            if sys.platform != 'win32':
250
 
                raise
251
 
            self.knownFailure('Foo:Bar cannot be used as a file-id on windows'
252
 
                              ' in repo format 7')
253
 
            return
254
 
        self.assertEqualDiff(
255
 
            '# bzr weave file v5\n'
256
 
            'i\n'
257
 
            '1 7fe70820e08a1aac0ef224d9c66ab66831cc4ab1\n'
258
 
            'n first\n'
259
 
            '\n'
260
 
            'w\n'
261
 
            '{ 0\n'
262
 
            '. content\n'
263
 
            '}\n'
264
 
            'W\n',
265
 
            t.get('weaves/74/Foo%3ABar.weave').read())
266
 
 
267
 
    def test_shared_disk_layout(self):
268
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
269
 
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
270
 
        # we want:
271
 
        # format 'Bazaar-NG Repository format 7'
272
 
        # inventory.weave == empty_weave
273
 
        # empty revision-store directory
274
 
        # empty weaves directory
275
 
        # a 'shared-storage' marker file.
276
 
        # lock is not present when unlocked
277
 
        t = control.get_repository_transport(None)
278
 
        self.assertEqualDiff('Bazaar-NG Repository format 7',
279
 
                             t.get('format').read())
280
 
        self.assertEqualDiff('', t.get('shared-storage').read())
281
 
        self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
282
 
        self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
283
 
        self.assertEqualDiff('# bzr weave file v5\n'
284
 
                             'w\n'
285
 
                             'W\n',
286
 
                             t.get('inventory.weave').read())
287
 
        self.assertFalse(t.has('branch-lock'))
288
 
 
289
 
    def test_creates_lockdir(self):
290
 
        """Make sure it appears to be controlled by a LockDir existence"""
291
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
292
 
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
293
 
        t = control.get_repository_transport(None)
294
 
        # TODO: Should check there is a 'lock' toplevel directory,
295
 
        # regardless of contents
296
 
        self.assertFalse(t.has('lock/held/info'))
297
 
        repo.lock_write()
298
 
        try:
299
 
            self.assertTrue(t.has('lock/held/info'))
300
 
        finally:
301
 
            # unlock so we don't get a warning about failing to do so
302
 
            repo.unlock()
303
 
 
304
 
    def test_uses_lockdir(self):
305
 
        """repo format 7 actually locks on lockdir"""
306
 
        base_url = self.get_url()
307
 
        control = bzrdir.BzrDirMetaFormat1().initialize(base_url)
308
 
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
309
 
        t = control.get_repository_transport(None)
310
 
        repo.lock_write()
311
 
        repo.unlock()
312
 
        del repo
313
 
        # make sure the same lock is created by opening it
314
 
        repo = repository.Repository.open(base_url)
315
 
        repo.lock_write()
316
 
        self.assertTrue(t.has('lock/held/info'))
317
 
        repo.unlock()
318
 
        self.assertFalse(t.has('lock/held/info'))
319
 
 
320
 
    def test_shared_no_tree_disk_layout(self):
321
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
322
 
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
323
 
        repo.set_make_working_trees(False)
324
 
        # we want:
325
 
        # format 'Bazaar-NG Repository format 7'
326
 
        # lock ''
327
 
        # inventory.weave == empty_weave
328
 
        # empty revision-store directory
329
 
        # empty weaves directory
330
 
        # a 'shared-storage' marker file.
331
 
        t = control.get_repository_transport(None)
332
 
        self.assertEqualDiff('Bazaar-NG Repository format 7',
333
 
                             t.get('format').read())
334
 
        ## self.assertEqualDiff('', t.get('lock').read())
335
 
        self.assertEqualDiff('', t.get('shared-storage').read())
336
 
        self.assertEqualDiff('', t.get('no-working-trees').read())
337
 
        repo.set_make_working_trees(True)
338
 
        self.assertFalse(t.has('no-working-trees'))
339
 
        self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
340
 
        self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
341
 
        self.assertEqualDiff('# bzr weave file v5\n'
342
 
                             'w\n'
343
 
                             'W\n',
344
 
                             t.get('inventory.weave').read())
345
 
 
346
 
    def test_supports_external_lookups(self):
347
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
348
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
349
 
        self.assertFalse(repo._format.supports_external_lookups)
 
171
        self.applyDeprecated(symbol_versioning.deprecated_in((2, 4, 0)),
 
172
            repository.RepositoryFormat.unregister_format, format)
 
173
 
 
174
 
 
175
class TestRepositoryFormatRegistry(TestCase):
 
176
 
 
177
    def setUp(self):
 
178
        super(TestRepositoryFormatRegistry, self).setUp()
 
179
        self.registry = repository.RepositoryFormatRegistry()
 
180
 
 
181
    def test_register_unregister_format(self):
 
182
        format = SampleRepositoryFormat()
 
183
        self.registry.register(format)
 
184
        self.assertEquals(format, self.registry.get("Sample .bzr repository format."))
 
185
        self.registry.remove(format)
 
186
        self.assertRaises(KeyError, self.registry.get, "Sample .bzr repository format.")
 
187
 
 
188
    def test_get_all(self):
 
189
        format = SampleRepositoryFormat()
 
190
        self.assertEquals([], self.registry._get_all())
 
191
        self.registry.register(format)
 
192
        self.assertEquals([format], self.registry._get_all())
 
193
 
 
194
    def test_register_extra(self):
 
195
        format = SampleExtraRepositoryFormat()
 
196
        self.assertEquals([], self.registry._get_all())
 
197
        self.registry.register_extra(format)
 
198
        self.assertEquals([format], self.registry._get_all())
 
199
 
 
200
    def test_register_extra_lazy(self):
 
201
        self.assertEquals([], self.registry._get_all())
 
202
        self.registry.register_extra_lazy("bzrlib.tests.test_repository",
 
203
            "SampleExtraRepositoryFormat")
 
204
        formats = self.registry._get_all()
 
205
        self.assertEquals(1, len(formats))
 
206
        self.assertIsInstance(formats[0], SampleExtraRepositoryFormat)
350
207
 
351
208
 
352
209
class TestFormatKnit1(TestCaseWithTransport):
571
428
        self.assertGetsDefaultInterRepository(dummy_a, dummy_b)
572
429
 
573
430
 
574
 
class TestInterWeaveRepo(TestCaseWithTransport):
575
 
 
576
 
    def test_is_compatible_and_registered(self):
577
 
        # InterWeaveRepo is compatible when either side
578
 
        # is a format 5/6/7 branch
579
 
        from bzrlib.repofmt import knitrepo, weaverepo
580
 
        formats = [weaverepo.RepositoryFormat5(),
581
 
                   weaverepo.RepositoryFormat6(),
582
 
                   weaverepo.RepositoryFormat7()]
583
 
        incompatible_formats = [weaverepo.RepositoryFormat4(),
584
 
                                knitrepo.RepositoryFormatKnit1(),
585
 
                                ]
586
 
        repo_a = self.make_repository('a')
587
 
        repo_b = self.make_repository('b')
588
 
        is_compatible = repository.InterWeaveRepo.is_compatible
589
 
        for source in incompatible_formats:
590
 
            # force incompatible left then right
591
 
            repo_a._format = source
592
 
            repo_b._format = formats[0]
593
 
            self.assertFalse(is_compatible(repo_a, repo_b))
594
 
            self.assertFalse(is_compatible(repo_b, repo_a))
595
 
        for source in formats:
596
 
            repo_a._format = source
597
 
            for target in formats:
598
 
                repo_b._format = target
599
 
                self.assertTrue(is_compatible(repo_a, repo_b))
600
 
        self.assertEqual(repository.InterWeaveRepo,
601
 
                         repository.InterRepository.get(repo_a,
602
 
                                                        repo_b).__class__)
 
431
class TestRepositoryFormat1(knitrepo.RepositoryFormatKnit1):
 
432
 
 
433
    def get_format_string(self):
 
434
        return "Test Format 1"
 
435
 
 
436
 
 
437
class TestRepositoryFormat2(knitrepo.RepositoryFormatKnit1):
 
438
 
 
439
    def get_format_string(self):
 
440
        return "Test Format 2"
603
441
 
604
442
 
605
443
class TestRepositoryConverter(TestCaseWithTransport):
606
444
 
607
445
    def test_convert_empty(self):
608
 
        t = get_transport(self.get_url('.'))
 
446
        source_format = TestRepositoryFormat1()
 
447
        target_format = TestRepositoryFormat2()
 
448
        repository.format_registry.register(source_format)
 
449
        self.addCleanup(repository.format_registry.remove,
 
450
            source_format)
 
451
        repository.format_registry.register(target_format)
 
452
        self.addCleanup(repository.format_registry.remove,
 
453
            target_format)
 
454
        t = self.get_transport()
609
455
        t.mkdir('repository')
610
456
        repo_dir = bzrdir.BzrDirMetaFormat1().initialize('repository')
611
 
        repo = weaverepo.RepositoryFormat7().initialize(repo_dir)
612
 
        target_format = knitrepo.RepositoryFormatKnit1()
 
457
        repo = TestRepositoryFormat1().initialize(repo_dir)
613
458
        converter = repository.CopyConverter(target_format)
614
459
        pb = bzrlib.ui.ui_factory.nested_progress_bar()
615
460
        try:
620
465
        self.assertTrue(isinstance(target_format, repo._format.__class__))
621
466
 
622
467
 
623
 
class TestMisc(TestCase):
624
 
 
625
 
    def test_unescape_xml(self):
626
 
        """We get some kind of error when malformed entities are passed"""
627
 
        self.assertRaises(KeyError, repository._unescape_xml, 'foo&bar;')
628
 
 
629
 
 
630
468
class TestRepositoryFormatKnit3(TestCaseWithTransport):
631
469
 
632
470
    def test_attribute__fetch_order(self):
681
519
 
682
520
class Test2a(tests.TestCaseWithMemoryTransport):
683
521
 
 
522
    def test_chk_bytes_uses_custom_btree_parser(self):
 
523
        mt = self.make_branch_and_memory_tree('test', format='2a')
 
524
        mt.lock_write()
 
525
        self.addCleanup(mt.unlock)
 
526
        mt.add([''], ['root-id'])
 
527
        mt.commit('first')
 
528
        index = mt.branch.repository.chk_bytes._index._graph_index._indices[0]
 
529
        self.assertEqual(btree_index._gcchk_factory, index._leaf_factory)
 
530
        # It should also work if we re-open the repo
 
531
        repo = mt.branch.repository.bzrdir.open_repository()
 
532
        repo.lock_read()
 
533
        self.addCleanup(repo.unlock)
 
534
        index = repo.chk_bytes._index._graph_index._indices[0]
 
535
        self.assertEqual(btree_index._gcchk_factory, index._leaf_factory)
 
536
 
684
537
    def test_fetch_combines_groups(self):
685
538
        builder = self.make_branch_builder('source', format='2a')
686
539
        builder.start_series()
956
809
 
957
810
    def setUp(self):
958
811
        super(TestDevelopment6FindParentIdsOfRevisions, self).setUp()
959
 
        self.builder = self.make_branch_builder('source',
960
 
            format='development6-rich-root')
 
812
        self.builder = self.make_branch_builder('source')
961
813
        self.builder.start_series()
962
814
        self.builder.build_snapshot('initial', None,
963
815
            [('add', ('', 'tree-root', 'directory', None))])
1622
1474
        self.assertTrue(new_pack.signature_index._optimize_for_size)
1623
1475
 
1624
1476
 
 
1477
class TestGCCHKPacker(TestCaseWithTransport):
 
1478
 
 
1479
    def make_abc_branch(self):
 
1480
        builder = self.make_branch_builder('source')
 
1481
        builder.start_series()
 
1482
        builder.build_snapshot('A', None, [
 
1483
            ('add', ('', 'root-id', 'directory', None)),
 
1484
            ('add', ('file', 'file-id', 'file', 'content\n')),
 
1485
            ])
 
1486
        builder.build_snapshot('B', ['A'], [
 
1487
            ('add', ('dir', 'dir-id', 'directory', None))])
 
1488
        builder.build_snapshot('C', ['B'], [
 
1489
            ('modify', ('file-id', 'new content\n'))])
 
1490
        builder.finish_series()
 
1491
        return builder.get_branch()
 
1492
 
 
1493
    def make_branch_with_disjoint_inventory_and_revision(self):
 
1494
        """a repo with separate packs for a revisions Revision and Inventory.
 
1495
 
 
1496
        There will be one pack file that holds the Revision content, and one
 
1497
        for the Inventory content.
 
1498
 
 
1499
        :return: (repository,
 
1500
                  pack_name_with_rev_A_Revision,
 
1501
                  pack_name_with_rev_A_Inventory,
 
1502
                  pack_name_with_rev_C_content)
 
1503
        """
 
1504
        b_source = self.make_abc_branch()
 
1505
        b_base = b_source.bzrdir.sprout('base', revision_id='A').open_branch()
 
1506
        b_stacked = b_base.bzrdir.sprout('stacked', stacked=True).open_branch()
 
1507
        b_stacked.lock_write()
 
1508
        self.addCleanup(b_stacked.unlock)
 
1509
        b_stacked.fetch(b_source, 'B')
 
1510
        # Now re-open the stacked repo directly (no fallbacks) so that we can
 
1511
        # fill in the A rev.
 
1512
        repo_not_stacked = b_stacked.bzrdir.open_repository()
 
1513
        repo_not_stacked.lock_write()
 
1514
        self.addCleanup(repo_not_stacked.unlock)
 
1515
        # Now we should have a pack file with A's inventory, but not its
 
1516
        # Revision
 
1517
        self.assertEqual([('A',), ('B',)],
 
1518
                         sorted(repo_not_stacked.inventories.keys()))
 
1519
        self.assertEqual([('B',)],
 
1520
                         sorted(repo_not_stacked.revisions.keys()))
 
1521
        stacked_pack_names = repo_not_stacked._pack_collection.names()
 
1522
        # We have a couple names here, figure out which has A's inventory
 
1523
        for name in stacked_pack_names:
 
1524
            pack = repo_not_stacked._pack_collection.get_pack_by_name(name)
 
1525
            keys = [n[1] for n in pack.inventory_index.iter_all_entries()]
 
1526
            if ('A',) in keys:
 
1527
                inv_a_pack_name = name
 
1528
                break
 
1529
        else:
 
1530
            self.fail('Could not find pack containing A\'s inventory')
 
1531
        repo_not_stacked.fetch(b_source.repository, 'A')
 
1532
        self.assertEqual([('A',), ('B',)],
 
1533
                         sorted(repo_not_stacked.revisions.keys()))
 
1534
        new_pack_names = set(repo_not_stacked._pack_collection.names())
 
1535
        rev_a_pack_names = new_pack_names.difference(stacked_pack_names)
 
1536
        self.assertEqual(1, len(rev_a_pack_names))
 
1537
        rev_a_pack_name = list(rev_a_pack_names)[0]
 
1538
        # Now fetch 'C', so we have a couple pack files to join
 
1539
        repo_not_stacked.fetch(b_source.repository, 'C')
 
1540
        rev_c_pack_names = set(repo_not_stacked._pack_collection.names())
 
1541
        rev_c_pack_names = rev_c_pack_names.difference(new_pack_names)
 
1542
        self.assertEqual(1, len(rev_c_pack_names))
 
1543
        rev_c_pack_name = list(rev_c_pack_names)[0]
 
1544
        return (repo_not_stacked, rev_a_pack_name, inv_a_pack_name,
 
1545
                rev_c_pack_name)
 
1546
 
 
1547
    def test_pack_with_distant_inventories(self):
 
1548
        # See https://bugs.launchpad.net/bzr/+bug/437003
 
1549
        # When repacking, it is possible to have an inventory in a different
 
1550
        # pack file than the associated revision. An autopack can then come
 
1551
        # along, and miss that inventory, and complain.
 
1552
        (repo, rev_a_pack_name, inv_a_pack_name, rev_c_pack_name
 
1553
         ) = self.make_branch_with_disjoint_inventory_and_revision()
 
1554
        a_pack = repo._pack_collection.get_pack_by_name(rev_a_pack_name)
 
1555
        c_pack = repo._pack_collection.get_pack_by_name(rev_c_pack_name)
 
1556
        packer = groupcompress_repo.GCCHKPacker(repo._pack_collection,
 
1557
                    [a_pack, c_pack], '.test-pack')
 
1558
        # This would raise ValueError in bug #437003, but should not raise an
 
1559
        # error once fixed.
 
1560
        packer.pack()
 
1561
 
 
1562
    def test_pack_with_missing_inventory(self):
 
1563
        # Similar to test_pack_with_missing_inventory, but this time, we force
 
1564
        # the A inventory to actually be gone from the repository.
 
1565
        (repo, rev_a_pack_name, inv_a_pack_name, rev_c_pack_name
 
1566
         ) = self.make_branch_with_disjoint_inventory_and_revision()
 
1567
        inv_a_pack = repo._pack_collection.get_pack_by_name(inv_a_pack_name)
 
1568
        repo._pack_collection._remove_pack_from_memory(inv_a_pack)
 
1569
        packer = groupcompress_repo.GCCHKPacker(repo._pack_collection,
 
1570
            repo._pack_collection.all_packs(), '.test-pack')
 
1571
        e = self.assertRaises(ValueError, packer.pack)
 
1572
        packer.new_pack.abort()
 
1573
        self.assertContainsRe(str(e),
 
1574
            r"We are missing inventories for revisions: .*'A'")
 
1575
 
 
1576
 
1625
1577
class TestCrossFormatPacks(TestCaseWithTransport):
1626
1578
 
1627
1579
    def log_pack(self, hint=None):
1644
1596
        self.orig_pack = target.pack
1645
1597
        target.pack = self.log_pack
1646
1598
        search = target.search_missing_revision_ids(
1647
 
            source_tree.branch.repository, tip)
 
1599
            source_tree.branch.repository, revision_ids=[tip])
1648
1600
        stream = source.get_stream(search)
1649
1601
        from_format = source_tree.branch.repository._format
1650
1602
        sink = target._get_sink()