~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_repository.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2008-03-16 14:01:20 UTC
  • mfrom: (3280.2.5 integration)
  • Revision ID: pqm@pqm.ubuntu.com-20080316140120-i3yq8yr1l66m11h7
Start 1.4 development

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006, 2007 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""Tests for the Repository facility that are not interface tests.
18
18
 
19
 
For interface tests see tests/per_repository/*.py.
 
19
For interface tests see tests/repository_implementations/*.py.
20
20
 
21
21
For concrete class tests see this file, and for storage formats tests
22
22
also see this file.
23
23
"""
24
24
 
 
25
import md5
25
26
from stat import S_ISDIR
26
 
import sys
 
27
from StringIO import StringIO
27
28
 
28
29
import bzrlib
29
 
from bzrlib.errors import (NoSuchFile,
 
30
from bzrlib.errors import (NotBranchError,
 
31
                           NoSuchFile,
30
32
                           UnknownFormatError,
31
33
                           UnsupportedFormatError,
32
34
                           )
33
 
from bzrlib import (
34
 
    btree_index,
35
 
    graph,
36
 
    tests,
37
 
    )
38
 
from bzrlib.btree_index import BTreeBuilder, BTreeGraphIndex
39
 
from bzrlib.index import GraphIndex
 
35
from bzrlib import graph
 
36
from bzrlib.index import GraphIndex, InMemoryGraphIndex
40
37
from bzrlib.repository import RepositoryFormat
 
38
from bzrlib.smart import server
41
39
from bzrlib.tests import (
42
40
    TestCase,
43
41
    TestCaseWithTransport,
44
 
    )
45
 
from bzrlib.transport import (
46
 
    get_transport,
47
 
    )
 
42
    test_knit,
 
43
    )
 
44
from bzrlib.transport import get_transport
 
45
from bzrlib.transport.memory import MemoryServer
 
46
from bzrlib.util import bencode
48
47
from bzrlib import (
49
48
    bzrdir,
50
49
    errors,
51
50
    inventory,
52
 
    osutils,
 
51
    progress,
53
52
    repository,
54
53
    revision as _mod_revision,
 
54
    symbol_versioning,
55
55
    upgrade,
56
 
    versionedfile,
57
56
    workingtree,
58
57
    )
59
 
from bzrlib.repofmt import (
60
 
    groupcompress_repo,
61
 
    knitrepo,
62
 
    pack_repo,
63
 
    weaverepo,
64
 
    )
 
58
from bzrlib.repofmt import knitrepo, weaverepo, pack_repo
65
59
 
66
60
 
67
61
class TestDefaultFormat(TestCase):
96
90
class SampleRepositoryFormat(repository.RepositoryFormat):
97
91
    """A sample format
98
92
 
99
 
    this format is initializable, unsupported to aid in testing the
 
93
    this format is initializable, unsupported to aid in testing the 
100
94
    open and open(unsupported=True) routines.
101
95
    """
102
96
 
123
117
    def test_find_format(self):
124
118
        # is the right format object found for a repository?
125
119
        # create a branch with a few known format objects.
126
 
        # this is not quite the same as
 
120
        # this is not quite the same as 
127
121
        self.build_tree(["foo/", "bar/"])
128
122
        def check_format(format, url):
129
123
            dir = format._matchingbzrdir.initialize(url)
132
126
            found_format = repository.RepositoryFormat.find_format(dir)
133
127
            self.failUnless(isinstance(found_format, format.__class__))
134
128
        check_format(weaverepo.RepositoryFormat7(), "bar")
135
 
 
 
129
        
136
130
    def test_find_format_no_repository(self):
137
131
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
138
132
        self.assertRaises(errors.NoRepositoryPresent,
164
158
 
165
159
class TestFormat6(TestCaseWithTransport):
166
160
 
167
 
    def test_attribute__fetch_order(self):
168
 
        """Weaves need topological data insertion."""
169
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
170
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
171
 
        self.assertEqual('topological', repo._format._fetch_order)
172
 
 
173
 
    def test_attribute__fetch_uses_deltas(self):
174
 
        """Weaves do not reuse deltas."""
175
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
176
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
177
 
        self.assertEqual(False, repo._format._fetch_uses_deltas)
178
 
 
179
 
    def test_attribute__fetch_reconcile(self):
180
 
        """Weave repositories need a reconcile after fetch."""
181
 
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
182
 
        repo = weaverepo.RepositoryFormat6().initialize(control)
183
 
        self.assertEqual(True, repo._format._fetch_reconcile)
184
 
 
185
161
    def test_no_ancestry_weave(self):
186
162
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
187
163
        repo = weaverepo.RepositoryFormat6().initialize(control)
191
167
                          control.transport.get,
192
168
                          'ancestry.weave')
193
169
 
 
170
    def test_exposed_versioned_files_are_marked_dirty(self):
 
171
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
 
172
        repo = weaverepo.RepositoryFormat6().initialize(control)
 
173
        repo.lock_write()
 
174
        inv = repo.get_inventory_weave()
 
175
        repo.unlock()
 
176
        self.assertRaises(errors.OutSideTransaction,
 
177
            inv.add_lines, 'foo', [], [])
 
178
 
194
179
    def test_supports_external_lookups(self):
195
180
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
196
181
        repo = weaverepo.RepositoryFormat6().initialize(control)
198
183
 
199
184
 
200
185
class TestFormat7(TestCaseWithTransport):
201
 
 
202
 
    def test_attribute__fetch_order(self):
203
 
        """Weaves need topological data insertion."""
204
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
205
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
206
 
        self.assertEqual('topological', repo._format._fetch_order)
207
 
 
208
 
    def test_attribute__fetch_uses_deltas(self):
209
 
        """Weaves do not reuse deltas."""
210
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
211
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
212
 
        self.assertEqual(False, repo._format._fetch_uses_deltas)
213
 
 
214
 
    def test_attribute__fetch_reconcile(self):
215
 
        """Weave repositories need a reconcile after fetch."""
216
 
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
217
 
        repo = weaverepo.RepositoryFormat7().initialize(control)
218
 
        self.assertEqual(True, repo._format._fetch_reconcile)
219
 
 
 
186
    
220
187
    def test_disk_layout(self):
221
188
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
222
189
        repo = weaverepo.RepositoryFormat7().initialize(control)
238
205
                             'w\n'
239
206
                             'W\n',
240
207
                             t.get('inventory.weave').read())
241
 
        # Creating a file with id Foo:Bar results in a non-escaped file name on
242
 
        # disk.
243
 
        control.create_branch()
244
 
        tree = control.create_workingtree()
245
 
        tree.add(['foo'], ['Foo:Bar'], ['file'])
246
 
        tree.put_file_bytes_non_atomic('Foo:Bar', 'content\n')
247
 
        try:
248
 
            tree.commit('first post', rev_id='first')
249
 
        except errors.IllegalPath:
250
 
            if sys.platform != 'win32':
251
 
                raise
252
 
            self.knownFailure('Foo:Bar cannot be used as a file-id on windows'
253
 
                              ' in repo format 7')
254
 
            return
255
 
        self.assertEqualDiff(
256
 
            '# bzr weave file v5\n'
257
 
            'i\n'
258
 
            '1 7fe70820e08a1aac0ef224d9c66ab66831cc4ab1\n'
259
 
            'n first\n'
260
 
            '\n'
261
 
            'w\n'
262
 
            '{ 0\n'
263
 
            '. content\n'
264
 
            '}\n'
265
 
            'W\n',
266
 
            t.get('weaves/74/Foo%3ABar.weave').read())
267
208
 
268
209
    def test_shared_disk_layout(self):
269
210
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
292
233
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
293
234
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
294
235
        t = control.get_repository_transport(None)
295
 
        # TODO: Should check there is a 'lock' toplevel directory,
 
236
        # TODO: Should check there is a 'lock' toplevel directory, 
296
237
        # regardless of contents
297
238
        self.assertFalse(t.has('lock/held/info'))
298
239
        repo.lock_write()
344
285
                             'W\n',
345
286
                             t.get('inventory.weave').read())
346
287
 
 
288
    def test_exposed_versioned_files_are_marked_dirty(self):
 
289
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
 
290
        repo = weaverepo.RepositoryFormat7().initialize(control)
 
291
        repo.lock_write()
 
292
        inv = repo.get_inventory_weave()
 
293
        repo.unlock()
 
294
        self.assertRaises(errors.OutSideTransaction,
 
295
            inv.add_lines, 'foo', [], [])
 
296
 
347
297
    def test_supports_external_lookups(self):
348
298
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
349
299
        repo = weaverepo.RepositoryFormat7().initialize(control)
351
301
 
352
302
 
353
303
class TestFormatKnit1(TestCaseWithTransport):
354
 
 
355
 
    def test_attribute__fetch_order(self):
356
 
        """Knits need topological data insertion."""
357
 
        repo = self.make_repository('.',
358
 
                format=bzrdir.format_registry.get('knit')())
359
 
        self.assertEqual('topological', repo._format._fetch_order)
360
 
 
361
 
    def test_attribute__fetch_uses_deltas(self):
362
 
        """Knits reuse deltas."""
363
 
        repo = self.make_repository('.',
364
 
                format=bzrdir.format_registry.get('knit')())
365
 
        self.assertEqual(True, repo._format._fetch_uses_deltas)
366
 
 
 
304
    
367
305
    def test_disk_layout(self):
368
306
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
369
307
        repo = knitrepo.RepositoryFormatKnit1().initialize(control)
383
321
        # self.assertEqualDiff('', t.get('lock').read())
384
322
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
385
323
        self.check_knits(t)
386
 
        # Check per-file knits.
387
 
        branch = control.create_branch()
388
 
        tree = control.create_workingtree()
389
 
        tree.add(['foo'], ['Nasty-IdC:'], ['file'])
390
 
        tree.put_file_bytes_non_atomic('Nasty-IdC:', '')
391
 
        tree.commit('1st post', rev_id='foo')
392
 
        self.assertHasKnit(t, 'knits/e8/%254easty-%2549d%2543%253a',
393
 
            '\nfoo fulltext 0 81  :')
394
324
 
395
 
    def assertHasKnit(self, t, knit_name, extra_content=''):
 
325
    def assertHasKnit(self, t, knit_name):
396
326
        """Assert that knit_name exists on t."""
397
 
        self.assertEqualDiff('# bzr knit index 8\n' + extra_content,
 
327
        self.assertEqualDiff('# bzr knit index 8\n',
398
328
                             t.get(knit_name + '.kndx').read())
 
329
        # no default content
 
330
        self.assertTrue(t.has(knit_name + '.knit'))
399
331
 
400
332
    def check_knits(self, t):
401
333
        """check knit content for a repository."""
445
377
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
446
378
        self.check_knits(t)
447
379
 
 
380
    def test_exposed_versioned_files_are_marked_dirty(self):
 
381
        format = bzrdir.BzrDirMetaFormat1()
 
382
        format.repository_format = knitrepo.RepositoryFormatKnit1()
 
383
        repo = self.make_repository('.', format=format)
 
384
        repo.lock_write()
 
385
        inv = repo.get_inventory_weave()
 
386
        repo.unlock()
 
387
        self.assertRaises(errors.OutSideTransaction,
 
388
            inv.add_lines, 'foo', [], [])
 
389
 
448
390
    def test_deserialise_sets_root_revision(self):
449
391
        """We must have a inventory.root.revision
450
392
 
455
397
        repo = self.make_repository('.',
456
398
                format=bzrdir.format_registry.get('knit')())
457
399
        inv_xml = '<inventory format="5">\n</inventory>\n'
458
 
        inv = repo._deserialise_inventory('test-rev-id', inv_xml)
 
400
        inv = repo.deserialise_inventory('test-rev-id', inv_xml)
459
401
        self.assertEqual('test-rev-id', inv.root.revision)
460
402
 
461
403
    def test_deserialise_uses_global_revision_id(self):
467
409
        # Arguably, the deserialise_inventory should detect a mismatch, and
468
410
        # raise an error, rather than silently using one revision_id over the
469
411
        # other.
470
 
        self.assertRaises(AssertionError, repo._deserialise_inventory,
 
412
        self.assertRaises(AssertionError, repo.deserialise_inventory,
471
413
            'test-rev-id', inv_xml)
472
 
        inv = repo._deserialise_inventory('other-rev-id', inv_xml)
 
414
        inv = repo.deserialise_inventory('other-rev-id', inv_xml)
473
415
        self.assertEqual('other-rev-id', inv.root.revision)
474
416
 
475
417
    def test_supports_external_lookups(self):
478
420
        self.assertFalse(repo._format.supports_external_lookups)
479
421
 
480
422
 
 
423
class KnitRepositoryStreamTests(test_knit.KnitTests):
 
424
    """Tests for knitrepo._get_stream_as_bytes."""
 
425
 
 
426
    def test_get_stream_as_bytes(self):
 
427
        # Make a simple knit
 
428
        k1 = self.make_test_knit()
 
429
        k1.add_lines('text-a', [], test_knit.split_lines(test_knit.TEXT_1))
 
430
        
 
431
        # Serialise it, check the output.
 
432
        bytes = knitrepo._get_stream_as_bytes(k1, ['text-a'])
 
433
        data = bencode.bdecode(bytes)
 
434
        format, record = data
 
435
        self.assertEqual('knit-plain', format)
 
436
        self.assertEqual(['text-a', ['fulltext'], []], record[:3])
 
437
        self.assertRecordContentEqual(k1, 'text-a', record[3])
 
438
 
 
439
    def test_get_stream_as_bytes_all(self):
 
440
        """Get a serialised data stream for all the records in a knit.
 
441
 
 
442
        Much like test_get_stream_all, except for get_stream_as_bytes.
 
443
        """
 
444
        k1 = self.make_test_knit()
 
445
        # Insert the same data as BasicKnitTests.test_knit_join, as they seem
 
446
        # to cover a range of cases (no parents, one parent, multiple parents).
 
447
        test_data = [
 
448
            ('text-a', [], test_knit.TEXT_1),
 
449
            ('text-b', ['text-a'], test_knit.TEXT_1),
 
450
            ('text-c', [], test_knit.TEXT_1),
 
451
            ('text-d', ['text-c'], test_knit.TEXT_1),
 
452
            ('text-m', ['text-b', 'text-d'], test_knit.TEXT_1),
 
453
           ]
 
454
        # This test is actually a bit strict as the order in which they're
 
455
        # returned is not defined.  This matches the current (deterministic)
 
456
        # behaviour.
 
457
        expected_data_list = [
 
458
            # version, options, parents
 
459
            ('text-a', ['fulltext'], []),
 
460
            ('text-b', ['line-delta'], ['text-a']),
 
461
            ('text-m', ['line-delta'], ['text-b', 'text-d']),
 
462
            ('text-c', ['fulltext'], []),
 
463
            ('text-d', ['line-delta'], ['text-c']),
 
464
            ]
 
465
        for version_id, parents, lines in test_data:
 
466
            k1.add_lines(version_id, parents, test_knit.split_lines(lines))
 
467
 
 
468
        bytes = knitrepo._get_stream_as_bytes(
 
469
            k1, ['text-a', 'text-b', 'text-m', 'text-c', 'text-d', ])
 
470
 
 
471
        data = bencode.bdecode(bytes)
 
472
        format = data.pop(0)
 
473
        self.assertEqual('knit-plain', format)
 
474
 
 
475
        for expected, actual in zip(expected_data_list, data):
 
476
            expected_version = expected[0]
 
477
            expected_options = expected[1]
 
478
            expected_parents = expected[2]
 
479
            version, options, parents, bytes = actual
 
480
            self.assertEqual(expected_version, version)
 
481
            self.assertEqual(expected_options, options)
 
482
            self.assertEqual(expected_parents, parents)
 
483
            self.assertRecordContentEqual(k1, version, bytes)
 
484
 
 
485
 
481
486
class DummyRepository(object):
482
487
    """A dummy repository for testing."""
483
488
 
484
 
    _format = None
485
489
    _serializer = None
486
490
 
487
491
    def supports_rich_root(self):
488
 
        if self._format is not None:
489
 
            return self._format.rich_root_data
490
492
        return False
491
493
 
492
 
    def get_graph(self):
493
 
        raise NotImplementedError
494
 
 
495
 
    def get_parent_map(self, revision_ids):
496
 
        raise NotImplementedError
497
 
 
498
494
 
499
495
class InterDummy(repository.InterRepository):
500
496
    """An inter-repository optimised code path for DummyRepository.
507
503
    @staticmethod
508
504
    def is_compatible(repo_source, repo_target):
509
505
        """InterDummy is compatible with DummyRepository."""
510
 
        return (isinstance(repo_source, DummyRepository) and
 
506
        return (isinstance(repo_source, DummyRepository) and 
511
507
            isinstance(repo_target, DummyRepository))
512
508
 
513
509
 
526
522
 
527
523
    def assertGetsDefaultInterRepository(self, repo_a, repo_b):
528
524
        """Asserts that InterRepository.get(repo_a, repo_b) -> the default.
529
 
 
 
525
        
530
526
        The effective default is now InterSameDataRepository because there is
531
527
        no actual sane default in the presence of incompatible data models.
532
528
        """
543
539
        # pair that it returns true on for the is_compatible static method
544
540
        # check
545
541
        dummy_a = DummyRepository()
546
 
        dummy_a._format = RepositoryFormat()
547
542
        dummy_b = DummyRepository()
548
 
        dummy_b._format = RepositoryFormat()
549
543
        repo = self.make_repository('.')
550
544
        # hack dummies to look like repo somewhat.
551
545
        dummy_a._serializer = repo._serializer
552
 
        dummy_a._format.supports_tree_reference = repo._format.supports_tree_reference
553
 
        dummy_a._format.rich_root_data = repo._format.rich_root_data
554
546
        dummy_b._serializer = repo._serializer
555
 
        dummy_b._format.supports_tree_reference = repo._format.supports_tree_reference
556
 
        dummy_b._format.rich_root_data = repo._format.rich_root_data
557
547
        repository.InterRepository.register_optimiser(InterDummy)
558
548
        try:
559
549
            # we should get the default for something InterDummy returns False
603
593
                                                        repo_b).__class__)
604
594
 
605
595
 
 
596
class TestInterRemoteToOther(TestCaseWithTransport):
 
597
 
 
598
    def make_remote_repository(self, path, backing_format=None):
 
599
        """Make a RemoteRepository object backed by a real repository that will
 
600
        be created at the given path."""
 
601
        self.make_repository(path, format=backing_format)
 
602
        smart_server = server.SmartTCPServer_for_testing()
 
603
        smart_server.setUp()
 
604
        remote_transport = get_transport(smart_server.get_url()).clone(path)
 
605
        self.addCleanup(smart_server.tearDown)
 
606
        remote_bzrdir = bzrdir.BzrDir.open_from_transport(remote_transport)
 
607
        remote_repo = remote_bzrdir.open_repository()
 
608
        return remote_repo
 
609
 
 
610
    def test_is_compatible_same_format(self):
 
611
        """InterRemoteToOther is compatible with a remote repository and a
 
612
        second repository that have the same format."""
 
613
        local_repo = self.make_repository('local')
 
614
        remote_repo = self.make_remote_repository('remote')
 
615
        is_compatible = repository.InterRemoteToOther.is_compatible
 
616
        self.assertTrue(
 
617
            is_compatible(remote_repo, local_repo),
 
618
            "InterRemoteToOther(%r, %r) is false" % (remote_repo, local_repo))
 
619
          
 
620
    def test_is_incompatible_different_format(self):
 
621
        local_repo = self.make_repository('local', 'dirstate')
 
622
        remote_repo = self.make_remote_repository('a', 'dirstate-with-subtree')
 
623
        is_compatible = repository.InterRemoteToOther.is_compatible
 
624
        self.assertFalse(
 
625
            is_compatible(remote_repo, local_repo),
 
626
            "InterRemoteToOther(%r, %r) is true" % (local_repo, remote_repo))
 
627
 
 
628
    def test_is_incompatible_different_format_both_remote(self):
 
629
        remote_repo_a = self.make_remote_repository(
 
630
            'a', 'dirstate-with-subtree')
 
631
        remote_repo_b = self.make_remote_repository('b', 'dirstate')
 
632
        is_compatible = repository.InterRemoteToOther.is_compatible
 
633
        self.assertFalse(
 
634
            is_compatible(remote_repo_a, remote_repo_b),
 
635
            "InterRemoteToOther(%r, %r) is true"
 
636
            % (remote_repo_a, remote_repo_b))
 
637
 
 
638
 
606
639
class TestRepositoryConverter(TestCaseWithTransport):
607
640
 
608
641
    def test_convert_empty(self):
622
655
 
623
656
 
624
657
class TestMisc(TestCase):
625
 
 
 
658
    
626
659
    def test_unescape_xml(self):
627
660
        """We get some kind of error when malformed entities are passed"""
628
 
        self.assertRaises(KeyError, repository._unescape_xml, 'foo&bar;')
 
661
        self.assertRaises(KeyError, repository._unescape_xml, 'foo&bar;') 
629
662
 
630
663
 
631
664
class TestRepositoryFormatKnit3(TestCaseWithTransport):
632
665
 
633
 
    def test_attribute__fetch_order(self):
634
 
        """Knits need topological data insertion."""
635
 
        format = bzrdir.BzrDirMetaFormat1()
636
 
        format.repository_format = knitrepo.RepositoryFormatKnit3()
637
 
        repo = self.make_repository('.', format=format)
638
 
        self.assertEqual('topological', repo._format._fetch_order)
639
 
 
640
 
    def test_attribute__fetch_uses_deltas(self):
641
 
        """Knits reuse deltas."""
642
 
        format = bzrdir.BzrDirMetaFormat1()
643
 
        format.repository_format = knitrepo.RepositoryFormatKnit3()
644
 
        repo = self.make_repository('.', format=format)
645
 
        self.assertEqual(True, repo._format._fetch_uses_deltas)
646
 
 
647
666
    def test_convert(self):
648
667
        """Ensure the upgrade adds weaves for roots"""
649
668
        format = bzrdir.BzrDirMetaFormat1()
651
670
        tree = self.make_branch_and_tree('.', format)
652
671
        tree.commit("Dull commit", rev_id="dull")
653
672
        revision_tree = tree.branch.repository.revision_tree('dull')
654
 
        revision_tree.lock_read()
655
 
        try:
656
 
            self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
657
 
                revision_tree.inventory.root.file_id)
658
 
        finally:
659
 
            revision_tree.unlock()
 
673
        self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
 
674
            revision_tree.inventory.root.file_id)
660
675
        format = bzrdir.BzrDirMetaFormat1()
661
676
        format.repository_format = knitrepo.RepositoryFormatKnit3()
662
677
        upgrade.Convert('.', format)
663
678
        tree = workingtree.WorkingTree.open('.')
664
679
        revision_tree = tree.branch.repository.revision_tree('dull')
665
 
        revision_tree.lock_read()
666
 
        try:
667
 
            revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
668
 
        finally:
669
 
            revision_tree.unlock()
 
680
        revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
670
681
        tree.commit("Another dull commit", rev_id='dull2')
671
682
        revision_tree = tree.branch.repository.revision_tree('dull2')
672
 
        revision_tree.lock_read()
673
 
        self.addCleanup(revision_tree.unlock)
674
683
        self.assertEqual('dull', revision_tree.inventory.root.revision)
675
684
 
 
685
    def test_exposed_versioned_files_are_marked_dirty(self):
 
686
        format = bzrdir.BzrDirMetaFormat1()
 
687
        format.repository_format = knitrepo.RepositoryFormatKnit3()
 
688
        repo = self.make_repository('.', format=format)
 
689
        repo.lock_write()
 
690
        inv = repo.get_inventory_weave()
 
691
        repo.unlock()
 
692
        self.assertRaises(errors.OutSideTransaction,
 
693
            inv.add_lines, 'foo', [], [])
 
694
 
676
695
    def test_supports_external_lookups(self):
677
696
        format = bzrdir.BzrDirMetaFormat1()
678
697
        format.repository_format = knitrepo.RepositoryFormatKnit3()
680
699
        self.assertFalse(repo._format.supports_external_lookups)
681
700
 
682
701
 
683
 
class Test2a(tests.TestCaseWithMemoryTransport):
684
 
 
685
 
    def test_chk_bytes_uses_custom_btree_parser(self):
686
 
        mt = self.make_branch_and_memory_tree('test', format='2a')
687
 
        mt.lock_write()
688
 
        self.addCleanup(mt.unlock)
689
 
        mt.add([''], ['root-id'])
690
 
        mt.commit('first')
691
 
        index = mt.branch.repository.chk_bytes._index._graph_index._indices[0]
692
 
        self.assertEqual(btree_index._gcchk_factory, index._leaf_factory)
693
 
        # It should also work if we re-open the repo
694
 
        repo = mt.branch.repository.bzrdir.open_repository()
695
 
        repo.lock_read()
696
 
        self.addCleanup(repo.unlock)
697
 
        index = repo.chk_bytes._index._graph_index._indices[0]
698
 
        self.assertEqual(btree_index._gcchk_factory, index._leaf_factory)
699
 
 
700
 
    def test_fetch_combines_groups(self):
701
 
        builder = self.make_branch_builder('source', format='2a')
702
 
        builder.start_series()
703
 
        builder.build_snapshot('1', None, [
704
 
            ('add', ('', 'root-id', 'directory', '')),
705
 
            ('add', ('file', 'file-id', 'file', 'content\n'))])
706
 
        builder.build_snapshot('2', ['1'], [
707
 
            ('modify', ('file-id', 'content-2\n'))])
708
 
        builder.finish_series()
709
 
        source = builder.get_branch()
710
 
        target = self.make_repository('target', format='2a')
711
 
        target.fetch(source.repository)
712
 
        target.lock_read()
713
 
        self.addCleanup(target.unlock)
714
 
        details = target.texts._index.get_build_details(
715
 
            [('file-id', '1',), ('file-id', '2',)])
716
 
        file_1_details = details[('file-id', '1')]
717
 
        file_2_details = details[('file-id', '2')]
718
 
        # The index, and what to read off disk, should be the same for both
719
 
        # versions of the file.
720
 
        self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
721
 
 
722
 
    def test_fetch_combines_groups(self):
723
 
        builder = self.make_branch_builder('source', format='2a')
724
 
        builder.start_series()
725
 
        builder.build_snapshot('1', None, [
726
 
            ('add', ('', 'root-id', 'directory', '')),
727
 
            ('add', ('file', 'file-id', 'file', 'content\n'))])
728
 
        builder.build_snapshot('2', ['1'], [
729
 
            ('modify', ('file-id', 'content-2\n'))])
730
 
        builder.finish_series()
731
 
        source = builder.get_branch()
732
 
        target = self.make_repository('target', format='2a')
733
 
        target.fetch(source.repository)
734
 
        target.lock_read()
735
 
        self.addCleanup(target.unlock)
736
 
        details = target.texts._index.get_build_details(
737
 
            [('file-id', '1',), ('file-id', '2',)])
738
 
        file_1_details = details[('file-id', '1')]
739
 
        file_2_details = details[('file-id', '2')]
740
 
        # The index, and what to read off disk, should be the same for both
741
 
        # versions of the file.
742
 
        self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
743
 
 
744
 
    def test_fetch_combines_groups(self):
745
 
        builder = self.make_branch_builder('source', format='2a')
746
 
        builder.start_series()
747
 
        builder.build_snapshot('1', None, [
748
 
            ('add', ('', 'root-id', 'directory', '')),
749
 
            ('add', ('file', 'file-id', 'file', 'content\n'))])
750
 
        builder.build_snapshot('2', ['1'], [
751
 
            ('modify', ('file-id', 'content-2\n'))])
752
 
        builder.finish_series()
753
 
        source = builder.get_branch()
754
 
        target = self.make_repository('target', format='2a')
755
 
        target.fetch(source.repository)
756
 
        target.lock_read()
757
 
        self.addCleanup(target.unlock)
758
 
        details = target.texts._index.get_build_details(
759
 
            [('file-id', '1',), ('file-id', '2',)])
760
 
        file_1_details = details[('file-id', '1')]
761
 
        file_2_details = details[('file-id', '2')]
762
 
        # The index, and what to read off disk, should be the same for both
763
 
        # versions of the file.
764
 
        self.assertEqual(file_1_details[0][:3], file_2_details[0][:3])
765
 
 
766
 
    def test_format_pack_compresses_True(self):
767
 
        repo = self.make_repository('repo', format='2a')
768
 
        self.assertTrue(repo._format.pack_compresses)
769
 
 
770
 
    def test_inventories_use_chk_map_with_parent_base_dict(self):
771
 
        tree = self.make_branch_and_memory_tree('repo', format="2a")
772
 
        tree.lock_write()
773
 
        tree.add([''], ['TREE_ROOT'])
774
 
        revid = tree.commit("foo")
775
 
        tree.unlock()
776
 
        tree.lock_read()
777
 
        self.addCleanup(tree.unlock)
778
 
        inv = tree.branch.repository.get_inventory(revid)
779
 
        self.assertNotEqual(None, inv.parent_id_basename_to_file_id)
780
 
        inv.parent_id_basename_to_file_id._ensure_root()
781
 
        inv.id_to_entry._ensure_root()
782
 
        self.assertEqual(65536, inv.id_to_entry._root_node.maximum_size)
783
 
        self.assertEqual(65536,
784
 
            inv.parent_id_basename_to_file_id._root_node.maximum_size)
785
 
 
786
 
    def test_autopack_unchanged_chk_nodes(self):
787
 
        # at 20 unchanged commits, chk pages are packed that are split into
788
 
        # two groups such that the new pack being made doesn't have all its
789
 
        # pages in the source packs (though they are in the repository).
790
 
        # Use a memory backed repository, we don't need to hit disk for this
791
 
        tree = self.make_branch_and_memory_tree('tree', format='2a')
792
 
        tree.lock_write()
793
 
        self.addCleanup(tree.unlock)
794
 
        tree.add([''], ['TREE_ROOT'])
795
 
        for pos in range(20):
796
 
            tree.commit(str(pos))
797
 
 
798
 
    def test_pack_with_hint(self):
799
 
        tree = self.make_branch_and_memory_tree('tree', format='2a')
800
 
        tree.lock_write()
801
 
        self.addCleanup(tree.unlock)
802
 
        tree.add([''], ['TREE_ROOT'])
803
 
        # 1 commit to leave untouched
804
 
        tree.commit('1')
805
 
        to_keep = tree.branch.repository._pack_collection.names()
806
 
        # 2 to combine
807
 
        tree.commit('2')
808
 
        tree.commit('3')
809
 
        all = tree.branch.repository._pack_collection.names()
810
 
        combine = list(set(all) - set(to_keep))
811
 
        self.assertLength(3, all)
812
 
        self.assertLength(2, combine)
813
 
        tree.branch.repository.pack(hint=combine)
814
 
        final = tree.branch.repository._pack_collection.names()
815
 
        self.assertLength(2, final)
816
 
        self.assertFalse(combine[0] in final)
817
 
        self.assertFalse(combine[1] in final)
818
 
        self.assertSubset(to_keep, final)
819
 
 
820
 
    def test_stream_source_to_gc(self):
821
 
        source = self.make_repository('source', format='2a')
822
 
        target = self.make_repository('target', format='2a')
823
 
        stream = source._get_source(target._format)
824
 
        self.assertIsInstance(stream, groupcompress_repo.GroupCHKStreamSource)
825
 
 
826
 
    def test_stream_source_to_non_gc(self):
827
 
        source = self.make_repository('source', format='2a')
828
 
        target = self.make_repository('target', format='rich-root-pack')
829
 
        stream = source._get_source(target._format)
830
 
        # We don't want the child GroupCHKStreamSource
831
 
        self.assertIs(type(stream), repository.StreamSource)
832
 
 
833
 
    def test_get_stream_for_missing_keys_includes_all_chk_refs(self):
834
 
        source_builder = self.make_branch_builder('source',
835
 
                            format='2a')
836
 
        # We have to build a fairly large tree, so that we are sure the chk
837
 
        # pages will have split into multiple pages.
838
 
        entries = [('add', ('', 'a-root-id', 'directory', None))]
839
 
        for i in 'abcdefghijklmnopqrstuvwxyz123456789':
840
 
            for j in 'abcdefghijklmnopqrstuvwxyz123456789':
841
 
                fname = i + j
842
 
                fid = fname + '-id'
843
 
                content = 'content for %s\n' % (fname,)
844
 
                entries.append(('add', (fname, fid, 'file', content)))
845
 
        source_builder.start_series()
846
 
        source_builder.build_snapshot('rev-1', None, entries)
847
 
        # Now change a few of them, so we get a few new pages for the second
848
 
        # revision
849
 
        source_builder.build_snapshot('rev-2', ['rev-1'], [
850
 
            ('modify', ('aa-id', 'new content for aa-id\n')),
851
 
            ('modify', ('cc-id', 'new content for cc-id\n')),
852
 
            ('modify', ('zz-id', 'new content for zz-id\n')),
853
 
            ])
854
 
        source_builder.finish_series()
855
 
        source_branch = source_builder.get_branch()
856
 
        source_branch.lock_read()
857
 
        self.addCleanup(source_branch.unlock)
858
 
        target = self.make_repository('target', format='2a')
859
 
        source = source_branch.repository._get_source(target._format)
860
 
        self.assertIsInstance(source, groupcompress_repo.GroupCHKStreamSource)
861
 
 
862
 
        # On a regular pass, getting the inventories and chk pages for rev-2
863
 
        # would only get the newly created chk pages
864
 
        search = graph.SearchResult(set(['rev-2']), set(['rev-1']), 1,
865
 
                                    set(['rev-2']))
866
 
        simple_chk_records = []
867
 
        for vf_name, substream in source.get_stream(search):
868
 
            if vf_name == 'chk_bytes':
869
 
                for record in substream:
870
 
                    simple_chk_records.append(record.key)
871
 
            else:
872
 
                for _ in substream:
873
 
                    continue
874
 
        # 3 pages, the root (InternalNode), + 2 pages which actually changed
875
 
        self.assertEqual([('sha1:91481f539e802c76542ea5e4c83ad416bf219f73',),
876
 
                          ('sha1:4ff91971043668583985aec83f4f0ab10a907d3f',),
877
 
                          ('sha1:81e7324507c5ca132eedaf2d8414ee4bb2226187',),
878
 
                          ('sha1:b101b7da280596c71a4540e9a1eeba8045985ee0',)],
879
 
                         simple_chk_records)
880
 
        # Now, when we do a similar call using 'get_stream_for_missing_keys'
881
 
        # we should get a much larger set of pages.
882
 
        missing = [('inventories', 'rev-2')]
883
 
        full_chk_records = []
884
 
        for vf_name, substream in source.get_stream_for_missing_keys(missing):
885
 
            if vf_name == 'inventories':
886
 
                for record in substream:
887
 
                    self.assertEqual(('rev-2',), record.key)
888
 
            elif vf_name == 'chk_bytes':
889
 
                for record in substream:
890
 
                    full_chk_records.append(record.key)
891
 
            else:
892
 
                self.fail('Should not be getting a stream of %s' % (vf_name,))
893
 
        # We have 257 records now. This is because we have 1 root page, and 256
894
 
        # leaf pages in a complete listing.
895
 
        self.assertEqual(257, len(full_chk_records))
896
 
        self.assertSubset(simple_chk_records, full_chk_records)
897
 
 
898
 
    def test_inconsistency_fatal(self):
899
 
        repo = self.make_repository('repo', format='2a')
900
 
        self.assertTrue(repo.revisions._index._inconsistency_fatal)
901
 
        self.assertFalse(repo.texts._index._inconsistency_fatal)
902
 
        self.assertFalse(repo.inventories._index._inconsistency_fatal)
903
 
        self.assertFalse(repo.signatures._index._inconsistency_fatal)
904
 
        self.assertFalse(repo.chk_bytes._index._inconsistency_fatal)
905
 
 
906
 
 
907
 
class TestKnitPackStreamSource(tests.TestCaseWithMemoryTransport):
908
 
 
909
 
    def test_source_to_exact_pack_092(self):
910
 
        source = self.make_repository('source', format='pack-0.92')
911
 
        target = self.make_repository('target', format='pack-0.92')
912
 
        stream_source = source._get_source(target._format)
913
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
914
 
 
915
 
    def test_source_to_exact_pack_rich_root_pack(self):
916
 
        source = self.make_repository('source', format='rich-root-pack')
917
 
        target = self.make_repository('target', format='rich-root-pack')
918
 
        stream_source = source._get_source(target._format)
919
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
920
 
 
921
 
    def test_source_to_exact_pack_19(self):
922
 
        source = self.make_repository('source', format='1.9')
923
 
        target = self.make_repository('target', format='1.9')
924
 
        stream_source = source._get_source(target._format)
925
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
926
 
 
927
 
    def test_source_to_exact_pack_19_rich_root(self):
928
 
        source = self.make_repository('source', format='1.9-rich-root')
929
 
        target = self.make_repository('target', format='1.9-rich-root')
930
 
        stream_source = source._get_source(target._format)
931
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
932
 
 
933
 
    def test_source_to_remote_exact_pack_19(self):
934
 
        trans = self.make_smart_server('target')
935
 
        trans.ensure_base()
936
 
        source = self.make_repository('source', format='1.9')
937
 
        target = self.make_repository('target', format='1.9')
938
 
        target = repository.Repository.open(trans.base)
939
 
        stream_source = source._get_source(target._format)
940
 
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
941
 
 
942
 
    def test_stream_source_to_non_exact(self):
943
 
        source = self.make_repository('source', format='pack-0.92')
944
 
        target = self.make_repository('target', format='1.9')
945
 
        stream = source._get_source(target._format)
946
 
        self.assertIs(type(stream), repository.StreamSource)
947
 
 
948
 
    def test_stream_source_to_non_exact_rich_root(self):
949
 
        source = self.make_repository('source', format='1.9')
950
 
        target = self.make_repository('target', format='1.9-rich-root')
951
 
        stream = source._get_source(target._format)
952
 
        self.assertIs(type(stream), repository.StreamSource)
953
 
 
954
 
    def test_source_to_remote_non_exact_pack_19(self):
955
 
        trans = self.make_smart_server('target')
956
 
        trans.ensure_base()
957
 
        source = self.make_repository('source', format='1.9')
958
 
        target = self.make_repository('target', format='1.6')
959
 
        target = repository.Repository.open(trans.base)
960
 
        stream_source = source._get_source(target._format)
961
 
        self.assertIs(type(stream_source), repository.StreamSource)
962
 
 
963
 
    def test_stream_source_to_knit(self):
964
 
        source = self.make_repository('source', format='pack-0.92')
965
 
        target = self.make_repository('target', format='dirstate')
966
 
        stream = source._get_source(target._format)
967
 
        self.assertIs(type(stream), repository.StreamSource)
968
 
 
969
 
 
970
 
class TestDevelopment6FindParentIdsOfRevisions(TestCaseWithTransport):
971
 
    """Tests for _find_parent_ids_of_revisions."""
972
 
 
973
 
    def setUp(self):
974
 
        super(TestDevelopment6FindParentIdsOfRevisions, self).setUp()
975
 
        self.builder = self.make_branch_builder('source',
976
 
            format='development6-rich-root')
977
 
        self.builder.start_series()
978
 
        self.builder.build_snapshot('initial', None,
979
 
            [('add', ('', 'tree-root', 'directory', None))])
980
 
        self.repo = self.builder.get_branch().repository
981
 
        self.addCleanup(self.builder.finish_series)
982
 
 
983
 
    def assertParentIds(self, expected_result, rev_set):
984
 
        self.assertEqual(sorted(expected_result),
985
 
            sorted(self.repo._find_parent_ids_of_revisions(rev_set)))
986
 
 
987
 
    def test_simple(self):
988
 
        self.builder.build_snapshot('revid1', None, [])
989
 
        self.builder.build_snapshot('revid2', ['revid1'], [])
990
 
        rev_set = ['revid2']
991
 
        self.assertParentIds(['revid1'], rev_set)
992
 
 
993
 
    def test_not_first_parent(self):
994
 
        self.builder.build_snapshot('revid1', None, [])
995
 
        self.builder.build_snapshot('revid2', ['revid1'], [])
996
 
        self.builder.build_snapshot('revid3', ['revid2'], [])
997
 
        rev_set = ['revid3', 'revid2']
998
 
        self.assertParentIds(['revid1'], rev_set)
999
 
 
1000
 
    def test_not_null(self):
1001
 
        rev_set = ['initial']
1002
 
        self.assertParentIds([], rev_set)
1003
 
 
1004
 
    def test_not_null_set(self):
1005
 
        self.builder.build_snapshot('revid1', None, [])
1006
 
        rev_set = [_mod_revision.NULL_REVISION]
1007
 
        self.assertParentIds([], rev_set)
1008
 
 
1009
 
    def test_ghost(self):
1010
 
        self.builder.build_snapshot('revid1', None, [])
1011
 
        rev_set = ['ghost', 'revid1']
1012
 
        self.assertParentIds(['initial'], rev_set)
1013
 
 
1014
 
    def test_ghost_parent(self):
1015
 
        self.builder.build_snapshot('revid1', None, [])
1016
 
        self.builder.build_snapshot('revid2', ['revid1', 'ghost'], [])
1017
 
        rev_set = ['revid2', 'revid1']
1018
 
        self.assertParentIds(['ghost', 'initial'], rev_set)
1019
 
 
1020
 
    def test_righthand_parent(self):
1021
 
        self.builder.build_snapshot('revid1', None, [])
1022
 
        self.builder.build_snapshot('revid2a', ['revid1'], [])
1023
 
        self.builder.build_snapshot('revid2b', ['revid1'], [])
1024
 
        self.builder.build_snapshot('revid3', ['revid2a', 'revid2b'], [])
1025
 
        rev_set = ['revid3', 'revid2a']
1026
 
        self.assertParentIds(['revid1', 'revid2b'], rev_set)
1027
 
 
1028
 
 
1029
702
class TestWithBrokenRepo(TestCaseWithTransport):
1030
703
    """These tests seem to be more appropriate as interface tests?"""
1031
704
 
1044
717
            inv = inventory.Inventory(revision_id='rev1a')
1045
718
            inv.root.revision = 'rev1a'
1046
719
            self.add_file(repo, inv, 'file1', 'rev1a', [])
1047
 
            repo.texts.add_lines((inv.root.file_id, 'rev1a'), [], [])
1048
720
            repo.add_inventory('rev1a', inv, [])
1049
721
            revision = _mod_revision.Revision('rev1a',
1050
722
                committer='jrandom@example.com', timestamp=0,
1085
757
    def add_revision(self, repo, revision_id, inv, parent_ids):
1086
758
        inv.revision_id = revision_id
1087
759
        inv.root.revision = revision_id
1088
 
        repo.texts.add_lines((inv.root.file_id, revision_id), [], [])
1089
760
        repo.add_inventory(revision_id, inv, parent_ids)
1090
761
        revision = _mod_revision.Revision(revision_id,
1091
762
            committer='jrandom@example.com', timestamp=0, inventory_sha1='',
1098
769
        entry.revision = revision
1099
770
        entry.text_size = 0
1100
771
        inv.add(entry)
1101
 
        text_key = (file_id, revision)
1102
 
        parent_keys = [(file_id, parent) for parent in parents]
1103
 
        repo.texts.add_lines(text_key, parent_keys, ['line\n'])
 
772
        vf = repo.weave_store.get_weave_or_empty(file_id,
 
773
                                                 repo.get_transaction())
 
774
        vf.add_lines(revision, parents, ['line\n'])
1104
775
 
1105
776
    def test_insert_from_broken_repo(self):
1106
777
        """Inserting a data stream from a broken repository won't silently
1108
779
        """
1109
780
        broken_repo = self.make_broken_repository()
1110
781
        empty_repo = self.make_repository('empty-repo')
1111
 
        try:
1112
 
            empty_repo.fetch(broken_repo)
1113
 
        except (errors.RevisionNotPresent, errors.BzrCheckError):
1114
 
            # Test successful: compression parent not being copied leads to
1115
 
            # error.
1116
 
            return
1117
 
        empty_repo.lock_read()
 
782
        search = graph.SearchResult(set(['rev1a', 'rev2', 'rev3']),
 
783
            set(), 3, ['rev1a', 'rev2', 'rev3'])
 
784
        stream = broken_repo.get_data_stream_for_search(search)
 
785
        empty_repo.lock_write()
1118
786
        self.addCleanup(empty_repo.unlock)
1119
 
        text = empty_repo.texts.get_record_stream(
1120
 
            [('file2-id', 'rev3')], 'topological', True).next()
1121
 
        self.assertEqual('line\n', text.get_bytes_as('fulltext'))
 
787
        empty_repo.start_write_group()
 
788
        try:
 
789
            self.assertRaises(
 
790
                errors.KnitCorrupt, empty_repo.insert_data_stream, stream)
 
791
        finally:
 
792
            empty_repo.abort_write_group()
 
793
 
 
794
 
 
795
class TestKnitPackNoSubtrees(TestCaseWithTransport):
 
796
 
 
797
    def get_format(self):
 
798
        return bzrdir.format_registry.make_bzrdir('pack-0.92')
 
799
 
 
800
    def test_disk_layout(self):
 
801
        format = self.get_format()
 
802
        repo = self.make_repository('.', format=format)
 
803
        # in case of side effects of locking.
 
804
        repo.lock_write()
 
805
        repo.unlock()
 
806
        t = repo.bzrdir.get_repository_transport(None)
 
807
        self.check_format(t)
 
808
        # XXX: no locks left when unlocked at the moment
 
809
        # self.assertEqualDiff('', t.get('lock').read())
 
810
        self.check_databases(t)
 
811
 
 
812
    def check_format(self, t):
 
813
        self.assertEqualDiff(
 
814
            "Bazaar pack repository format 1 (needs bzr 0.92)\n",
 
815
                             t.get('format').read())
 
816
 
 
817
    def assertHasKndx(self, t, knit_name):
 
818
        """Assert that knit_name exists on t."""
 
819
        self.assertEqualDiff('# bzr knit index 8\n',
 
820
                             t.get(knit_name + '.kndx').read())
 
821
 
 
822
    def assertHasNoKndx(self, t, knit_name):
 
823
        """Assert that knit_name has no index on t."""
 
824
        self.assertFalse(t.has(knit_name + '.kndx'))
 
825
 
 
826
    def assertHasNoKnit(self, t, knit_name):
 
827
        """Assert that knit_name exists on t."""
 
828
        # no default content
 
829
        self.assertFalse(t.has(knit_name + '.knit'))
 
830
 
 
831
    def check_databases(self, t):
 
832
        """check knit content for a repository."""
 
833
        # check conversion worked
 
834
        self.assertHasNoKndx(t, 'inventory')
 
835
        self.assertHasNoKnit(t, 'inventory')
 
836
        self.assertHasNoKndx(t, 'revisions')
 
837
        self.assertHasNoKnit(t, 'revisions')
 
838
        self.assertHasNoKndx(t, 'signatures')
 
839
        self.assertHasNoKnit(t, 'signatures')
 
840
        self.assertFalse(t.has('knits'))
 
841
        # revision-indexes file-container directory
 
842
        self.assertEqual([],
 
843
            list(GraphIndex(t, 'pack-names', None).iter_all_entries()))
 
844
        self.assertTrue(S_ISDIR(t.stat('packs').st_mode))
 
845
        self.assertTrue(S_ISDIR(t.stat('upload').st_mode))
 
846
        self.assertTrue(S_ISDIR(t.stat('indices').st_mode))
 
847
        self.assertTrue(S_ISDIR(t.stat('obsolete_packs').st_mode))
 
848
 
 
849
    def test_shared_disk_layout(self):
 
850
        format = self.get_format()
 
851
        repo = self.make_repository('.', shared=True, format=format)
 
852
        # we want:
 
853
        t = repo.bzrdir.get_repository_transport(None)
 
854
        self.check_format(t)
 
855
        # XXX: no locks left when unlocked at the moment
 
856
        # self.assertEqualDiff('', t.get('lock').read())
 
857
        # We should have a 'shared-storage' marker file.
 
858
        self.assertEqualDiff('', t.get('shared-storage').read())
 
859
        self.check_databases(t)
 
860
 
 
861
    def test_shared_no_tree_disk_layout(self):
 
862
        format = self.get_format()
 
863
        repo = self.make_repository('.', shared=True, format=format)
 
864
        repo.set_make_working_trees(False)
 
865
        # we want:
 
866
        t = repo.bzrdir.get_repository_transport(None)
 
867
        self.check_format(t)
 
868
        # XXX: no locks left when unlocked at the moment
 
869
        # self.assertEqualDiff('', t.get('lock').read())
 
870
        # We should have a 'shared-storage' marker file.
 
871
        self.assertEqualDiff('', t.get('shared-storage').read())
 
872
        # We should have a marker for the no-working-trees flag.
 
873
        self.assertEqualDiff('', t.get('no-working-trees').read())
 
874
        # The marker should go when we toggle the setting.
 
875
        repo.set_make_working_trees(True)
 
876
        self.assertFalse(t.has('no-working-trees'))
 
877
        self.check_databases(t)
 
878
 
 
879
    def test_adding_revision_creates_pack_indices(self):
 
880
        format = self.get_format()
 
881
        tree = self.make_branch_and_tree('.', format=format)
 
882
        trans = tree.branch.repository.bzrdir.get_repository_transport(None)
 
883
        self.assertEqual([],
 
884
            list(GraphIndex(trans, 'pack-names', None).iter_all_entries()))
 
885
        tree.commit('foobarbaz')
 
886
        index = GraphIndex(trans, 'pack-names', None)
 
887
        index_nodes = list(index.iter_all_entries())
 
888
        self.assertEqual(1, len(index_nodes))
 
889
        node = index_nodes[0]
 
890
        name = node[1][0]
 
891
        # the pack sizes should be listed in the index
 
892
        pack_value = node[2]
 
893
        sizes = [int(digits) for digits in pack_value.split(' ')]
 
894
        for size, suffix in zip(sizes, ['.rix', '.iix', '.tix', '.six']):
 
895
            stat = trans.stat('indices/%s%s' % (name, suffix))
 
896
            self.assertEqual(size, stat.st_size)
 
897
 
 
898
    def test_pulling_nothing_leads_to_no_new_names(self):
 
899
        format = self.get_format()
 
900
        tree1 = self.make_branch_and_tree('1', format=format)
 
901
        tree2 = self.make_branch_and_tree('2', format=format)
 
902
        tree1.branch.repository.fetch(tree2.branch.repository)
 
903
        trans = tree1.branch.repository.bzrdir.get_repository_transport(None)
 
904
        self.assertEqual([],
 
905
            list(GraphIndex(trans, 'pack-names', None).iter_all_entries()))
 
906
 
 
907
    def test_commit_across_pack_shape_boundary_autopacks(self):
 
908
        format = self.get_format()
 
909
        tree = self.make_branch_and_tree('.', format=format)
 
910
        trans = tree.branch.repository.bzrdir.get_repository_transport(None)
 
911
        # This test could be a little cheaper by replacing the packs
 
912
        # attribute on the repository to allow a different pack distribution
 
913
        # and max packs policy - so we are checking the policy is honoured
 
914
        # in the test. But for now 11 commits is not a big deal in a single
 
915
        # test.
 
916
        for x in range(9):
 
917
            tree.commit('commit %s' % x)
 
918
        # there should be 9 packs:
 
919
        index = GraphIndex(trans, 'pack-names', None)
 
920
        self.assertEqual(9, len(list(index.iter_all_entries())))
 
921
        # insert some files in obsolete_packs which should be removed by pack.
 
922
        trans.put_bytes('obsolete_packs/foo', '123')
 
923
        trans.put_bytes('obsolete_packs/bar', '321')
 
924
        # committing one more should coalesce to 1 of 10.
 
925
        tree.commit('commit triggering pack')
 
926
        index = GraphIndex(trans, 'pack-names', None)
 
927
        self.assertEqual(1, len(list(index.iter_all_entries())))
 
928
        # packing should not damage data
 
929
        tree = tree.bzrdir.open_workingtree()
 
930
        check_result = tree.branch.repository.check(
 
931
            [tree.branch.last_revision()])
 
932
        # We should have 50 (10x5) files in the obsolete_packs directory.
 
933
        obsolete_files = list(trans.list_dir('obsolete_packs'))
 
934
        self.assertFalse('foo' in obsolete_files)
 
935
        self.assertFalse('bar' in obsolete_files)
 
936
        self.assertEqual(50, len(obsolete_files))
 
937
        # XXX: Todo check packs obsoleted correctly - old packs and indices
 
938
        # in the obsolete_packs directory.
 
939
        large_pack_name = list(index.iter_all_entries())[0][1][0]
 
940
        # finally, committing again should not touch the large pack.
 
941
        tree.commit('commit not triggering pack')
 
942
        index = GraphIndex(trans, 'pack-names', None)
 
943
        self.assertEqual(2, len(list(index.iter_all_entries())))
 
944
        pack_names = [node[1][0] for node in index.iter_all_entries()]
 
945
        self.assertTrue(large_pack_name in pack_names)
 
946
 
 
947
    def test_pack_after_two_commits_packs_everything(self):
 
948
        format = self.get_format()
 
949
        tree = self.make_branch_and_tree('.', format=format)
 
950
        trans = tree.branch.repository.bzrdir.get_repository_transport(None)
 
951
        tree.commit('start')
 
952
        tree.commit('more work')
 
953
        tree.branch.repository.pack()
 
954
        # there should be 1 pack:
 
955
        index = GraphIndex(trans, 'pack-names', None)
 
956
        self.assertEqual(1, len(list(index.iter_all_entries())))
 
957
        self.assertEqual(2, len(tree.branch.repository.all_revision_ids()))
 
958
 
 
959
    def test_pack_layout(self):
 
960
        format = self.get_format()
 
961
        tree = self.make_branch_and_tree('.', format=format)
 
962
        trans = tree.branch.repository.bzrdir.get_repository_transport(None)
 
963
        tree.commit('start', rev_id='1')
 
964
        tree.commit('more work', rev_id='2')
 
965
        tree.branch.repository.pack()
 
966
        tree.lock_read()
 
967
        self.addCleanup(tree.unlock)
 
968
        pack = tree.branch.repository._pack_collection.get_pack_by_name(
 
969
            tree.branch.repository._pack_collection.names()[0])
 
970
        # revision access tends to be tip->ancestor, so ordering that way on 
 
971
        # disk is a good idea.
 
972
        for _1, key, val, refs in pack.revision_index.iter_all_entries():
 
973
            if key == ('1',):
 
974
                pos_1 = int(val[1:].split()[0])
 
975
            else:
 
976
                pos_2 = int(val[1:].split()[0])
 
977
        self.assertTrue(pos_2 < pos_1)
 
978
 
 
979
    def test_pack_repositories_support_multiple_write_locks(self):
 
980
        format = self.get_format()
 
981
        self.make_repository('.', shared=True, format=format)
 
982
        r1 = repository.Repository.open('.')
 
983
        r2 = repository.Repository.open('.')
 
984
        r1.lock_write()
 
985
        self.addCleanup(r1.unlock)
 
986
        r2.lock_write()
 
987
        r2.unlock()
 
988
 
 
989
    def _add_text(self, repo, fileid):
 
990
        """Add a text to the repository within a write group."""
 
991
        vf =repo.weave_store.get_weave(fileid, repo.get_transaction())
 
992
        vf.add_lines('samplerev+' + fileid, [], [])
 
993
 
 
994
    def test_concurrent_writers_merge_new_packs(self):
 
995
        format = self.get_format()
 
996
        self.make_repository('.', shared=True, format=format)
 
997
        r1 = repository.Repository.open('.')
 
998
        r2 = repository.Repository.open('.')
 
999
        r1.lock_write()
 
1000
        try:
 
1001
            # access enough data to load the names list
 
1002
            list(r1.all_revision_ids())
 
1003
            r2.lock_write()
 
1004
            try:
 
1005
                # access enough data to load the names list
 
1006
                list(r2.all_revision_ids())
 
1007
                r1.start_write_group()
 
1008
                try:
 
1009
                    r2.start_write_group()
 
1010
                    try:
 
1011
                        self._add_text(r1, 'fileidr1')
 
1012
                        self._add_text(r2, 'fileidr2')
 
1013
                    except:
 
1014
                        r2.abort_write_group()
 
1015
                        raise
 
1016
                except:
 
1017
                    r1.abort_write_group()
 
1018
                    raise
 
1019
                # both r1 and r2 have open write groups with data in them
 
1020
                # created while the other's write group was open.
 
1021
                # Commit both which requires a merge to the pack-names.
 
1022
                try:
 
1023
                    r1.commit_write_group()
 
1024
                except:
 
1025
                    r1.abort_write_group()
 
1026
                    r2.abort_write_group()
 
1027
                    raise
 
1028
                r2.commit_write_group()
 
1029
                # tell r1 to reload from disk
 
1030
                r1._pack_collection.reset()
 
1031
                # Now both repositories should know about both names
 
1032
                r1._pack_collection.ensure_loaded()
 
1033
                r2._pack_collection.ensure_loaded()
 
1034
                self.assertEqual(r1._pack_collection.names(), r2._pack_collection.names())
 
1035
                self.assertEqual(2, len(r1._pack_collection.names()))
 
1036
            finally:
 
1037
                r2.unlock()
 
1038
        finally:
 
1039
            r1.unlock()
 
1040
 
 
1041
    def test_concurrent_writer_second_preserves_dropping_a_pack(self):
 
1042
        format = self.get_format()
 
1043
        self.make_repository('.', shared=True, format=format)
 
1044
        r1 = repository.Repository.open('.')
 
1045
        r2 = repository.Repository.open('.')
 
1046
        # add a pack to drop
 
1047
        r1.lock_write()
 
1048
        try:
 
1049
            r1.start_write_group()
 
1050
            try:
 
1051
                self._add_text(r1, 'fileidr1')
 
1052
            except:
 
1053
                r1.abort_write_group()
 
1054
                raise
 
1055
            else:
 
1056
                r1.commit_write_group()
 
1057
            r1._pack_collection.ensure_loaded()
 
1058
            name_to_drop = r1._pack_collection.all_packs()[0].name
 
1059
        finally:
 
1060
            r1.unlock()
 
1061
        r1.lock_write()
 
1062
        try:
 
1063
            # access enough data to load the names list
 
1064
            list(r1.all_revision_ids())
 
1065
            r2.lock_write()
 
1066
            try:
 
1067
                # access enough data to load the names list
 
1068
                list(r2.all_revision_ids())
 
1069
                r1._pack_collection.ensure_loaded()
 
1070
                try:
 
1071
                    r2.start_write_group()
 
1072
                    try:
 
1073
                        # in r1, drop the pack
 
1074
                        r1._pack_collection._remove_pack_from_memory(
 
1075
                            r1._pack_collection.get_pack_by_name(name_to_drop))
 
1076
                        # in r2, add a pack
 
1077
                        self._add_text(r2, 'fileidr2')
 
1078
                    except:
 
1079
                        r2.abort_write_group()
 
1080
                        raise
 
1081
                except:
 
1082
                    r1._pack_collection.reset()
 
1083
                    raise
 
1084
                # r1 has a changed names list, and r2 an open write groups with
 
1085
                # changes.
 
1086
                # save r1, and then commit the r2 write group, which requires a
 
1087
                # merge to the pack-names, which should not reinstate
 
1088
                # name_to_drop
 
1089
                try:
 
1090
                    r1._pack_collection._save_pack_names()
 
1091
                    r1._pack_collection.reset()
 
1092
                except:
 
1093
                    r2.abort_write_group()
 
1094
                    raise
 
1095
                try:
 
1096
                    r2.commit_write_group()
 
1097
                except:
 
1098
                    r2.abort_write_group()
 
1099
                    raise
 
1100
                # Now both repositories should now about just one name.
 
1101
                r1._pack_collection.ensure_loaded()
 
1102
                r2._pack_collection.ensure_loaded()
 
1103
                self.assertEqual(r1._pack_collection.names(), r2._pack_collection.names())
 
1104
                self.assertEqual(1, len(r1._pack_collection.names()))
 
1105
                self.assertFalse(name_to_drop in r1._pack_collection.names())
 
1106
            finally:
 
1107
                r2.unlock()
 
1108
        finally:
 
1109
            r1.unlock()
 
1110
 
 
1111
    def test_lock_write_does_not_physically_lock(self):
 
1112
        repo = self.make_repository('.', format=self.get_format())
 
1113
        repo.lock_write()
 
1114
        self.addCleanup(repo.unlock)
 
1115
        self.assertFalse(repo.get_physical_lock_status())
 
1116
 
 
1117
    def prepare_for_break_lock(self):
 
1118
        # Setup the global ui factory state so that a break-lock method call
 
1119
        # will find usable input in the input stream.
 
1120
        old_factory = bzrlib.ui.ui_factory
 
1121
        def restoreFactory():
 
1122
            bzrlib.ui.ui_factory = old_factory
 
1123
        self.addCleanup(restoreFactory)
 
1124
        bzrlib.ui.ui_factory = bzrlib.ui.SilentUIFactory()
 
1125
        bzrlib.ui.ui_factory.stdin = StringIO("y\n")
 
1126
 
 
1127
    def test_break_lock_breaks_physical_lock(self):
 
1128
        repo = self.make_repository('.', format=self.get_format())
 
1129
        repo._pack_collection.lock_names()
 
1130
        repo2 = repository.Repository.open('.')
 
1131
        self.assertTrue(repo.get_physical_lock_status())
 
1132
        self.prepare_for_break_lock()
 
1133
        repo2.break_lock()
 
1134
        self.assertFalse(repo.get_physical_lock_status())
 
1135
 
 
1136
    def test_broken_physical_locks_error_on__unlock_names_lock(self):
 
1137
        repo = self.make_repository('.', format=self.get_format())
 
1138
        repo._pack_collection.lock_names()
 
1139
        self.assertTrue(repo.get_physical_lock_status())
 
1140
        repo2 = repository.Repository.open('.')
 
1141
        self.prepare_for_break_lock()
 
1142
        repo2.break_lock()
 
1143
        self.assertRaises(errors.LockBroken, repo._pack_collection._unlock_names)
 
1144
 
 
1145
    def test_fetch_without_find_ghosts_ignores_ghosts(self):
 
1146
        # we want two repositories at this point:
 
1147
        # one with a revision that is a ghost in the other
 
1148
        # repository.
 
1149
        # 'ghost' is present in has_ghost, 'ghost' is absent in 'missing_ghost'.
 
1150
        # 'references' is present in both repositories, and 'tip' is present
 
1151
        # just in has_ghost.
 
1152
        # has_ghost       missing_ghost
 
1153
        #------------------------------
 
1154
        # 'ghost'             -
 
1155
        # 'references'    'references'
 
1156
        # 'tip'               -
 
1157
        # In this test we fetch 'tip' which should not fetch 'ghost'
 
1158
        has_ghost = self.make_repository('has_ghost', format=self.get_format())
 
1159
        missing_ghost = self.make_repository('missing_ghost',
 
1160
            format=self.get_format())
 
1161
 
 
1162
        def add_commit(repo, revision_id, parent_ids):
 
1163
            repo.lock_write()
 
1164
            repo.start_write_group()
 
1165
            inv = inventory.Inventory(revision_id=revision_id)
 
1166
            inv.root.revision = revision_id
 
1167
            root_id = inv.root.file_id
 
1168
            sha1 = repo.add_inventory(revision_id, inv, [])
 
1169
            vf = repo.weave_store.get_weave_or_empty(root_id,
 
1170
                repo.get_transaction())
 
1171
            vf.add_lines(revision_id, [], [])
 
1172
            rev = bzrlib.revision.Revision(timestamp=0,
 
1173
                                           timezone=None,
 
1174
                                           committer="Foo Bar <foo@example.com>",
 
1175
                                           message="Message",
 
1176
                                           inventory_sha1=sha1,
 
1177
                                           revision_id=revision_id)
 
1178
            rev.parent_ids = parent_ids
 
1179
            repo.add_revision(revision_id, rev)
 
1180
            repo.commit_write_group()
 
1181
            repo.unlock()
 
1182
        add_commit(has_ghost, 'ghost', [])
 
1183
        add_commit(has_ghost, 'references', ['ghost'])
 
1184
        add_commit(missing_ghost, 'references', ['ghost'])
 
1185
        add_commit(has_ghost, 'tip', ['references'])
 
1186
        missing_ghost.fetch(has_ghost, 'tip')
 
1187
        # missing ghost now has tip and not ghost.
 
1188
        rev = missing_ghost.get_revision('tip')
 
1189
        inv = missing_ghost.get_inventory('tip')
 
1190
        self.assertRaises(errors.NoSuchRevision,
 
1191
            missing_ghost.get_revision, 'ghost')
 
1192
        self.assertRaises(errors.RevisionNotPresent,
 
1193
            missing_ghost.get_inventory, 'ghost')
 
1194
 
 
1195
    def test_supports_external_lookups(self):
 
1196
        repo = self.make_repository('.', format=self.get_format())
 
1197
        self.assertFalse(repo._format.supports_external_lookups)
 
1198
 
 
1199
 
 
1200
class TestKnitPackSubtrees(TestKnitPackNoSubtrees):
 
1201
 
 
1202
    def get_format(self):
 
1203
        return bzrdir.format_registry.make_bzrdir(
 
1204
            'pack-0.92-subtree')
 
1205
 
 
1206
    def check_format(self, t):
 
1207
        self.assertEqualDiff(
 
1208
            "Bazaar pack repository format 1 with subtree support (needs bzr 0.92)\n",
 
1209
            t.get('format').read())
 
1210
 
 
1211
 
 
1212
class TestDevelopment0(TestKnitPackNoSubtrees):
 
1213
 
 
1214
    def get_format(self):
 
1215
        return bzrdir.format_registry.make_bzrdir(
 
1216
            'development')
 
1217
 
 
1218
    def check_format(self, t):
 
1219
        self.assertEqualDiff(
 
1220
            "Bazaar development format 0 (needs bzr.dev from before 1.3)\n",
 
1221
            t.get('format').read())
 
1222
 
 
1223
 
 
1224
class TestDevelopment0Subtree(TestKnitPackNoSubtrees):
 
1225
 
 
1226
    def get_format(self):
 
1227
        return bzrdir.format_registry.make_bzrdir(
 
1228
            'development-subtree')
 
1229
 
 
1230
    def check_format(self, t):
 
1231
        self.assertEqualDiff(
 
1232
            "Bazaar development format 0 with subtree support "
 
1233
            "(needs bzr.dev from before 1.3)\n",
 
1234
            t.get('format').read())
1122
1235
 
1123
1236
 
1124
1237
class TestRepositoryPackCollection(TestCaseWithTransport):
1126
1239
    def get_format(self):
1127
1240
        return bzrdir.format_registry.make_bzrdir('pack-0.92')
1128
1241
 
1129
 
    def get_packs(self):
1130
 
        format = self.get_format()
1131
 
        repo = self.make_repository('.', format=format)
1132
 
        return repo._pack_collection
1133
 
 
1134
 
    def make_packs_and_alt_repo(self, write_lock=False):
1135
 
        """Create a pack repo with 3 packs, and access it via a second repo."""
1136
 
        tree = self.make_branch_and_tree('.', format=self.get_format())
1137
 
        tree.lock_write()
1138
 
        self.addCleanup(tree.unlock)
1139
 
        rev1 = tree.commit('one')
1140
 
        rev2 = tree.commit('two')
1141
 
        rev3 = tree.commit('three')
1142
 
        r = repository.Repository.open('.')
1143
 
        if write_lock:
1144
 
            r.lock_write()
1145
 
        else:
1146
 
            r.lock_read()
1147
 
        self.addCleanup(r.unlock)
1148
 
        packs = r._pack_collection
1149
 
        packs.ensure_loaded()
1150
 
        return tree, r, packs, [rev1, rev2, rev3]
1151
 
 
1152
 
    def test__clear_obsolete_packs(self):
1153
 
        packs = self.get_packs()
1154
 
        obsolete_pack_trans = packs.transport.clone('obsolete_packs')
1155
 
        obsolete_pack_trans.put_bytes('a-pack.pack', 'content\n')
1156
 
        obsolete_pack_trans.put_bytes('a-pack.rix', 'content\n')
1157
 
        obsolete_pack_trans.put_bytes('a-pack.iix', 'content\n')
1158
 
        obsolete_pack_trans.put_bytes('another-pack.pack', 'foo\n')
1159
 
        obsolete_pack_trans.put_bytes('not-a-pack.rix', 'foo\n')
1160
 
        res = packs._clear_obsolete_packs()
1161
 
        self.assertEqual(['a-pack', 'another-pack'], sorted(res))
1162
 
        self.assertEqual([], obsolete_pack_trans.list_dir('.'))
1163
 
 
1164
 
    def test__clear_obsolete_packs_preserve(self):
1165
 
        packs = self.get_packs()
1166
 
        obsolete_pack_trans = packs.transport.clone('obsolete_packs')
1167
 
        obsolete_pack_trans.put_bytes('a-pack.pack', 'content\n')
1168
 
        obsolete_pack_trans.put_bytes('a-pack.rix', 'content\n')
1169
 
        obsolete_pack_trans.put_bytes('a-pack.iix', 'content\n')
1170
 
        obsolete_pack_trans.put_bytes('another-pack.pack', 'foo\n')
1171
 
        obsolete_pack_trans.put_bytes('not-a-pack.rix', 'foo\n')
1172
 
        res = packs._clear_obsolete_packs(preserve=set(['a-pack']))
1173
 
        self.assertEqual(['a-pack', 'another-pack'], sorted(res))
1174
 
        self.assertEqual(['a-pack.iix', 'a-pack.pack', 'a-pack.rix'],
1175
 
                         sorted(obsolete_pack_trans.list_dir('.')))
1176
 
 
1177
1242
    def test__max_pack_count(self):
1178
1243
        """The maximum pack count is a function of the number of revisions."""
 
1244
        format = self.get_format()
 
1245
        repo = self.make_repository('.', format=format)
 
1246
        packs = repo._pack_collection
1179
1247
        # no revisions - one pack, so that we can have a revision free repo
1180
1248
        # without it blowing up
1181
 
        packs = self.get_packs()
1182
1249
        self.assertEqual(1, packs._max_pack_count(0))
1183
1250
        # after that the sum of the digits, - check the first 1-9
1184
1251
        self.assertEqual(1, packs._max_pack_count(1))
1199
1266
        # check some arbitrary big numbers
1200
1267
        self.assertEqual(25, packs._max_pack_count(112894))
1201
1268
 
1202
 
    def test_repr(self):
1203
 
        packs = self.get_packs()
1204
 
        self.assertContainsRe(repr(packs),
1205
 
            'RepositoryPackCollection(.*Repository(.*))')
1206
 
 
1207
 
    def test__obsolete_packs(self):
1208
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1209
 
        names = packs.names()
1210
 
        pack = packs.get_pack_by_name(names[0])
1211
 
        # Schedule this one for removal
1212
 
        packs._remove_pack_from_memory(pack)
1213
 
        # Simulate a concurrent update by renaming the .pack file and one of
1214
 
        # the indices
1215
 
        packs.transport.rename('packs/%s.pack' % (names[0],),
1216
 
                               'obsolete_packs/%s.pack' % (names[0],))
1217
 
        packs.transport.rename('indices/%s.iix' % (names[0],),
1218
 
                               'obsolete_packs/%s.iix' % (names[0],))
1219
 
        # Now trigger the obsoletion, and ensure that all the remaining files
1220
 
        # are still renamed
1221
 
        packs._obsolete_packs([pack])
1222
 
        self.assertEqual([n + '.pack' for n in names[1:]],
1223
 
                         sorted(packs._pack_transport.list_dir('.')))
1224
 
        # names[0] should not be present in the index anymore
1225
 
        self.assertEqual(names[1:],
1226
 
            sorted(set([osutils.splitext(n)[0] for n in
1227
 
                        packs._index_transport.list_dir('.')])))
1228
 
 
1229
1269
    def test_pack_distribution_zero(self):
1230
 
        packs = self.get_packs()
 
1270
        format = self.get_format()
 
1271
        repo = self.make_repository('.', format=format)
 
1272
        packs = repo._pack_collection
1231
1273
        self.assertEqual([0], packs.pack_distribution(0))
1232
1274
 
1233
1275
    def test_ensure_loaded_unlocked(self):
1234
 
        packs = self.get_packs()
 
1276
        format = self.get_format()
 
1277
        repo = self.make_repository('.', format=format)
1235
1278
        self.assertRaises(errors.ObjectNotLocked,
1236
 
                          packs.ensure_loaded)
 
1279
                          repo._pack_collection.ensure_loaded)
1237
1280
 
1238
1281
    def test_pack_distribution_one_to_nine(self):
1239
 
        packs = self.get_packs()
 
1282
        format = self.get_format()
 
1283
        repo = self.make_repository('.', format=format)
 
1284
        packs = repo._pack_collection
1240
1285
        self.assertEqual([1],
1241
1286
            packs.pack_distribution(1))
1242
1287
        self.assertEqual([1, 1],
1258
1303
 
1259
1304
    def test_pack_distribution_stable_at_boundaries(self):
1260
1305
        """When there are multi-rev packs the counts are stable."""
1261
 
        packs = self.get_packs()
 
1306
        format = self.get_format()
 
1307
        repo = self.make_repository('.', format=format)
 
1308
        packs = repo._pack_collection
1262
1309
        # in 10s:
1263
1310
        self.assertEqual([10], packs.pack_distribution(10))
1264
1311
        self.assertEqual([10, 1], packs.pack_distribution(11))
1273
1320
        self.assertEqual([100, 100, 10, 1], packs.pack_distribution(211))
1274
1321
 
1275
1322
    def test_plan_pack_operations_2009_revisions_skip_all_packs(self):
1276
 
        packs = self.get_packs()
 
1323
        format = self.get_format()
 
1324
        repo = self.make_repository('.', format=format)
 
1325
        packs = repo._pack_collection
1277
1326
        existing_packs = [(2000, "big"), (9, "medium")]
1278
1327
        # rev count - 2009 -> 2x1000 + 9x1
1279
1328
        pack_operations = packs.plan_autopack_combinations(
1281
1330
        self.assertEqual([], pack_operations)
1282
1331
 
1283
1332
    def test_plan_pack_operations_2010_revisions_skip_all_packs(self):
1284
 
        packs = self.get_packs()
 
1333
        format = self.get_format()
 
1334
        repo = self.make_repository('.', format=format)
 
1335
        packs = repo._pack_collection
1285
1336
        existing_packs = [(2000, "big"), (9, "medium"), (1, "single")]
1286
1337
        # rev count - 2010 -> 2x1000 + 1x10
1287
1338
        pack_operations = packs.plan_autopack_combinations(
1289
1340
        self.assertEqual([], pack_operations)
1290
1341
 
1291
1342
    def test_plan_pack_operations_2010_combines_smallest_two(self):
1292
 
        packs = self.get_packs()
 
1343
        format = self.get_format()
 
1344
        repo = self.make_repository('.', format=format)
 
1345
        packs = repo._pack_collection
1293
1346
        existing_packs = [(1999, "big"), (9, "medium"), (1, "single2"),
1294
1347
            (1, "single1")]
1295
1348
        # rev count - 2010 -> 2x1000 + 1x10 (3)
1296
1349
        pack_operations = packs.plan_autopack_combinations(
1297
1350
            existing_packs, [1000, 1000, 10])
1298
 
        self.assertEqual([[2, ["single2", "single1"]]], pack_operations)
1299
 
 
1300
 
    def test_plan_pack_operations_creates_a_single_op(self):
1301
 
        packs = self.get_packs()
1302
 
        existing_packs = [(50, 'a'), (40, 'b'), (30, 'c'), (10, 'd'),
1303
 
                          (10, 'e'), (6, 'f'), (4, 'g')]
1304
 
        # rev count 150 -> 1x100 and 5x10
1305
 
        # The two size 10 packs do not need to be touched. The 50, 40, 30 would
1306
 
        # be combined into a single 120 size pack, and the 6 & 4 would
1307
 
        # becombined into a size 10 pack. However, if we have to rewrite them,
1308
 
        # we save a pack file with no increased I/O by putting them into the
1309
 
        # same file.
1310
 
        distribution = packs.pack_distribution(150)
1311
 
        pack_operations = packs.plan_autopack_combinations(existing_packs,
1312
 
                                                           distribution)
1313
 
        self.assertEqual([[130, ['a', 'b', 'c', 'f', 'g']]], pack_operations)
 
1351
        self.assertEqual([[2, ["single2", "single1"]], [0, []]], pack_operations)
1314
1352
 
1315
1353
    def test_all_packs_none(self):
1316
1354
        format = self.get_format()
1354
1392
        tree.lock_read()
1355
1393
        self.addCleanup(tree.unlock)
1356
1394
        packs = tree.branch.repository._pack_collection
1357
 
        packs.reset()
1358
1395
        packs.ensure_loaded()
1359
1396
        name = packs.names()[0]
1360
1397
        pack_1 = packs.get_pack_by_name(name)
1361
1398
        # the pack should be correctly initialised
1362
 
        sizes = packs._names[name]
1363
 
        rev_index = GraphIndex(packs._index_transport, name + '.rix', sizes[0])
1364
 
        inv_index = GraphIndex(packs._index_transport, name + '.iix', sizes[1])
1365
 
        txt_index = GraphIndex(packs._index_transport, name + '.tix', sizes[2])
1366
 
        sig_index = GraphIndex(packs._index_transport, name + '.six', sizes[3])
 
1399
        rev_index = GraphIndex(packs._index_transport, name + '.rix',
 
1400
            packs._names[name][0])
 
1401
        inv_index = GraphIndex(packs._index_transport, name + '.iix',
 
1402
            packs._names[name][1])
 
1403
        txt_index = GraphIndex(packs._index_transport, name + '.tix',
 
1404
            packs._names[name][2])
 
1405
        sig_index = GraphIndex(packs._index_transport, name + '.six',
 
1406
            packs._names[name][3])
1367
1407
        self.assertEqual(pack_repo.ExistingPack(packs._pack_transport,
1368
1408
            name, rev_index, inv_index, txt_index, sig_index), pack_1)
1369
1409
        # and the same instance should be returned on successive calls.
1370
1410
        self.assertTrue(pack_1 is packs.get_pack_by_name(name))
1371
1411
 
1372
 
    def test_reload_pack_names_new_entry(self):
1373
 
        tree, r, packs, revs = self.make_packs_and_alt_repo()
1374
 
        names = packs.names()
1375
 
        # Add a new pack file into the repository
1376
 
        rev4 = tree.commit('four')
1377
 
        new_names = tree.branch.repository._pack_collection.names()
1378
 
        new_name = set(new_names).difference(names)
1379
 
        self.assertEqual(1, len(new_name))
1380
 
        new_name = new_name.pop()
1381
 
        # The old collection hasn't noticed yet
1382
 
        self.assertEqual(names, packs.names())
1383
 
        self.assertTrue(packs.reload_pack_names())
1384
 
        self.assertEqual(new_names, packs.names())
1385
 
        # And the repository can access the new revision
1386
 
        self.assertEqual({rev4:(revs[-1],)}, r.get_parent_map([rev4]))
1387
 
        self.assertFalse(packs.reload_pack_names())
1388
 
 
1389
 
    def test_reload_pack_names_added_and_removed(self):
1390
 
        tree, r, packs, revs = self.make_packs_and_alt_repo()
1391
 
        names = packs.names()
1392
 
        # Now repack the whole thing
1393
 
        tree.branch.repository.pack()
1394
 
        new_names = tree.branch.repository._pack_collection.names()
1395
 
        # The other collection hasn't noticed yet
1396
 
        self.assertEqual(names, packs.names())
1397
 
        self.assertTrue(packs.reload_pack_names())
1398
 
        self.assertEqual(new_names, packs.names())
1399
 
        self.assertEqual({revs[-1]:(revs[-2],)}, r.get_parent_map([revs[-1]]))
1400
 
        self.assertFalse(packs.reload_pack_names())
1401
 
 
1402
 
    def test_reload_pack_names_preserves_pending(self):
1403
 
        # TODO: Update this to also test for pending-deleted names
1404
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1405
 
        # We will add one pack (via start_write_group + insert_record_stream),
1406
 
        # and remove another pack (via _remove_pack_from_memory)
1407
 
        orig_names = packs.names()
1408
 
        orig_at_load = packs._packs_at_load
1409
 
        to_remove_name = iter(orig_names).next()
1410
 
        r.start_write_group()
1411
 
        self.addCleanup(r.abort_write_group)
1412
 
        r.texts.insert_record_stream([versionedfile.FulltextContentFactory(
1413
 
            ('text', 'rev'), (), None, 'content\n')])
1414
 
        new_pack = packs._new_pack
1415
 
        self.assertTrue(new_pack.data_inserted())
1416
 
        new_pack.finish()
1417
 
        packs.allocate(new_pack)
1418
 
        packs._new_pack = None
1419
 
        removed_pack = packs.get_pack_by_name(to_remove_name)
1420
 
        packs._remove_pack_from_memory(removed_pack)
1421
 
        names = packs.names()
1422
 
        all_nodes, deleted_nodes, new_nodes, _ = packs._diff_pack_names()
1423
 
        new_names = set([x[0][0] for x in new_nodes])
1424
 
        self.assertEqual(names, sorted([x[0][0] for x in all_nodes]))
1425
 
        self.assertEqual(set(names) - set(orig_names), new_names)
1426
 
        self.assertEqual(set([new_pack.name]), new_names)
1427
 
        self.assertEqual([to_remove_name],
1428
 
                         sorted([x[0][0] for x in deleted_nodes]))
1429
 
        packs.reload_pack_names()
1430
 
        reloaded_names = packs.names()
1431
 
        self.assertEqual(orig_at_load, packs._packs_at_load)
1432
 
        self.assertEqual(names, reloaded_names)
1433
 
        all_nodes, deleted_nodes, new_nodes, _ = packs._diff_pack_names()
1434
 
        new_names = set([x[0][0] for x in new_nodes])
1435
 
        self.assertEqual(names, sorted([x[0][0] for x in all_nodes]))
1436
 
        self.assertEqual(set(names) - set(orig_names), new_names)
1437
 
        self.assertEqual(set([new_pack.name]), new_names)
1438
 
        self.assertEqual([to_remove_name],
1439
 
                         sorted([x[0][0] for x in deleted_nodes]))
1440
 
 
1441
 
    def test_autopack_obsoletes_new_pack(self):
1442
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1443
 
        packs._max_pack_count = lambda x: 1
1444
 
        packs.pack_distribution = lambda x: [10]
1445
 
        r.start_write_group()
1446
 
        r.revisions.insert_record_stream([versionedfile.FulltextContentFactory(
1447
 
            ('bogus-rev',), (), None, 'bogus-content\n')])
1448
 
        # This should trigger an autopack, which will combine everything into a
1449
 
        # single pack file.
1450
 
        new_names = r.commit_write_group()
1451
 
        names = packs.names()
1452
 
        self.assertEqual(1, len(names))
1453
 
        self.assertEqual([names[0] + '.pack'],
1454
 
                         packs._pack_transport.list_dir('.'))
1455
 
 
1456
 
    def test_autopack_reloads_and_stops(self):
1457
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1458
 
        # After we have determined what needs to be autopacked, trigger a
1459
 
        # full-pack via the other repo which will cause us to re-evaluate and
1460
 
        # decide we don't need to do anything
1461
 
        orig_execute = packs._execute_pack_operations
1462
 
        def _munged_execute_pack_ops(*args, **kwargs):
1463
 
            tree.branch.repository.pack()
1464
 
            return orig_execute(*args, **kwargs)
1465
 
        packs._execute_pack_operations = _munged_execute_pack_ops
1466
 
        packs._max_pack_count = lambda x: 1
1467
 
        packs.pack_distribution = lambda x: [10]
1468
 
        self.assertFalse(packs.autopack())
1469
 
        self.assertEqual(1, len(packs.names()))
1470
 
        self.assertEqual(tree.branch.repository._pack_collection.names(),
1471
 
                         packs.names())
1472
 
 
1473
 
    def test__save_pack_names(self):
1474
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1475
 
        names = packs.names()
1476
 
        pack = packs.get_pack_by_name(names[0])
1477
 
        packs._remove_pack_from_memory(pack)
1478
 
        packs._save_pack_names(obsolete_packs=[pack])
1479
 
        cur_packs = packs._pack_transport.list_dir('.')
1480
 
        self.assertEqual([n + '.pack' for n in names[1:]], sorted(cur_packs))
1481
 
        # obsolete_packs will also have stuff like .rix and .iix present.
1482
 
        obsolete_packs = packs.transport.list_dir('obsolete_packs')
1483
 
        obsolete_names = set([osutils.splitext(n)[0] for n in obsolete_packs])
1484
 
        self.assertEqual([pack.name], sorted(obsolete_names))
1485
 
 
1486
 
    def test__save_pack_names_already_obsoleted(self):
1487
 
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1488
 
        names = packs.names()
1489
 
        pack = packs.get_pack_by_name(names[0])
1490
 
        packs._remove_pack_from_memory(pack)
1491
 
        # We are going to simulate a concurrent autopack by manually obsoleting
1492
 
        # the pack directly.
1493
 
        packs._obsolete_packs([pack])
1494
 
        packs._save_pack_names(clear_obsolete_packs=True,
1495
 
                               obsolete_packs=[pack])
1496
 
        cur_packs = packs._pack_transport.list_dir('.')
1497
 
        self.assertEqual([n + '.pack' for n in names[1:]], sorted(cur_packs))
1498
 
        # Note that while we set clear_obsolete_packs=True, it should not
1499
 
        # delete a pack file that we have also scheduled for obsoletion.
1500
 
        obsolete_packs = packs.transport.list_dir('obsolete_packs')
1501
 
        obsolete_names = set([osutils.splitext(n)[0] for n in obsolete_packs])
1502
 
        self.assertEqual([pack.name], sorted(obsolete_names))
1503
 
 
1504
 
 
1505
1412
 
1506
1413
class TestPack(TestCaseWithTransport):
1507
1414
    """Tests for the Pack object."""
1561
1468
        pack_transport = self.get_transport('pack')
1562
1469
        index_transport = self.get_transport('index')
1563
1470
        upload_transport.mkdir('.')
1564
 
        collection = pack_repo.RepositoryPackCollection(
1565
 
            repo=None,
1566
 
            transport=self.get_transport('.'),
1567
 
            index_transport=index_transport,
1568
 
            upload_transport=upload_transport,
1569
 
            pack_transport=pack_transport,
1570
 
            index_builder_class=BTreeBuilder,
1571
 
            index_class=BTreeGraphIndex,
1572
 
            use_chk_index=False)
1573
 
        pack = pack_repo.NewPack(collection)
1574
 
        self.addCleanup(pack.abort) # Make sure the write stream gets closed
1575
 
        self.assertIsInstance(pack.revision_index, BTreeBuilder)
1576
 
        self.assertIsInstance(pack.inventory_index, BTreeBuilder)
1577
 
        self.assertIsInstance(pack._hash, type(osutils.md5()))
 
1471
        pack = pack_repo.NewPack(upload_transport, index_transport,
 
1472
            pack_transport)
 
1473
        self.assertIsInstance(pack.revision_index, InMemoryGraphIndex)
 
1474
        self.assertIsInstance(pack.inventory_index, InMemoryGraphIndex)
 
1475
        self.assertIsInstance(pack._hash, type(md5.new()))
1578
1476
        self.assertTrue(pack.upload_transport is upload_transport)
1579
1477
        self.assertTrue(pack.index_transport is index_transport)
1580
1478
        self.assertTrue(pack.pack_transport is pack_transport)
1587
1485
class TestPacker(TestCaseWithTransport):
1588
1486
    """Tests for the packs repository Packer class."""
1589
1487
 
1590
 
    def test_pack_optimizes_pack_order(self):
1591
 
        builder = self.make_branch_builder('.', format="1.9")
1592
 
        builder.start_series()
1593
 
        builder.build_snapshot('A', None, [
1594
 
            ('add', ('', 'root-id', 'directory', None)),
1595
 
            ('add', ('f', 'f-id', 'file', 'content\n'))])
1596
 
        builder.build_snapshot('B', ['A'],
1597
 
            [('modify', ('f-id', 'new-content\n'))])
1598
 
        builder.build_snapshot('C', ['B'],
1599
 
            [('modify', ('f-id', 'third-content\n'))])
1600
 
        builder.build_snapshot('D', ['C'],
1601
 
            [('modify', ('f-id', 'fourth-content\n'))])
1602
 
        b = builder.get_branch()
1603
 
        b.lock_read()
1604
 
        builder.finish_series()
1605
 
        self.addCleanup(b.unlock)
1606
 
        # At this point, we should have 4 pack files available
1607
 
        # Because of how they were built, they correspond to
1608
 
        # ['D', 'C', 'B', 'A']
1609
 
        packs = b.repository._pack_collection.packs
1610
 
        packer = pack_repo.Packer(b.repository._pack_collection,
1611
 
                                  packs, 'testing',
1612
 
                                  revision_ids=['B', 'C'])
1613
 
        # Now, when we are copying the B & C revisions, their pack files should
1614
 
        # be moved to the front of the stack
1615
 
        # The new ordering moves B & C to the front of the .packs attribute,
1616
 
        # and leaves the others in the original order.
1617
 
        new_packs = [packs[1], packs[2], packs[0], packs[3]]
1618
 
        new_pack = packer.pack()
1619
 
        self.assertEqual(new_packs, packer.packs)
1620
 
 
1621
 
 
1622
 
class TestOptimisingPacker(TestCaseWithTransport):
1623
 
    """Tests for the OptimisingPacker class."""
1624
 
 
1625
 
    def get_pack_collection(self):
1626
 
        repo = self.make_repository('.')
1627
 
        return repo._pack_collection
1628
 
 
1629
 
    def test_open_pack_will_optimise(self):
1630
 
        packer = pack_repo.OptimisingPacker(self.get_pack_collection(),
1631
 
                                            [], '.test')
1632
 
        new_pack = packer.open_pack()
1633
 
        self.addCleanup(new_pack.abort) # ensure cleanup
1634
 
        self.assertIsInstance(new_pack, pack_repo.NewPack)
1635
 
        self.assertTrue(new_pack.revision_index._optimize_for_size)
1636
 
        self.assertTrue(new_pack.inventory_index._optimize_for_size)
1637
 
        self.assertTrue(new_pack.text_index._optimize_for_size)
1638
 
        self.assertTrue(new_pack.signature_index._optimize_for_size)
1639
 
 
1640
 
 
1641
 
class TestCrossFormatPacks(TestCaseWithTransport):
1642
 
 
1643
 
    def log_pack(self, hint=None):
1644
 
        self.calls.append(('pack', hint))
1645
 
        self.orig_pack(hint=hint)
1646
 
        if self.expect_hint:
1647
 
            self.assertTrue(hint)
1648
 
 
1649
 
    def run_stream(self, src_fmt, target_fmt, expect_pack_called):
1650
 
        self.expect_hint = expect_pack_called
1651
 
        self.calls = []
1652
 
        source_tree = self.make_branch_and_tree('src', format=src_fmt)
1653
 
        source_tree.lock_write()
1654
 
        self.addCleanup(source_tree.unlock)
1655
 
        tip = source_tree.commit('foo')
1656
 
        target = self.make_repository('target', format=target_fmt)
1657
 
        target.lock_write()
1658
 
        self.addCleanup(target.unlock)
1659
 
        source = source_tree.branch.repository._get_source(target._format)
1660
 
        self.orig_pack = target.pack
1661
 
        target.pack = self.log_pack
1662
 
        search = target.search_missing_revision_ids(
1663
 
            source_tree.branch.repository, tip)
1664
 
        stream = source.get_stream(search)
1665
 
        from_format = source_tree.branch.repository._format
1666
 
        sink = target._get_sink()
1667
 
        sink.insert_stream(stream, from_format, [])
1668
 
        if expect_pack_called:
1669
 
            self.assertLength(1, self.calls)
1670
 
        else:
1671
 
            self.assertLength(0, self.calls)
1672
 
 
1673
 
    def run_fetch(self, src_fmt, target_fmt, expect_pack_called):
1674
 
        self.expect_hint = expect_pack_called
1675
 
        self.calls = []
1676
 
        source_tree = self.make_branch_and_tree('src', format=src_fmt)
1677
 
        source_tree.lock_write()
1678
 
        self.addCleanup(source_tree.unlock)
1679
 
        tip = source_tree.commit('foo')
1680
 
        target = self.make_repository('target', format=target_fmt)
1681
 
        target.lock_write()
1682
 
        self.addCleanup(target.unlock)
1683
 
        source = source_tree.branch.repository
1684
 
        self.orig_pack = target.pack
1685
 
        target.pack = self.log_pack
1686
 
        target.fetch(source)
1687
 
        if expect_pack_called:
1688
 
            self.assertLength(1, self.calls)
1689
 
        else:
1690
 
            self.assertLength(0, self.calls)
1691
 
 
1692
 
    def test_sink_format_hint_no(self):
1693
 
        # When the target format says packing makes no difference, pack is not
1694
 
        # called.
1695
 
        self.run_stream('1.9', 'rich-root-pack', False)
1696
 
 
1697
 
    def test_sink_format_hint_yes(self):
1698
 
        # When the target format says packing makes a difference, pack is
1699
 
        # called.
1700
 
        self.run_stream('1.9', '2a', True)
1701
 
 
1702
 
    def test_sink_format_same_no(self):
1703
 
        # When the formats are the same, pack is not called.
1704
 
        self.run_stream('2a', '2a', False)
1705
 
 
1706
 
    def test_IDS_format_hint_no(self):
1707
 
        # When the target format says packing makes no difference, pack is not
1708
 
        # called.
1709
 
        self.run_fetch('1.9', 'rich-root-pack', False)
1710
 
 
1711
 
    def test_IDS_format_hint_yes(self):
1712
 
        # When the target format says packing makes a difference, pack is
1713
 
        # called.
1714
 
        self.run_fetch('1.9', '2a', True)
1715
 
 
1716
 
    def test_IDS_format_same_no(self):
1717
 
        # When the formats are the same, pack is not called.
1718
 
        self.run_fetch('2a', '2a', False)
 
1488
    # To date, this class has been factored out and nothing new added to it;
 
1489
    # thus there are not yet any tests.
 
1490
 
 
1491
 
 
1492
class TestInterDifferingSerializer(TestCaseWithTransport):
 
1493
 
 
1494
    def test_progress_bar(self):
 
1495
        tree = self.make_branch_and_tree('tree')
 
1496
        tree.commit('rev1', rev_id='rev-1')
 
1497
        tree.commit('rev2', rev_id='rev-2')
 
1498
        tree.commit('rev3', rev_id='rev-3')
 
1499
        repo = self.make_repository('repo')
 
1500
        inter_repo = repository.InterDifferingSerializer(
 
1501
            tree.branch.repository, repo)
 
1502
        pb = progress.InstrumentedProgress(to_file=StringIO())
 
1503
        pb.never_throttle = True
 
1504
        inter_repo.fetch('rev-1', pb)
 
1505
        self.assertEqual('Transferring revisions', pb.last_msg)
 
1506
        self.assertEqual(1, pb.last_cnt)
 
1507
        self.assertEqual(1, pb.last_total)
 
1508
        inter_repo.fetch('rev-3', pb)
 
1509
        self.assertEqual(2, pb.last_cnt)
 
1510
        self.assertEqual(2, pb.last_total)