← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_repository.py

Merge the 0.17 fixes back into bzr.dev

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, 2007 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""Tests for the Repository facility that are not interface tests.
18
18
 
19
 
For interface tests see tests/per_repository/*.py.
 
19
For interface tests see tests/repository_implementations/*.py.
20
20
 
21
21
For concrete class tests see this file, and for storage formats tests
22
22
also see this file.
24
24
 
25
25
from stat import S_ISDIR
26
26
from StringIO import StringIO
27
 
import sys
28
27
 
 
28
from bzrlib import symbol_versioning
29
29
import bzrlib
 
30
import bzrlib.bzrdir as bzrdir
 
31
import bzrlib.errors as errors
30
32
from bzrlib.errors import (NotBranchError,
31
33
                           NoSuchFile,
32
34
                           UnknownFormatError,
33
35
                           UnsupportedFormatError,
34
36
                           )
35
 
from bzrlib import (
36
 
    graph,
37
 
    tests,
38
 
    )
39
 
from bzrlib.branchbuilder import BranchBuilder
40
 
from bzrlib.btree_index import BTreeBuilder, BTreeGraphIndex
41
 
from bzrlib.index import GraphIndex, InMemoryGraphIndex
42
37
from bzrlib.repository import RepositoryFormat
43
 
from bzrlib.smart import server
44
 
from bzrlib.tests import (
45
 
    TestCase,
46
 
    TestCaseWithTransport,
47
 
    TestSkipped,
48
 
    test_knit,
49
 
    )
50
 
from bzrlib.transport import (
51
 
    fakenfs,
52
 
    get_transport,
53
 
    )
 
38
from bzrlib.tests import TestCase, TestCaseWithTransport
 
39
from bzrlib.transport import get_transport
 
40
from bzrlib.transport.memory import MemoryServer
54
41
from bzrlib import (
55
 
    bencode,
56
 
    bzrdir,
57
 
    errors,
58
 
    inventory,
59
 
    osutils,
60
 
    progress,
61
42
    repository,
62
 
    revision as _mod_revision,
63
 
    symbol_versioning,
64
43
    upgrade,
65
 
    versionedfile,
66
44
    workingtree,
67
45
    )
68
 
from bzrlib.repofmt import (
69
 
    groupcompress_repo,
70
 
    knitrepo,
71
 
    pack_repo,
72
 
    weaverepo,
73
 
    )
 
46
from bzrlib.repofmt import knitrepo, weaverepo
74
47
 
75
48
 
76
49
class TestDefaultFormat(TestCase):
105
78
class SampleRepositoryFormat(repository.RepositoryFormat):
106
79
    """A sample format
107
80
 
108
 
    this format is initializable, unsupported to aid in testing the
 
81
    this format is initializable, unsupported to aid in testing the 
109
82
    open and open(unsupported=True) routines.
110
83
    """
111
84
 
132
105
    def test_find_format(self):
133
106
        # is the right format object found for a repository?
134
107
        # create a branch with a few known format objects.
135
 
        # this is not quite the same as
 
108
        # this is not quite the same as 
136
109
        self.build_tree(["foo/", "bar/"])
137
110
        def check_format(format, url):
138
111
            dir = format._matchingbzrdir.initialize(url)
141
114
            found_format = repository.RepositoryFormat.find_format(dir)
142
115
            self.failUnless(isinstance(found_format, format.__class__))
143
116
        check_format(weaverepo.RepositoryFormat7(), "bar")
144
 
 
 
117
        
145
118
    def test_find_format_no_repository(self):
146
119
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
147
120
        self.assertRaises(errors.NoRepositoryPresent,
173
146
 
174
147
class TestFormat6(TestCaseWithTransport):
175
148
 
176
 
    def test_attribute__fetch_order(self):
177
 
        """Weaves need topological data insertion."""
178
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
179
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
180
 
        self.assertEqual('topological', repo._format._fetch_order)
181
 
 
182
 
    def test_attribute__fetch_uses_deltas(self):
183
 
        """Weaves do not reuse deltas."""
184
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
185
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
186
 
        self.assertEqual(False, repo._format._fetch_uses_deltas)
187
 
 
188
 
    def test_attribute__fetch_reconcile(self):
189
 
        """Weave repositories need a reconcile after fetch."""
190
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
191
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
192
 
        self.assertEqual(True, repo._format._fetch_reconcile)
193
 
 
194
149
    def test_no_ancestry_weave(self):
195
150
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
196
151
        repo = weaverepo.RepositoryFormat6().initialize(control)
200
155
                          control.transport.get,
201
156
                          'ancestry.weave')
202
157
 
203
 
    def test_supports_external_lookups(self):
204
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
205
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
206
 
        self.assertFalse(repo._format.supports_external_lookups)
207
 
 
208
158
 
209
159
class TestFormat7(TestCaseWithTransport):
210
 
 
211
 
    def test_attribute__fetch_order(self):
212
 
        """Weaves need topological data insertion."""
213
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
214
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
215
 
        self.assertEqual('topological', repo._format._fetch_order)
216
 
 
217
 
    def test_attribute__fetch_uses_deltas(self):
218
 
        """Weaves do not reuse deltas."""
219
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
220
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
221
 
        self.assertEqual(False, repo._format._fetch_uses_deltas)
222
 
 
223
 
    def test_attribute__fetch_reconcile(self):
224
 
        """Weave repositories need a reconcile after fetch."""
225
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
226
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
227
 
        self.assertEqual(True, repo._format._fetch_reconcile)
228
 
 
 
160
    
229
161
    def test_disk_layout(self):
230
162
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
231
163
        repo = weaverepo.RepositoryFormat7().initialize(control)
247
179
                             'w\n'
248
180
                             'W\n',
249
181
                             t.get('inventory.weave').read())
250
 
        # Creating a file with id Foo:Bar results in a non-escaped file name on
251
 
        # disk.
252
 
        control.create_branch()
253
 
        tree = control.create_workingtree()
254
 
        tree.add(['foo'], ['Foo:Bar'], ['file'])
255
 
        tree.put_file_bytes_non_atomic('Foo:Bar', 'content\n')
256
 
        try:
257
 
            tree.commit('first post', rev_id='first')
258
 
        except errors.IllegalPath:
259
 
            if sys.platform != 'win32':
260
 
                raise
261
 
            self.knownFailure('Foo:Bar cannot be used as a file-id on windows'
262
 
                              ' in repo format 7')
263
 
            return
264
 
        self.assertEqualDiff(
265
 
            '# bzr weave file v5\n'
266
 
            'i\n'
267
 
            '1 7fe70820e08a1aac0ef224d9c66ab66831cc4ab1\n'
268
 
            'n first\n'
269
 
            '\n'
270
 
            'w\n'
271
 
            '{ 0\n'
272
 
            '. content\n'
273
 
            '}\n'
274
 
            'W\n',
275
 
            t.get('weaves/74/Foo%3ABar.weave').read())
276
182
 
277
183
    def test_shared_disk_layout(self):
278
184
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
301
207
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
302
208
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
303
209
        t = control.get_repository_transport(None)
304
 
        # TODO: Should check there is a 'lock' toplevel directory,
 
210
        # TODO: Should check there is a 'lock' toplevel directory, 
305
211
        # regardless of contents
306
212
        self.assertFalse(t.has('lock/held/info'))
307
213
        repo.lock_write()
353
259
                             'W\n',
354
260
                             t.get('inventory.weave').read())
355
261
 
356
 
    def test_supports_external_lookups(self):
357
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
358
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
359
 
        self.assertFalse(repo._format.supports_external_lookups)
360
 
 
361
262
 
362
263
class TestFormatKnit1(TestCaseWithTransport):
363
 
 
364
 
    def test_attribute__fetch_order(self):
365
 
        """Knits need topological data insertion."""
366
 
        repo = self.make_repository('.',
367
 
                format=bzrdir.format_registry.get('knit')())
368
 
        self.assertEqual('topological', repo._format._fetch_order)
369
 
 
370
 
    def test_attribute__fetch_uses_deltas(self):
371
 
        """Knits reuse deltas."""
372
 
        repo = self.make_repository('.',
373
 
                format=bzrdir.format_registry.get('knit')())
374
 
        self.assertEqual(True, repo._format._fetch_uses_deltas)
375
 
 
 
264
    
376
265
    def test_disk_layout(self):
377
266
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
378
267
        repo = knitrepo.RepositoryFormatKnit1().initialize(control)
392
281
        # self.assertEqualDiff('', t.get('lock').read())
393
282
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
394
283
        self.check_knits(t)
395
 
        # Check per-file knits.
396
 
        branch = control.create_branch()
397
 
        tree = control.create_workingtree()
398
 
        tree.add(['foo'], ['Nasty-IdC:'], ['file'])
399
 
        tree.put_file_bytes_non_atomic('Nasty-IdC:', '')
400
 
        tree.commit('1st post', rev_id='foo')
401
 
        self.assertHasKnit(t, 'knits/e8/%254easty-%2549d%2543%253a',
402
 
            '\nfoo fulltext 0 81  :')
403
284
 
404
 
    def assertHasKnit(self, t, knit_name, extra_content=''):
 
285
    def assertHasKnit(self, t, knit_name):
405
286
        """Assert that knit_name exists on t."""
406
 
        self.assertEqualDiff('# bzr knit index 8\n' + extra_content,
 
287
        self.assertEqualDiff('# bzr knit index 8\n',
407
288
                             t.get(knit_name + '.kndx').read())
 
289
        # no default content
 
290
        self.assertTrue(t.has(knit_name + '.knit'))
408
291
 
409
292
    def check_knits(self, t):
410
293
        """check knit content for a repository."""
454
337
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
455
338
        self.check_knits(t)
456
339
 
457
 
    def test_deserialise_sets_root_revision(self):
458
 
        """We must have a inventory.root.revision
459
 
 
460
 
        Old versions of the XML5 serializer did not set the revision_id for
461
 
        the whole inventory. So we grab the one from the expected text. Which
462
 
        is valid when the api is not being abused.
463
 
        """
464
 
        repo = self.make_repository('.',
465
 
                format=bzrdir.format_registry.get('knit')())
466
 
        inv_xml = '<inventory format="5">\n</inventory>\n'
467
 
        inv = repo._deserialise_inventory('test-rev-id', inv_xml)
468
 
        self.assertEqual('test-rev-id', inv.root.revision)
469
 
 
470
 
    def test_deserialise_uses_global_revision_id(self):
471
 
        """If it is set, then we re-use the global revision id"""
472
 
        repo = self.make_repository('.',
473
 
                format=bzrdir.format_registry.get('knit')())
474
 
        inv_xml = ('<inventory format="5" revision_id="other-rev-id">\n'
475
 
                   '</inventory>\n')
476
 
        # Arguably, the deserialise_inventory should detect a mismatch, and
477
 
        # raise an error, rather than silently using one revision_id over the
478
 
        # other.
479
 
        self.assertRaises(AssertionError, repo._deserialise_inventory,
480
 
            'test-rev-id', inv_xml)
481
 
        inv = repo._deserialise_inventory('other-rev-id', inv_xml)
482
 
        self.assertEqual('other-rev-id', inv.root.revision)
483
 
 
484
 
    def test_supports_external_lookups(self):
485
 
        repo = self.make_repository('.',
486
 
                format=bzrdir.format_registry.get('knit')())
487
 
        self.assertFalse(repo._format.supports_external_lookups)
488
 
 
489
340
 
490
341
class DummyRepository(object):
491
342
    """A dummy repository for testing."""
492
343
 
493
 
    _format = None
494
344
    _serializer = None
495
345
 
496
346
    def supports_rich_root(self):
497
 
        if self._format is not None:
498
 
            return self._format.rich_root_data
499
347
        return False
500
348
 
501
 
    def get_graph(self):
502
 
        raise NotImplementedError
503
 
 
504
 
    def get_parent_map(self, revision_ids):
505
 
        raise NotImplementedError
506
 
 
507
349
 
508
350
class InterDummy(repository.InterRepository):
509
351
    """An inter-repository optimised code path for DummyRepository.
510
352
 
511
353
    This is for use during testing where we use DummyRepository as repositories
512
354
    so that none of the default regsitered inter-repository classes will
513
 
    MATCH.
 
355
    match.
514
356
    """
515
357
 
516
358
    @staticmethod
517
359
    def is_compatible(repo_source, repo_target):
518
360
        """InterDummy is compatible with DummyRepository."""
519
 
        return (isinstance(repo_source, DummyRepository) and
 
361
        return (isinstance(repo_source, DummyRepository) and 
520
362
            isinstance(repo_target, DummyRepository))
521
363
 
522
364
 
535
377
 
536
378
    def assertGetsDefaultInterRepository(self, repo_a, repo_b):
537
379
        """Asserts that InterRepository.get(repo_a, repo_b) -> the default.
538
 
 
 
380
        
539
381
        The effective default is now InterSameDataRepository because there is
540
382
        no actual sane default in the presence of incompatible data models.
541
383
        """
552
394
        # pair that it returns true on for the is_compatible static method
553
395
        # check
554
396
        dummy_a = DummyRepository()
555
 
        dummy_a._format = RepositoryFormat()
556
397
        dummy_b = DummyRepository()
557
 
        dummy_b._format = RepositoryFormat()
558
398
        repo = self.make_repository('.')
559
399
        # hack dummies to look like repo somewhat.
560
400
        dummy_a._serializer = repo._serializer
561
 
        dummy_a._format.supports_tree_reference = repo._format.supports_tree_reference
562
 
        dummy_a._format.rich_root_data = repo._format.rich_root_data
563
401
        dummy_b._serializer = repo._serializer
564
 
        dummy_b._format.supports_tree_reference = repo._format.supports_tree_reference
565
 
        dummy_b._format.rich_root_data = repo._format.rich_root_data
566
402
        repository.InterRepository.register_optimiser(InterDummy)
567
403
        try:
568
404
            # we should get the default for something InterDummy returns False
631
467
 
632
468
 
633
469
class TestMisc(TestCase):
634
 
 
 
470
    
635
471
    def test_unescape_xml(self):
636
472
        """We get some kind of error when malformed entities are passed"""
637
 
        self.assertRaises(KeyError, repository._unescape_xml, 'foo&bar;')
 
473
        self.assertRaises(KeyError, repository._unescape_xml, 'foo&bar;') 
638
474
 
639
475
 
640
476
class TestRepositoryFormatKnit3(TestCaseWithTransport):
641
477
 
642
 
    def test_attribute__fetch_order(self):
643
 
        """Knits need topological data insertion."""
644
 
        format = bzrdir.BzrDirMetaFormat1()
645
 
        format.repository_format = knitrepo.RepositoryFormatKnit3()
646
 
        repo = self.make_repository('.', format=format)
647
 
        self.assertEqual('topological', repo._format._fetch_order)
648
 
 
649
 
    def test_attribute__fetch_uses_deltas(self):
650
 
        """Knits reuse deltas."""
651
 
        format = bzrdir.BzrDirMetaFormat1()
652
 
        format.repository_format = knitrepo.RepositoryFormatKnit3()
653
 
        repo = self.make_repository('.', format=format)
654
 
        self.assertEqual(True, repo._format._fetch_uses_deltas)
655
 
 
656
478
    def test_convert(self):
657
479
        """Ensure the upgrade adds weaves for roots"""
658
480
        format = bzrdir.BzrDirMetaFormat1()
660
482
        tree = self.make_branch_and_tree('.', format)
661
483
        tree.commit("Dull commit", rev_id="dull")
662
484
        revision_tree = tree.branch.repository.revision_tree('dull')
663
 
        revision_tree.lock_read()
664
 
        try:
665
 
            self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
666
 
                revision_tree.inventory.root.file_id)
667
 
        finally:
668
 
            revision_tree.unlock()
 
485
        self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
 
486
            revision_tree.inventory.root.file_id)
669
487
        format = bzrdir.BzrDirMetaFormat1()
670
488
        format.repository_format = knitrepo.RepositoryFormatKnit3()
671
489
        upgrade.Convert('.', format)
672
490
        tree = workingtree.WorkingTree.open('.')
673
491
        revision_tree = tree.branch.repository.revision_tree('dull')
674
 
        revision_tree.lock_read()
675
 
        try:
676
 
            revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
677
 
        finally:
678
 
            revision_tree.unlock()
 
492
        revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
679
493
        tree.commit("Another dull commit", rev_id='dull2')
680
494
        revision_tree = tree.branch.repository.revision_tree('dull2')
681
 
        revision_tree.lock_read()
682
 
        self.addCleanup(revision_tree.unlock)
683
495
        self.assertEqual('dull', revision_tree.inventory.root.revision)
684
496
 
685
 
    def test_supports_external_lookups(self):
686
 
        format = bzrdir.BzrDirMetaFormat1()
687
 
        format.repository_format = knitrepo.RepositoryFormatKnit3()
688
 
        repo = self.make_repository('.', format=format)
689
 
        self.assertFalse(repo._format.supports_external_lookups)
690
 
 
691
 
 
692
 
class Test2a(tests.TestCaseWithMemoryTransport):
693
 
 
694
 
    def test_fetch_combines_groups(self):
695
 
        builder = self.make_branch_builder('source', format='2a')
696
 
        builder.start_series()
697
 
        builder.build_snapshot('1', None, [
698
 
            ('add', ('', 'root-id', 'directory', '')),
699
 
            ('add', ('file', 'file-id', 'file', 'content\n'))])
700
 
        builder.build_snapshot('2', ['1'], [
701
 
            ('modify', ('file-id', 'content-2\n'))])
702
 
        builder.finish_series()
703
 
        source = builder.get_branch()
704
 
        target = self.make_repository('target', format='2a')
705
 
        target.fetch(source.repository)
706
 
        target.lock_read()
707
 
        self.addCleanup(target.unlock)
708
 
        details = target.texts._index.get_build_details(
709
 
            [('file-id', '1',), ('file-id', '2',)])
710
 
        file_1_details = details[('file-id', '1')]
711
 
        file_2_details = details[('file-id', '2')]
712
 
        # The index, and what to read off disk, should be the same for both
713
 
        # versions of the file.
714
 
        self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
715
 
 
716
 
    def test_fetch_combines_groups(self):
717
 
        builder = self.make_branch_builder('source', format='2a')
718
 
        builder.start_series()
719
 
        builder.build_snapshot('1', None, [
720
 
            ('add', ('', 'root-id', 'directory', '')),
721
 
            ('add', ('file', 'file-id', 'file', 'content\n'))])
722
 
        builder.build_snapshot('2', ['1'], [
723
 
            ('modify', ('file-id', 'content-2\n'))])
724
 
        builder.finish_series()
725
 
        source = builder.get_branch()
726
 
        target = self.make_repository('target', format='2a')
727
 
        target.fetch(source.repository)
728
 
        target.lock_read()
729
 
        self.addCleanup(target.unlock)
730
 
        details = target.texts._index.get_build_details(
731
 
            [('file-id', '1',), ('file-id', '2',)])
732
 
        file_1_details = details[('file-id', '1')]
733
 
        file_2_details = details[('file-id', '2')]
734
 
        # The index, and what to read off disk, should be the same for both
735
 
        # versions of the file.
736
 
        self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
737
 
 
738
 
    def test_fetch_combines_groups(self):
739
 
        builder = self.make_branch_builder('source', format='2a')
740
 
        builder.start_series()
741
 
        builder.build_snapshot('1', None, [
742
 
            ('add', ('', 'root-id', 'directory', '')),
743
 
            ('add', ('file', 'file-id', 'file', 'content\n'))])
744
 
        builder.build_snapshot('2', ['1'], [
745
 
            ('modify', ('file-id', 'content-2\n'))])
746
 
        builder.finish_series()
747
 
        source = builder.get_branch()
748
 
        target = self.make_repository('target', format='2a')
749
 
        target.fetch(source.repository)
750
 
        target.lock_read()
751
 
        self.addCleanup(target.unlock)
752
 
        details = target.texts._index.get_build_details(
753
 
            [('file-id', '1',), ('file-id', '2',)])
754
 
        file_1_details = details[('file-id', '1')]
755
 
        file_2_details = details[('file-id', '2')]
756
 
        # The index, and what to read off disk, should be the same for both
757
 
        # versions of the file.
758
 
        self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
759
 
 
760
 
    def test_format_pack_compresses_True(self):
761
 
        repo = self.make_repository('repo', format='2a')
762
 
        self.assertTrue(repo._format.pack_compresses)
763
 
 
764
 
    def test_inventories_use_chk_map_with_parent_base_dict(self):
765
 
        tree = self.make_branch_and_memory_tree('repo', format="2a")
766
 
        tree.lock_write()
767
 
        tree.add([''], ['TREE_ROOT'])
768
 
        revid = tree.commit("foo")
769
 
        tree.unlock()
770
 
        tree.lock_read()
771
 
        self.addCleanup(tree.unlock)
772
 
        inv = tree.branch.repository.get_inventory(revid)
773
 
        self.assertNotEqual(None, inv.parent_id_basename_to_file_id)
774
 
        inv.parent_id_basename_to_file_id._ensure_root()
775
 
        inv.id_to_entry._ensure_root()
776
 
        self.assertEqual(65536, inv.id_to_entry._root_node.maximum_size)
777
 
        self.assertEqual(65536,
778
 
            inv.parent_id_basename_to_file_id._root_node.maximum_size)
779
 
 
780
 
    def test_autopack_unchanged_chk_nodes(self):
781
 
        # at 20 unchanged commits, chk pages are packed that are split into
782
 
        # two groups such that the new pack being made doesn't have all its
783
 
        # pages in the source packs (though they are in the repository).
784
 
        # Use a memory backed repository, we don't need to hit disk for this
785
 
        tree = self.make_branch_and_memory_tree('tree', format='2a')
786
 
        tree.lock_write()
787
 
        self.addCleanup(tree.unlock)
788
 
        tree.add([''], ['TREE_ROOT'])
789
 
        for pos in range(20):
790
 
            tree.commit(str(pos))
791
 
 
792
 
    def test_pack_with_hint(self):
793
 
        tree = self.make_branch_and_memory_tree('tree', format='2a')
794
 
        tree.lock_write()
795
 
        self.addCleanup(tree.unlock)
796
 
        tree.add([''], ['TREE_ROOT'])
797
 
        # 1 commit to leave untouched
798
 
        tree.commit('1')
799
 
        to_keep = tree.branch.repository._pack_collection.names()
800
 
        # 2 to combine
801
 
        tree.commit('2')
802
 
        tree.commit('3')
803
 
        all = tree.branch.repository._pack_collection.names()
804
 
        combine = list(set(all) - set(to_keep))
805
 
        self.assertLength(3, all)
806
 
        self.assertLength(2, combine)
807
 
        tree.branch.repository.pack(hint=combine)
808
 
        final = tree.branch.repository._pack_collection.names()
809
 
        self.assertLength(2, final)
810
 
        self.assertFalse(combine[0] in final)
811
 
        self.assertFalse(combine[1] in final)
812
 
        self.assertSubset(to_keep, final)
813
 
 
814
 
    def test_stream_source_to_gc(self):
815
 
        source = self.make_repository('source', format='2a')
816
 
        target = self.make_repository('target', format='2a')
817
 
        stream = source._get_source(target._format)
818
 
        self.assertIsInstance(stream, groupcompress_repo.GroupCHKStreamSource)
819
 
 
820
 
    def test_stream_source_to_non_gc(self):
821
 
        source = self.make_repository('source', format='2a')
822
 
        target = self.make_repository('target', format='rich-root-pack')
823
 
        stream = source._get_source(target._format)
824
 
        # We don't want the child GroupCHKStreamSource
825
 
        self.assertIs(type(stream), repository.StreamSource)
826
 
 
827
 
    def test_get_stream_for_missing_keys_includes_all_chk_refs(self):
828
 
        source_builder = self.make_branch_builder('source',
829
 
                            format='2a')
830
 
        # We have to build a fairly large tree, so that we are sure the chk
831
 
        # pages will have split into multiple pages.
832
 
        entries = [('add', ('', 'a-root-id', 'directory', None))]
833
 
        for i in 'abcdefghijklmnopqrstuvwxyz123456789':
834
 
            for j in 'abcdefghijklmnopqrstuvwxyz123456789':
835
 
                fname = i + j
836
 
                fid = fname + '-id'
837
 
                content = 'content for %s\n' % (fname,)
838
 
                entries.append(('add', (fname, fid, 'file', content)))
839
 
        source_builder.start_series()
840
 
        source_builder.build_snapshot('rev-1', None, entries)
841
 
        # Now change a few of them, so we get a few new pages for the second
842
 
        # revision
843
 
        source_builder.build_snapshot('rev-2', ['rev-1'], [
844
 
            ('modify', ('aa-id', 'new content for aa-id\n')),
845
 
            ('modify', ('cc-id', 'new content for cc-id\n')),
846
 
            ('modify', ('zz-id', 'new content for zz-id\n')),
847
 
            ])
848
 
        source_builder.finish_series()
849
 
        source_branch = source_builder.get_branch()
850
 
        source_branch.lock_read()
851
 
        self.addCleanup(source_branch.unlock)
852
 
        target = self.make_repository('target', format='2a')
853
 
        source = source_branch.repository._get_source(target._format)
854
 
        self.assertIsInstance(source, groupcompress_repo.GroupCHKStreamSource)
855
 
 
856
 
        # On a regular pass, getting the inventories and chk pages for rev-2
857
 
        # would only get the newly created chk pages
858
 
        search = graph.SearchResult(set(['rev-2']), set(['rev-1']), 1,
859
 
                                    set(['rev-2']))
860
 
        simple_chk_records = []
861
 
        for vf_name, substream in source.get_stream(search):
862
 
            if vf_name == 'chk_bytes':
863
 
                for record in substream:
864
 
                    simple_chk_records.append(record.key)
865
 
            else:
866
 
                for _ in substream:
867
 
                    continue
868
 
        # 3 pages, the root (InternalNode), + 2 pages which actually changed
869
 
        self.assertEqual([('sha1:91481f539e802c76542ea5e4c83ad416bf219f73',),
870
 
                          ('sha1:4ff91971043668583985aec83f4f0ab10a907d3f',),
871
 
                          ('sha1:81e7324507c5ca132eedaf2d8414ee4bb2226187',),
872
 
                          ('sha1:b101b7da280596c71a4540e9a1eeba8045985ee0',)],
873
 
                         simple_chk_records)
874
 
        # Now, when we do a similar call using 'get_stream_for_missing_keys'
875
 
        # we should get a much larger set of pages.
876
 
        missing = [('inventories', 'rev-2')]
877
 
        full_chk_records = []
878
 
        for vf_name, substream in source.get_stream_for_missing_keys(missing):
879
 
            if vf_name == 'inventories':
880
 
                for record in substream:
881
 
                    self.assertEqual(('rev-2',), record.key)
882
 
            elif vf_name == 'chk_bytes':
883
 
                for record in substream:
884
 
                    full_chk_records.append(record.key)
885
 
            else:
886
 
                self.fail('Should not be getting a stream of %s' % (vf_name,))
887
 
        # We have 257 records now. This is because we have 1 root page, and 256
888
 
        # leaf pages in a complete listing.
889
 
        self.assertEqual(257, len(full_chk_records))
890
 
        self.assertSubset(simple_chk_records, full_chk_records)
891
 
 
892
 
    def test_inconsistency_fatal(self):
893
 
        repo = self.make_repository('repo', format='2a')
894
 
        self.assertTrue(repo.revisions._index._inconsistency_fatal)
895
 
        self.assertFalse(repo.texts._index._inconsistency_fatal)
896
 
        self.assertFalse(repo.inventories._index._inconsistency_fatal)
897
 
        self.assertFalse(repo.signatures._index._inconsistency_fatal)
898
 
        self.assertFalse(repo.chk_bytes._index._inconsistency_fatal)
899
 
 
900
 
 
901
 
class TestKnitPackStreamSource(tests.TestCaseWithMemoryTransport):
902
 
 
903
 
    def test_source_to_exact_pack_092(self):
904
 
        source = self.make_repository('source', format='pack-0.92')
905
 
        target = self.make_repository('target', format='pack-0.92')
906
 
        stream_source = source._get_source(target._format)
907
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
908
 
 
909
 
    def test_source_to_exact_pack_rich_root_pack(self):
910
 
        source = self.make_repository('source', format='rich-root-pack')
911
 
        target = self.make_repository('target', format='rich-root-pack')
912
 
        stream_source = source._get_source(target._format)
913
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
914
 
 
915
 
    def test_source_to_exact_pack_19(self):
916
 
        source = self.make_repository('source', format='1.9')
917
 
        target = self.make_repository('target', format='1.9')
918
 
        stream_source = source._get_source(target._format)
919
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
920
 
 
921
 
    def test_source_to_exact_pack_19_rich_root(self):
922
 
        source = self.make_repository('source', format='1.9-rich-root')
923
 
        target = self.make_repository('target', format='1.9-rich-root')
924
 
        stream_source = source._get_source(target._format)
925
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
926
 
 
927
 
    def test_source_to_remote_exact_pack_19(self):
928
 
        trans = self.make_smart_server('target')
929
 
        trans.ensure_base()
930
 
        source = self.make_repository('source', format='1.9')
931
 
        target = self.make_repository('target', format='1.9')
932
 
        target = repository.Repository.open(trans.base)
933
 
        stream_source = source._get_source(target._format)
934
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
935
 
 
936
 
    def test_stream_source_to_non_exact(self):
937
 
        source = self.make_repository('source', format='pack-0.92')
938
 
        target = self.make_repository('target', format='1.9')
939
 
        stream = source._get_source(target._format)
940
 
        self.assertIs(type(stream), repository.StreamSource)
941
 
 
942
 
    def test_stream_source_to_non_exact_rich_root(self):
943
 
        source = self.make_repository('source', format='1.9')
944
 
        target = self.make_repository('target', format='1.9-rich-root')
945
 
        stream = source._get_source(target._format)
946
 
        self.assertIs(type(stream), repository.StreamSource)
947
 
 
948
 
    def test_source_to_remote_non_exact_pack_19(self):
949
 
        trans = self.make_smart_server('target')
950
 
        trans.ensure_base()
951
 
        source = self.make_repository('source', format='1.9')
952
 
        target = self.make_repository('target', format='1.6')
953
 
        target = repository.Repository.open(trans.base)
954
 
        stream_source = source._get_source(target._format)
955
 
        self.assertIs(type(stream_source), repository.StreamSource)
956
 
 
957
 
    def test_stream_source_to_knit(self):
958
 
        source = self.make_repository('source', format='pack-0.92')
959
 
        target = self.make_repository('target', format='dirstate')
960
 
        stream = source._get_source(target._format)
961
 
        self.assertIs(type(stream), repository.StreamSource)
962
 
 
963
 
 
964
 
class TestDevelopment6FindParentIdsOfRevisions(TestCaseWithTransport):
965
 
    """Tests for _find_parent_ids_of_revisions."""
966
 
 
967
 
    def setUp(self):
968
 
        super(TestDevelopment6FindParentIdsOfRevisions, self).setUp()
969
 
        self.builder = self.make_branch_builder('source',
970
 
            format='development6-rich-root')
971
 
        self.builder.start_series()
972
 
        self.builder.build_snapshot('initial', None,
973
 
            [('add', ('', 'tree-root', 'directory', None))])
974
 
        self.repo = self.builder.get_branch().repository
975
 
        self.addCleanup(self.builder.finish_series)
976
 
 
977
 
    def assertParentIds(self, expected_result, rev_set):
978
 
        self.assertEqual(sorted(expected_result),
979
 
            sorted(self.repo._find_parent_ids_of_revisions(rev_set)))
980
 
 
981
 
    def test_simple(self):
982
 
        self.builder.build_snapshot('revid1', None, [])
983
 
        self.builder.build_snapshot('revid2', ['revid1'], [])
984
 
        rev_set = ['revid2']
985
 
        self.assertParentIds(['revid1'], rev_set)
986
 
 
987
 
    def test_not_first_parent(self):
988
 
        self.builder.build_snapshot('revid1', None, [])
989
 
        self.builder.build_snapshot('revid2', ['revid1'], [])
990
 
        self.builder.build_snapshot('revid3', ['revid2'], [])
991
 
        rev_set = ['revid3', 'revid2']
992
 
        self.assertParentIds(['revid1'], rev_set)
993
 
 
994
 
    def test_not_null(self):
995
 
        rev_set = ['initial']
996
 
        self.assertParentIds([], rev_set)
997
 
 
998
 
    def test_not_null_set(self):
999
 
        self.builder.build_snapshot('revid1', None, [])
1000
 
        rev_set = [_mod_revision.NULL_REVISION]
1001
 
        self.assertParentIds([], rev_set)
1002
 
 
1003
 
    def test_ghost(self):
1004
 
        self.builder.build_snapshot('revid1', None, [])
1005
 
        rev_set = ['ghost', 'revid1']
1006
 
        self.assertParentIds(['initial'], rev_set)
1007
 
 
1008
 
    def test_ghost_parent(self):
1009
 
        self.builder.build_snapshot('revid1', None, [])
1010
 
        self.builder.build_snapshot('revid2', ['revid1', 'ghost'], [])
1011
 
        rev_set = ['revid2', 'revid1']
1012
 
        self.assertParentIds(['ghost', 'initial'], rev_set)
1013
 
 
1014
 
    def test_righthand_parent(self):
1015
 
        self.builder.build_snapshot('revid1', None, [])
1016
 
        self.builder.build_snapshot('revid2a', ['revid1'], [])
1017
 
        self.builder.build_snapshot('revid2b', ['revid1'], [])
1018
 
        self.builder.build_snapshot('revid3', ['revid2a', 'revid2b'], [])
1019
 
        rev_set = ['revid3', 'revid2a']
1020
 
        self.assertParentIds(['revid1', 'revid2b'], rev_set)
1021
 
 
1022
 
 
1023
 
class TestWithBrokenRepo(TestCaseWithTransport):
1024
 
    """These tests seem to be more appropriate as interface tests?"""
1025
 
 
1026
 
    def make_broken_repository(self):
1027
 
        # XXX: This function is borrowed from Aaron's "Reconcile can fix bad
1028
 
        # parent references" branch which is due to land in bzr.dev soon.  Once
1029
 
        # it does, this duplication should be removed.
1030
 
        repo = self.make_repository('broken-repo')
1031
 
        cleanups = []
1032
 
        try:
1033
 
            repo.lock_write()
1034
 
            cleanups.append(repo.unlock)
1035
 
            repo.start_write_group()
1036
 
            cleanups.append(repo.commit_write_group)
1037
 
            # make rev1a: A well-formed revision, containing 'file1'
1038
 
            inv = inventory.Inventory(revision_id='rev1a')
1039
 
            inv.root.revision = 'rev1a'
1040
 
            self.add_file(repo, inv, 'file1', 'rev1a', [])
1041
 
            repo.texts.add_lines((inv.root.file_id, 'rev1a'), [], [])
1042
 
            repo.add_inventory('rev1a', inv, [])
1043
 
            revision = _mod_revision.Revision('rev1a',
1044
 
                committer='jrandom@example.com', timestamp=0,
1045
 
                inventory_sha1='', timezone=0, message='foo', parent_ids=[])
1046
 
            repo.add_revision('rev1a',revision, inv)
1047
 
 
1048
 
            # make rev1b, which has no Revision, but has an Inventory, and
1049
 
            # file1
1050
 
            inv = inventory.Inventory(revision_id='rev1b')
1051
 
            inv.root.revision = 'rev1b'
1052
 
            self.add_file(repo, inv, 'file1', 'rev1b', [])
1053
 
            repo.add_inventory('rev1b', inv, [])
1054
 
 
1055
 
            # make rev2, with file1 and file2
1056
 
            # file2 is sane
1057
 
            # file1 has 'rev1b' as an ancestor, even though this is not
1058
 
            # mentioned by 'rev1a', making it an unreferenced ancestor
1059
 
            inv = inventory.Inventory()
1060
 
            self.add_file(repo, inv, 'file1', 'rev2', ['rev1a', 'rev1b'])
1061
 
            self.add_file(repo, inv, 'file2', 'rev2', [])
1062
 
            self.add_revision(repo, 'rev2', inv, ['rev1a'])
1063
 
 
1064
 
            # make ghost revision rev1c
1065
 
            inv = inventory.Inventory()
1066
 
            self.add_file(repo, inv, 'file2', 'rev1c', [])
1067
 
 
1068
 
            # make rev3 with file2
1069
 
            # file2 refers to 'rev1c', which is a ghost in this repository, so
1070
 
            # file2 cannot have rev1c as its ancestor.
1071
 
            inv = inventory.Inventory()
1072
 
            self.add_file(repo, inv, 'file2', 'rev3', ['rev1c'])
1073
 
            self.add_revision(repo, 'rev3', inv, ['rev1c'])
1074
 
            return repo
1075
 
        finally:
1076
 
            for cleanup in reversed(cleanups):
1077
 
                cleanup()
1078
 
 
1079
 
    def add_revision(self, repo, revision_id, inv, parent_ids):
1080
 
        inv.revision_id = revision_id
1081
 
        inv.root.revision = revision_id
1082
 
        repo.texts.add_lines((inv.root.file_id, revision_id), [], [])
1083
 
        repo.add_inventory(revision_id, inv, parent_ids)
1084
 
        revision = _mod_revision.Revision(revision_id,
1085
 
            committer='jrandom@example.com', timestamp=0, inventory_sha1='',
1086
 
            timezone=0, message='foo', parent_ids=parent_ids)
1087
 
        repo.add_revision(revision_id,revision, inv)
1088
 
 
1089
 
    def add_file(self, repo, inv, filename, revision, parents):
1090
 
        file_id = filename + '-id'
1091
 
        entry = inventory.InventoryFile(file_id, filename, 'TREE_ROOT')
1092
 
        entry.revision = revision
1093
 
        entry.text_size = 0
1094
 
        inv.add(entry)
1095
 
        text_key = (file_id, revision)
1096
 
        parent_keys = [(file_id, parent) for parent in parents]
1097
 
        repo.texts.add_lines(text_key, parent_keys, ['line\n'])
1098
 
 
1099
 
    def test_insert_from_broken_repo(self):
1100
 
        """Inserting a data stream from a broken repository won't silently
1101
 
        corrupt the target repository.
1102
 
        """
1103
 
        broken_repo = self.make_broken_repository()
1104
 
        empty_repo = self.make_repository('empty-repo')
1105
 
        try:
1106
 
            empty_repo.fetch(broken_repo)
1107
 
        except (errors.RevisionNotPresent, errors.BzrCheckError):
1108
 
            # Test successful: compression parent not being copied leads to
1109
 
            # error.
1110
 
            return
1111
 
        empty_repo.lock_read()
1112
 
        self.addCleanup(empty_repo.unlock)
1113
 
        text = empty_repo.texts.get_record_stream(
1114
 
            [('file2-id', 'rev3')], 'topological', True).next()
1115
 
        self.assertEqual('line\n', text.get_bytes_as('fulltext'))
1116
 
 
1117
 
 
1118
 
class TestRepositoryPackCollection(TestCaseWithTransport):
1119
 
 
1120
 
    def get_format(self):
1121
 
        return bzrdir.format_registry.make_bzrdir('pack-0.92')
1122
 
 
1123
 
    def get_packs(self):
1124
 
        format = self.get_format()
1125
 
        repo = self.make_repository('.', format=format)
1126
 
        return repo._pack_collection
1127
 
 
1128
 
    def make_packs_and_alt_repo(self, write_lock=False):
1129
 
        """Create a pack repo with 3 packs, and access it via a second repo."""
1130
 
        tree = self.make_branch_and_tree('.', format=self.get_format())
1131
 
        tree.lock_write()
1132
 
        self.addCleanup(tree.unlock)
1133
 
        rev1 = tree.commit('one')
1134
 
        rev2 = tree.commit('two')
1135
 
        rev3 = tree.commit('three')
1136
 
        r = repository.Repository.open('.')
1137
 
        if write_lock:
1138
 
            r.lock_write()
1139
 
        else:
1140
 
            r.lock_read()
1141
 
        self.addCleanup(r.unlock)
1142
 
        packs = r._pack_collection
1143
 
        packs.ensure_loaded()
1144
 
        return tree, r, packs, [rev1, rev2, rev3]
1145
 
 
1146
 
    def test__clear_obsolete_packs(self):
1147
 
        packs = self.get_packs()
1148
 
        obsolete_pack_trans = packs.transport.clone('obsolete_packs')
1149
 
        obsolete_pack_trans.put_bytes('a-pack.pack', 'content\n')
1150
 
        obsolete_pack_trans.put_bytes('a-pack.rix', 'content\n')
1151
 
        obsolete_pack_trans.put_bytes('a-pack.iix', 'content\n')
1152
 
        obsolete_pack_trans.put_bytes('another-pack.pack', 'foo\n')
1153
 
        obsolete_pack_trans.put_bytes('not-a-pack.rix', 'foo\n')
1154
 
        res = packs._clear_obsolete_packs()
1155
 
        self.assertEqual(['a-pack', 'another-pack'], sorted(res))
1156
 
        self.assertEqual([], obsolete_pack_trans.list_dir('.'))
1157
 
 
1158
 
    def test__clear_obsolete_packs_preserve(self):
1159
 
        packs = self.get_packs()
1160
 
        obsolete_pack_trans = packs.transport.clone('obsolete_packs')
1161
 
        obsolete_pack_trans.put_bytes('a-pack.pack', 'content\n')
1162
 
        obsolete_pack_trans.put_bytes('a-pack.rix', 'content\n')
1163
 
        obsolete_pack_trans.put_bytes('a-pack.iix', 'content\n')
1164
 
        obsolete_pack_trans.put_bytes('another-pack.pack', 'foo\n')
1165
 
        obsolete_pack_trans.put_bytes('not-a-pack.rix', 'foo\n')
1166
 
        res = packs._clear_obsolete_packs(preserve=set(['a-pack']))
1167
 
        self.assertEqual(['a-pack', 'another-pack'], sorted(res))
1168
 
        self.assertEqual(['a-pack.iix', 'a-pack.pack', 'a-pack.rix'],
1169
 
                         sorted(obsolete_pack_trans.list_dir('.')))
1170
 
 
1171
 
    def test__max_pack_count(self):
1172
 
        """The maximum pack count is a function of the number of revisions."""
1173
 
        # no revisions - one pack, so that we can have a revision free repo
1174
 
        # without it blowing up
1175
 
        packs = self.get_packs()
1176
 
        self.assertEqual(1, packs._max_pack_count(0))
1177
 
        # after that the sum of the digits, - check the first 1-9
1178
 
        self.assertEqual(1, packs._max_pack_count(1))
1179
 
        self.assertEqual(2, packs._max_pack_count(2))
1180
 
        self.assertEqual(3, packs._max_pack_count(3))
1181
 
        self.assertEqual(4, packs._max_pack_count(4))
1182
 
        self.assertEqual(5, packs._max_pack_count(5))
1183
 
        self.assertEqual(6, packs._max_pack_count(6))
1184
 
        self.assertEqual(7, packs._max_pack_count(7))
1185
 
        self.assertEqual(8, packs._max_pack_count(8))
1186
 
        self.assertEqual(9, packs._max_pack_count(9))
1187
 
        # check the boundary cases with two digits for the next decade
1188
 
        self.assertEqual(1, packs._max_pack_count(10))
1189
 
        self.assertEqual(2, packs._max_pack_count(11))
1190
 
        self.assertEqual(10, packs._max_pack_count(19))
1191
 
        self.assertEqual(2, packs._max_pack_count(20))
1192
 
        self.assertEqual(3, packs._max_pack_count(21))
1193
 
        # check some arbitrary big numbers
1194
 
        self.assertEqual(25, packs._max_pack_count(112894))
1195
 
 
1196
 
    def test_repr(self):
1197
 
        packs = self.get_packs()
1198
 
        self.assertContainsRe(repr(packs),
1199
 
            'RepositoryPackCollection(.*Repository(.*))')
1200
 
 
1201
 
    def test__obsolete_packs(self):
1202
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1203
 
        names = packs.names()
1204
 
        pack = packs.get_pack_by_name(names[0])
1205
 
        # Schedule this one for removal
1206
 
        packs._remove_pack_from_memory(pack)
1207
 
        # Simulate a concurrent update by renaming the .pack file and one of
1208
 
        # the indices
1209
 
        packs.transport.rename('packs/%s.pack' % (names[0],),
1210
 
                               'obsolete_packs/%s.pack' % (names[0],))
1211
 
        packs.transport.rename('indices/%s.iix' % (names[0],),
1212
 
                               'obsolete_packs/%s.iix' % (names[0],))
1213
 
        # Now trigger the obsoletion, and ensure that all the remaining files
1214
 
        # are still renamed
1215
 
        packs._obsolete_packs([pack])
1216
 
        self.assertEqual([n + '.pack' for n in names[1:]],
1217
 
                         sorted(packs._pack_transport.list_dir('.')))
1218
 
        # names[0] should not be present in the index anymore
1219
 
        self.assertEqual(names[1:],
1220
 
            sorted(set([osutils.splitext(n)[0] for n in
1221
 
                        packs._index_transport.list_dir('.')])))
1222
 
 
1223
 
    def test_pack_distribution_zero(self):
1224
 
        packs = self.get_packs()
1225
 
        self.assertEqual([0], packs.pack_distribution(0))
1226
 
 
1227
 
    def test_ensure_loaded_unlocked(self):
1228
 
        packs = self.get_packs()
1229
 
        self.assertRaises(errors.ObjectNotLocked,
1230
 
                          packs.ensure_loaded)
1231
 
 
1232
 
    def test_pack_distribution_one_to_nine(self):
1233
 
        packs = self.get_packs()
1234
 
        self.assertEqual([1],
1235
 
            packs.pack_distribution(1))
1236
 
        self.assertEqual([1, 1],
1237
 
            packs.pack_distribution(2))
1238
 
        self.assertEqual([1, 1, 1],
1239
 
            packs.pack_distribution(3))
1240
 
        self.assertEqual([1, 1, 1, 1],
1241
 
            packs.pack_distribution(4))
1242
 
        self.assertEqual([1, 1, 1, 1, 1],
1243
 
            packs.pack_distribution(5))
1244
 
        self.assertEqual([1, 1, 1, 1, 1, 1],
1245
 
            packs.pack_distribution(6))
1246
 
        self.assertEqual([1, 1, 1, 1, 1, 1, 1],
1247
 
            packs.pack_distribution(7))
1248
 
        self.assertEqual([1, 1, 1, 1, 1, 1, 1, 1],
1249
 
            packs.pack_distribution(8))
1250
 
        self.assertEqual([1, 1, 1, 1, 1, 1, 1, 1, 1],
1251
 
            packs.pack_distribution(9))
1252
 
 
1253
 
    def test_pack_distribution_stable_at_boundaries(self):
1254
 
        """When there are multi-rev packs the counts are stable."""
1255
 
        packs = self.get_packs()
1256
 
        # in 10s:
1257
 
        self.assertEqual([10], packs.pack_distribution(10))
1258
 
        self.assertEqual([10, 1], packs.pack_distribution(11))
1259
 
        self.assertEqual([10, 10], packs.pack_distribution(20))
1260
 
        self.assertEqual([10, 10, 1], packs.pack_distribution(21))
1261
 
        # 100s
1262
 
        self.assertEqual([100], packs.pack_distribution(100))
1263
 
        self.assertEqual([100, 1], packs.pack_distribution(101))
1264
 
        self.assertEqual([100, 10, 1], packs.pack_distribution(111))
1265
 
        self.assertEqual([100, 100], packs.pack_distribution(200))
1266
 
        self.assertEqual([100, 100, 1], packs.pack_distribution(201))
1267
 
        self.assertEqual([100, 100, 10, 1], packs.pack_distribution(211))
1268
 
 
1269
 
    def test_plan_pack_operations_2009_revisions_skip_all_packs(self):
1270
 
        packs = self.get_packs()
1271
 
        existing_packs = [(2000, "big"), (9, "medium")]
1272
 
        # rev count - 2009 -> 2x1000 + 9x1
1273
 
        pack_operations = packs.plan_autopack_combinations(
1274
 
            existing_packs, [1000, 1000, 1, 1, 1, 1, 1, 1, 1, 1, 1])
1275
 
        self.assertEqual([], pack_operations)
1276
 
 
1277
 
    def test_plan_pack_operations_2010_revisions_skip_all_packs(self):
1278
 
        packs = self.get_packs()
1279
 
        existing_packs = [(2000, "big"), (9, "medium"), (1, "single")]
1280
 
        # rev count - 2010 -> 2x1000 + 1x10
1281
 
        pack_operations = packs.plan_autopack_combinations(
1282
 
            existing_packs, [1000, 1000, 10])
1283
 
        self.assertEqual([], pack_operations)
1284
 
 
1285
 
    def test_plan_pack_operations_2010_combines_smallest_two(self):
1286
 
        packs = self.get_packs()
1287
 
        existing_packs = [(1999, "big"), (9, "medium"), (1, "single2"),
1288
 
            (1, "single1")]
1289
 
        # rev count - 2010 -> 2x1000 + 1x10 (3)
1290
 
        pack_operations = packs.plan_autopack_combinations(
1291
 
            existing_packs, [1000, 1000, 10])
1292
 
        self.assertEqual([[2, ["single2", "single1"]]], pack_operations)
1293
 
 
1294
 
    def test_plan_pack_operations_creates_a_single_op(self):
1295
 
        packs = self.get_packs()
1296
 
        existing_packs = [(50, 'a'), (40, 'b'), (30, 'c'), (10, 'd'),
1297
 
                          (10, 'e'), (6, 'f'), (4, 'g')]
1298
 
        # rev count 150 -> 1x100 and 5x10
1299
 
        # The two size 10 packs do not need to be touched. The 50, 40, 30 would
1300
 
        # be combined into a single 120 size pack, and the 6 & 4 would
1301
 
        # becombined into a size 10 pack. However, if we have to rewrite them,
1302
 
        # we save a pack file with no increased I/O by putting them into the
1303
 
        # same file.
1304
 
        distribution = packs.pack_distribution(150)
1305
 
        pack_operations = packs.plan_autopack_combinations(existing_packs,
1306
 
                                                           distribution)
1307
 
        self.assertEqual([[130, ['a', 'b', 'c', 'f', 'g']]], pack_operations)
1308
 
 
1309
 
    def test_all_packs_none(self):
1310
 
        format = self.get_format()
1311
 
        tree = self.make_branch_and_tree('.', format=format)
1312
 
        tree.lock_read()
1313
 
        self.addCleanup(tree.unlock)
1314
 
        packs = tree.branch.repository._pack_collection
1315
 
        packs.ensure_loaded()
1316
 
        self.assertEqual([], packs.all_packs())
1317
 
 
1318
 
    def test_all_packs_one(self):
1319
 
        format = self.get_format()
1320
 
        tree = self.make_branch_and_tree('.', format=format)
1321
 
        tree.commit('start')
1322
 
        tree.lock_read()
1323
 
        self.addCleanup(tree.unlock)
1324
 
        packs = tree.branch.repository._pack_collection
1325
 
        packs.ensure_loaded()
1326
 
        self.assertEqual([
1327
 
            packs.get_pack_by_name(packs.names()[0])],
1328
 
            packs.all_packs())
1329
 
 
1330
 
    def test_all_packs_two(self):
1331
 
        format = self.get_format()
1332
 
        tree = self.make_branch_and_tree('.', format=format)
1333
 
        tree.commit('start')
1334
 
        tree.commit('continue')
1335
 
        tree.lock_read()
1336
 
        self.addCleanup(tree.unlock)
1337
 
        packs = tree.branch.repository._pack_collection
1338
 
        packs.ensure_loaded()
1339
 
        self.assertEqual([
1340
 
            packs.get_pack_by_name(packs.names()[0]),
1341
 
            packs.get_pack_by_name(packs.names()[1]),
1342
 
            ], packs.all_packs())
1343
 
 
1344
 
    def test_get_pack_by_name(self):
1345
 
        format = self.get_format()
1346
 
        tree = self.make_branch_and_tree('.', format=format)
1347
 
        tree.commit('start')
1348
 
        tree.lock_read()
1349
 
        self.addCleanup(tree.unlock)
1350
 
        packs = tree.branch.repository._pack_collection
1351
 
        packs.reset()
1352
 
        packs.ensure_loaded()
1353
 
        name = packs.names()[0]
1354
 
        pack_1 = packs.get_pack_by_name(name)
1355
 
        # the pack should be correctly initialised
1356
 
        sizes = packs._names[name]
1357
 
        rev_index = GraphIndex(packs._index_transport, name + '.rix', sizes[0])
1358
 
        inv_index = GraphIndex(packs._index_transport, name + '.iix', sizes[1])
1359
 
        txt_index = GraphIndex(packs._index_transport, name + '.tix', sizes[2])
1360
 
        sig_index = GraphIndex(packs._index_transport, name + '.six', sizes[3])
1361
 
        self.assertEqual(pack_repo.ExistingPack(packs._pack_transport,
1362
 
            name, rev_index, inv_index, txt_index, sig_index), pack_1)
1363
 
        # and the same instance should be returned on successive calls.
1364
 
        self.assertTrue(pack_1 is packs.get_pack_by_name(name))
1365
 
 
1366
 
    def test_reload_pack_names_new_entry(self):
1367
 
        tree, r, packs, revs = self.make_packs_and_alt_repo()
1368
 
        names = packs.names()
1369
 
        # Add a new pack file into the repository
1370
 
        rev4 = tree.commit('four')
1371
 
        new_names = tree.branch.repository._pack_collection.names()
1372
 
        new_name = set(new_names).difference(names)
1373
 
        self.assertEqual(1, len(new_name))
1374
 
        new_name = new_name.pop()
1375
 
        # The old collection hasn't noticed yet
1376
 
        self.assertEqual(names, packs.names())
1377
 
        self.assertTrue(packs.reload_pack_names())
1378
 
        self.assertEqual(new_names, packs.names())
1379
 
        # And the repository can access the new revision
1380
 
        self.assertEqual({rev4:(revs[-1],)}, r.get_parent_map([rev4]))
1381
 
        self.assertFalse(packs.reload_pack_names())
1382
 
 
1383
 
    def test_reload_pack_names_added_and_removed(self):
1384
 
        tree, r, packs, revs = self.make_packs_and_alt_repo()
1385
 
        names = packs.names()
1386
 
        # Now repack the whole thing
1387
 
        tree.branch.repository.pack()
1388
 
        new_names = tree.branch.repository._pack_collection.names()
1389
 
        # The other collection hasn't noticed yet
1390
 
        self.assertEqual(names, packs.names())
1391
 
        self.assertTrue(packs.reload_pack_names())
1392
 
        self.assertEqual(new_names, packs.names())
1393
 
        self.assertEqual({revs[-1]:(revs[-2],)}, r.get_parent_map([revs[-1]]))
1394
 
        self.assertFalse(packs.reload_pack_names())
1395
 
 
1396
 
    def test_reload_pack_names_preserves_pending(self):
1397
 
        # TODO: Update this to also test for pending-deleted names
1398
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1399
 
        # We will add one pack (via start_write_group + insert_record_stream),
1400
 
        # and remove another pack (via _remove_pack_from_memory)
1401
 
        orig_names = packs.names()
1402
 
        orig_at_load = packs._packs_at_load
1403
 
        to_remove_name = iter(orig_names).next()
1404
 
        r.start_write_group()
1405
 
        self.addCleanup(r.abort_write_group)
1406
 
        r.texts.insert_record_stream([versionedfile.FulltextContentFactory(
1407
 
            ('text', 'rev'), (), None, 'content\n')])
1408
 
        new_pack = packs._new_pack
1409
 
        self.assertTrue(new_pack.data_inserted())
1410
 
        new_pack.finish()
1411
 
        packs.allocate(new_pack)
1412
 
        packs._new_pack = None
1413
 
        removed_pack = packs.get_pack_by_name(to_remove_name)
1414
 
        packs._remove_pack_from_memory(removed_pack)
1415
 
        names = packs.names()
1416
 
        all_nodes, deleted_nodes, new_nodes, _ = packs._diff_pack_names()
1417
 
        new_names = set([x[0][0] for x in new_nodes])
1418
 
        self.assertEqual(names, sorted([x[0][0] for x in all_nodes]))
1419
 
        self.assertEqual(set(names) - set(orig_names), new_names)
1420
 
        self.assertEqual(set([new_pack.name]), new_names)
1421
 
        self.assertEqual([to_remove_name],
1422
 
                         sorted([x[0][0] for x in deleted_nodes]))
1423
 
        packs.reload_pack_names()
1424
 
        reloaded_names = packs.names()
1425
 
        self.assertEqual(orig_at_load, packs._packs_at_load)
1426
 
        self.assertEqual(names, reloaded_names)
1427
 
        all_nodes, deleted_nodes, new_nodes, _ = packs._diff_pack_names()
1428
 
        new_names = set([x[0][0] for x in new_nodes])
1429
 
        self.assertEqual(names, sorted([x[0][0] for x in all_nodes]))
1430
 
        self.assertEqual(set(names) - set(orig_names), new_names)
1431
 
        self.assertEqual(set([new_pack.name]), new_names)
1432
 
        self.assertEqual([to_remove_name],
1433
 
                         sorted([x[0][0] for x in deleted_nodes]))
1434
 
 
1435
 
    def test_autopack_obsoletes_new_pack(self):
1436
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1437
 
        packs._max_pack_count = lambda x: 1
1438
 
        packs.pack_distribution = lambda x: [10]
1439
 
        r.start_write_group()
1440
 
        r.revisions.insert_record_stream([versionedfile.FulltextContentFactory(
1441
 
            ('bogus-rev',), (), None, 'bogus-content\n')])
1442
 
        # This should trigger an autopack, which will combine everything into a
1443
 
        # single pack file.
1444
 
        new_names = r.commit_write_group()
1445
 
        names = packs.names()
1446
 
        self.assertEqual(1, len(names))
1447
 
        self.assertEqual([names[0] + '.pack'],
1448
 
                         packs._pack_transport.list_dir('.'))
1449
 
 
1450
 
    def test_autopack_reloads_and_stops(self):
1451
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1452
 
        # After we have determined what needs to be autopacked, trigger a
1453
 
        # full-pack via the other repo which will cause us to re-evaluate and
1454
 
        # decide we don't need to do anything
1455
 
        orig_execute = packs._execute_pack_operations
1456
 
        def _munged_execute_pack_ops(*args, **kwargs):
1457
 
            tree.branch.repository.pack()
1458
 
            return orig_execute(*args, **kwargs)
1459
 
        packs._execute_pack_operations = _munged_execute_pack_ops
1460
 
        packs._max_pack_count = lambda x: 1
1461
 
        packs.pack_distribution = lambda x: [10]
1462
 
        self.assertFalse(packs.autopack())
1463
 
        self.assertEqual(1, len(packs.names()))
1464
 
        self.assertEqual(tree.branch.repository._pack_collection.names(),
1465
 
                         packs.names())
1466
 
 
1467
 
    def test__save_pack_names(self):
1468
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1469
 
        names = packs.names()
1470
 
        pack = packs.get_pack_by_name(names[0])
1471
 
        packs._remove_pack_from_memory(pack)
1472
 
        packs._save_pack_names(obsolete_packs=[pack])
1473
 
        cur_packs = packs._pack_transport.list_dir('.')
1474
 
        self.assertEqual([n + '.pack' for n in names[1:]], sorted(cur_packs))
1475
 
        # obsolete_packs will also have stuff like .rix and .iix present.
1476
 
        obsolete_packs = packs.transport.list_dir('obsolete_packs')
1477
 
        obsolete_names = set([osutils.splitext(n)[0] for n in obsolete_packs])
1478
 
        self.assertEqual([pack.name], sorted(obsolete_names))
1479
 
 
1480
 
    def test__save_pack_names_already_obsoleted(self):
1481
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1482
 
        names = packs.names()
1483
 
        pack = packs.get_pack_by_name(names[0])
1484
 
        packs._remove_pack_from_memory(pack)
1485
 
        # We are going to simulate a concurrent autopack by manually obsoleting
1486
 
        # the pack directly.
1487
 
        packs._obsolete_packs([pack])
1488
 
        packs._save_pack_names(clear_obsolete_packs=True,
1489
 
                               obsolete_packs=[pack])
1490
 
        cur_packs = packs._pack_transport.list_dir('.')
1491
 
        self.assertEqual([n + '.pack' for n in names[1:]], sorted(cur_packs))
1492
 
        # Note that while we set clear_obsolete_packs=True, it should not
1493
 
        # delete a pack file that we have also scheduled for obsoletion.
1494
 
        obsolete_packs = packs.transport.list_dir('obsolete_packs')
1495
 
        obsolete_names = set([osutils.splitext(n)[0] for n in obsolete_packs])
1496
 
        self.assertEqual([pack.name], sorted(obsolete_names))
1497
 
 
1498
 
 
1499
 
 
1500
 
class TestPack(TestCaseWithTransport):
1501
 
    """Tests for the Pack object."""
1502
 
 
1503
 
    def assertCurrentlyEqual(self, left, right):
1504
 
        self.assertTrue(left == right)
1505
 
        self.assertTrue(right == left)
1506
 
        self.assertFalse(left != right)
1507
 
        self.assertFalse(right != left)
1508
 
 
1509
 
    def assertCurrentlyNotEqual(self, left, right):
1510
 
        self.assertFalse(left == right)
1511
 
        self.assertFalse(right == left)
1512
 
        self.assertTrue(left != right)
1513
 
        self.assertTrue(right != left)
1514
 
 
1515
 
    def test___eq____ne__(self):
1516
 
        left = pack_repo.ExistingPack('', '', '', '', '', '')
1517
 
        right = pack_repo.ExistingPack('', '', '', '', '', '')
1518
 
        self.assertCurrentlyEqual(left, right)
1519
 
        # change all attributes and ensure equality changes as we do.
1520
 
        left.revision_index = 'a'
1521
 
        self.assertCurrentlyNotEqual(left, right)
1522
 
        right.revision_index = 'a'
1523
 
        self.assertCurrentlyEqual(left, right)
1524
 
        left.inventory_index = 'a'
1525
 
        self.assertCurrentlyNotEqual(left, right)
1526
 
        right.inventory_index = 'a'
1527
 
        self.assertCurrentlyEqual(left, right)
1528
 
        left.text_index = 'a'
1529
 
        self.assertCurrentlyNotEqual(left, right)
1530
 
        right.text_index = 'a'
1531
 
        self.assertCurrentlyEqual(left, right)
1532
 
        left.signature_index = 'a'
1533
 
        self.assertCurrentlyNotEqual(left, right)
1534
 
        right.signature_index = 'a'
1535
 
        self.assertCurrentlyEqual(left, right)
1536
 
        left.name = 'a'
1537
 
        self.assertCurrentlyNotEqual(left, right)
1538
 
        right.name = 'a'
1539
 
        self.assertCurrentlyEqual(left, right)
1540
 
        left.transport = 'a'
1541
 
        self.assertCurrentlyNotEqual(left, right)
1542
 
        right.transport = 'a'
1543
 
        self.assertCurrentlyEqual(left, right)
1544
 
 
1545
 
    def test_file_name(self):
1546
 
        pack = pack_repo.ExistingPack('', 'a_name', '', '', '', '')
1547
 
        self.assertEqual('a_name.pack', pack.file_name())
1548
 
 
1549
 
 
1550
 
class TestNewPack(TestCaseWithTransport):
1551
 
    """Tests for pack_repo.NewPack."""
1552
 
 
1553
 
    def test_new_instance_attributes(self):
1554
 
        upload_transport = self.get_transport('upload')
1555
 
        pack_transport = self.get_transport('pack')
1556
 
        index_transport = self.get_transport('index')
1557
 
        upload_transport.mkdir('.')
1558
 
        collection = pack_repo.RepositoryPackCollection(
1559
 
            repo=None,
1560
 
            transport=self.get_transport('.'),
1561
 
            index_transport=index_transport,
1562
 
            upload_transport=upload_transport,
1563
 
            pack_transport=pack_transport,
1564
 
            index_builder_class=BTreeBuilder,
1565
 
            index_class=BTreeGraphIndex,
1566
 
            use_chk_index=False)
1567
 
        pack = pack_repo.NewPack(collection)
1568
 
        self.addCleanup(pack.abort) # Make sure the write stream gets closed
1569
 
        self.assertIsInstance(pack.revision_index, BTreeBuilder)
1570
 
        self.assertIsInstance(pack.inventory_index, BTreeBuilder)
1571
 
        self.assertIsInstance(pack._hash, type(osutils.md5()))
1572
 
        self.assertTrue(pack.upload_transport is upload_transport)
1573
 
        self.assertTrue(pack.index_transport is index_transport)
1574
 
        self.assertTrue(pack.pack_transport is pack_transport)
1575
 
        self.assertEqual(None, pack.index_sizes)
1576
 
        self.assertEqual(20, len(pack.random_name))
1577
 
        self.assertIsInstance(pack.random_name, str)
1578
 
        self.assertIsInstance(pack.start_time, float)
1579
 
 
1580
 
 
1581
 
class TestPacker(TestCaseWithTransport):
1582
 
    """Tests for the packs repository Packer class."""
1583
 
 
1584
 
    def test_pack_optimizes_pack_order(self):
1585
 
        builder = self.make_branch_builder('.', format="1.9")
1586
 
        builder.start_series()
1587
 
        builder.build_snapshot('A', None, [
1588
 
            ('add', ('', 'root-id', 'directory', None)),
1589
 
            ('add', ('f', 'f-id', 'file', 'content\n'))])
1590
 
        builder.build_snapshot('B', ['A'],
1591
 
            [('modify', ('f-id', 'new-content\n'))])
1592
 
        builder.build_snapshot('C', ['B'],
1593
 
            [('modify', ('f-id', 'third-content\n'))])
1594
 
        builder.build_snapshot('D', ['C'],
1595
 
            [('modify', ('f-id', 'fourth-content\n'))])
1596
 
        b = builder.get_branch()
1597
 
        b.lock_read()
1598
 
        builder.finish_series()
1599
 
        self.addCleanup(b.unlock)
1600
 
        # At this point, we should have 4 pack files available
1601
 
        # Because of how they were built, they correspond to
1602
 
        # ['D', 'C', 'B', 'A']
1603
 
        packs = b.repository._pack_collection.packs
1604
 
        packer = pack_repo.Packer(b.repository._pack_collection,
1605
 
                                  packs, 'testing',
1606
 
                                  revision_ids=['B', 'C'])
1607
 
        # Now, when we are copying the B & C revisions, their pack files should
1608
 
        # be moved to the front of the stack
1609
 
        # The new ordering moves B & C to the front of the .packs attribute,
1610
 
        # and leaves the others in the original order.
1611
 
        new_packs = [packs[1], packs[2], packs[0], packs[3]]
1612
 
        new_pack = packer.pack()
1613
 
        self.assertEqual(new_packs, packer.packs)
1614
 
 
1615
 
 
1616
 
class TestOptimisingPacker(TestCaseWithTransport):
1617
 
    """Tests for the OptimisingPacker class."""
1618
 
 
1619
 
    def get_pack_collection(self):
1620
 
        repo = self.make_repository('.')
1621
 
        return repo._pack_collection
1622
 
 
1623
 
    def test_open_pack_will_optimise(self):
1624
 
        packer = pack_repo.OptimisingPacker(self.get_pack_collection(),
1625
 
                                            [], '.test')
1626
 
        new_pack = packer.open_pack()
1627
 
        self.addCleanup(new_pack.abort) # ensure cleanup
1628
 
        self.assertIsInstance(new_pack, pack_repo.NewPack)
1629
 
        self.assertTrue(new_pack.revision_index._optimize_for_size)
1630
 
        self.assertTrue(new_pack.inventory_index._optimize_for_size)
1631
 
        self.assertTrue(new_pack.text_index._optimize_for_size)
1632
 
        self.assertTrue(new_pack.signature_index._optimize_for_size)
1633
 
 
1634
 
 
1635
 
class TestCrossFormatPacks(TestCaseWithTransport):
1636
 
 
1637
 
    def log_pack(self, hint=None):
1638
 
        self.calls.append(('pack', hint))
1639
 
        self.orig_pack(hint=hint)
1640
 
        if self.expect_hint:
1641
 
            self.assertTrue(hint)
1642
 
 
1643
 
    def run_stream(self, src_fmt, target_fmt, expect_pack_called):
1644
 
        self.expect_hint = expect_pack_called
1645
 
        self.calls = []
1646
 
        source_tree = self.make_branch_and_tree('src', format=src_fmt)
1647
 
        source_tree.lock_write()
1648
 
        self.addCleanup(source_tree.unlock)
1649
 
        tip = source_tree.commit('foo')
1650
 
        target = self.make_repository('target', format=target_fmt)
1651
 
        target.lock_write()
1652
 
        self.addCleanup(target.unlock)
1653
 
        source = source_tree.branch.repository._get_source(target._format)
1654
 
        self.orig_pack = target.pack
1655
 
        target.pack = self.log_pack
1656
 
        search = target.search_missing_revision_ids(
1657
 
            source_tree.branch.repository, tip)
1658
 
        stream = source.get_stream(search)
1659
 
        from_format = source_tree.branch.repository._format
1660
 
        sink = target._get_sink()
1661
 
        sink.insert_stream(stream, from_format, [])
1662
 
        if expect_pack_called:
1663
 
            self.assertLength(1, self.calls)
1664
 
        else:
1665
 
            self.assertLength(0, self.calls)
1666
 
 
1667
 
    def run_fetch(self, src_fmt, target_fmt, expect_pack_called):
1668
 
        self.expect_hint = expect_pack_called
1669
 
        self.calls = []
1670
 
        source_tree = self.make_branch_and_tree('src', format=src_fmt)
1671
 
        source_tree.lock_write()
1672
 
        self.addCleanup(source_tree.unlock)
1673
 
        tip = source_tree.commit('foo')
1674
 
        target = self.make_repository('target', format=target_fmt)
1675
 
        target.lock_write()
1676
 
        self.addCleanup(target.unlock)
1677
 
        source = source_tree.branch.repository
1678
 
        self.orig_pack = target.pack
1679
 
        target.pack = self.log_pack
1680
 
        target.fetch(source)
1681
 
        if expect_pack_called:
1682
 
            self.assertLength(1, self.calls)
1683
 
        else:
1684
 
            self.assertLength(0, self.calls)
1685
 
 
1686
 
    def test_sink_format_hint_no(self):
1687
 
        # When the target format says packing makes no difference, pack is not
1688
 
        # called.
1689
 
        self.run_stream('1.9', 'rich-root-pack', False)
1690
 
 
1691
 
    def test_sink_format_hint_yes(self):
1692
 
        # When the target format says packing makes a difference, pack is
1693
 
        # called.
1694
 
        self.run_stream('1.9', '2a', True)
1695
 
 
1696
 
    def test_sink_format_same_no(self):
1697
 
        # When the formats are the same, pack is not called.
1698
 
        self.run_stream('2a', '2a', False)
1699
 
 
1700
 
    def test_IDS_format_hint_no(self):
1701
 
        # When the target format says packing makes no difference, pack is not
1702
 
        # called.
1703
 
        self.run_fetch('1.9', 'rich-root-pack', False)
1704
 
 
1705
 
    def test_IDS_format_hint_yes(self):
1706
 
        # When the target format says packing makes a difference, pack is
1707
 
        # called.
1708
 
        self.run_fetch('1.9', '2a', True)
1709
 
 
1710
 
    def test_IDS_format_same_no(self):
1711
 
        # When the formats are the same, pack is not called.
1712
 
        self.run_fetch('2a', '2a', False)