109
118
return "opened repository."
121
class SampleExtraRepositoryFormat(repository.RepositoryFormat):
122
"""A sample format that can not be used in a metadir
126
def get_format_string(self):
127
raise NotImplementedError
112
130
class TestRepositoryFormat(TestCaseWithTransport):
113
131
"""Tests for the Repository format detection used by the bzr meta dir facility.BzrBranchFormat facility."""
115
133
def test_find_format(self):
116
134
# is the right format object found for a repository?
117
135
# create a branch with a few known format objects.
118
# this is not quite the same as
136
# this is not quite the same as
119
137
self.build_tree(["foo/", "bar/"])
120
138
def check_format(format, url):
121
139
dir = format._matchingbzrdir.initialize(url)
122
140
format.initialize(dir)
123
t = get_transport(url)
124
found_format = repository.RepositoryFormat.find_format(dir)
125
self.failUnless(isinstance(found_format, format.__class__))
126
check_format(weaverepo.RepositoryFormat7(), "bar")
141
t = transport.get_transport_from_path(url)
142
found_format = repository.RepositoryFormatMetaDir.find_format(dir)
143
self.assertIsInstance(found_format, format.__class__)
144
check_format(repository.format_registry.get_default(), "bar")
128
146
def test_find_format_no_repository(self):
129
147
dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
130
148
self.assertRaises(errors.NoRepositoryPresent,
131
repository.RepositoryFormat.find_format,
149
repository.RepositoryFormatMetaDir.find_format,
152
def test_from_string(self):
153
self.assertIsInstance(
154
SampleRepositoryFormat.from_string(
155
"Sample .bzr repository format."),
156
SampleRepositoryFormat)
157
self.assertRaises(AssertionError,
158
SampleRepositoryFormat.from_string,
159
"Different .bzr repository format.")
134
161
def test_find_format_unknown_format(self):
135
162
dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
136
163
SampleRepositoryFormat().initialize(dir)
137
164
self.assertRaises(UnknownFormatError,
138
repository.RepositoryFormat.find_format,
165
repository.RepositoryFormatMetaDir.find_format,
168
def test_find_format_with_features(self):
169
tree = self.make_branch_and_tree('.', format='2a')
170
tree.branch.repository.update_feature_flags({"name": "necessity"})
171
found_format = repository.RepositoryFormatMetaDir.find_format(tree.bzrdir)
172
self.assertIsInstance(found_format, repository.RepositoryFormatMetaDir)
173
self.assertEqual(found_format.features.get("name"), "necessity")
174
self.assertRaises(errors.MissingFeature, found_format.check_support_status,
176
self.addCleanup(repository.RepositoryFormatMetaDir.unregister_feature,
178
repository.RepositoryFormatMetaDir.register_feature("name")
179
found_format.check_support_status(True)
182
class TestRepositoryFormatRegistry(TestCase):
185
super(TestRepositoryFormatRegistry, self).setUp()
186
self.registry = repository.RepositoryFormatRegistry()
141
188
def test_register_unregister_format(self):
142
189
format = SampleRepositoryFormat()
144
dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
146
format.initialize(dir)
147
# register a format for it.
148
repository.RepositoryFormat.register_format(format)
149
# which repository.Open will refuse (not supported)
150
self.assertRaises(UnsupportedFormatError, repository.Repository.open, self.get_url())
151
# but open(unsupported) will work
152
self.assertEqual(format.open(dir), "opened repository.")
153
# unregister the format
154
repository.RepositoryFormat.unregister_format(format)
157
class TestFormat6(TestCaseWithTransport):
159
def test_no_ancestry_weave(self):
160
control = bzrdir.BzrDirFormat6().initialize(self.get_url())
161
repo = weaverepo.RepositoryFormat6().initialize(control)
162
# We no longer need to create the ancestry.weave file
163
# since it is *never* used.
164
self.assertRaises(NoSuchFile,
165
control.transport.get,
168
def test_exposed_versioned_files_are_marked_dirty(self):
169
control = bzrdir.BzrDirFormat6().initialize(self.get_url())
170
repo = weaverepo.RepositoryFormat6().initialize(control)
172
inv = repo.get_inventory_weave()
174
self.assertRaises(errors.OutSideTransaction,
175
inv.add_lines, 'foo', [], [])
178
class TestFormat7(TestCaseWithTransport):
180
def test_disk_layout(self):
181
control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
182
repo = weaverepo.RepositoryFormat7().initialize(control)
183
# in case of side effects of locking.
187
# format 'Bazaar-NG Repository format 7'
189
# inventory.weave == empty_weave
190
# empty revision-store directory
191
# empty weaves directory
192
t = control.get_repository_transport(None)
193
self.assertEqualDiff('Bazaar-NG Repository format 7',
194
t.get('format').read())
195
self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
196
self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
197
self.assertEqualDiff('# bzr weave file v5\n'
200
t.get('inventory.weave').read())
202
def test_shared_disk_layout(self):
203
control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
204
repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
206
# format 'Bazaar-NG Repository format 7'
207
# inventory.weave == empty_weave
208
# empty revision-store directory
209
# empty weaves directory
210
# a 'shared-storage' marker file.
211
# lock is not present when unlocked
212
t = control.get_repository_transport(None)
213
self.assertEqualDiff('Bazaar-NG Repository format 7',
214
t.get('format').read())
215
self.assertEqualDiff('', t.get('shared-storage').read())
216
self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
217
self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
218
self.assertEqualDiff('# bzr weave file v5\n'
221
t.get('inventory.weave').read())
222
self.assertFalse(t.has('branch-lock'))
224
def test_creates_lockdir(self):
225
"""Make sure it appears to be controlled by a LockDir existence"""
226
control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
227
repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
228
t = control.get_repository_transport(None)
229
# TODO: Should check there is a 'lock' toplevel directory,
230
# regardless of contents
231
self.assertFalse(t.has('lock/held/info'))
234
self.assertTrue(t.has('lock/held/info'))
236
# unlock so we don't get a warning about failing to do so
239
def test_uses_lockdir(self):
240
"""repo format 7 actually locks on lockdir"""
241
base_url = self.get_url()
242
control = bzrdir.BzrDirMetaFormat1().initialize(base_url)
243
repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
244
t = control.get_repository_transport(None)
248
# make sure the same lock is created by opening it
249
repo = repository.Repository.open(base_url)
251
self.assertTrue(t.has('lock/held/info'))
253
self.assertFalse(t.has('lock/held/info'))
255
def test_shared_no_tree_disk_layout(self):
256
control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
257
repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
258
repo.set_make_working_trees(False)
260
# format 'Bazaar-NG Repository format 7'
262
# inventory.weave == empty_weave
263
# empty revision-store directory
264
# empty weaves directory
265
# a 'shared-storage' marker file.
266
t = control.get_repository_transport(None)
267
self.assertEqualDiff('Bazaar-NG Repository format 7',
268
t.get('format').read())
269
## self.assertEqualDiff('', t.get('lock').read())
270
self.assertEqualDiff('', t.get('shared-storage').read())
271
self.assertEqualDiff('', t.get('no-working-trees').read())
272
repo.set_make_working_trees(True)
273
self.assertFalse(t.has('no-working-trees'))
274
self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
275
self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
276
self.assertEqualDiff('# bzr weave file v5\n'
279
t.get('inventory.weave').read())
281
def test_exposed_versioned_files_are_marked_dirty(self):
282
control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
283
repo = weaverepo.RepositoryFormat7().initialize(control)
285
inv = repo.get_inventory_weave()
287
self.assertRaises(errors.OutSideTransaction,
288
inv.add_lines, 'foo', [], [])
190
self.registry.register(format)
191
self.assertEqual(format, self.registry.get("Sample .bzr repository format."))
192
self.registry.remove(format)
193
self.assertRaises(KeyError, self.registry.get, "Sample .bzr repository format.")
195
def test_get_all(self):
196
format = SampleRepositoryFormat()
197
self.assertEqual([], self.registry._get_all())
198
self.registry.register(format)
199
self.assertEqual([format], self.registry._get_all())
201
def test_register_extra(self):
202
format = SampleExtraRepositoryFormat()
203
self.assertEqual([], self.registry._get_all())
204
self.registry.register_extra(format)
205
self.assertEqual([format], self.registry._get_all())
207
def test_register_extra_lazy(self):
208
self.assertEqual([], self.registry._get_all())
209
self.registry.register_extra_lazy("bzrlib.tests.test_repository",
210
"SampleExtraRepositoryFormat")
211
formats = self.registry._get_all()
212
self.assertEqual(1, len(formats))
213
self.assertIsInstance(formats[0], SampleExtraRepositoryFormat)
291
216
class TestFormatKnit1(TestCaseWithTransport):
218
def test_attribute__fetch_order(self):
219
"""Knits need topological data insertion."""
220
repo = self.make_repository('.',
221
format=controldir.format_registry.get('knit')())
222
self.assertEqual('topological', repo._format._fetch_order)
224
def test_attribute__fetch_uses_deltas(self):
225
"""Knits reuse deltas."""
226
repo = self.make_repository('.',
227
format=controldir.format_registry.get('knit')())
228
self.assertEqual(True, repo._format._fetch_uses_deltas)
293
230
def test_disk_layout(self):
294
231
control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
295
232
repo = knitrepo.RepositoryFormatKnit1().initialize(control)
383
316
is valid when the api is not being abused.
385
318
repo = self.make_repository('.',
386
format=bzrdir.format_registry.get('knit')())
319
format=controldir.format_registry.get('knit')())
387
320
inv_xml = '<inventory format="5">\n</inventory>\n'
388
inv = repo.deserialise_inventory('test-rev-id', inv_xml)
321
inv = repo._deserialise_inventory('test-rev-id', inv_xml)
389
322
self.assertEqual('test-rev-id', inv.root.revision)
391
324
def test_deserialise_uses_global_revision_id(self):
392
325
"""If it is set, then we re-use the global revision id"""
393
326
repo = self.make_repository('.',
394
format=bzrdir.format_registry.get('knit')())
327
format=controldir.format_registry.get('knit')())
395
328
inv_xml = ('<inventory format="5" revision_id="other-rev-id">\n'
396
329
'</inventory>\n')
397
330
# Arguably, the deserialise_inventory should detect a mismatch, and
398
331
# raise an error, rather than silently using one revision_id over the
400
inv = repo.deserialise_inventory('test-rev-id', inv_xml)
333
self.assertRaises(AssertionError, repo._deserialise_inventory,
334
'test-rev-id', inv_xml)
335
inv = repo._deserialise_inventory('other-rev-id', inv_xml)
401
336
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
expected_data_list = [
436
# version, options, parents
437
('text-a', ['fulltext'], []),
438
('text-b', ['line-delta'], ['text-a']),
439
('text-c', ['fulltext'], []),
440
('text-d', ['line-delta'], ['text-c']),
441
('text-m', ['line-delta'], ['text-b', 'text-d']),
443
for version_id, parents, lines in test_data:
444
k1.add_lines(version_id, parents, test_knit.split_lines(lines))
446
bytes = knitrepo._get_stream_as_bytes(
447
k1, ['text-a', 'text-b', 'text-c', 'text-d', 'text-m'])
449
data = bencode.bdecode(bytes)
451
self.assertEqual('knit-plain', format)
453
for expected, actual in zip(expected_data_list, data):
454
expected_version = expected[0]
455
expected_options = expected[1]
456
expected_parents = expected[2]
457
version, options, parents, bytes = actual
458
self.assertEqual(expected_version, version)
459
self.assertEqual(expected_options, options)
460
self.assertEqual(expected_parents, parents)
461
self.assertRecordContentEqual(k1, version, bytes)
338
def test_supports_external_lookups(self):
339
repo = self.make_repository('.',
340
format=controldir.format_registry.get('knit')())
341
self.assertFalse(repo._format.supports_external_lookups)
464
344
class DummyRepository(object):
465
345
"""A dummy repository for testing."""
467
348
_serializer = None
469
350
def supports_rich_root(self):
351
if self._format is not None:
352
return self._format.rich_root_data
356
raise NotImplementedError
358
def get_parent_map(self, revision_ids):
359
raise NotImplementedError
473
362
class InterDummy(repository.InterRepository):
474
363
"""An inter-repository optimised code path for DummyRepository.
540
441
self.assertGetsDefaultInterRepository(dummy_a, dummy_b)
543
class TestInterWeaveRepo(TestCaseWithTransport):
545
def test_is_compatible_and_registered(self):
546
# InterWeaveRepo is compatible when either side
547
# is a format 5/6/7 branch
548
from bzrlib.repofmt import knitrepo, weaverepo
549
formats = [weaverepo.RepositoryFormat5(),
550
weaverepo.RepositoryFormat6(),
551
weaverepo.RepositoryFormat7()]
552
incompatible_formats = [weaverepo.RepositoryFormat4(),
553
knitrepo.RepositoryFormatKnit1(),
555
repo_a = self.make_repository('a')
556
repo_b = self.make_repository('b')
557
is_compatible = repository.InterWeaveRepo.is_compatible
558
for source in incompatible_formats:
559
# force incompatible left then right
560
repo_a._format = source
561
repo_b._format = formats[0]
562
self.assertFalse(is_compatible(repo_a, repo_b))
563
self.assertFalse(is_compatible(repo_b, repo_a))
564
for source in formats:
565
repo_a._format = source
566
for target in formats:
567
repo_b._format = target
568
self.assertTrue(is_compatible(repo_a, repo_b))
569
self.assertEqual(repository.InterWeaveRepo,
570
repository.InterRepository.get(repo_a,
574
class TestInterRemoteToOther(TestCaseWithTransport):
576
def make_remote_repository(self, path, backing_format=None):
577
"""Make a RemoteRepository object backed by a real repository that will
578
be created at the given path."""
579
self.make_repository(path, format=backing_format)
580
smart_server = server.SmartTCPServer_for_testing()
582
remote_transport = get_transport(smart_server.get_url()).clone(path)
583
self.addCleanup(smart_server.tearDown)
584
remote_bzrdir = bzrdir.BzrDir.open_from_transport(remote_transport)
585
remote_repo = remote_bzrdir.open_repository()
588
def test_is_compatible_same_format(self):
589
"""InterRemoteToOther is compatible with a remote repository and a
590
second repository that have the same format."""
591
local_repo = self.make_repository('local')
592
remote_repo = self.make_remote_repository('remote')
593
is_compatible = repository.InterRemoteToOther.is_compatible
595
is_compatible(remote_repo, local_repo),
596
"InterRemoteToOther(%r, %r) is false" % (remote_repo, local_repo))
598
def test_is_incompatible_different_format(self):
599
local_repo = self.make_repository('local', 'dirstate')
600
remote_repo = self.make_remote_repository('a', 'dirstate-with-subtree')
601
is_compatible = repository.InterRemoteToOther.is_compatible
603
is_compatible(remote_repo, local_repo),
604
"InterRemoteToOther(%r, %r) is true" % (local_repo, remote_repo))
606
def test_is_incompatible_different_format_both_remote(self):
607
remote_repo_a = self.make_remote_repository(
608
'a', 'dirstate-with-subtree')
609
remote_repo_b = self.make_remote_repository('b', 'dirstate')
610
is_compatible = repository.InterRemoteToOther.is_compatible
612
is_compatible(remote_repo_a, remote_repo_b),
613
"InterRemoteToOther(%r, %r) is true"
614
% (remote_repo_a, remote_repo_b))
444
class TestRepositoryFormat1(knitrepo.RepositoryFormatKnit1):
447
def get_format_string(cls):
448
return "Test Format 1"
451
class TestRepositoryFormat2(knitrepo.RepositoryFormatKnit1):
454
def get_format_string(cls):
455
return "Test Format 2"
617
458
class TestRepositoryConverter(TestCaseWithTransport):
619
460
def test_convert_empty(self):
620
t = get_transport(self.get_url('.'))
461
source_format = TestRepositoryFormat1()
462
target_format = TestRepositoryFormat2()
463
repository.format_registry.register(source_format)
464
self.addCleanup(repository.format_registry.remove,
466
repository.format_registry.register(target_format)
467
self.addCleanup(repository.format_registry.remove,
469
t = self.get_transport()
621
470
t.mkdir('repository')
622
471
repo_dir = bzrdir.BzrDirMetaFormat1().initialize('repository')
623
repo = weaverepo.RepositoryFormat7().initialize(repo_dir)
624
target_format = knitrepo.RepositoryFormatKnit1()
472
repo = TestRepositoryFormat1().initialize(repo_dir)
625
473
converter = repository.CopyConverter(target_format)
626
474
pb = bzrlib.ui.ui_factory.nested_progress_bar()
648
503
tree = self.make_branch_and_tree('.', format)
649
504
tree.commit("Dull commit", rev_id="dull")
650
505
revision_tree = tree.branch.repository.revision_tree('dull')
651
self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
652
revision_tree.inventory.root.file_id)
506
revision_tree.lock_read()
508
self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
509
revision_tree.get_root_id())
511
revision_tree.unlock()
653
512
format = bzrdir.BzrDirMetaFormat1()
654
513
format.repository_format = knitrepo.RepositoryFormatKnit3()
655
514
upgrade.Convert('.', format)
656
515
tree = workingtree.WorkingTree.open('.')
657
516
revision_tree = tree.branch.repository.revision_tree('dull')
658
revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
517
revision_tree.lock_read()
519
revision_tree.get_file_lines(revision_tree.get_root_id())
521
revision_tree.unlock()
659
522
tree.commit("Another dull commit", rev_id='dull2')
660
523
revision_tree = tree.branch.repository.revision_tree('dull2')
661
self.assertEqual('dull', revision_tree.inventory.root.revision)
524
revision_tree.lock_read()
525
self.addCleanup(revision_tree.unlock)
526
self.assertEqual('dull',
527
revision_tree.get_file_revision(revision_tree.get_root_id()))
663
def test_exposed_versioned_files_are_marked_dirty(self):
529
def test_supports_external_lookups(self):
664
530
format = bzrdir.BzrDirMetaFormat1()
665
531
format.repository_format = knitrepo.RepositoryFormatKnit3()
666
532
repo = self.make_repository('.', format=format)
668
inv = repo.get_inventory_weave()
670
self.assertRaises(errors.OutSideTransaction,
671
inv.add_lines, 'foo', [], [])
533
self.assertFalse(repo._format.supports_external_lookups)
536
class Test2a(tests.TestCaseWithMemoryTransport):
538
def test_chk_bytes_uses_custom_btree_parser(self):
539
mt = self.make_branch_and_memory_tree('test', format='2a')
541
self.addCleanup(mt.unlock)
542
mt.add([''], ['root-id'])
544
index = mt.branch.repository.chk_bytes._index._graph_index._indices[0]
545
self.assertEqual(btree_index._gcchk_factory, index._leaf_factory)
546
# It should also work if we re-open the repo
547
repo = mt.branch.repository.bzrdir.open_repository()
549
self.addCleanup(repo.unlock)
550
index = repo.chk_bytes._index._graph_index._indices[0]
551
self.assertEqual(btree_index._gcchk_factory, index._leaf_factory)
553
def test_fetch_combines_groups(self):
554
builder = self.make_branch_builder('source', format='2a')
555
builder.start_series()
556
builder.build_snapshot('1', None, [
557
('add', ('', 'root-id', 'directory', '')),
558
('add', ('file', 'file-id', 'file', 'content\n'))])
559
builder.build_snapshot('2', ['1'], [
560
('modify', ('file-id', 'content-2\n'))])
561
builder.finish_series()
562
source = builder.get_branch()
563
target = self.make_repository('target', format='2a')
564
target.fetch(source.repository)
566
self.addCleanup(target.unlock)
567
details = target.texts._index.get_build_details(
568
[('file-id', '1',), ('file-id', '2',)])
569
file_1_details = details[('file-id', '1')]
570
file_2_details = details[('file-id', '2')]
571
# The index, and what to read off disk, should be the same for both
572
# versions of the file.
573
self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
575
def test_fetch_combines_groups(self):
576
builder = self.make_branch_builder('source', format='2a')
577
builder.start_series()
578
builder.build_snapshot('1', None, [
579
('add', ('', 'root-id', 'directory', '')),
580
('add', ('file', 'file-id', 'file', 'content\n'))])
581
builder.build_snapshot('2', ['1'], [
582
('modify', ('file-id', 'content-2\n'))])
583
builder.finish_series()
584
source = builder.get_branch()
585
target = self.make_repository('target', format='2a')
586
target.fetch(source.repository)
588
self.addCleanup(target.unlock)
589
details = target.texts._index.get_build_details(
590
[('file-id', '1',), ('file-id', '2',)])
591
file_1_details = details[('file-id', '1')]
592
file_2_details = details[('file-id', '2')]
593
# The index, and what to read off disk, should be the same for both
594
# versions of the file.
595
self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
597
def test_fetch_combines_groups(self):
598
builder = self.make_branch_builder('source', format='2a')
599
builder.start_series()
600
builder.build_snapshot('1', None, [
601
('add', ('', 'root-id', 'directory', '')),
602
('add', ('file', 'file-id', 'file', 'content\n'))])
603
builder.build_snapshot('2', ['1'], [
604
('modify', ('file-id', 'content-2\n'))])
605
builder.finish_series()
606
source = builder.get_branch()
607
target = self.make_repository('target', format='2a')
608
target.fetch(source.repository)
610
self.addCleanup(target.unlock)
611
details = target.texts._index.get_build_details(
612
[('file-id', '1',), ('file-id', '2',)])
613
file_1_details = details[('file-id', '1')]
614
file_2_details = details[('file-id', '2')]
615
# The index, and what to read off disk, should be the same for both
616
# versions of the file.
617
self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
619
def test_format_pack_compresses_True(self):
620
repo = self.make_repository('repo', format='2a')
621
self.assertTrue(repo._format.pack_compresses)
623
def test_inventories_use_chk_map_with_parent_base_dict(self):
624
tree = self.make_branch_and_memory_tree('repo', format="2a")
626
tree.add([''], ['TREE_ROOT'])
627
revid = tree.commit("foo")
630
self.addCleanup(tree.unlock)
631
inv = tree.branch.repository.get_inventory(revid)
632
self.assertNotEqual(None, inv.parent_id_basename_to_file_id)
633
inv.parent_id_basename_to_file_id._ensure_root()
634
inv.id_to_entry._ensure_root()
635
self.assertEqual(65536, inv.id_to_entry._root_node.maximum_size)
636
self.assertEqual(65536,
637
inv.parent_id_basename_to_file_id._root_node.maximum_size)
639
def test_autopack_unchanged_chk_nodes(self):
640
# at 20 unchanged commits, chk pages are packed that are split into
641
# two groups such that the new pack being made doesn't have all its
642
# pages in the source packs (though they are in the repository).
643
# Use a memory backed repository, we don't need to hit disk for this
644
tree = self.make_branch_and_memory_tree('tree', format='2a')
646
self.addCleanup(tree.unlock)
647
tree.add([''], ['TREE_ROOT'])
648
for pos in range(20):
649
tree.commit(str(pos))
651
def test_pack_with_hint(self):
652
tree = self.make_branch_and_memory_tree('tree', format='2a')
654
self.addCleanup(tree.unlock)
655
tree.add([''], ['TREE_ROOT'])
656
# 1 commit to leave untouched
658
to_keep = tree.branch.repository._pack_collection.names()
662
all = tree.branch.repository._pack_collection.names()
663
combine = list(set(all) - set(to_keep))
664
self.assertLength(3, all)
665
self.assertLength(2, combine)
666
tree.branch.repository.pack(hint=combine)
667
final = tree.branch.repository._pack_collection.names()
668
self.assertLength(2, final)
669
self.assertFalse(combine[0] in final)
670
self.assertFalse(combine[1] in final)
671
self.assertSubset(to_keep, final)
673
def test_stream_source_to_gc(self):
674
source = self.make_repository('source', format='2a')
675
target = self.make_repository('target', format='2a')
676
stream = source._get_source(target._format)
677
self.assertIsInstance(stream, groupcompress_repo.GroupCHKStreamSource)
679
def test_stream_source_to_non_gc(self):
680
source = self.make_repository('source', format='2a')
681
target = self.make_repository('target', format='rich-root-pack')
682
stream = source._get_source(target._format)
683
# We don't want the child GroupCHKStreamSource
684
self.assertIs(type(stream), vf_repository.StreamSource)
686
def test_get_stream_for_missing_keys_includes_all_chk_refs(self):
687
source_builder = self.make_branch_builder('source',
689
# We have to build a fairly large tree, so that we are sure the chk
690
# pages will have split into multiple pages.
691
entries = [('add', ('', 'a-root-id', 'directory', None))]
692
for i in 'abcdefghijklmnopqrstuvwxyz123456789':
693
for j in 'abcdefghijklmnopqrstuvwxyz123456789':
696
content = 'content for %s\n' % (fname,)
697
entries.append(('add', (fname, fid, 'file', content)))
698
source_builder.start_series()
699
source_builder.build_snapshot('rev-1', None, entries)
700
# Now change a few of them, so we get a few new pages for the second
702
source_builder.build_snapshot('rev-2', ['rev-1'], [
703
('modify', ('aa-id', 'new content for aa-id\n')),
704
('modify', ('cc-id', 'new content for cc-id\n')),
705
('modify', ('zz-id', 'new content for zz-id\n')),
707
source_builder.finish_series()
708
source_branch = source_builder.get_branch()
709
source_branch.lock_read()
710
self.addCleanup(source_branch.unlock)
711
target = self.make_repository('target', format='2a')
712
source = source_branch.repository._get_source(target._format)
713
self.assertIsInstance(source, groupcompress_repo.GroupCHKStreamSource)
715
# On a regular pass, getting the inventories and chk pages for rev-2
716
# would only get the newly created chk pages
717
search = vf_search.SearchResult(set(['rev-2']), set(['rev-1']), 1,
719
simple_chk_records = []
720
for vf_name, substream in source.get_stream(search):
721
if vf_name == 'chk_bytes':
722
for record in substream:
723
simple_chk_records.append(record.key)
727
# 3 pages, the root (InternalNode), + 2 pages which actually changed
728
self.assertEqual([('sha1:91481f539e802c76542ea5e4c83ad416bf219f73',),
729
('sha1:4ff91971043668583985aec83f4f0ab10a907d3f',),
730
('sha1:81e7324507c5ca132eedaf2d8414ee4bb2226187',),
731
('sha1:b101b7da280596c71a4540e9a1eeba8045985ee0',)],
733
# Now, when we do a similar call using 'get_stream_for_missing_keys'
734
# we should get a much larger set of pages.
735
missing = [('inventories', 'rev-2')]
736
full_chk_records = []
737
for vf_name, substream in source.get_stream_for_missing_keys(missing):
738
if vf_name == 'inventories':
739
for record in substream:
740
self.assertEqual(('rev-2',), record.key)
741
elif vf_name == 'chk_bytes':
742
for record in substream:
743
full_chk_records.append(record.key)
745
self.fail('Should not be getting a stream of %s' % (vf_name,))
746
# We have 257 records now. This is because we have 1 root page, and 256
747
# leaf pages in a complete listing.
748
self.assertEqual(257, len(full_chk_records))
749
self.assertSubset(simple_chk_records, full_chk_records)
751
def test_inconsistency_fatal(self):
752
repo = self.make_repository('repo', format='2a')
753
self.assertTrue(repo.revisions._index._inconsistency_fatal)
754
self.assertFalse(repo.texts._index._inconsistency_fatal)
755
self.assertFalse(repo.inventories._index._inconsistency_fatal)
756
self.assertFalse(repo.signatures._index._inconsistency_fatal)
757
self.assertFalse(repo.chk_bytes._index._inconsistency_fatal)
760
class TestKnitPackStreamSource(tests.TestCaseWithMemoryTransport):
762
def test_source_to_exact_pack_092(self):
763
source = self.make_repository('source', format='pack-0.92')
764
target = self.make_repository('target', format='pack-0.92')
765
stream_source = source._get_source(target._format)
766
self.assertIsInstance(stream_source, knitpack_repo.KnitPackStreamSource)
768
def test_source_to_exact_pack_rich_root_pack(self):
769
source = self.make_repository('source', format='rich-root-pack')
770
target = self.make_repository('target', format='rich-root-pack')
771
stream_source = source._get_source(target._format)
772
self.assertIsInstance(stream_source, knitpack_repo.KnitPackStreamSource)
774
def test_source_to_exact_pack_19(self):
775
source = self.make_repository('source', format='1.9')
776
target = self.make_repository('target', format='1.9')
777
stream_source = source._get_source(target._format)
778
self.assertIsInstance(stream_source, knitpack_repo.KnitPackStreamSource)
780
def test_source_to_exact_pack_19_rich_root(self):
781
source = self.make_repository('source', format='1.9-rich-root')
782
target = self.make_repository('target', format='1.9-rich-root')
783
stream_source = source._get_source(target._format)
784
self.assertIsInstance(stream_source, knitpack_repo.KnitPackStreamSource)
786
def test_source_to_remote_exact_pack_19(self):
787
trans = self.make_smart_server('target')
789
source = self.make_repository('source', format='1.9')
790
target = self.make_repository('target', format='1.9')
791
target = repository.Repository.open(trans.base)
792
stream_source = source._get_source(target._format)
793
self.assertIsInstance(stream_source, knitpack_repo.KnitPackStreamSource)
795
def test_stream_source_to_non_exact(self):
796
source = self.make_repository('source', format='pack-0.92')
797
target = self.make_repository('target', format='1.9')
798
stream = source._get_source(target._format)
799
self.assertIs(type(stream), vf_repository.StreamSource)
801
def test_stream_source_to_non_exact_rich_root(self):
802
source = self.make_repository('source', format='1.9')
803
target = self.make_repository('target', format='1.9-rich-root')
804
stream = source._get_source(target._format)
805
self.assertIs(type(stream), vf_repository.StreamSource)
807
def test_source_to_remote_non_exact_pack_19(self):
808
trans = self.make_smart_server('target')
810
source = self.make_repository('source', format='1.9')
811
target = self.make_repository('target', format='1.6')
812
target = repository.Repository.open(trans.base)
813
stream_source = source._get_source(target._format)
814
self.assertIs(type(stream_source), vf_repository.StreamSource)
816
def test_stream_source_to_knit(self):
817
source = self.make_repository('source', format='pack-0.92')
818
target = self.make_repository('target', format='dirstate')
819
stream = source._get_source(target._format)
820
self.assertIs(type(stream), vf_repository.StreamSource)
823
class TestDevelopment6FindParentIdsOfRevisions(TestCaseWithTransport):
824
"""Tests for _find_parent_ids_of_revisions."""
827
super(TestDevelopment6FindParentIdsOfRevisions, self).setUp()
828
self.builder = self.make_branch_builder('source')
829
self.builder.start_series()
830
self.builder.build_snapshot('initial', None,
831
[('add', ('', 'tree-root', 'directory', None))])
832
self.repo = self.builder.get_branch().repository
833
self.addCleanup(self.builder.finish_series)
835
def assertParentIds(self, expected_result, rev_set):
836
self.assertEqual(sorted(expected_result),
837
sorted(self.repo._find_parent_ids_of_revisions(rev_set)))
839
def test_simple(self):
840
self.builder.build_snapshot('revid1', None, [])
841
self.builder.build_snapshot('revid2', ['revid1'], [])
843
self.assertParentIds(['revid1'], rev_set)
845
def test_not_first_parent(self):
846
self.builder.build_snapshot('revid1', None, [])
847
self.builder.build_snapshot('revid2', ['revid1'], [])
848
self.builder.build_snapshot('revid3', ['revid2'], [])
849
rev_set = ['revid3', 'revid2']
850
self.assertParentIds(['revid1'], rev_set)
852
def test_not_null(self):
853
rev_set = ['initial']
854
self.assertParentIds([], rev_set)
856
def test_not_null_set(self):
857
self.builder.build_snapshot('revid1', None, [])
858
rev_set = [_mod_revision.NULL_REVISION]
859
self.assertParentIds([], rev_set)
861
def test_ghost(self):
862
self.builder.build_snapshot('revid1', None, [])
863
rev_set = ['ghost', 'revid1']
864
self.assertParentIds(['initial'], rev_set)
866
def test_ghost_parent(self):
867
self.builder.build_snapshot('revid1', None, [])
868
self.builder.build_snapshot('revid2', ['revid1', 'ghost'], [])
869
rev_set = ['revid2', 'revid1']
870
self.assertParentIds(['ghost', 'initial'], rev_set)
872
def test_righthand_parent(self):
873
self.builder.build_snapshot('revid1', None, [])
874
self.builder.build_snapshot('revid2a', ['revid1'], [])
875
self.builder.build_snapshot('revid2b', ['revid1'], [])
876
self.builder.build_snapshot('revid3', ['revid2a', 'revid2b'], [])
877
rev_set = ['revid3', 'revid2a']
878
self.assertParentIds(['revid1', 'revid2b'], rev_set)
674
881
class TestWithBrokenRepo(TestCaseWithTransport):
752
961
broken_repo = self.make_broken_repository()
753
962
empty_repo = self.make_repository('empty-repo')
754
stream = broken_repo.get_data_stream(['rev1a', 'rev2', 'rev3'])
755
empty_repo.lock_write()
964
empty_repo.fetch(broken_repo)
965
except (errors.RevisionNotPresent, errors.BzrCheckError):
966
# Test successful: compression parent not being copied leads to
969
empty_repo.lock_read()
756
970
self.addCleanup(empty_repo.unlock)
757
empty_repo.start_write_group()
760
errors.KnitCorrupt, empty_repo.insert_data_stream, stream)
762
empty_repo.abort_write_group()
765
class TestKnitPackNoSubtrees(TestCaseWithTransport):
767
def get_format(self):
768
return bzrdir.format_registry.make_bzrdir('pack-0.92')
770
def test_disk_layout(self):
771
format = self.get_format()
772
repo = self.make_repository('.', format=format)
773
# in case of side effects of locking.
776
t = repo.bzrdir.get_repository_transport(None)
778
# XXX: no locks left when unlocked at the moment
779
# self.assertEqualDiff('', t.get('lock').read())
780
self.check_databases(t)
782
def check_format(self, t):
783
self.assertEqualDiff(
784
"Bazaar pack repository format 1 (needs bzr 0.92)\n",
785
t.get('format').read())
787
def assertHasKndx(self, t, knit_name):
788
"""Assert that knit_name exists on t."""
789
self.assertEqualDiff('# bzr knit index 8\n',
790
t.get(knit_name + '.kndx').read())
792
def assertHasNoKndx(self, t, knit_name):
793
"""Assert that knit_name has no index on t."""
794
self.assertFalse(t.has(knit_name + '.kndx'))
796
def assertHasNoKnit(self, t, knit_name):
797
"""Assert that knit_name exists on t."""
799
self.assertFalse(t.has(knit_name + '.knit'))
801
def check_databases(self, t):
802
"""check knit content for a repository."""
803
# check conversion worked
804
self.assertHasNoKndx(t, 'inventory')
805
self.assertHasNoKnit(t, 'inventory')
806
self.assertHasNoKndx(t, 'revisions')
807
self.assertHasNoKnit(t, 'revisions')
808
self.assertHasNoKndx(t, 'signatures')
809
self.assertHasNoKnit(t, 'signatures')
810
self.assertFalse(t.has('knits'))
811
# revision-indexes file-container directory
813
list(GraphIndex(t, 'pack-names', None).iter_all_entries()))
814
self.assertTrue(S_ISDIR(t.stat('packs').st_mode))
815
self.assertTrue(S_ISDIR(t.stat('upload').st_mode))
816
self.assertTrue(S_ISDIR(t.stat('indices').st_mode))
817
self.assertTrue(S_ISDIR(t.stat('obsolete_packs').st_mode))
819
def test_shared_disk_layout(self):
820
format = self.get_format()
821
repo = self.make_repository('.', shared=True, format=format)
823
t = repo.bzrdir.get_repository_transport(None)
825
# XXX: no locks left when unlocked at the moment
826
# self.assertEqualDiff('', t.get('lock').read())
827
# We should have a 'shared-storage' marker file.
828
self.assertEqualDiff('', t.get('shared-storage').read())
829
self.check_databases(t)
831
def test_shared_no_tree_disk_layout(self):
832
format = self.get_format()
833
repo = self.make_repository('.', shared=True, format=format)
834
repo.set_make_working_trees(False)
836
t = repo.bzrdir.get_repository_transport(None)
838
# XXX: no locks left when unlocked at the moment
839
# self.assertEqualDiff('', t.get('lock').read())
840
# We should have a 'shared-storage' marker file.
841
self.assertEqualDiff('', t.get('shared-storage').read())
842
# We should have a marker for the no-working-trees flag.
843
self.assertEqualDiff('', t.get('no-working-trees').read())
844
# The marker should go when we toggle the setting.
845
repo.set_make_working_trees(True)
846
self.assertFalse(t.has('no-working-trees'))
847
self.check_databases(t)
849
def test_adding_revision_creates_pack_indices(self):
850
format = self.get_format()
851
tree = self.make_branch_and_tree('.', format=format)
852
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
854
list(GraphIndex(trans, 'pack-names', None).iter_all_entries()))
855
tree.commit('foobarbaz')
856
index = GraphIndex(trans, 'pack-names', None)
857
index_nodes = list(index.iter_all_entries())
858
self.assertEqual(1, len(index_nodes))
859
node = index_nodes[0]
861
# the pack sizes should be listed in the index
863
sizes = [int(digits) for digits in pack_value.split(' ')]
864
for size, suffix in zip(sizes, ['.rix', '.iix', '.tix', '.six']):
865
stat = trans.stat('indices/%s%s' % (name, suffix))
866
self.assertEqual(size, stat.st_size)
868
def test_pulling_nothing_leads_to_no_new_names(self):
869
format = self.get_format()
870
tree1 = self.make_branch_and_tree('1', format=format)
871
tree2 = self.make_branch_and_tree('2', format=format)
872
tree1.branch.repository.fetch(tree2.branch.repository)
873
trans = tree1.branch.repository.bzrdir.get_repository_transport(None)
875
list(GraphIndex(trans, 'pack-names', None).iter_all_entries()))
877
def test_commit_across_pack_shape_boundary_autopacks(self):
878
format = self.get_format()
879
tree = self.make_branch_and_tree('.', format=format)
880
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
881
# This test could be a little cheaper by replacing the packs
882
# attribute on the repository to allow a different pack distribution
883
# and max packs policy - so we are checking the policy is honoured
884
# in the test. But for now 11 commits is not a big deal in a single
887
tree.commit('commit %s' % x)
888
# there should be 9 packs:
889
index = GraphIndex(trans, 'pack-names', None)
890
self.assertEqual(9, len(list(index.iter_all_entries())))
891
# insert some files in obsolete_packs which should be removed by pack.
892
trans.put_bytes('obsolete_packs/foo', '123')
893
trans.put_bytes('obsolete_packs/bar', '321')
894
# committing one more should coalesce to 1 of 10.
895
tree.commit('commit triggering pack')
896
index = GraphIndex(trans, 'pack-names', None)
897
self.assertEqual(1, len(list(index.iter_all_entries())))
898
# packing should not damage data
899
tree = tree.bzrdir.open_workingtree()
900
check_result = tree.branch.repository.check(
901
[tree.branch.last_revision()])
902
# We should have 50 (10x5) files in the obsolete_packs directory.
903
obsolete_files = list(trans.list_dir('obsolete_packs'))
904
self.assertFalse('foo' in obsolete_files)
905
self.assertFalse('bar' in obsolete_files)
906
self.assertEqual(50, len(obsolete_files))
907
# XXX: Todo check packs obsoleted correctly - old packs and indices
908
# in the obsolete_packs directory.
909
large_pack_name = list(index.iter_all_entries())[0][1][0]
910
# finally, committing again should not touch the large pack.
911
tree.commit('commit not triggering pack')
912
index = GraphIndex(trans, 'pack-names', None)
913
self.assertEqual(2, len(list(index.iter_all_entries())))
914
pack_names = [node[1][0] for node in index.iter_all_entries()]
915
self.assertTrue(large_pack_name in pack_names)
917
def test_pack_after_two_commits_packs_everything(self):
918
format = self.get_format()
919
tree = self.make_branch_and_tree('.', format=format)
920
trans = tree.branch.repository.bzrdir.get_repository_transport(None)
922
tree.commit('more work')
923
tree.branch.repository.pack()
924
# there should be 1 pack:
925
index = GraphIndex(trans, 'pack-names', None)
926
self.assertEqual(1, len(list(index.iter_all_entries())))
927
self.assertEqual(2, len(tree.branch.repository.all_revision_ids()))
929
def test_pack_repositories_support_multiple_write_locks(self):
930
format = self.get_format()
931
self.make_repository('.', shared=True, format=format)
932
r1 = repository.Repository.open('.')
933
r2 = repository.Repository.open('.')
935
self.addCleanup(r1.unlock)
939
def _add_text(self, repo, fileid):
940
"""Add a text to the repository within a write group."""
941
vf =repo.weave_store.get_weave(fileid, repo.get_transaction())
942
vf.add_lines('samplerev+' + fileid, [], [])
944
def test_concurrent_writers_merge_new_packs(self):
945
format = self.get_format()
946
self.make_repository('.', shared=True, format=format)
947
r1 = repository.Repository.open('.')
948
r2 = repository.Repository.open('.')
951
# access enough data to load the names list
952
list(r1.all_revision_ids())
955
# access enough data to load the names list
956
list(r2.all_revision_ids())
957
r1.start_write_group()
959
r2.start_write_group()
961
self._add_text(r1, 'fileidr1')
962
self._add_text(r2, 'fileidr2')
964
r2.abort_write_group()
967
r1.abort_write_group()
969
# both r1 and r2 have open write groups with data in them
970
# created while the other's write group was open.
971
# Commit both which requires a merge to the pack-names.
973
r1.commit_write_group()
975
r1.abort_write_group()
976
r2.abort_write_group()
978
r2.commit_write_group()
979
# tell r1 to reload from disk
980
r1._pack_collection.reset()
981
# Now both repositories should know about both names
982
r1._pack_collection.ensure_loaded()
983
r2._pack_collection.ensure_loaded()
984
self.assertEqual(r1._pack_collection.names(), r2._pack_collection.names())
985
self.assertEqual(2, len(r1._pack_collection.names()))
991
def test_concurrent_writer_second_preserves_dropping_a_pack(self):
992
format = self.get_format()
993
self.make_repository('.', shared=True, format=format)
994
r1 = repository.Repository.open('.')
995
r2 = repository.Repository.open('.')
999
r1.start_write_group()
1001
self._add_text(r1, 'fileidr1')
1003
r1.abort_write_group()
1006
r1.commit_write_group()
1007
r1._pack_collection.ensure_loaded()
1008
name_to_drop = r1._pack_collection.all_packs()[0].name
1013
# access enough data to load the names list
1014
list(r1.all_revision_ids())
1017
# access enough data to load the names list
1018
list(r2.all_revision_ids())
1019
r1._pack_collection.ensure_loaded()
1021
r2.start_write_group()
1023
# in r1, drop the pack
1024
r1._pack_collection._remove_pack_from_memory(
1025
r1._pack_collection.get_pack_by_name(name_to_drop))
1027
self._add_text(r2, 'fileidr2')
1029
r2.abort_write_group()
1032
r1._pack_collection.reset()
1034
# r1 has a changed names list, and r2 an open write groups with
1036
# save r1, and then commit the r2 write group, which requires a
1037
# merge to the pack-names, which should not reinstate
1040
r1._pack_collection._save_pack_names()
1041
r1._pack_collection.reset()
1043
r2.abort_write_group()
1046
r2.commit_write_group()
1048
r2.abort_write_group()
1050
# Now both repositories should now about just one name.
1051
r1._pack_collection.ensure_loaded()
1052
r2._pack_collection.ensure_loaded()
1053
self.assertEqual(r1._pack_collection.names(), r2._pack_collection.names())
1054
self.assertEqual(1, len(r1._pack_collection.names()))
1055
self.assertFalse(name_to_drop in r1._pack_collection.names())
1061
def test_lock_write_does_not_physically_lock(self):
1062
repo = self.make_repository('.', format=self.get_format())
1064
self.addCleanup(repo.unlock)
1065
self.assertFalse(repo.get_physical_lock_status())
1067
def prepare_for_break_lock(self):
1068
# Setup the global ui factory state so that a break-lock method call
1069
# will find usable input in the input stream.
1070
old_factory = bzrlib.ui.ui_factory
1071
def restoreFactory():
1072
bzrlib.ui.ui_factory = old_factory
1073
self.addCleanup(restoreFactory)
1074
bzrlib.ui.ui_factory = bzrlib.ui.SilentUIFactory()
1075
bzrlib.ui.ui_factory.stdin = StringIO("y\n")
1077
def test_break_lock_breaks_physical_lock(self):
1078
repo = self.make_repository('.', format=self.get_format())
1079
repo._pack_collection.lock_names()
1080
repo2 = repository.Repository.open('.')
1081
self.assertTrue(repo.get_physical_lock_status())
1082
self.prepare_for_break_lock()
1084
self.assertFalse(repo.get_physical_lock_status())
1086
def test_broken_physical_locks_error_on__unlock_names_lock(self):
1087
repo = self.make_repository('.', format=self.get_format())
1088
repo._pack_collection.lock_names()
1089
self.assertTrue(repo.get_physical_lock_status())
1090
repo2 = repository.Repository.open('.')
1091
self.prepare_for_break_lock()
1093
self.assertRaises(errors.LockBroken, repo._pack_collection._unlock_names)
1095
def test_fetch_without_find_ghosts_ignores_ghosts(self):
1096
# we want two repositories at this point:
1097
# one with a revision that is a ghost in the other
1099
# 'ghost' is present in has_ghost, 'ghost' is absent in 'missing_ghost'.
1100
# 'references' is present in both repositories, and 'tip' is present
1101
# just in has_ghost.
1102
# has_ghost missing_ghost
1103
#------------------------------
1105
# 'references' 'references'
1107
# In this test we fetch 'tip' which should not fetch 'ghost'
1108
has_ghost = self.make_repository('has_ghost', format=self.get_format())
1109
missing_ghost = self.make_repository('missing_ghost',
1110
format=self.get_format())
1112
def add_commit(repo, revision_id, parent_ids):
1114
repo.start_write_group()
1115
inv = inventory.Inventory(revision_id=revision_id)
1116
inv.root.revision = revision_id
1117
root_id = inv.root.file_id
1118
sha1 = repo.add_inventory(revision_id, inv, [])
1119
vf = repo.weave_store.get_weave_or_empty(root_id,
1120
repo.get_transaction())
1121
vf.add_lines(revision_id, [], [])
1122
rev = bzrlib.revision.Revision(timestamp=0,
1124
committer="Foo Bar <foo@example.com>",
1126
inventory_sha1=sha1,
1127
revision_id=revision_id)
1128
rev.parent_ids = parent_ids
1129
repo.add_revision(revision_id, rev)
1130
repo.commit_write_group()
1132
add_commit(has_ghost, 'ghost', [])
1133
add_commit(has_ghost, 'references', ['ghost'])
1134
add_commit(missing_ghost, 'references', ['ghost'])
1135
add_commit(has_ghost, 'tip', ['references'])
1136
missing_ghost.fetch(has_ghost, 'tip')
1137
# missing ghost now has tip and not ghost.
1138
rev = missing_ghost.get_revision('tip')
1139
inv = missing_ghost.get_inventory('tip')
1140
self.assertRaises(errors.NoSuchRevision,
1141
missing_ghost.get_revision, 'ghost')
1142
self.assertRaises(errors.RevisionNotPresent,
1143
missing_ghost.get_inventory, 'ghost')
1146
class TestKnitPackSubtrees(TestKnitPackNoSubtrees):
1148
def get_format(self):
1149
return bzrdir.format_registry.make_bzrdir(
1150
'pack-0.92-subtree')
1152
def check_format(self, t):
1153
self.assertEqualDiff(
1154
"Bazaar pack repository format 1 with subtree support (needs bzr 0.92)\n",
1155
t.get('format').read())
971
text = empty_repo.texts.get_record_stream(
972
[('file2-id', 'rev3')], 'topological', True).next()
973
self.assertEqual('line\n', text.get_bytes_as('fulltext'))
1158
976
class TestRepositoryPackCollection(TestCaseWithTransport):
1160
978
def get_format(self):
1161
return bzrdir.format_registry.make_bzrdir('pack-0.92')
979
return controldir.format_registry.make_bzrdir('pack-0.92')
982
format = self.get_format()
983
repo = self.make_repository('.', format=format)
984
return repo._pack_collection
986
def make_packs_and_alt_repo(self, write_lock=False):
987
"""Create a pack repo with 3 packs, and access it via a second repo."""
988
tree = self.make_branch_and_tree('.', format=self.get_format())
990
self.addCleanup(tree.unlock)
991
rev1 = tree.commit('one')
992
rev2 = tree.commit('two')
993
rev3 = tree.commit('three')
994
r = repository.Repository.open('.')
999
self.addCleanup(r.unlock)
1000
packs = r._pack_collection
1001
packs.ensure_loaded()
1002
return tree, r, packs, [rev1, rev2, rev3]
1004
def test__clear_obsolete_packs(self):
1005
packs = self.get_packs()
1006
obsolete_pack_trans = packs.transport.clone('obsolete_packs')
1007
obsolete_pack_trans.put_bytes('a-pack.pack', 'content\n')
1008
obsolete_pack_trans.put_bytes('a-pack.rix', 'content\n')
1009
obsolete_pack_trans.put_bytes('a-pack.iix', 'content\n')
1010
obsolete_pack_trans.put_bytes('another-pack.pack', 'foo\n')
1011
obsolete_pack_trans.put_bytes('not-a-pack.rix', 'foo\n')
1012
res = packs._clear_obsolete_packs()
1013
self.assertEqual(['a-pack', 'another-pack'], sorted(res))
1014
self.assertEqual([], obsolete_pack_trans.list_dir('.'))
1016
def test__clear_obsolete_packs_preserve(self):
1017
packs = self.get_packs()
1018
obsolete_pack_trans = packs.transport.clone('obsolete_packs')
1019
obsolete_pack_trans.put_bytes('a-pack.pack', 'content\n')
1020
obsolete_pack_trans.put_bytes('a-pack.rix', 'content\n')
1021
obsolete_pack_trans.put_bytes('a-pack.iix', 'content\n')
1022
obsolete_pack_trans.put_bytes('another-pack.pack', 'foo\n')
1023
obsolete_pack_trans.put_bytes('not-a-pack.rix', 'foo\n')
1024
res = packs._clear_obsolete_packs(preserve=set(['a-pack']))
1025
self.assertEqual(['a-pack', 'another-pack'], sorted(res))
1026
self.assertEqual(['a-pack.iix', 'a-pack.pack', 'a-pack.rix'],
1027
sorted(obsolete_pack_trans.list_dir('.')))
1163
1029
def test__max_pack_count(self):
1164
1030
"""The maximum pack count is a function of the number of revisions."""
1165
format = self.get_format()
1166
repo = self.make_repository('.', format=format)
1167
packs = repo._pack_collection
1168
1031
# no revisions - one pack, so that we can have a revision free repo
1169
1032
# without it blowing up
1033
packs = self.get_packs()
1170
1034
self.assertEqual(1, packs._max_pack_count(0))
1171
1035
# after that the sum of the digits, - check the first 1-9
1172
1036
self.assertEqual(1, packs._max_pack_count(1))
1307
1223
tree.lock_read()
1308
1224
self.addCleanup(tree.unlock)
1309
1225
packs = tree.branch.repository._pack_collection
1310
1227
packs.ensure_loaded()
1311
1228
name = packs.names()[0]
1312
1229
pack_1 = packs.get_pack_by_name(name)
1313
1230
# the pack should be correctly initialised
1314
rev_index = GraphIndex(packs._index_transport, name + '.rix',
1315
packs._names[name][0])
1316
inv_index = GraphIndex(packs._index_transport, name + '.iix',
1317
packs._names[name][1])
1318
txt_index = GraphIndex(packs._index_transport, name + '.tix',
1319
packs._names[name][2])
1320
sig_index = GraphIndex(packs._index_transport, name + '.six',
1321
packs._names[name][3])
1231
sizes = packs._names[name]
1232
rev_index = GraphIndex(packs._index_transport, name + '.rix', sizes[0])
1233
inv_index = GraphIndex(packs._index_transport, name + '.iix', sizes[1])
1234
txt_index = GraphIndex(packs._index_transport, name + '.tix', sizes[2])
1235
sig_index = GraphIndex(packs._index_transport, name + '.six', sizes[3])
1322
1236
self.assertEqual(pack_repo.ExistingPack(packs._pack_transport,
1323
1237
name, rev_index, inv_index, txt_index, sig_index), pack_1)
1324
1238
# and the same instance should be returned on successive calls.
1325
1239
self.assertTrue(pack_1 is packs.get_pack_by_name(name))
1241
def test_reload_pack_names_new_entry(self):
1242
tree, r, packs, revs = self.make_packs_and_alt_repo()
1243
names = packs.names()
1244
# Add a new pack file into the repository
1245
rev4 = tree.commit('four')
1246
new_names = tree.branch.repository._pack_collection.names()
1247
new_name = set(new_names).difference(names)
1248
self.assertEqual(1, len(new_name))
1249
new_name = new_name.pop()
1250
# The old collection hasn't noticed yet
1251
self.assertEqual(names, packs.names())
1252
self.assertTrue(packs.reload_pack_names())
1253
self.assertEqual(new_names, packs.names())
1254
# And the repository can access the new revision
1255
self.assertEqual({rev4:(revs[-1],)}, r.get_parent_map([rev4]))
1256
self.assertFalse(packs.reload_pack_names())
1258
def test_reload_pack_names_added_and_removed(self):
1259
tree, r, packs, revs = self.make_packs_and_alt_repo()
1260
names = packs.names()
1261
# Now repack the whole thing
1262
tree.branch.repository.pack()
1263
new_names = tree.branch.repository._pack_collection.names()
1264
# The other collection hasn't noticed yet
1265
self.assertEqual(names, packs.names())
1266
self.assertTrue(packs.reload_pack_names())
1267
self.assertEqual(new_names, packs.names())
1268
self.assertEqual({revs[-1]:(revs[-2],)}, r.get_parent_map([revs[-1]]))
1269
self.assertFalse(packs.reload_pack_names())
1271
def test_reload_pack_names_preserves_pending(self):
1272
# TODO: Update this to also test for pending-deleted names
1273
tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1274
# We will add one pack (via start_write_group + insert_record_stream),
1275
# and remove another pack (via _remove_pack_from_memory)
1276
orig_names = packs.names()
1277
orig_at_load = packs._packs_at_load
1278
to_remove_name = iter(orig_names).next()
1279
r.start_write_group()
1280
self.addCleanup(r.abort_write_group)
1281
r.texts.insert_record_stream([versionedfile.FulltextContentFactory(
1282
('text', 'rev'), (), None, 'content\n')])
1283
new_pack = packs._new_pack
1284
self.assertTrue(new_pack.data_inserted())
1286
packs.allocate(new_pack)
1287
packs._new_pack = None
1288
removed_pack = packs.get_pack_by_name(to_remove_name)
1289
packs._remove_pack_from_memory(removed_pack)
1290
names = packs.names()
1291
all_nodes, deleted_nodes, new_nodes, _ = packs._diff_pack_names()
1292
new_names = set([x[0][0] for x in new_nodes])
1293
self.assertEqual(names, sorted([x[0][0] for x in all_nodes]))
1294
self.assertEqual(set(names) - set(orig_names), new_names)
1295
self.assertEqual(set([new_pack.name]), new_names)
1296
self.assertEqual([to_remove_name],
1297
sorted([x[0][0] for x in deleted_nodes]))
1298
packs.reload_pack_names()
1299
reloaded_names = packs.names()
1300
self.assertEqual(orig_at_load, packs._packs_at_load)
1301
self.assertEqual(names, reloaded_names)
1302
all_nodes, deleted_nodes, new_nodes, _ = packs._diff_pack_names()
1303
new_names = set([x[0][0] for x in new_nodes])
1304
self.assertEqual(names, sorted([x[0][0] for x in all_nodes]))
1305
self.assertEqual(set(names) - set(orig_names), new_names)
1306
self.assertEqual(set([new_pack.name]), new_names)
1307
self.assertEqual([to_remove_name],
1308
sorted([x[0][0] for x in deleted_nodes]))
1310
def test_autopack_obsoletes_new_pack(self):
1311
tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1312
packs._max_pack_count = lambda x: 1
1313
packs.pack_distribution = lambda x: [10]
1314
r.start_write_group()
1315
r.revisions.insert_record_stream([versionedfile.FulltextContentFactory(
1316
('bogus-rev',), (), None, 'bogus-content\n')])
1317
# This should trigger an autopack, which will combine everything into a
1319
new_names = r.commit_write_group()
1320
names = packs.names()
1321
self.assertEqual(1, len(names))
1322
self.assertEqual([names[0] + '.pack'],
1323
packs._pack_transport.list_dir('.'))
1325
def test_autopack_reloads_and_stops(self):
1326
tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1327
# After we have determined what needs to be autopacked, trigger a
1328
# full-pack via the other repo which will cause us to re-evaluate and
1329
# decide we don't need to do anything
1330
orig_execute = packs._execute_pack_operations
1331
def _munged_execute_pack_ops(*args, **kwargs):
1332
tree.branch.repository.pack()
1333
return orig_execute(*args, **kwargs)
1334
packs._execute_pack_operations = _munged_execute_pack_ops
1335
packs._max_pack_count = lambda x: 1
1336
packs.pack_distribution = lambda x: [10]
1337
self.assertFalse(packs.autopack())
1338
self.assertEqual(1, len(packs.names()))
1339
self.assertEqual(tree.branch.repository._pack_collection.names(),
1342
def test__save_pack_names(self):
1343
tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1344
names = packs.names()
1345
pack = packs.get_pack_by_name(names[0])
1346
packs._remove_pack_from_memory(pack)
1347
packs._save_pack_names(obsolete_packs=[pack])
1348
cur_packs = packs._pack_transport.list_dir('.')
1349
self.assertEqual([n + '.pack' for n in names[1:]], sorted(cur_packs))
1350
# obsolete_packs will also have stuff like .rix and .iix present.
1351
obsolete_packs = packs.transport.list_dir('obsolete_packs')
1352
obsolete_names = set([osutils.splitext(n)[0] for n in obsolete_packs])
1353
self.assertEqual([pack.name], sorted(obsolete_names))
1355
def test__save_pack_names_already_obsoleted(self):
1356
tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1357
names = packs.names()
1358
pack = packs.get_pack_by_name(names[0])
1359
packs._remove_pack_from_memory(pack)
1360
# We are going to simulate a concurrent autopack by manually obsoleting
1361
# the pack directly.
1362
packs._obsolete_packs([pack])
1363
packs._save_pack_names(clear_obsolete_packs=True,
1364
obsolete_packs=[pack])
1365
cur_packs = packs._pack_transport.list_dir('.')
1366
self.assertEqual([n + '.pack' for n in names[1:]], sorted(cur_packs))
1367
# Note that while we set clear_obsolete_packs=True, it should not
1368
# delete a pack file that we have also scheduled for obsoletion.
1369
obsolete_packs = packs.transport.list_dir('obsolete_packs')
1370
obsolete_names = set([osutils.splitext(n)[0] for n in obsolete_packs])
1371
self.assertEqual([pack.name], sorted(obsolete_names))
1373
def test_pack_no_obsolete_packs_directory(self):
1374
"""Bug #314314, don't fail if obsolete_packs directory does
1376
tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1377
r.control_transport.rmdir('obsolete_packs')
1378
packs._clear_obsolete_packs()
1328
1381
class TestPack(TestCaseWithTransport):
1329
1382
"""Tests for the Pack object."""
1400
1462
class TestPacker(TestCaseWithTransport):
1401
1463
"""Tests for the packs repository Packer class."""
1403
# To date, this class has been factored out and nothing new added to it;
1404
# thus there are not yet any tests.
1465
def test_pack_optimizes_pack_order(self):
1466
builder = self.make_branch_builder('.', format="1.9")
1467
builder.start_series()
1468
builder.build_snapshot('A', None, [
1469
('add', ('', 'root-id', 'directory', None)),
1470
('add', ('f', 'f-id', 'file', 'content\n'))])
1471
builder.build_snapshot('B', ['A'],
1472
[('modify', ('f-id', 'new-content\n'))])
1473
builder.build_snapshot('C', ['B'],
1474
[('modify', ('f-id', 'third-content\n'))])
1475
builder.build_snapshot('D', ['C'],
1476
[('modify', ('f-id', 'fourth-content\n'))])
1477
b = builder.get_branch()
1479
builder.finish_series()
1480
self.addCleanup(b.unlock)
1481
# At this point, we should have 4 pack files available
1482
# Because of how they were built, they correspond to
1483
# ['D', 'C', 'B', 'A']
1484
packs = b.repository._pack_collection.packs
1485
packer = knitpack_repo.KnitPacker(b.repository._pack_collection,
1487
revision_ids=['B', 'C'])
1488
# Now, when we are copying the B & C revisions, their pack files should
1489
# be moved to the front of the stack
1490
# The new ordering moves B & C to the front of the .packs attribute,
1491
# and leaves the others in the original order.
1492
new_packs = [packs[1], packs[2], packs[0], packs[3]]
1493
new_pack = packer.pack()
1494
self.assertEqual(new_packs, packer.packs)
1497
class TestOptimisingPacker(TestCaseWithTransport):
1498
"""Tests for the OptimisingPacker class."""
1500
def get_pack_collection(self):
1501
repo = self.make_repository('.')
1502
return repo._pack_collection
1504
def test_open_pack_will_optimise(self):
1505
packer = knitpack_repo.OptimisingKnitPacker(self.get_pack_collection(),
1507
new_pack = packer.open_pack()
1508
self.addCleanup(new_pack.abort) # ensure cleanup
1509
self.assertIsInstance(new_pack, pack_repo.NewPack)
1510
self.assertTrue(new_pack.revision_index._optimize_for_size)
1511
self.assertTrue(new_pack.inventory_index._optimize_for_size)
1512
self.assertTrue(new_pack.text_index._optimize_for_size)
1513
self.assertTrue(new_pack.signature_index._optimize_for_size)
1516
class TestGCCHKPacker(TestCaseWithTransport):
1518
def make_abc_branch(self):
1519
builder = self.make_branch_builder('source')
1520
builder.start_series()
1521
builder.build_snapshot('A', None, [
1522
('add', ('', 'root-id', 'directory', None)),
1523
('add', ('file', 'file-id', 'file', 'content\n')),
1525
builder.build_snapshot('B', ['A'], [
1526
('add', ('dir', 'dir-id', 'directory', None))])
1527
builder.build_snapshot('C', ['B'], [
1528
('modify', ('file-id', 'new content\n'))])
1529
builder.finish_series()
1530
return builder.get_branch()
1532
def make_branch_with_disjoint_inventory_and_revision(self):
1533
"""a repo with separate packs for a revisions Revision and Inventory.
1535
There will be one pack file that holds the Revision content, and one
1536
for the Inventory content.
1538
:return: (repository,
1539
pack_name_with_rev_A_Revision,
1540
pack_name_with_rev_A_Inventory,
1541
pack_name_with_rev_C_content)
1543
b_source = self.make_abc_branch()
1544
b_base = b_source.bzrdir.sprout('base', revision_id='A').open_branch()
1545
b_stacked = b_base.bzrdir.sprout('stacked', stacked=True).open_branch()
1546
b_stacked.lock_write()
1547
self.addCleanup(b_stacked.unlock)
1548
b_stacked.fetch(b_source, 'B')
1549
# Now re-open the stacked repo directly (no fallbacks) so that we can
1550
# fill in the A rev.
1551
repo_not_stacked = b_stacked.bzrdir.open_repository()
1552
repo_not_stacked.lock_write()
1553
self.addCleanup(repo_not_stacked.unlock)
1554
# Now we should have a pack file with A's inventory, but not its
1556
self.assertEqual([('A',), ('B',)],
1557
sorted(repo_not_stacked.inventories.keys()))
1558
self.assertEqual([('B',)],
1559
sorted(repo_not_stacked.revisions.keys()))
1560
stacked_pack_names = repo_not_stacked._pack_collection.names()
1561
# We have a couple names here, figure out which has A's inventory
1562
for name in stacked_pack_names:
1563
pack = repo_not_stacked._pack_collection.get_pack_by_name(name)
1564
keys = [n[1] for n in pack.inventory_index.iter_all_entries()]
1566
inv_a_pack_name = name
1569
self.fail('Could not find pack containing A\'s inventory')
1570
repo_not_stacked.fetch(b_source.repository, 'A')
1571
self.assertEqual([('A',), ('B',)],
1572
sorted(repo_not_stacked.revisions.keys()))
1573
new_pack_names = set(repo_not_stacked._pack_collection.names())
1574
rev_a_pack_names = new_pack_names.difference(stacked_pack_names)
1575
self.assertEqual(1, len(rev_a_pack_names))
1576
rev_a_pack_name = list(rev_a_pack_names)[0]
1577
# Now fetch 'C', so we have a couple pack files to join
1578
repo_not_stacked.fetch(b_source.repository, 'C')
1579
rev_c_pack_names = set(repo_not_stacked._pack_collection.names())
1580
rev_c_pack_names = rev_c_pack_names.difference(new_pack_names)
1581
self.assertEqual(1, len(rev_c_pack_names))
1582
rev_c_pack_name = list(rev_c_pack_names)[0]
1583
return (repo_not_stacked, rev_a_pack_name, inv_a_pack_name,
1586
def test_pack_with_distant_inventories(self):
1587
# See https://bugs.launchpad.net/bzr/+bug/437003
1588
# When repacking, it is possible to have an inventory in a different
1589
# pack file than the associated revision. An autopack can then come
1590
# along, and miss that inventory, and complain.
1591
(repo, rev_a_pack_name, inv_a_pack_name, rev_c_pack_name
1592
) = self.make_branch_with_disjoint_inventory_and_revision()
1593
a_pack = repo._pack_collection.get_pack_by_name(rev_a_pack_name)
1594
c_pack = repo._pack_collection.get_pack_by_name(rev_c_pack_name)
1595
packer = groupcompress_repo.GCCHKPacker(repo._pack_collection,
1596
[a_pack, c_pack], '.test-pack')
1597
# This would raise ValueError in bug #437003, but should not raise an
1601
def test_pack_with_missing_inventory(self):
1602
# Similar to test_pack_with_missing_inventory, but this time, we force
1603
# the A inventory to actually be gone from the repository.
1604
(repo, rev_a_pack_name, inv_a_pack_name, rev_c_pack_name
1605
) = self.make_branch_with_disjoint_inventory_and_revision()
1606
inv_a_pack = repo._pack_collection.get_pack_by_name(inv_a_pack_name)
1607
repo._pack_collection._remove_pack_from_memory(inv_a_pack)
1608
packer = groupcompress_repo.GCCHKPacker(repo._pack_collection,
1609
repo._pack_collection.all_packs(), '.test-pack')
1610
e = self.assertRaises(ValueError, packer.pack)
1611
packer.new_pack.abort()
1612
self.assertContainsRe(str(e),
1613
r"We are missing inventories for revisions: .*'A'")
1616
class TestCrossFormatPacks(TestCaseWithTransport):
1618
def log_pack(self, hint=None):
1619
self.calls.append(('pack', hint))
1620
self.orig_pack(hint=hint)
1621
if self.expect_hint:
1622
self.assertTrue(hint)
1624
def run_stream(self, src_fmt, target_fmt, expect_pack_called):
1625
self.expect_hint = expect_pack_called
1627
source_tree = self.make_branch_and_tree('src', format=src_fmt)
1628
source_tree.lock_write()
1629
self.addCleanup(source_tree.unlock)
1630
tip = source_tree.commit('foo')
1631
target = self.make_repository('target', format=target_fmt)
1633
self.addCleanup(target.unlock)
1634
source = source_tree.branch.repository._get_source(target._format)
1635
self.orig_pack = target.pack
1636
self.overrideAttr(target, "pack", self.log_pack)
1637
search = target.search_missing_revision_ids(
1638
source_tree.branch.repository, revision_ids=[tip])
1639
stream = source.get_stream(search)
1640
from_format = source_tree.branch.repository._format
1641
sink = target._get_sink()
1642
sink.insert_stream(stream, from_format, [])
1643
if expect_pack_called:
1644
self.assertLength(1, self.calls)
1646
self.assertLength(0, self.calls)
1648
def run_fetch(self, src_fmt, target_fmt, expect_pack_called):
1649
self.expect_hint = expect_pack_called
1651
source_tree = self.make_branch_and_tree('src', format=src_fmt)
1652
source_tree.lock_write()
1653
self.addCleanup(source_tree.unlock)
1654
tip = source_tree.commit('foo')
1655
target = self.make_repository('target', format=target_fmt)
1657
self.addCleanup(target.unlock)
1658
source = source_tree.branch.repository
1659
self.orig_pack = target.pack
1660
self.overrideAttr(target, "pack", self.log_pack)
1661
target.fetch(source)
1662
if expect_pack_called:
1663
self.assertLength(1, self.calls)
1665
self.assertLength(0, self.calls)
1667
def test_sink_format_hint_no(self):
1668
# When the target format says packing makes no difference, pack is not
1670
self.run_stream('1.9', 'rich-root-pack', False)
1672
def test_sink_format_hint_yes(self):
1673
# When the target format says packing makes a difference, pack is
1675
self.run_stream('1.9', '2a', True)
1677
def test_sink_format_same_no(self):
1678
# When the formats are the same, pack is not called.
1679
self.run_stream('2a', '2a', False)
1681
def test_IDS_format_hint_no(self):
1682
# When the target format says packing makes no difference, pack is not
1684
self.run_fetch('1.9', 'rich-root-pack', False)
1686
def test_IDS_format_hint_yes(self):
1687
# When the target format says packing makes a difference, pack is
1689
self.run_fetch('1.9', '2a', True)
1691
def test_IDS_format_same_no(self):
1692
# When the formats are the same, pack is not called.
1693
self.run_fetch('2a', '2a', False)
1696
class Test_LazyListJoin(tests.TestCase):
1698
def test__repr__(self):
1699
lazy = repository._LazyListJoin(['a'], ['b'])
1700
self.assertEqual("bzrlib.repository._LazyListJoin((['a'], ['b']))",
1704
class TestFeatures(tests.TestCaseWithTransport):
1706
def test_open_with_present_feature(self):
1708
repository.RepositoryFormatMetaDir.unregister_feature,
1709
"makes-cheese-sandwich")
1710
repository.RepositoryFormatMetaDir.register_feature(
1711
"makes-cheese-sandwich")
1712
repo = self.make_repository('.')
1714
repo._format.features["makes-cheese-sandwich"] = "required"
1715
repo._format.check_support_status(False)
1718
def test_open_with_missing_required_feature(self):
1719
repo = self.make_repository('.')
1721
repo._format.features["makes-cheese-sandwich"] = "required"
1722
self.assertRaises(errors.MissingFeature,
1723
repo._format.check_support_status, False)
added
removed
bzrlib/tests/test_repository.py
