397
476
# Arguably, the deserialise_inventory should detect a mismatch, and
398
477
# raise an error, rather than silently using one revision_id over the
400
inv = repo.deserialise_inventory('test-rev-id', inv_xml)
479
self.assertRaises(AssertionError, repo._deserialise_inventory,
480
'test-rev-id', inv_xml)
481
inv = repo._deserialise_inventory('other-rev-id', inv_xml)
401
482
self.assertEqual('other-rev-id', inv.root.revision)
404
class KnitRepositoryStreamTests(test_knit.KnitTests):
405
"""Tests for knitrepo._get_stream_as_bytes."""
407
def test_get_stream_as_bytes(self):
409
k1 = self.make_test_knit()
410
k1.add_lines('text-a', [], test_knit.split_lines(test_knit.TEXT_1))
412
# Serialise it, check the output.
413
bytes = knitrepo._get_stream_as_bytes(k1, ['text-a'])
414
data = bencode.bdecode(bytes)
415
format, record = data
416
self.assertEqual('knit-plain', format)
417
self.assertEqual(['text-a', ['fulltext'], []], record[:3])
418
self.assertRecordContentEqual(k1, 'text-a', record[3])
420
def test_get_stream_as_bytes_all(self):
421
"""Get a serialised data stream for all the records in a knit.
423
Much like test_get_stream_all, except for get_stream_as_bytes.
425
k1 = self.make_test_knit()
426
# Insert the same data as BasicKnitTests.test_knit_join, as they seem
427
# to cover a range of cases (no parents, one parent, multiple parents).
429
('text-a', [], test_knit.TEXT_1),
430
('text-b', ['text-a'], test_knit.TEXT_1),
431
('text-c', [], test_knit.TEXT_1),
432
('text-d', ['text-c'], test_knit.TEXT_1),
433
('text-m', ['text-b', 'text-d'], test_knit.TEXT_1),
435
# This test is actually a bit strict as the order in which they're
436
# returned is not defined. This matches the current (deterministic)
438
expected_data_list = [
439
# version, options, parents
440
('text-a', ['fulltext'], []),
441
('text-b', ['line-delta'], ['text-a']),
442
('text-m', ['line-delta'], ['text-b', 'text-d']),
443
('text-c', ['fulltext'], []),
444
('text-d', ['line-delta'], ['text-c']),
446
for version_id, parents, lines in test_data:
447
k1.add_lines(version_id, parents, test_knit.split_lines(lines))
449
bytes = knitrepo._get_stream_as_bytes(
450
k1, ['text-a', 'text-b', 'text-m', 'text-c', 'text-d', ])
452
data = bencode.bdecode(bytes)
454
self.assertEqual('knit-plain', format)
456
for expected, actual in zip(expected_data_list, data):
457
expected_version = expected[0]
458
expected_options = expected[1]
459
expected_parents = expected[2]
460
version, options, parents, bytes = actual
461
self.assertEqual(expected_version, version)
462
self.assertEqual(expected_options, options)
463
self.assertEqual(expected_parents, parents)
464
self.assertRecordContentEqual(k1, version, bytes)
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)
467
490
class DummyRepository(object):
468
491
"""A dummy repository for testing."""
470
494
_serializer = None
472
496
def supports_rich_root(self):
497
if self._format is not None:
498
return self._format.rich_root_data
502
raise NotImplementedError
504
def get_parent_map(self, revision_ids):
505
raise NotImplementedError
476
508
class InterDummy(repository.InterRepository):
477
509
"""An inter-repository optimised code path for DummyRepository.
651
660
tree = self.make_branch_and_tree('.', format)
652
661
tree.commit("Dull commit", rev_id="dull")
653
662
revision_tree = tree.branch.repository.revision_tree('dull')
654
self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
655
revision_tree.inventory.root.file_id)
663
revision_tree.lock_read()
665
self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
666
revision_tree.inventory.root.file_id)
668
revision_tree.unlock()
656
669
format = bzrdir.BzrDirMetaFormat1()
657
670
format.repository_format = knitrepo.RepositoryFormatKnit3()
658
671
upgrade.Convert('.', format)
659
672
tree = workingtree.WorkingTree.open('.')
660
673
revision_tree = tree.branch.repository.revision_tree('dull')
661
revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
674
revision_tree.lock_read()
676
revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
678
revision_tree.unlock()
662
679
tree.commit("Another dull commit", rev_id='dull2')
663
680
revision_tree = tree.branch.repository.revision_tree('dull2')
681
revision_tree.lock_read()
682
self.addCleanup(revision_tree.unlock)
664
683
self.assertEqual('dull', revision_tree.inventory.root.revision)
666
def test_exposed_versioned_files_are_marked_dirty(self):
685
def test_supports_external_lookups(self):
667
686
format = bzrdir.BzrDirMetaFormat1()
668
687
format.repository_format = knitrepo.RepositoryFormatKnit3()
669
688
repo = self.make_repository('.', format=format)
671
inv = repo.get_inventory_weave()
673
self.assertRaises(errors.OutSideTransaction,
674
inv.add_lines, 'foo', [], [])
689
self.assertFalse(repo._format.supports_external_lookups)
692
class Test2a(tests.TestCaseWithMemoryTransport):
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)
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])
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)
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])
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)
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])
760
def test_format_pack_compresses_True(self):
761
repo = self.make_repository('repo', format='2a')
762
self.assertTrue(repo._format.pack_compresses)
764
def test_inventories_use_chk_map_with_parent_base_dict(self):
765
tree = self.make_branch_and_memory_tree('repo', format="2a")
767
tree.add([''], ['TREE_ROOT'])
768
revid = tree.commit("foo")
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)
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')
787
self.addCleanup(tree.unlock)
788
tree.add([''], ['TREE_ROOT'])
789
for pos in range(20):
790
tree.commit(str(pos))
792
def test_pack_with_hint(self):
793
tree = self.make_branch_and_memory_tree('tree', format='2a')
795
self.addCleanup(tree.unlock)
796
tree.add([''], ['TREE_ROOT'])
797
# 1 commit to leave untouched
799
to_keep = tree.branch.repository._pack_collection.names()
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)
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)
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)
827
def test_get_stream_for_missing_keys_includes_all_chk_refs(self):
828
source_builder = self.make_branch_builder('source',
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':
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
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')),
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)
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,
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)
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',)],
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)
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)
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)
901
class TestKnitPackStreamSource(tests.TestCaseWithMemoryTransport):
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)
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)
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)
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)
927
def test_source_to_remote_exact_pack_19(self):
928
trans = self.make_smart_server('target')
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)
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)
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)
948
def test_source_to_remote_non_exact_pack_19(self):
949
trans = self.make_smart_server('target')
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)
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)
964
class TestDevelopment6FindParentIdsOfRevisions(TestCaseWithTransport):
965
"""Tests for _find_parent_ids_of_revisions."""
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)
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)))
981
def test_simple(self):
982
self.builder.build_snapshot('revid1', None, [])
983
self.builder.build_snapshot('revid2', ['revid1'], [])
985
self.assertParentIds(['revid1'], rev_set)
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)
994
def test_not_null(self):
995
rev_set = ['initial']
996
self.assertParentIds([], rev_set)
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)
1003
def test_ghost(self):
1004
self.builder.build_snapshot('revid1', None, [])
1005
rev_set = ['ghost', 'revid1']
1006
self.assertParentIds(['initial'], rev_set)
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)
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)
677
1023
class TestWithBrokenRepo(TestCaseWithTransport):
755
1103
broken_repo = self.make_broken_repository()
756
1104
empty_repo = self.make_repository('empty-repo')
757
stream = broken_repo.get_data_stream(['rev1a', 'rev2', 'rev3'])
758
empty_repo.lock_write()
1106
empty_repo.fetch(broken_repo)
1107
except (errors.RevisionNotPresent, errors.BzrCheckError):
1108
# Test successful: compression parent not being copied leads to
1111
empty_repo.lock_read()
759
1112
self.addCleanup(empty_repo.unlock)
760
empty_repo.start_write_group()
763
errors.KnitCorrupt, empty_repo.insert_data_stream, stream)
765
empty_repo.abort_write_group()
768
class TestKnitPackNoSubtrees(TestCaseWithTransport):
770
def get_format(self):
771
return bzrdir.format_registry.make_bzrdir('pack-0.92')
773
def test_disk_layout(self):
774
format = self.get_format()
775
repo = self.make_repository('.', format=format)
776
# in case of side effects of locking.
779
t = repo.bzrdir.get_repository_transport(None)
781
# XXX: no locks left when unlocked at the moment
782
# self.assertEqualDiff('', t.get('lock').read())
783
self.check_databases(t)
785
def check_format(self, t):
786
self.assertEqualDiff(
787
"Bazaar pack repository format 1 (needs bzr 0.92)\n",
788
t.get('format').read())
790
def assertHasKndx(self, t, knit_name):
791
"""Assert that knit_name exists on t."""
792
self.assertEqualDiff('# bzr knit index 8\n',
793
t.get(knit_name + '.kndx').read())
795
def assertHasNoKndx(self, t, knit_name):
796
"""Assert that knit_name has no index on t."""
797
self.assertFalse(t.has(knit_name + '.kndx'))
799
def assertHasNoKnit(self, t, knit_name):
800
"""Assert that knit_name exists on t."""
802
self.assertFalse(t.has(knit_name + '.knit'))
804
def check_databases(self, t):
805
"""check knit content for a repository."""
806
# check conversion worked
807
self.assertHasNoKndx(t, 'inventory')
808
self.assertHasNoKnit(t, 'inventory')
809
self.assertHasNoKndx(t, 'revisions')
810
self.assertHasNoKnit(t, 'revisions')
811
self.assertHasNoKndx(t, 'signatures')
812
self.assertHasNoKnit(t, 'signatures')
813
self.assertFalse(t.has('knits'))
814
# revision-indexes file-container directory
816
list(GraphIndex(t, 'pack-names', None).iter_all_entries()))
817
self.assertTrue(S_ISDIR(t.stat('packs').st_mode))
818
self.assertTrue(S_ISDIR(t.stat('upload').st_mode))
819
self.assertTrue(S_ISDIR(t.stat('indices').st_mode))
820
self.assertTrue(S_ISDIR(t.stat('obsolete_packs').st_mode))
822
def test_shared_disk_layout(self):
823
format = self.get_format()
824
repo = self.make_repository('.', shared=True, format=format)
826
t = repo.bzrdir.get_repository_transport(None)
828
# XXX: no locks left when unlocked at the moment
829
# self.assertEqualDiff('', t.get('lock').read())
830
# We should have a 'shared-storage' marker file.
831
self.assertEqualDiff('', t.get('shared-storage').read())
832
self.check_databases(t)
834
def test_shared_no_tree_disk_layout(self):
835
format = self.get_format()
836
repo = self.make_repository('.', shared=True, format=format)
837
repo.set_make_working_trees(False)
839
t = repo.bzrdir.get_repository_transport(None)
841
# XXX: no locks left when unlocked at the moment
842
# self.assertEqualDiff('', t.get('lock').read())
843
# We should have a 'shared-storage' marker file.
844
self.assertEqualDiff('', t.get('shared-storage').read())
845
# We should have a marker for the no-working-trees flag.
846
self.assertEqualDiff('', t.get('no-working-trees').read())
847
# The marker should go when we toggle the setting.
848
repo.set_make_working_trees(True)
849
self.assertFalse(t.has('no-working-trees'))
850
self.check_databases(t)
852
def test_adding_revision_creates_pack_indices(self):
853
format = self.get_format()
854
tree = self.make_branch_and_tree('.', format=format)
855
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
857
list(GraphIndex(trans, 'pack-names', None).iter_all_entries()))
858
tree.commit('foobarbaz')
859
index = GraphIndex(trans, 'pack-names', None)
860
index_nodes = list(index.iter_all_entries())
861
self.assertEqual(1, len(index_nodes))
862
node = index_nodes[0]
864
# the pack sizes should be listed in the index
866
sizes = [int(digits) for digits in pack_value.split(' ')]
867
for size, suffix in zip(sizes, ['.rix', '.iix', '.tix', '.six']):
868
stat = trans.stat('indices/%s%s' % (name, suffix))
869
self.assertEqual(size, stat.st_size)
871
def test_pulling_nothing_leads_to_no_new_names(self):
872
format = self.get_format()
873
tree1 = self.make_branch_and_tree('1', format=format)
874
tree2 = self.make_branch_and_tree('2', format=format)
875
tree1.branch.repository.fetch(tree2.branch.repository)
876
trans = tree1.branch.repository.bzrdir.get_repository_transport(None)
878
list(GraphIndex(trans, 'pack-names', None).iter_all_entries()))
880
def test_commit_across_pack_shape_boundary_autopacks(self):
881
format = self.get_format()
882
tree = self.make_branch_and_tree('.', format=format)
883
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
884
# This test could be a little cheaper by replacing the packs
885
# attribute on the repository to allow a different pack distribution
886
# and max packs policy - so we are checking the policy is honoured
887
# in the test. But for now 11 commits is not a big deal in a single
890
tree.commit('commit %s' % x)
891
# there should be 9 packs:
892
index = GraphIndex(trans, 'pack-names', None)
893
self.assertEqual(9, len(list(index.iter_all_entries())))
894
# insert some files in obsolete_packs which should be removed by pack.
895
trans.put_bytes('obsolete_packs/foo', '123')
896
trans.put_bytes('obsolete_packs/bar', '321')
897
# committing one more should coalesce to 1 of 10.
898
tree.commit('commit triggering pack')
899
index = GraphIndex(trans, 'pack-names', None)
900
self.assertEqual(1, len(list(index.iter_all_entries())))
901
# packing should not damage data
902
tree = tree.bzrdir.open_workingtree()
903
check_result = tree.branch.repository.check(
904
[tree.branch.last_revision()])
905
# We should have 50 (10x5) files in the obsolete_packs directory.
906
obsolete_files = list(trans.list_dir('obsolete_packs'))
907
self.assertFalse('foo' in obsolete_files)
908
self.assertFalse('bar' in obsolete_files)
909
self.assertEqual(50, len(obsolete_files))
910
# XXX: Todo check packs obsoleted correctly - old packs and indices
911
# in the obsolete_packs directory.
912
large_pack_name = list(index.iter_all_entries())[0][1][0]
913
# finally, committing again should not touch the large pack.
914
tree.commit('commit not triggering pack')
915
index = GraphIndex(trans, 'pack-names', None)
916
self.assertEqual(2, len(list(index.iter_all_entries())))
917
pack_names = [node[1][0] for node in index.iter_all_entries()]
918
self.assertTrue(large_pack_name in pack_names)
920
def test_pack_after_two_commits_packs_everything(self):
921
format = self.get_format()
922
tree = self.make_branch_and_tree('.', format=format)
923
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
925
tree.commit('more work')
926
tree.branch.repository.pack()
927
# there should be 1 pack:
928
index = GraphIndex(trans, 'pack-names', None)
929
self.assertEqual(1, len(list(index.iter_all_entries())))
930
self.assertEqual(2, len(tree.branch.repository.all_revision_ids()))
932
def test_pack_layout(self):
933
format = self.get_format()
934
tree = self.make_branch_and_tree('.', format=format)
935
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
936
tree.commit('start', rev_id='1')
937
tree.commit('more work', rev_id='2')
938
tree.branch.repository.pack()
940
self.addCleanup(tree.unlock)
941
pack = tree.branch.repository._pack_collection.get_pack_by_name(
942
tree.branch.repository._pack_collection.names()[0])
943
# revision access tends to be tip->ancestor, so ordering that way on
944
# disk is a good idea.
945
for _1, key, val, refs in pack.revision_index.iter_all_entries():
947
pos_1 = int(val[1:].split()[0])
949
pos_2 = int(val[1:].split()[0])
950
self.assertTrue(pos_2 < pos_1)
952
def test_pack_repositories_support_multiple_write_locks(self):
953
format = self.get_format()
954
self.make_repository('.', shared=True, format=format)
955
r1 = repository.Repository.open('.')
956
r2 = repository.Repository.open('.')
958
self.addCleanup(r1.unlock)
962
def _add_text(self, repo, fileid):
963
"""Add a text to the repository within a write group."""
964
vf =repo.weave_store.get_weave(fileid, repo.get_transaction())
965
vf.add_lines('samplerev+' + fileid, [], [])
967
def test_concurrent_writers_merge_new_packs(self):
968
format = self.get_format()
969
self.make_repository('.', shared=True, format=format)
970
r1 = repository.Repository.open('.')
971
r2 = repository.Repository.open('.')
974
# access enough data to load the names list
975
list(r1.all_revision_ids())
978
# access enough data to load the names list
979
list(r2.all_revision_ids())
980
r1.start_write_group()
982
r2.start_write_group()
984
self._add_text(r1, 'fileidr1')
985
self._add_text(r2, 'fileidr2')
987
r2.abort_write_group()
990
r1.abort_write_group()
992
# both r1 and r2 have open write groups with data in them
993
# created while the other's write group was open.
994
# Commit both which requires a merge to the pack-names.
996
r1.commit_write_group()
998
r1.abort_write_group()
999
r2.abort_write_group()
1001
r2.commit_write_group()
1002
# tell r1 to reload from disk
1003
r1._pack_collection.reset()
1004
# Now both repositories should know about both names
1005
r1._pack_collection.ensure_loaded()
1006
r2._pack_collection.ensure_loaded()
1007
self.assertEqual(r1._pack_collection.names(), r2._pack_collection.names())
1008
self.assertEqual(2, len(r1._pack_collection.names()))
1014
def test_concurrent_writer_second_preserves_dropping_a_pack(self):
1015
format = self.get_format()
1016
self.make_repository('.', shared=True, format=format)
1017
r1 = repository.Repository.open('.')
1018
r2 = repository.Repository.open('.')
1019
# add a pack to drop
1022
r1.start_write_group()
1024
self._add_text(r1, 'fileidr1')
1026
r1.abort_write_group()
1029
r1.commit_write_group()
1030
r1._pack_collection.ensure_loaded()
1031
name_to_drop = r1._pack_collection.all_packs()[0].name
1036
# access enough data to load the names list
1037
list(r1.all_revision_ids())
1040
# access enough data to load the names list
1041
list(r2.all_revision_ids())
1042
r1._pack_collection.ensure_loaded()
1044
r2.start_write_group()
1046
# in r1, drop the pack
1047
r1._pack_collection._remove_pack_from_memory(
1048
r1._pack_collection.get_pack_by_name(name_to_drop))
1050
self._add_text(r2, 'fileidr2')
1052
r2.abort_write_group()
1055
r1._pack_collection.reset()
1057
# r1 has a changed names list, and r2 an open write groups with
1059
# save r1, and then commit the r2 write group, which requires a
1060
# merge to the pack-names, which should not reinstate
1063
r1._pack_collection._save_pack_names()
1064
r1._pack_collection.reset()
1066
r2.abort_write_group()
1069
r2.commit_write_group()
1071
r2.abort_write_group()
1073
# Now both repositories should now about just one name.
1074
r1._pack_collection.ensure_loaded()
1075
r2._pack_collection.ensure_loaded()
1076
self.assertEqual(r1._pack_collection.names(), r2._pack_collection.names())
1077
self.assertEqual(1, len(r1._pack_collection.names()))
1078
self.assertFalse(name_to_drop in r1._pack_collection.names())
1084
def test_lock_write_does_not_physically_lock(self):
1085
repo = self.make_repository('.', format=self.get_format())
1087
self.addCleanup(repo.unlock)
1088
self.assertFalse(repo.get_physical_lock_status())
1090
def prepare_for_break_lock(self):
1091
# Setup the global ui factory state so that a break-lock method call
1092
# will find usable input in the input stream.
1093
old_factory = bzrlib.ui.ui_factory
1094
def restoreFactory():
1095
bzrlib.ui.ui_factory = old_factory
1096
self.addCleanup(restoreFactory)
1097
bzrlib.ui.ui_factory = bzrlib.ui.SilentUIFactory()
1098
bzrlib.ui.ui_factory.stdin = StringIO("y\n")
1100
def test_break_lock_breaks_physical_lock(self):
1101
repo = self.make_repository('.', format=self.get_format())
1102
repo._pack_collection.lock_names()
1103
repo2 = repository.Repository.open('.')
1104
self.assertTrue(repo.get_physical_lock_status())
1105
self.prepare_for_break_lock()
1107
self.assertFalse(repo.get_physical_lock_status())
1109
def test_broken_physical_locks_error_on__unlock_names_lock(self):
1110
repo = self.make_repository('.', format=self.get_format())
1111
repo._pack_collection.lock_names()
1112
self.assertTrue(repo.get_physical_lock_status())
1113
repo2 = repository.Repository.open('.')
1114
self.prepare_for_break_lock()
1116
self.assertRaises(errors.LockBroken, repo._pack_collection._unlock_names)
1118
def test_fetch_without_find_ghosts_ignores_ghosts(self):
1119
# we want two repositories at this point:
1120
# one with a revision that is a ghost in the other
1122
# 'ghost' is present in has_ghost, 'ghost' is absent in 'missing_ghost'.
1123
# 'references' is present in both repositories, and 'tip' is present
1124
# just in has_ghost.
1125
# has_ghost missing_ghost
1126
#------------------------------
1128
# 'references' 'references'
1130
# In this test we fetch 'tip' which should not fetch 'ghost'
1131
has_ghost = self.make_repository('has_ghost', format=self.get_format())
1132
missing_ghost = self.make_repository('missing_ghost',
1133
format=self.get_format())
1135
def add_commit(repo, revision_id, parent_ids):
1137
repo.start_write_group()
1138
inv = inventory.Inventory(revision_id=revision_id)
1139
inv.root.revision = revision_id
1140
root_id = inv.root.file_id
1141
sha1 = repo.add_inventory(revision_id, inv, [])
1142
vf = repo.weave_store.get_weave_or_empty(root_id,
1143
repo.get_transaction())
1144
vf.add_lines(revision_id, [], [])
1145
rev = bzrlib.revision.Revision(timestamp=0,
1147
committer="Foo Bar <foo@example.com>",
1149
inventory_sha1=sha1,
1150
revision_id=revision_id)
1151
rev.parent_ids = parent_ids
1152
repo.add_revision(revision_id, rev)
1153
repo.commit_write_group()
1155
add_commit(has_ghost, 'ghost', [])
1156
add_commit(has_ghost, 'references', ['ghost'])
1157
add_commit(missing_ghost, 'references', ['ghost'])
1158
add_commit(has_ghost, 'tip', ['references'])
1159
missing_ghost.fetch(has_ghost, 'tip')
1160
# missing ghost now has tip and not ghost.
1161
rev = missing_ghost.get_revision('tip')
1162
inv = missing_ghost.get_inventory('tip')
1163
self.assertRaises(errors.NoSuchRevision,
1164
missing_ghost.get_revision, 'ghost')
1165
self.assertRaises(errors.RevisionNotPresent,
1166
missing_ghost.get_inventory, 'ghost')
1169
class TestKnitPackSubtrees(TestKnitPackNoSubtrees):
1171
def get_format(self):
1172
return bzrdir.format_registry.make_bzrdir(
1173
'pack-0.92-subtree')
1175
def check_format(self, t):
1176
self.assertEqualDiff(
1177
"Bazaar pack repository format 1 with subtree support (needs bzr 0.92)\n",
1178
t.get('format').read())
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'))
1181
1118
class TestRepositoryPackCollection(TestCaseWithTransport):
1336
1348
tree.lock_read()
1337
1349
self.addCleanup(tree.unlock)
1338
1350
packs = tree.branch.repository._pack_collection
1339
1352
packs.ensure_loaded()
1340
1353
name = packs.names()[0]
1341
1354
pack_1 = packs.get_pack_by_name(name)
1342
1355
# the pack should be correctly initialised
1343
rev_index = GraphIndex(packs._index_transport, name + '.rix',
1344
packs._names[name][0])
1345
inv_index = GraphIndex(packs._index_transport, name + '.iix',
1346
packs._names[name][1])
1347
txt_index = GraphIndex(packs._index_transport, name + '.tix',
1348
packs._names[name][2])
1349
sig_index = GraphIndex(packs._index_transport, name + '.six',
1350
packs._names[name][3])
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])
1351
1361
self.assertEqual(pack_repo.ExistingPack(packs._pack_transport,
1352
1362
name, rev_index, inv_index, txt_index, sig_index), pack_1)
1353
1363
# and the same instance should be returned on successive calls.
1354
1364
self.assertTrue(pack_1 is packs.get_pack_by_name(name))
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())
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())
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())
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]))
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
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('.'))
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(),
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))
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))
1357
1500
class TestPack(TestCaseWithTransport):
1358
1501
"""Tests for the Pack object."""
1429
1581
class TestPacker(TestCaseWithTransport):
1430
1582
"""Tests for the packs repository Packer class."""
1432
# To date, this class has been factored out and nothing new added to it;
1433
# thus there are not yet any tests.
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()
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,
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)
1616
class TestOptimisingPacker(TestCaseWithTransport):
1617
"""Tests for the OptimisingPacker class."""
1619
def get_pack_collection(self):
1620
repo = self.make_repository('.')
1621
return repo._pack_collection
1623
def test_open_pack_will_optimise(self):
1624
packer = pack_repo.OptimisingPacker(self.get_pack_collection(),
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)
1635
class TestCrossFormatPacks(TestCaseWithTransport):
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)
1643
def run_stream(self, src_fmt, target_fmt, expect_pack_called):
1644
self.expect_hint = expect_pack_called
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)
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)
1665
self.assertLength(0, self.calls)
1667
def run_fetch(self, src_fmt, target_fmt, expect_pack_called):
1668
self.expect_hint = expect_pack_called
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)
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)
1684
self.assertLength(0, self.calls)
1686
def test_sink_format_hint_no(self):
1687
# When the target format says packing makes no difference, pack is not
1689
self.run_stream('1.9', 'rich-root-pack', False)
1691
def test_sink_format_hint_yes(self):
1692
# When the target format says packing makes a difference, pack is
1694
self.run_stream('1.9', '2a', True)
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)
1700
def test_IDS_format_hint_no(self):
1701
# When the target format says packing makes no difference, pack is not
1703
self.run_fetch('1.9', 'rich-root-pack', False)
1705
def test_IDS_format_hint_yes(self):
1706
# When the target format says packing makes a difference, pack is
1708
self.run_fetch('1.9', '2a', True)
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)
added
removed
bzrlib/tests/test_repository.py
