/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
4110.2.26 by Martin Pool
Remove outdated progress bar test
1
# Copyright (C) 2006, 2007, 2008, 2009 Canonical Ltd
1685.1.63 by Martin Pool
Small Transport fixups
2
#
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1685.1.63 by Martin Pool
Small Transport fixups
7
#
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1685.1.63 by Martin Pool
Small Transport fixups
12
#
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
16
17
"""Tests for the Repository facility that are not interface tests.
18
3689.1.4 by John Arbash Meinel
Doc strings that reference repository_implementations
19
For interface tests see tests/per_repository/*.py.
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
20
21
For concrete class tests see this file, and for storage formats tests
22
also see this file.
23
"""
24
1773.4.1 by Martin Pool
Add pyflakes makefile target; fix many warnings
25
from stat import S_ISDIR
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
26
from StringIO import StringIO
27
1556.1.4 by Robert Collins
Add a new format for what will become knit, and the surrounding logic to upgrade repositories within metadirs, and tests for the same.
28
import bzrlib
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
29
from bzrlib.errors import (NotBranchError,
1534.4.41 by Robert Collins
Branch now uses BzrDir reasonably sanely.
30
                           NoSuchFile,
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
31
                           UnknownFormatError,
32
                           UnsupportedFormatError,
33
                           )
4360.4.3 by John Arbash Meinel
Introduce a KnitPackStreamSource which is used when
34
from bzrlib import (
35
    graph,
36
    tests,
37
    )
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
38
from bzrlib.branchbuilder import BranchBuilder
3735.1.1 by Robert Collins
Add development2 formats using BTree indices.
39
from bzrlib.btree_index import BTreeBuilder, BTreeGraphIndex
2592.3.192 by Robert Collins
Move new revision index management to NewPack.
40
from bzrlib.index import GraphIndex, InMemoryGraphIndex
2241.1.1 by Martin Pool
Change RepositoryFormat to use a Registry rather than ad-hoc dictionary
41
from bzrlib.repository import RepositoryFormat
2535.3.41 by Andrew Bennetts
Add tests for InterRemoteToOther.is_compatible.
42
from bzrlib.smart import server
2670.3.5 by Andrew Bennetts
Remove get_stream_as_bytes from KnitVersionedFile's API, make it a function in knitrepo.py instead.
43
from bzrlib.tests import (
44
    TestCase,
45
    TestCaseWithTransport,
3446.2.1 by Martin Pool
Failure to delete an obsolete pack file should not be fatal.
46
    TestSkipped,
2670.3.5 by Andrew Bennetts
Remove get_stream_as_bytes from KnitVersionedFile's API, make it a function in knitrepo.py instead.
47
    test_knit,
48
    )
3446.2.1 by Martin Pool
Failure to delete an obsolete pack file should not be fatal.
49
from bzrlib.transport import (
50
    fakenfs,
51
    get_transport,
52
    )
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
53
from bzrlib.transport.memory import MemoryServer
2241.1.1 by Martin Pool
Change RepositoryFormat to use a Registry rather than ad-hoc dictionary
54
from bzrlib import (
2694.5.4 by Jelmer Vernooij
Move bzrlib.util.bencode to bzrlib._bencode_py.
55
    bencode,
2535.3.41 by Andrew Bennetts
Add tests for InterRemoteToOther.is_compatible.
56
    bzrdir,
57
    errors,
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
58
    inventory,
2929.3.5 by Vincent Ladeuil
New files, same warnings, same fixes.
59
    osutils,
3146.6.1 by Aaron Bentley
InterDifferingSerializer shows a progress bar
60
    progress,
2241.1.1 by Martin Pool
Change RepositoryFormat to use a Registry rather than ad-hoc dictionary
61
    repository,
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
62
    revision as _mod_revision,
2535.3.41 by Andrew Bennetts
Add tests for InterRemoteToOther.is_compatible.
63
    symbol_versioning,
2241.1.1 by Martin Pool
Change RepositoryFormat to use a Registry rather than ad-hoc dictionary
64
    upgrade,
65
    workingtree,
66
    )
3735.42.5 by John Arbash Meinel
Change the tests so we now just use a direct test that _get_source is
67
from bzrlib.repofmt import (
68
    groupcompress_repo,
69
    knitrepo,
70
    pack_repo,
71
    weaverepo,
72
    )
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
73
74
75
class TestDefaultFormat(TestCase):
76
1534.4.41 by Robert Collins
Branch now uses BzrDir reasonably sanely.
77
    def test_get_set_default_format(self):
2204.5.3 by Aaron Bentley
zap old repository default handling
78
        old_default = bzrdir.format_registry.get('default')
79
        private_default = old_default().repository_format.__class__
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
80
        old_format = repository.RepositoryFormat.get_default_format()
1910.2.33 by Aaron Bentley
Fix default format test
81
        self.assertTrue(isinstance(old_format, private_default))
2204.5.3 by Aaron Bentley
zap old repository default handling
82
        def make_sample_bzrdir():
83
            my_bzrdir = bzrdir.BzrDirMetaFormat1()
84
            my_bzrdir.repository_format = SampleRepositoryFormat()
85
            return my_bzrdir
86
        bzrdir.format_registry.remove('default')
87
        bzrdir.format_registry.register('sample', make_sample_bzrdir, '')
88
        bzrdir.format_registry.set_default('sample')
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
89
        # creating a repository should now create an instrumented dir.
90
        try:
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
91
            # the default branch format is used by the meta dir format
92
            # which is not the default bzrdir format at this point
1685.1.63 by Martin Pool
Small Transport fixups
93
            dir = bzrdir.BzrDirMetaFormat1().initialize('memory:///')
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
94
            result = dir.create_repository()
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
95
            self.assertEqual(result, 'A bzr repository dir')
2241.1.1 by Martin Pool
Change RepositoryFormat to use a Registry rather than ad-hoc dictionary
96
        finally:
2204.5.3 by Aaron Bentley
zap old repository default handling
97
            bzrdir.format_registry.remove('default')
2363.5.14 by Aaron Bentley
Prevent repository.get_set_default_format from corrupting inventory
98
            bzrdir.format_registry.remove('sample')
2204.5.3 by Aaron Bentley
zap old repository default handling
99
            bzrdir.format_registry.register('default', old_default, '')
100
        self.assertIsInstance(repository.RepositoryFormat.get_default_format(),
101
                              old_format.__class__)
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
102
103
104
class SampleRepositoryFormat(repository.RepositoryFormat):
105
    """A sample format
106
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
107
    this format is initializable, unsupported to aid in testing the
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
108
    open and open(unsupported=True) routines.
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
109
    """
110
111
    def get_format_string(self):
112
        """See RepositoryFormat.get_format_string()."""
113
        return "Sample .bzr repository format."
114
1534.6.1 by Robert Collins
allow API creation of shared repositories
115
    def initialize(self, a_bzrdir, shared=False):
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
116
        """Initialize a repository in a BzrDir"""
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
117
        t = a_bzrdir.get_repository_transport(self)
1955.3.13 by John Arbash Meinel
Run the full test suite, and fix up any deprecation warnings.
118
        t.put_bytes('format', self.get_format_string())
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
119
        return 'A bzr repository dir'
120
121
    def is_supported(self):
122
        return False
123
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
124
    def open(self, a_bzrdir, _found=False):
1534.4.40 by Robert Collins
Add RepositoryFormats and allow bzrdir.open or create _repository to be used.
125
        return "opened repository."
1534.4.41 by Robert Collins
Branch now uses BzrDir reasonably sanely.
126
127
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
128
class TestRepositoryFormat(TestCaseWithTransport):
129
    """Tests for the Repository format detection used by the bzr meta dir facility.BzrBranchFormat facility."""
130
131
    def test_find_format(self):
132
        # is the right format object found for a repository?
133
        # create a branch with a few known format objects.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
134
        # this is not quite the same as
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
135
        self.build_tree(["foo/", "bar/"])
136
        def check_format(format, url):
137
            dir = format._matchingbzrdir.initialize(url)
138
            format.initialize(dir)
139
            t = get_transport(url)
140
            found_format = repository.RepositoryFormat.find_format(dir)
141
            self.failUnless(isinstance(found_format, format.__class__))
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
142
        check_format(weaverepo.RepositoryFormat7(), "bar")
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
143
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
144
    def test_find_format_no_repository(self):
145
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
146
        self.assertRaises(errors.NoRepositoryPresent,
147
                          repository.RepositoryFormat.find_format,
148
                          dir)
149
150
    def test_find_format_unknown_format(self):
151
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
152
        SampleRepositoryFormat().initialize(dir)
153
        self.assertRaises(UnknownFormatError,
154
                          repository.RepositoryFormat.find_format,
155
                          dir)
156
157
    def test_register_unregister_format(self):
158
        format = SampleRepositoryFormat()
159
        # make a control dir
160
        dir = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
161
        # make a repo
162
        format.initialize(dir)
163
        # register a format for it.
164
        repository.RepositoryFormat.register_format(format)
165
        # which repository.Open will refuse (not supported)
166
        self.assertRaises(UnsupportedFormatError, repository.Repository.open, self.get_url())
167
        # but open(unsupported) will work
168
        self.assertEqual(format.open(dir), "opened repository.")
169
        # unregister the format
170
        repository.RepositoryFormat.unregister_format(format)
171
172
1534.4.41 by Robert Collins
Branch now uses BzrDir reasonably sanely.
173
class TestFormat6(TestCaseWithTransport):
174
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
175
    def test_attribute__fetch_order(self):
176
        """Weaves need topological data insertion."""
177
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
178
        repo = weaverepo.RepositoryFormat6().initialize(control)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
179
        self.assertEqual('topological', repo._format._fetch_order)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
180
181
    def test_attribute__fetch_uses_deltas(self):
182
        """Weaves do not reuse deltas."""
183
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
184
        repo = weaverepo.RepositoryFormat6().initialize(control)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
185
        self.assertEqual(False, repo._format._fetch_uses_deltas)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
186
3565.3.4 by Robert Collins
Defer decision to reconcile to the repository being fetched into.
187
    def test_attribute__fetch_reconcile(self):
188
        """Weave repositories need a reconcile after fetch."""
189
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
190
        repo = weaverepo.RepositoryFormat6().initialize(control)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
191
        self.assertEqual(True, repo._format._fetch_reconcile)
3565.3.4 by Robert Collins
Defer decision to reconcile to the repository being fetched into.
192
1534.4.41 by Robert Collins
Branch now uses BzrDir reasonably sanely.
193
    def test_no_ancestry_weave(self):
194
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
195
        repo = weaverepo.RepositoryFormat6().initialize(control)
1534.4.41 by Robert Collins
Branch now uses BzrDir reasonably sanely.
196
        # We no longer need to create the ancestry.weave file
197
        # since it is *never* used.
198
        self.assertRaises(NoSuchFile,
199
                          control.transport.get,
200
                          'ancestry.weave')
201
3221.3.1 by Robert Collins
* Repository formats have a new supported-feature attribute
202
    def test_supports_external_lookups(self):
203
        control = bzrdir.BzrDirFormat6().initialize(self.get_url())
204
        repo = weaverepo.RepositoryFormat6().initialize(control)
205
        self.assertFalse(repo._format.supports_external_lookups)
206
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
207
208
class TestFormat7(TestCaseWithTransport):
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
209
210
    def test_attribute__fetch_order(self):
211
        """Weaves need topological data insertion."""
212
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
213
        repo = weaverepo.RepositoryFormat7().initialize(control)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
214
        self.assertEqual('topological', repo._format._fetch_order)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
215
216
    def test_attribute__fetch_uses_deltas(self):
217
        """Weaves do not reuse deltas."""
218
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
219
        repo = weaverepo.RepositoryFormat7().initialize(control)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
220
        self.assertEqual(False, repo._format._fetch_uses_deltas)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
221
3565.3.4 by Robert Collins
Defer decision to reconcile to the repository being fetched into.
222
    def test_attribute__fetch_reconcile(self):
223
        """Weave repositories need a reconcile after fetch."""
224
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
225
        repo = weaverepo.RepositoryFormat7().initialize(control)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
226
        self.assertEqual(True, repo._format._fetch_reconcile)
3565.3.4 by Robert Collins
Defer decision to reconcile to the repository being fetched into.
227
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
228
    def test_disk_layout(self):
229
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
230
        repo = weaverepo.RepositoryFormat7().initialize(control)
1534.5.3 by Robert Collins
Make format 4/5/6 branches share a single LockableFiles instance across wt/branch/repository.
231
        # in case of side effects of locking.
232
        repo.lock_write()
233
        repo.unlock()
1534.4.47 by Robert Collins
Split out repository into .bzr/repository
234
        # we want:
235
        # format 'Bazaar-NG Repository format 7'
236
        # lock ''
237
        # inventory.weave == empty_weave
238
        # empty revision-store directory
239
        # empty weaves directory
240
        t = control.get_repository_transport(None)
241
        self.assertEqualDiff('Bazaar-NG Repository format 7',
242
                             t.get('format').read())
243
        self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
244
        self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
245
        self.assertEqualDiff('# bzr weave file v5\n'
246
                             'w\n'
247
                             'W\n',
248
                             t.get('inventory.weave').read())
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
249
        # Creating a file with id Foo:Bar results in a non-escaped file name on
250
        # disk.
251
        control.create_branch()
252
        tree = control.create_workingtree()
253
        tree.add(['foo'], ['Foo:Bar'], ['file'])
254
        tree.put_file_bytes_non_atomic('Foo:Bar', 'content\n')
255
        tree.commit('first post', rev_id='first')
256
        self.assertEqualDiff(
257
            '# bzr weave file v5\n'
258
            'i\n'
259
            '1 7fe70820e08a1aac0ef224d9c66ab66831cc4ab1\n'
260
            'n first\n'
261
            '\n'
262
            'w\n'
263
            '{ 0\n'
264
            '. content\n'
265
            '}\n'
266
            'W\n',
267
            t.get('weaves/74/Foo%3ABar.weave').read())
1534.6.1 by Robert Collins
allow API creation of shared repositories
268
269
    def test_shared_disk_layout(self):
270
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
271
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
1534.6.1 by Robert Collins
allow API creation of shared repositories
272
        # we want:
273
        # format 'Bazaar-NG Repository format 7'
274
        # inventory.weave == empty_weave
275
        # empty revision-store directory
276
        # empty weaves directory
277
        # a 'shared-storage' marker file.
1553.5.49 by Martin Pool
Use LockDirs for repo format 7
278
        # lock is not present when unlocked
1534.6.1 by Robert Collins
allow API creation of shared repositories
279
        t = control.get_repository_transport(None)
280
        self.assertEqualDiff('Bazaar-NG Repository format 7',
281
                             t.get('format').read())
282
        self.assertEqualDiff('', t.get('shared-storage').read())
283
        self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
284
        self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
285
        self.assertEqualDiff('# bzr weave file v5\n'
286
                             'w\n'
287
                             'W\n',
288
                             t.get('inventory.weave').read())
1553.5.49 by Martin Pool
Use LockDirs for repo format 7
289
        self.assertFalse(t.has('branch-lock'))
290
1553.5.56 by Martin Pool
Format 7 repo now uses LockDir!
291
    def test_creates_lockdir(self):
1553.5.49 by Martin Pool
Use LockDirs for repo format 7
292
        """Make sure it appears to be controlled by a LockDir existence"""
293
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
294
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
1553.5.49 by Martin Pool
Use LockDirs for repo format 7
295
        t = control.get_repository_transport(None)
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
296
        # TODO: Should check there is a 'lock' toplevel directory,
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
297
        # regardless of contents
298
        self.assertFalse(t.has('lock/held/info'))
1553.5.49 by Martin Pool
Use LockDirs for repo format 7
299
        repo.lock_write()
1658.1.4 by Martin Pool
Quieten warning from TestFormat7.test_creates_lockdir about failing to unlock
300
        try:
301
            self.assertTrue(t.has('lock/held/info'))
302
        finally:
303
            # unlock so we don't get a warning about failing to do so
304
            repo.unlock()
1553.5.56 by Martin Pool
Format 7 repo now uses LockDir!
305
306
    def test_uses_lockdir(self):
307
        """repo format 7 actually locks on lockdir"""
308
        base_url = self.get_url()
309
        control = bzrdir.BzrDirMetaFormat1().initialize(base_url)
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
310
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
1553.5.56 by Martin Pool
Format 7 repo now uses LockDir!
311
        t = control.get_repository_transport(None)
312
        repo.lock_write()
313
        repo.unlock()
314
        del repo
315
        # make sure the same lock is created by opening it
316
        repo = repository.Repository.open(base_url)
317
        repo.lock_write()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
318
        self.assertTrue(t.has('lock/held/info'))
1553.5.56 by Martin Pool
Format 7 repo now uses LockDir!
319
        repo.unlock()
1553.5.58 by Martin Pool
Change LockDirs to format "lock-name/held/info"
320
        self.assertFalse(t.has('lock/held/info'))
1534.6.5 by Robert Collins
Cloning of repos preserves shared and make-working-tree attributes.
321
322
    def test_shared_no_tree_disk_layout(self):
323
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
324
        repo = weaverepo.RepositoryFormat7().initialize(control, shared=True)
1534.6.5 by Robert Collins
Cloning of repos preserves shared and make-working-tree attributes.
325
        repo.set_make_working_trees(False)
326
        # we want:
327
        # format 'Bazaar-NG Repository format 7'
328
        # lock ''
329
        # inventory.weave == empty_weave
330
        # empty revision-store directory
331
        # empty weaves directory
332
        # a 'shared-storage' marker file.
333
        t = control.get_repository_transport(None)
334
        self.assertEqualDiff('Bazaar-NG Repository format 7',
335
                             t.get('format').read())
1553.5.56 by Martin Pool
Format 7 repo now uses LockDir!
336
        ## self.assertEqualDiff('', t.get('lock').read())
1534.6.5 by Robert Collins
Cloning of repos preserves shared and make-working-tree attributes.
337
        self.assertEqualDiff('', t.get('shared-storage').read())
338
        self.assertEqualDiff('', t.get('no-working-trees').read())
339
        repo.set_make_working_trees(True)
340
        self.assertFalse(t.has('no-working-trees'))
341
        self.assertTrue(S_ISDIR(t.stat('revision-store').st_mode))
342
        self.assertTrue(S_ISDIR(t.stat('weaves').st_mode))
343
        self.assertEqualDiff('# bzr weave file v5\n'
344
                             'w\n'
345
                             'W\n',
346
                             t.get('inventory.weave').read())
1534.1.27 by Robert Collins
Start InterRepository with InterRepository.get.
347
3221.3.1 by Robert Collins
* Repository formats have a new supported-feature attribute
348
    def test_supports_external_lookups(self):
349
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
350
        repo = weaverepo.RepositoryFormat7().initialize(control)
351
        self.assertFalse(repo._format.supports_external_lookups)
352
1534.1.27 by Robert Collins
Start InterRepository with InterRepository.get.
353
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
354
class TestFormatKnit1(TestCaseWithTransport):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
355
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
356
    def test_attribute__fetch_order(self):
357
        """Knits need topological data insertion."""
358
        repo = self.make_repository('.',
359
                format=bzrdir.format_registry.get('knit')())
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
360
        self.assertEqual('topological', repo._format._fetch_order)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
361
362
    def test_attribute__fetch_uses_deltas(self):
363
        """Knits reuse deltas."""
364
        repo = self.make_repository('.',
365
                format=bzrdir.format_registry.get('knit')())
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
366
        self.assertEqual(True, repo._format._fetch_uses_deltas)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
367
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
368
    def test_disk_layout(self):
369
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.6 by Martin Pool
Move Knit repositories into the submodule bzrlib.repofmt.knitrepo and
370
        repo = knitrepo.RepositoryFormatKnit1().initialize(control)
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
371
        # in case of side effects of locking.
372
        repo.lock_write()
373
        repo.unlock()
374
        # we want:
375
        # format 'Bazaar-NG Knit Repository Format 1'
1553.5.62 by Martin Pool
Add tests that MetaDir repositories use LockDirs
376
        # lock: is a directory
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
377
        # inventory.weave == empty_weave
378
        # empty revision-store directory
379
        # empty weaves directory
380
        t = control.get_repository_transport(None)
381
        self.assertEqualDiff('Bazaar-NG Knit Repository Format 1',
382
                             t.get('format').read())
1553.5.57 by Martin Pool
[merge] sync from bzr.dev
383
        # XXX: no locks left when unlocked at the moment
384
        # self.assertEqualDiff('', t.get('lock').read())
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
385
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
1563.2.35 by Robert Collins
cleanup deprecation warnings and finish conversion so the inventory is knit based too.
386
        self.check_knits(t)
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
387
        # Check per-file knits.
388
        branch = control.create_branch()
389
        tree = control.create_workingtree()
390
        tree.add(['foo'], ['Nasty-IdC:'], ['file'])
391
        tree.put_file_bytes_non_atomic('Nasty-IdC:', '')
392
        tree.commit('1st post', rev_id='foo')
393
        self.assertHasKnit(t, 'knits/e8/%254easty-%2549d%2543%253a',
394
            '\nfoo fulltext 0 81  :')
1563.2.35 by Robert Collins
cleanup deprecation warnings and finish conversion so the inventory is knit based too.
395
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
396
    def assertHasKnit(self, t, knit_name, extra_content=''):
1654.1.3 by Robert Collins
Refactor repository knit tests slightly to remove duplication - add a assertHasKnit method.
397
        """Assert that knit_name exists on t."""
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
398
        self.assertEqualDiff('# bzr knit index 8\n' + extra_content,
1654.1.3 by Robert Collins
Refactor repository knit tests slightly to remove duplication - add a assertHasKnit method.
399
                             t.get(knit_name + '.kndx').read())
400
1563.2.35 by Robert Collins
cleanup deprecation warnings and finish conversion so the inventory is knit based too.
401
    def check_knits(self, t):
402
        """check knit content for a repository."""
1654.1.3 by Robert Collins
Refactor repository knit tests slightly to remove duplication - add a assertHasKnit method.
403
        self.assertHasKnit(t, 'inventory')
404
        self.assertHasKnit(t, 'revisions')
405
        self.assertHasKnit(t, 'signatures')
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
406
407
    def test_shared_disk_layout(self):
408
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.6 by Martin Pool
Move Knit repositories into the submodule bzrlib.repofmt.knitrepo and
409
        repo = knitrepo.RepositoryFormatKnit1().initialize(control, shared=True)
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
410
        # we want:
411
        # format 'Bazaar-NG Knit Repository Format 1'
1553.5.62 by Martin Pool
Add tests that MetaDir repositories use LockDirs
412
        # lock: is a directory
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
413
        # inventory.weave == empty_weave
414
        # empty revision-store directory
415
        # empty weaves directory
416
        # a 'shared-storage' marker file.
417
        t = control.get_repository_transport(None)
418
        self.assertEqualDiff('Bazaar-NG Knit Repository Format 1',
419
                             t.get('format').read())
1553.5.57 by Martin Pool
[merge] sync from bzr.dev
420
        # XXX: no locks left when unlocked at the moment
421
        # self.assertEqualDiff('', t.get('lock').read())
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
422
        self.assertEqualDiff('', t.get('shared-storage').read())
423
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
1563.2.35 by Robert Collins
cleanup deprecation warnings and finish conversion so the inventory is knit based too.
424
        self.check_knits(t)
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
425
426
    def test_shared_no_tree_disk_layout(self):
427
        control = bzrdir.BzrDirMetaFormat1().initialize(self.get_url())
2241.1.6 by Martin Pool
Move Knit repositories into the submodule bzrlib.repofmt.knitrepo and
428
        repo = knitrepo.RepositoryFormatKnit1().initialize(control, shared=True)
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
429
        repo.set_make_working_trees(False)
430
        # we want:
431
        # format 'Bazaar-NG Knit Repository Format 1'
432
        # lock ''
433
        # inventory.weave == empty_weave
434
        # empty revision-store directory
435
        # empty weaves directory
436
        # a 'shared-storage' marker file.
437
        t = control.get_repository_transport(None)
438
        self.assertEqualDiff('Bazaar-NG Knit Repository Format 1',
439
                             t.get('format').read())
1553.5.57 by Martin Pool
[merge] sync from bzr.dev
440
        # XXX: no locks left when unlocked at the moment
441
        # self.assertEqualDiff('', t.get('lock').read())
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
442
        self.assertEqualDiff('', t.get('shared-storage').read())
443
        self.assertEqualDiff('', t.get('no-working-trees').read())
444
        repo.set_make_working_trees(True)
445
        self.assertFalse(t.has('no-working-trees'))
446
        self.assertTrue(S_ISDIR(t.stat('knits').st_mode))
1563.2.35 by Robert Collins
cleanup deprecation warnings and finish conversion so the inventory is knit based too.
447
        self.check_knits(t)
1556.1.3 by Robert Collins
Rearrangment of Repository logic to be less type code driven, and bugfix InterRepository.missing_revision_ids
448
2917.2.1 by John Arbash Meinel
Fix bug #152360. The xml5 serializer should be using
449
    def test_deserialise_sets_root_revision(self):
450
        """We must have a inventory.root.revision
451
452
        Old versions of the XML5 serializer did not set the revision_id for
453
        the whole inventory. So we grab the one from the expected text. Which
454
        is valid when the api is not being abused.
455
        """
456
        repo = self.make_repository('.',
457
                format=bzrdir.format_registry.get('knit')())
458
        inv_xml = '<inventory format="5">\n</inventory>\n'
459
        inv = repo.deserialise_inventory('test-rev-id', inv_xml)
460
        self.assertEqual('test-rev-id', inv.root.revision)
461
462
    def test_deserialise_uses_global_revision_id(self):
463
        """If it is set, then we re-use the global revision id"""
464
        repo = self.make_repository('.',
465
                format=bzrdir.format_registry.get('knit')())
466
        inv_xml = ('<inventory format="5" revision_id="other-rev-id">\n'
467
                   '</inventory>\n')
468
        # Arguably, the deserialise_inventory should detect a mismatch, and
469
        # raise an error, rather than silently using one revision_id over the
470
        # other.
3169.2.2 by Robert Collins
Add a test to Repository.deserialise_inventory that the resulting ivnentory is the one asked for, and update relevant tests. Also tweak the model 1 to 2 regenerate inventories logic to use the revision trees parent marker which is more accurate in some cases.
471
        self.assertRaises(AssertionError, repo.deserialise_inventory,
472
            'test-rev-id', inv_xml)
473
        inv = repo.deserialise_inventory('other-rev-id', inv_xml)
2917.2.1 by John Arbash Meinel
Fix bug #152360. The xml5 serializer should be using
474
        self.assertEqual('other-rev-id', inv.root.revision)
475
3221.3.1 by Robert Collins
* Repository formats have a new supported-feature attribute
476
    def test_supports_external_lookups(self):
477
        repo = self.make_repository('.',
478
                format=bzrdir.format_registry.get('knit')())
479
        self.assertFalse(repo._format.supports_external_lookups)
480
2535.3.53 by Andrew Bennetts
Remove get_stream_as_bytes from KnitVersionedFile's API, make it a function in knitrepo.py instead.
481
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
482
class DummyRepository(object):
483
    """A dummy repository for testing."""
484
3452.2.11 by Andrew Bennetts
Merge thread.
485
    _format = None
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
486
    _serializer = None
487
488
    def supports_rich_root(self):
489
        return False
490
3709.5.10 by Andrew Bennetts
Fix test failure caused by missing attributes on DummyRepository.
491
    def get_graph(self):
492
        raise NotImplementedError
493
494
    def get_parent_map(self, revision_ids):
495
        raise NotImplementedError
496
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
497
498
class InterDummy(repository.InterRepository):
499
    """An inter-repository optimised code path for DummyRepository.
500
501
    This is for use during testing where we use DummyRepository as repositories
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
502
    so that none of the default regsitered inter-repository classes will
2818.4.2 by Robert Collins
Review feedback.
503
    MATCH.
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
504
    """
505
506
    @staticmethod
507
    def is_compatible(repo_source, repo_target):
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
508
        """InterDummy is compatible with DummyRepository."""
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
509
        return (isinstance(repo_source, DummyRepository) and
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
510
            isinstance(repo_target, DummyRepository))
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
511
512
1534.1.27 by Robert Collins
Start InterRepository with InterRepository.get.
513
class TestInterRepository(TestCaseWithTransport):
514
515
    def test_get_default_inter_repository(self):
516
        # test that the InterRepository.get(repo_a, repo_b) probes
517
        # for a inter_repo class where is_compatible(repo_a, repo_b) returns
518
        # true and returns a default inter_repo otherwise.
519
        # This also tests that the default registered optimised interrepository
520
        # classes do not barf inappropriately when a surprising repository type
521
        # is handed to them.
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
522
        dummy_a = DummyRepository()
523
        dummy_b = DummyRepository()
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
524
        self.assertGetsDefaultInterRepository(dummy_a, dummy_b)
525
526
    def assertGetsDefaultInterRepository(self, repo_a, repo_b):
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
527
        """Asserts that InterRepository.get(repo_a, repo_b) -> the default.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
528
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
529
        The effective default is now InterSameDataRepository because there is
530
        no actual sane default in the presence of incompatible data models.
531
        """
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
532
        inter_repo = repository.InterRepository.get(repo_a, repo_b)
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
533
        self.assertEqual(repository.InterSameDataRepository,
1534.1.27 by Robert Collins
Start InterRepository with InterRepository.get.
534
                         inter_repo.__class__)
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
535
        self.assertEqual(repo_a, inter_repo.source)
536
        self.assertEqual(repo_b, inter_repo.target)
537
538
    def test_register_inter_repository_class(self):
539
        # test that a optimised code path provider - a
540
        # InterRepository subclass can be registered and unregistered
541
        # and that it is correctly selected when given a repository
542
        # pair that it returns true on for the is_compatible static method
543
        # check
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
544
        dummy_a = DummyRepository()
545
        dummy_b = DummyRepository()
546
        repo = self.make_repository('.')
547
        # hack dummies to look like repo somewhat.
548
        dummy_a._serializer = repo._serializer
549
        dummy_b._serializer = repo._serializer
550
        repository.InterRepository.register_optimiser(InterDummy)
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
551
        try:
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
552
            # we should get the default for something InterDummy returns False
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
553
            # to
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
554
            self.assertFalse(InterDummy.is_compatible(dummy_a, repo))
555
            self.assertGetsDefaultInterRepository(dummy_a, repo)
556
            # and we should get an InterDummy for a pair it 'likes'
557
            self.assertTrue(InterDummy.is_compatible(dummy_a, dummy_b))
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
558
            inter_repo = repository.InterRepository.get(dummy_a, dummy_b)
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
559
            self.assertEqual(InterDummy, inter_repo.__class__)
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
560
            self.assertEqual(dummy_a, inter_repo.source)
561
            self.assertEqual(dummy_b, inter_repo.target)
562
        finally:
2305.2.3 by Andrew Bennetts
Bring across test_repository improvements from the hpss branch to fix the last test failures.
563
            repository.InterRepository.unregister_optimiser(InterDummy)
1534.1.28 by Robert Collins
Allow for optimised InterRepository selection.
564
        # now we should get the default InterRepository object again.
565
        self.assertGetsDefaultInterRepository(dummy_a, dummy_b)
1534.1.33 by Robert Collins
Move copy_content_into into InterRepository and InterWeaveRepo, and disable the default codepath test as we have optimised paths for all current combinations.
566
2241.1.17 by Martin Pool
Restore old InterWeave tests
567
568
class TestInterWeaveRepo(TestCaseWithTransport):
569
570
    def test_is_compatible_and_registered(self):
571
        # InterWeaveRepo is compatible when either side
572
        # is a format 5/6/7 branch
2241.1.20 by mbp at sourcefrog
update tests for new locations of weave repos
573
        from bzrlib.repofmt import knitrepo, weaverepo
574
        formats = [weaverepo.RepositoryFormat5(),
575
                   weaverepo.RepositoryFormat6(),
576
                   weaverepo.RepositoryFormat7()]
577
        incompatible_formats = [weaverepo.RepositoryFormat4(),
578
                                knitrepo.RepositoryFormatKnit1(),
2241.1.17 by Martin Pool
Restore old InterWeave tests
579
                                ]
580
        repo_a = self.make_repository('a')
581
        repo_b = self.make_repository('b')
582
        is_compatible = repository.InterWeaveRepo.is_compatible
583
        for source in incompatible_formats:
584
            # force incompatible left then right
585
            repo_a._format = source
586
            repo_b._format = formats[0]
587
            self.assertFalse(is_compatible(repo_a, repo_b))
588
            self.assertFalse(is_compatible(repo_b, repo_a))
589
        for source in formats:
590
            repo_a._format = source
591
            for target in formats:
592
                repo_b._format = target
593
                self.assertTrue(is_compatible(repo_a, repo_b))
594
        self.assertEqual(repository.InterWeaveRepo,
595
                         repository.InterRepository.get(repo_a,
596
                                                        repo_b).__class__)
597
1556.1.4 by Robert Collins
Add a new format for what will become knit, and the surrounding logic to upgrade repositories within metadirs, and tests for the same.
598
599
class TestRepositoryConverter(TestCaseWithTransport):
600
601
    def test_convert_empty(self):
602
        t = get_transport(self.get_url('.'))
603
        t.mkdir('repository')
604
        repo_dir = bzrdir.BzrDirMetaFormat1().initialize('repository')
2241.1.4 by Martin Pool
Moved old weave-based repository formats into bzrlib.repofmt.weaverepo.
605
        repo = weaverepo.RepositoryFormat7().initialize(repo_dir)
2241.1.6 by Martin Pool
Move Knit repositories into the submodule bzrlib.repofmt.knitrepo and
606
        target_format = knitrepo.RepositoryFormatKnit1()
1556.1.4 by Robert Collins
Add a new format for what will become knit, and the surrounding logic to upgrade repositories within metadirs, and tests for the same.
607
        converter = repository.CopyConverter(target_format)
1594.1.3 by Robert Collins
Fixup pb usage to use nested_progress_bar.
608
        pb = bzrlib.ui.ui_factory.nested_progress_bar()
609
        try:
610
            converter.convert(repo, pb)
611
        finally:
612
            pb.finished()
1556.1.4 by Robert Collins
Add a new format for what will become knit, and the surrounding logic to upgrade repositories within metadirs, and tests for the same.
613
        repo = repo_dir.open_repository()
614
        self.assertTrue(isinstance(target_format, repo._format.__class__))
1843.2.5 by Aaron Bentley
Add test of _unescape_xml
615
616
617
class TestMisc(TestCase):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
618
1843.2.5 by Aaron Bentley
Add test of _unescape_xml
619
    def test_unescape_xml(self):
620
        """We get some kind of error when malformed entities are passed"""
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
621
        self.assertRaises(KeyError, repository._unescape_xml, 'foo&bar;')
1910.2.13 by Aaron Bentley
Start work on converter
622
623
2255.2.211 by Robert Collins
Remove knit2 repository format- it has never been supported.
624
class TestRepositoryFormatKnit3(TestCaseWithTransport):
1910.2.13 by Aaron Bentley
Start work on converter
625
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
626
    def test_attribute__fetch_order(self):
627
        """Knits need topological data insertion."""
628
        format = bzrdir.BzrDirMetaFormat1()
629
        format.repository_format = knitrepo.RepositoryFormatKnit3()
630
        repo = self.make_repository('.', format=format)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
631
        self.assertEqual('topological', repo._format._fetch_order)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
632
633
    def test_attribute__fetch_uses_deltas(self):
634
        """Knits reuse deltas."""
635
        format = bzrdir.BzrDirMetaFormat1()
636
        format.repository_format = knitrepo.RepositoryFormatKnit3()
637
        repo = self.make_repository('.', format=format)
4053.1.4 by Robert Collins
Move the fetch control attributes from Repository to RepositoryFormat.
638
        self.assertEqual(True, repo._format._fetch_uses_deltas)
3565.3.1 by Robert Collins
* The generic fetch code now uses two attributes on Repository objects
639
1910.2.13 by Aaron Bentley
Start work on converter
640
    def test_convert(self):
641
        """Ensure the upgrade adds weaves for roots"""
1910.2.35 by Aaron Bentley
Better fix for convesion test
642
        format = bzrdir.BzrDirMetaFormat1()
2241.1.6 by Martin Pool
Move Knit repositories into the submodule bzrlib.repofmt.knitrepo and
643
        format.repository_format = knitrepo.RepositoryFormatKnit1()
1910.2.35 by Aaron Bentley
Better fix for convesion test
644
        tree = self.make_branch_and_tree('.', format)
1910.2.13 by Aaron Bentley
Start work on converter
645
        tree.commit("Dull commit", rev_id="dull")
646
        revision_tree = tree.branch.repository.revision_tree('dull')
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
647
        revision_tree.lock_read()
648
        try:
649
            self.assertRaises(errors.NoSuchFile, revision_tree.get_file_lines,
650
                revision_tree.inventory.root.file_id)
651
        finally:
652
            revision_tree.unlock()
1910.2.13 by Aaron Bentley
Start work on converter
653
        format = bzrdir.BzrDirMetaFormat1()
2255.2.211 by Robert Collins
Remove knit2 repository format- it has never been supported.
654
        format.repository_format = knitrepo.RepositoryFormatKnit3()
1910.2.13 by Aaron Bentley
Start work on converter
655
        upgrade.Convert('.', format)
1910.2.27 by Aaron Bentley
Fixed conversion test
656
        tree = workingtree.WorkingTree.open('.')
1910.2.13 by Aaron Bentley
Start work on converter
657
        revision_tree = tree.branch.repository.revision_tree('dull')
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
658
        revision_tree.lock_read()
659
        try:
660
            revision_tree.get_file_lines(revision_tree.inventory.root.file_id)
661
        finally:
662
            revision_tree.unlock()
1910.2.27 by Aaron Bentley
Fixed conversion test
663
        tree.commit("Another dull commit", rev_id='dull2')
664
        revision_tree = tree.branch.repository.revision_tree('dull2')
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
665
        revision_tree.lock_read()
666
        self.addCleanup(revision_tree.unlock)
1910.2.27 by Aaron Bentley
Fixed conversion test
667
        self.assertEqual('dull', revision_tree.inventory.root.revision)
2220.2.2 by Martin Pool
Add tag command and basic implementation
668
3221.3.1 by Robert Collins
* Repository formats have a new supported-feature attribute
669
    def test_supports_external_lookups(self):
670
        format = bzrdir.BzrDirMetaFormat1()
671
        format.repository_format = knitrepo.RepositoryFormatKnit3()
672
        repo = self.make_repository('.', format=format)
673
        self.assertFalse(repo._format.supports_external_lookups)
674
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
675
4431.3.7 by Jonathan Lange
Cherrypick bzr.dev 4470, resolving conflicts.
676
class Test2a(TestCaseWithTransport):
677
678
    def test_format_pack_compresses_True(self):
679
        repo = self.make_repository('repo', format='2a')
680
        self.assertTrue(repo._format.pack_compresses)
3735.2.40 by Robert Collins
Add development4 which has a parent_id to basename index on CHKInventory objects.
681
682
    def test_inventories_use_chk_map_with_parent_base_dict(self):
4431.3.7 by Jonathan Lange
Cherrypick bzr.dev 4470, resolving conflicts.
683
        tree = self.make_branch_and_tree('repo', format="2a")
3735.2.40 by Robert Collins
Add development4 which has a parent_id to basename index on CHKInventory objects.
684
        revid = tree.commit("foo")
685
        tree.lock_read()
686
        self.addCleanup(tree.unlock)
687
        inv = tree.branch.repository.get_inventory(revid)
3735.2.41 by Robert Collins
Make the parent_id_basename index be updated during CHKInventory.apply_delta.
688
        self.assertNotEqual(None, inv.parent_id_basename_to_file_id)
689
        inv.parent_id_basename_to_file_id._ensure_root()
3735.2.40 by Robert Collins
Add development4 which has a parent_id to basename index on CHKInventory objects.
690
        inv.id_to_entry._ensure_root()
4241.6.8 by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)
691
        self.assertEqual(65536, inv.id_to_entry._root_node.maximum_size)
692
        self.assertEqual(65536,
3735.2.41 by Robert Collins
Make the parent_id_basename index be updated during CHKInventory.apply_delta.
693
            inv.parent_id_basename_to_file_id._root_node.maximum_size)
3735.2.40 by Robert Collins
Add development4 which has a parent_id to basename index on CHKInventory objects.
694
4431.3.7 by Jonathan Lange
Cherrypick bzr.dev 4470, resolving conflicts.
695
    def test_autopack_unchanged_chk_nodes(self):
696
        # at 20 unchanged commits, chk pages are packed that are split into
697
        # two groups such that the new pack being made doesn't have all its
698
        # pages in the source packs (though they are in the repository).
699
        tree = self.make_branch_and_tree('tree', format='2a')
700
        for pos in range(20):
701
            tree.commit(str(pos))
702
703
    def test_pack_with_hint(self):
704
        tree = self.make_branch_and_tree('tree', format='2a')
705
        # 1 commit to leave untouched
706
        tree.commit('1')
707
        to_keep = tree.branch.repository._pack_collection.names()
708
        # 2 to combine
709
        tree.commit('2')
710
        tree.commit('3')
711
        all = tree.branch.repository._pack_collection.names()
712
        combine = list(set(all) - set(to_keep))
713
        self.assertLength(3, all)
714
        self.assertLength(2, combine)
715
        tree.branch.repository.pack(hint=combine)
716
        final = tree.branch.repository._pack_collection.names()
717
        self.assertLength(2, final)
718
        self.assertFalse(combine[0] in final)
719
        self.assertFalse(combine[1] in final)
720
        self.assertSubset(to_keep, final)
721
4360.4.3 by John Arbash Meinel
Introduce a KnitPackStreamSource which is used when
722
    def test_stream_source_to_gc(self):
4462.2.1 by Robert Collins
Add new attribute to RepositoryFormat pack_compresses, hinting when pack can be useful.
723
        source = self.make_repository('source', format='2a')
724
        target = self.make_repository('target', format='2a')
4360.4.3 by John Arbash Meinel
Introduce a KnitPackStreamSource which is used when
725
        stream = source._get_source(target._format)
726
        self.assertIsInstance(stream, groupcompress_repo.GroupCHKStreamSource)
727
728
    def test_stream_source_to_non_gc(self):
4462.2.1 by Robert Collins
Add new attribute to RepositoryFormat pack_compresses, hinting when pack can be useful.
729
        source = self.make_repository('source', format='2a')
4360.4.3 by John Arbash Meinel
Introduce a KnitPackStreamSource which is used when
730
        target = self.make_repository('target', format='rich-root-pack')
731
        stream = source._get_source(target._format)
732
        # We don't want the child GroupCHKStreamSource
733
        self.assertIs(type(stream), repository.StreamSource)
734
4360.4.9 by John Arbash Meinel
Merge bzr.dev, bringing in the gc stacking fixes.
735
    def test_get_stream_for_missing_keys_includes_all_chk_refs(self):
736
        source_builder = self.make_branch_builder('source',
4462.2.1 by Robert Collins
Add new attribute to RepositoryFormat pack_compresses, hinting when pack can be useful.
737
                            format='2a')
4360.4.9 by John Arbash Meinel
Merge bzr.dev, bringing in the gc stacking fixes.
738
        # We have to build a fairly large tree, so that we are sure the chk
739
        # pages will have split into multiple pages.
740
        entries = [('add', ('', 'a-root-id', 'directory', None))]
741
        for i in 'abcdefghijklmnopqrstuvwxyz123456789':
742
            for j in 'abcdefghijklmnopqrstuvwxyz123456789':
743
                fname = i + j
744
                fid = fname + '-id'
745
                content = 'content for %s\n' % (fname,)
746
                entries.append(('add', (fname, fid, 'file', content)))
747
        source_builder.start_series()
748
        source_builder.build_snapshot('rev-1', None, entries)
749
        # Now change a few of them, so we get a few new pages for the second
750
        # revision
751
        source_builder.build_snapshot('rev-2', ['rev-1'], [
752
            ('modify', ('aa-id', 'new content for aa-id\n')),
753
            ('modify', ('cc-id', 'new content for cc-id\n')),
754
            ('modify', ('zz-id', 'new content for zz-id\n')),
755
            ])
756
        source_builder.finish_series()
757
        source_branch = source_builder.get_branch()
758
        source_branch.lock_read()
759
        self.addCleanup(source_branch.unlock)
4462.2.1 by Robert Collins
Add new attribute to RepositoryFormat pack_compresses, hinting when pack can be useful.
760
        target = self.make_repository('target', format='2a')
4360.4.9 by John Arbash Meinel
Merge bzr.dev, bringing in the gc stacking fixes.
761
        source = source_branch.repository._get_source(target._format)
762
        self.assertIsInstance(source, groupcompress_repo.GroupCHKStreamSource)
763
764
        # On a regular pass, getting the inventories and chk pages for rev-2
765
        # would only get the newly created chk pages
766
        search = graph.SearchResult(set(['rev-2']), set(['rev-1']), 1,
767
                                    set(['rev-2']))
768
        simple_chk_records = []
769
        for vf_name, substream in source.get_stream(search):
770
            if vf_name == 'chk_bytes':
771
                for record in substream:
772
                    simple_chk_records.append(record.key)
773
            else:
774
                for _ in substream:
775
                    continue
776
        # 3 pages, the root (InternalNode), + 2 pages which actually changed
777
        self.assertEqual([('sha1:91481f539e802c76542ea5e4c83ad416bf219f73',),
778
                          ('sha1:4ff91971043668583985aec83f4f0ab10a907d3f',),
779
                          ('sha1:81e7324507c5ca132eedaf2d8414ee4bb2226187',),
780
                          ('sha1:b101b7da280596c71a4540e9a1eeba8045985ee0',)],
781
                         simple_chk_records)
782
        # Now, when we do a similar call using 'get_stream_for_missing_keys'
783
        # we should get a much larger set of pages.
784
        missing = [('inventories', 'rev-2')]
785
        full_chk_records = []
786
        for vf_name, substream in source.get_stream_for_missing_keys(missing):
787
            if vf_name == 'inventories':
788
                for record in substream:
789
                    self.assertEqual(('rev-2',), record.key)
790
            elif vf_name == 'chk_bytes':
791
                for record in substream:
792
                    full_chk_records.append(record.key)
793
            else:
794
                self.fail('Should not be getting a stream of %s' % (vf_name,))
795
        # We have 257 records now. This is because we have 1 root page, and 256
796
        # leaf pages in a complete listing.
797
        self.assertEqual(257, len(full_chk_records))
798
        self.assertSubset(simple_chk_records, full_chk_records)
799
4465.2.7 by Aaron Bentley
Move test_inconsistency_fatal to test_repository
800
    def test_inconsistency_fatal(self):
801
        repo = self.make_repository('repo', format='2a')
802
        self.assertTrue(repo.revisions._index._inconsistency_fatal)
803
        self.assertFalse(repo.texts._index._inconsistency_fatal)
804
        self.assertFalse(repo.inventories._index._inconsistency_fatal)
805
        self.assertFalse(repo.signatures._index._inconsistency_fatal)
806
        self.assertFalse(repo.chk_bytes._index._inconsistency_fatal)
807
4360.4.3 by John Arbash Meinel
Introduce a KnitPackStreamSource which is used when
808
809
class TestKnitPackStreamSource(tests.TestCaseWithMemoryTransport):
810
811
    def test_source_to_exact_pack_092(self):
812
        source = self.make_repository('source', format='pack-0.92')
813
        target = self.make_repository('target', format='pack-0.92')
814
        stream_source = source._get_source(target._format)
815
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
816
817
    def test_source_to_exact_pack_rich_root_pack(self):
818
        source = self.make_repository('source', format='rich-root-pack')
819
        target = self.make_repository('target', format='rich-root-pack')
820
        stream_source = source._get_source(target._format)
821
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
822
823
    def test_source_to_exact_pack_19(self):
824
        source = self.make_repository('source', format='1.9')
825
        target = self.make_repository('target', format='1.9')
826
        stream_source = source._get_source(target._format)
827
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
828
829
    def test_source_to_exact_pack_19_rich_root(self):
830
        source = self.make_repository('source', format='1.9-rich-root')
831
        target = self.make_repository('target', format='1.9-rich-root')
832
        stream_source = source._get_source(target._format)
833
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
834
835
    def test_source_to_remote_exact_pack_19(self):
836
        trans = self.make_smart_server('target')
837
        trans.ensure_base()
838
        source = self.make_repository('source', format='1.9')
839
        target = self.make_repository('target', format='1.9')
840
        target = repository.Repository.open(trans.base)
841
        stream_source = source._get_source(target._format)
842
        self.assertIsInstance(stream_source, pack_repo.KnitPackStreamSource)
843
844
    def test_stream_source_to_non_exact(self):
845
        source = self.make_repository('source', format='pack-0.92')
846
        target = self.make_repository('target', format='1.9')
847
        stream = source._get_source(target._format)
848
        self.assertIs(type(stream), repository.StreamSource)
849
850
    def test_stream_source_to_non_exact_rich_root(self):
851
        source = self.make_repository('source', format='1.9')
852
        target = self.make_repository('target', format='1.9-rich-root')
853
        stream = source._get_source(target._format)
854
        self.assertIs(type(stream), repository.StreamSource)
855
856
    def test_source_to_remote_non_exact_pack_19(self):
857
        trans = self.make_smart_server('target')
858
        trans.ensure_base()
859
        source = self.make_repository('source', format='1.9')
860
        target = self.make_repository('target', format='1.6')
861
        target = repository.Repository.open(trans.base)
862
        stream_source = source._get_source(target._format)
863
        self.assertIs(type(stream_source), repository.StreamSource)
864
865
    def test_stream_source_to_knit(self):
866
        source = self.make_repository('source', format='pack-0.92')
867
        target = self.make_repository('target', format='dirstate')
868
        stream = source._get_source(target._format)
869
        self.assertIs(type(stream), repository.StreamSource)
870
3735.2.40 by Robert Collins
Add development4 which has a parent_id to basename index on CHKInventory objects.
871
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
872
class TestDevelopment6FindParentIdsOfRevisions(TestCaseWithTransport):
873
    """Tests for _find_parent_ids_of_revisions."""
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
874
875
    def setUp(self):
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
876
        super(TestDevelopment6FindParentIdsOfRevisions, self).setUp()
4241.6.8 by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)
877
        self.builder = self.make_branch_builder('source',
878
            format='development6-rich-root')
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
879
        self.builder.start_series()
880
        self.builder.build_snapshot('initial', None,
881
            [('add', ('', 'tree-root', 'directory', None))])
882
        self.repo = self.builder.get_branch().repository
883
        self.addCleanup(self.builder.finish_series)
3735.2.99 by John Arbash Meinel
Merge bzr.dev 4034. Whitespace cleanup
884
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
885
    def assertParentIds(self, expected_result, rev_set):
886
        self.assertEqual(sorted(expected_result),
887
            sorted(self.repo._find_parent_ids_of_revisions(rev_set)))
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
888
889
    def test_simple(self):
890
        self.builder.build_snapshot('revid1', None, [])
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
891
        self.builder.build_snapshot('revid2', ['revid1'], [])
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
892
        rev_set = ['revid2']
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
893
        self.assertParentIds(['revid1'], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
894
895
    def test_not_first_parent(self):
896
        self.builder.build_snapshot('revid1', None, [])
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
897
        self.builder.build_snapshot('revid2', ['revid1'], [])
898
        self.builder.build_snapshot('revid3', ['revid2'], [])
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
899
        rev_set = ['revid3', 'revid2']
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
900
        self.assertParentIds(['revid1'], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
901
902
    def test_not_null(self):
903
        rev_set = ['initial']
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
904
        self.assertParentIds([], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
905
906
    def test_not_null_set(self):
907
        self.builder.build_snapshot('revid1', None, [])
908
        rev_set = [_mod_revision.NULL_REVISION]
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
909
        self.assertParentIds([], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
910
911
    def test_ghost(self):
912
        self.builder.build_snapshot('revid1', None, [])
913
        rev_set = ['ghost', 'revid1']
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
914
        self.assertParentIds(['initial'], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
915
916
    def test_ghost_parent(self):
917
        self.builder.build_snapshot('revid1', None, [])
918
        self.builder.build_snapshot('revid2', ['revid1', 'ghost'], [])
919
        rev_set = ['revid2', 'revid1']
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
920
        self.assertParentIds(['ghost', 'initial'], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
921
922
    def test_righthand_parent(self):
923
        self.builder.build_snapshot('revid1', None, [])
924
        self.builder.build_snapshot('revid2a', ['revid1'], [])
925
        self.builder.build_snapshot('revid2b', ['revid1'], [])
926
        self.builder.build_snapshot('revid3', ['revid2a', 'revid2b'], [])
927
        rev_set = ['revid3', 'revid2a']
4343.3.32 by John Arbash Meinel
Change the tests for _find_revision_outside_set to the new _find_parent_ids function.
928
        self.assertParentIds(['revid1', 'revid2b'], rev_set)
3735.4.1 by Andrew Bennetts
Add _find_revision_outside_set.
929
930
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
931
class TestWithBrokenRepo(TestCaseWithTransport):
2592.3.214 by Robert Collins
Merge bzr.dev.
932
    """These tests seem to be more appropriate as interface tests?"""
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
933
934
    def make_broken_repository(self):
935
        # XXX: This function is borrowed from Aaron's "Reconcile can fix bad
936
        # parent references" branch which is due to land in bzr.dev soon.  Once
937
        # it does, this duplication should be removed.
938
        repo = self.make_repository('broken-repo')
939
        cleanups = []
940
        try:
941
            repo.lock_write()
942
            cleanups.append(repo.unlock)
943
            repo.start_write_group()
944
            cleanups.append(repo.commit_write_group)
945
            # make rev1a: A well-formed revision, containing 'file1'
946
            inv = inventory.Inventory(revision_id='rev1a')
947
            inv.root.revision = 'rev1a'
948
            self.add_file(repo, inv, 'file1', 'rev1a', [])
949
            repo.add_inventory('rev1a', inv, [])
950
            revision = _mod_revision.Revision('rev1a',
951
                committer='jrandom@example.com', timestamp=0,
952
                inventory_sha1='', timezone=0, message='foo', parent_ids=[])
953
            repo.add_revision('rev1a',revision, inv)
954
955
            # make rev1b, which has no Revision, but has an Inventory, and
956
            # file1
957
            inv = inventory.Inventory(revision_id='rev1b')
958
            inv.root.revision = 'rev1b'
959
            self.add_file(repo, inv, 'file1', 'rev1b', [])
960
            repo.add_inventory('rev1b', inv, [])
961
962
            # make rev2, with file1 and file2
963
            # file2 is sane
964
            # file1 has 'rev1b' as an ancestor, even though this is not
965
            # mentioned by 'rev1a', making it an unreferenced ancestor
966
            inv = inventory.Inventory()
967
            self.add_file(repo, inv, 'file1', 'rev2', ['rev1a', 'rev1b'])
968
            self.add_file(repo, inv, 'file2', 'rev2', [])
969
            self.add_revision(repo, 'rev2', inv, ['rev1a'])
970
971
            # make ghost revision rev1c
972
            inv = inventory.Inventory()
973
            self.add_file(repo, inv, 'file2', 'rev1c', [])
974
975
            # make rev3 with file2
976
            # file2 refers to 'rev1c', which is a ghost in this repository, so
977
            # file2 cannot have rev1c as its ancestor.
978
            inv = inventory.Inventory()
979
            self.add_file(repo, inv, 'file2', 'rev3', ['rev1c'])
980
            self.add_revision(repo, 'rev3', inv, ['rev1c'])
981
            return repo
982
        finally:
983
            for cleanup in reversed(cleanups):
984
                cleanup()
985
986
    def add_revision(self, repo, revision_id, inv, parent_ids):
987
        inv.revision_id = revision_id
988
        inv.root.revision = revision_id
989
        repo.add_inventory(revision_id, inv, parent_ids)
990
        revision = _mod_revision.Revision(revision_id,
991
            committer='jrandom@example.com', timestamp=0, inventory_sha1='',
992
            timezone=0, message='foo', parent_ids=parent_ids)
993
        repo.add_revision(revision_id,revision, inv)
994
995
    def add_file(self, repo, inv, filename, revision, parents):
996
        file_id = filename + '-id'
997
        entry = inventory.InventoryFile(file_id, filename, 'TREE_ROOT')
998
        entry.revision = revision
2535.4.10 by Andrew Bennetts
Fix one failing test, disable another.
999
        entry.text_size = 0
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
1000
        inv.add(entry)
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
1001
        text_key = (file_id, revision)
1002
        parent_keys = [(file_id, parent) for parent in parents]
1003
        repo.texts.add_lines(text_key, parent_keys, ['line\n'])
2535.3.57 by Andrew Bennetts
Perform some sanity checking of data streams rather than blindly inserting them into our repository.
1004
1005
    def test_insert_from_broken_repo(self):
1006
        """Inserting a data stream from a broken repository won't silently
1007
        corrupt the target repository.
1008
        """
1009
        broken_repo = self.make_broken_repository()
1010
        empty_repo = self.make_repository('empty-repo')
4606.1.1 by Robert Collins
Change test_insert_from_broken_repo from a known failure to a working test.
1011
        try:
1012
            empty_repo.fetch(broken_repo)
1013
        except (errors.RevisionNotPresent, errors.BzrCheckError):
1014
            # Test successful: compression parent not being copied leads to
1015
            # error.
1016
            return
1017
        empty_repo.lock_read()
1018
        self.addCleanup(empty_repo.unlock)
1019
        text = empty_repo.texts.get_record_stream(
1020
            [('file2-id', 'rev3')], 'topological', True).next()
1021
        self.assertEqual('line\n', text.get_bytes_as('fulltext'))
2592.3.214 by Robert Collins
Merge bzr.dev.
1022
1023
2592.3.84 by Robert Collins
Start of autopacking logic.
1024
class TestRepositoryPackCollection(TestCaseWithTransport):
1025
1026
    def get_format(self):
3010.3.3 by Martin Pool
Merge trunk
1027
        return bzrdir.format_registry.make_bzrdir('pack-0.92')
2592.3.84 by Robert Collins
Start of autopacking logic.
1028
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1029
    def get_packs(self):
1030
        format = self.get_format()
1031
        repo = self.make_repository('.', format=format)
1032
        return repo._pack_collection
1033
3789.2.20 by John Arbash Meinel
The autopack code can now trigger itself to retry when _copy_revision_texts fails.
1034
    def make_packs_and_alt_repo(self, write_lock=False):
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1035
        """Create a pack repo with 3 packs, and access it via a second repo."""
1036
        tree = self.make_branch_and_tree('.')
1037
        tree.lock_write()
1038
        self.addCleanup(tree.unlock)
1039
        rev1 = tree.commit('one')
1040
        rev2 = tree.commit('two')
1041
        rev3 = tree.commit('three')
1042
        r = repository.Repository.open('.')
3789.2.20 by John Arbash Meinel
The autopack code can now trigger itself to retry when _copy_revision_texts fails.
1043
        if write_lock:
1044
            r.lock_write()
1045
        else:
1046
            r.lock_read()
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1047
        self.addCleanup(r.unlock)
1048
        packs = r._pack_collection
1049
        packs.ensure_loaded()
1050
        return tree, r, packs, [rev1, rev2, rev3]
1051
2592.3.84 by Robert Collins
Start of autopacking logic.
1052
    def test__max_pack_count(self):
2592.3.219 by Robert Collins
Review feedback.
1053
        """The maximum pack count is a function of the number of revisions."""
2592.3.84 by Robert Collins
Start of autopacking logic.
1054
        # no revisions - one pack, so that we can have a revision free repo
1055
        # without it blowing up
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1056
        packs = self.get_packs()
2592.3.84 by Robert Collins
Start of autopacking logic.
1057
        self.assertEqual(1, packs._max_pack_count(0))
1058
        # after that the sum of the digits, - check the first 1-9
1059
        self.assertEqual(1, packs._max_pack_count(1))
1060
        self.assertEqual(2, packs._max_pack_count(2))
1061
        self.assertEqual(3, packs._max_pack_count(3))
1062
        self.assertEqual(4, packs._max_pack_count(4))
1063
        self.assertEqual(5, packs._max_pack_count(5))
1064
        self.assertEqual(6, packs._max_pack_count(6))
1065
        self.assertEqual(7, packs._max_pack_count(7))
1066
        self.assertEqual(8, packs._max_pack_count(8))
1067
        self.assertEqual(9, packs._max_pack_count(9))
1068
        # check the boundary cases with two digits for the next decade
1069
        self.assertEqual(1, packs._max_pack_count(10))
1070
        self.assertEqual(2, packs._max_pack_count(11))
1071
        self.assertEqual(10, packs._max_pack_count(19))
1072
        self.assertEqual(2, packs._max_pack_count(20))
1073
        self.assertEqual(3, packs._max_pack_count(21))
1074
        # check some arbitrary big numbers
1075
        self.assertEqual(25, packs._max_pack_count(112894))
1076
1077
    def test_pack_distribution_zero(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1078
        packs = self.get_packs()
2592.3.84 by Robert Collins
Start of autopacking logic.
1079
        self.assertEqual([0], packs.pack_distribution(0))
3052.1.6 by John Arbash Meinel
Change the lock check to raise ObjectNotLocked.
1080
1081
    def test_ensure_loaded_unlocked(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1082
        packs = self.get_packs()
3052.1.6 by John Arbash Meinel
Change the lock check to raise ObjectNotLocked.
1083
        self.assertRaises(errors.ObjectNotLocked,
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1084
                          packs.ensure_loaded)
3052.1.6 by John Arbash Meinel
Change the lock check to raise ObjectNotLocked.
1085
2592.3.84 by Robert Collins
Start of autopacking logic.
1086
    def test_pack_distribution_one_to_nine(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1087
        packs = self.get_packs()
2592.3.84 by Robert Collins
Start of autopacking logic.
1088
        self.assertEqual([1],
1089
            packs.pack_distribution(1))
1090
        self.assertEqual([1, 1],
1091
            packs.pack_distribution(2))
1092
        self.assertEqual([1, 1, 1],
1093
            packs.pack_distribution(3))
1094
        self.assertEqual([1, 1, 1, 1],
1095
            packs.pack_distribution(4))
1096
        self.assertEqual([1, 1, 1, 1, 1],
1097
            packs.pack_distribution(5))
1098
        self.assertEqual([1, 1, 1, 1, 1, 1],
1099
            packs.pack_distribution(6))
1100
        self.assertEqual([1, 1, 1, 1, 1, 1, 1],
1101
            packs.pack_distribution(7))
1102
        self.assertEqual([1, 1, 1, 1, 1, 1, 1, 1],
1103
            packs.pack_distribution(8))
1104
        self.assertEqual([1, 1, 1, 1, 1, 1, 1, 1, 1],
1105
            packs.pack_distribution(9))
1106
1107
    def test_pack_distribution_stable_at_boundaries(self):
1108
        """When there are multi-rev packs the counts are stable."""
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1109
        packs = self.get_packs()
2592.3.84 by Robert Collins
Start of autopacking logic.
1110
        # in 10s:
1111
        self.assertEqual([10], packs.pack_distribution(10))
1112
        self.assertEqual([10, 1], packs.pack_distribution(11))
1113
        self.assertEqual([10, 10], packs.pack_distribution(20))
1114
        self.assertEqual([10, 10, 1], packs.pack_distribution(21))
1115
        # 100s
1116
        self.assertEqual([100], packs.pack_distribution(100))
1117
        self.assertEqual([100, 1], packs.pack_distribution(101))
1118
        self.assertEqual([100, 10, 1], packs.pack_distribution(111))
1119
        self.assertEqual([100, 100], packs.pack_distribution(200))
1120
        self.assertEqual([100, 100, 1], packs.pack_distribution(201))
1121
        self.assertEqual([100, 100, 10, 1], packs.pack_distribution(211))
1122
2592.3.85 by Robert Collins
Finish autopack corner cases.
1123
    def test_plan_pack_operations_2009_revisions_skip_all_packs(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1124
        packs = self.get_packs()
2592.3.85 by Robert Collins
Finish autopack corner cases.
1125
        existing_packs = [(2000, "big"), (9, "medium")]
1126
        # rev count - 2009 -> 2x1000 + 9x1
1127
        pack_operations = packs.plan_autopack_combinations(
1128
            existing_packs, [1000, 1000, 1, 1, 1, 1, 1, 1, 1, 1, 1])
1129
        self.assertEqual([], pack_operations)
1130
1131
    def test_plan_pack_operations_2010_revisions_skip_all_packs(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1132
        packs = self.get_packs()
2592.3.85 by Robert Collins
Finish autopack corner cases.
1133
        existing_packs = [(2000, "big"), (9, "medium"), (1, "single")]
1134
        # rev count - 2010 -> 2x1000 + 1x10
1135
        pack_operations = packs.plan_autopack_combinations(
1136
            existing_packs, [1000, 1000, 10])
1137
        self.assertEqual([], pack_operations)
1138
1139
    def test_plan_pack_operations_2010_combines_smallest_two(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1140
        packs = self.get_packs()
2592.3.85 by Robert Collins
Finish autopack corner cases.
1141
        existing_packs = [(1999, "big"), (9, "medium"), (1, "single2"),
1142
            (1, "single1")]
1143
        # rev count - 2010 -> 2x1000 + 1x10 (3)
1144
        pack_operations = packs.plan_autopack_combinations(
1145
            existing_packs, [1000, 1000, 10])
3711.4.2 by John Arbash Meinel
Change the logic to solve it in a different way.
1146
        self.assertEqual([[2, ["single2", "single1"]]], pack_operations)
2592.3.85 by Robert Collins
Finish autopack corner cases.
1147
3711.4.2 by John Arbash Meinel
Change the logic to solve it in a different way.
1148
    def test_plan_pack_operations_creates_a_single_op(self):
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1149
        packs = self.get_packs()
3711.4.2 by John Arbash Meinel
Change the logic to solve it in a different way.
1150
        existing_packs = [(50, 'a'), (40, 'b'), (30, 'c'), (10, 'd'),
1151
                          (10, 'e'), (6, 'f'), (4, 'g')]
1152
        # rev count 150 -> 1x100 and 5x10
1153
        # The two size 10 packs do not need to be touched. The 50, 40, 30 would
1154
        # be combined into a single 120 size pack, and the 6 & 4 would
1155
        # becombined into a size 10 pack. However, if we have to rewrite them,
1156
        # we save a pack file with no increased I/O by putting them into the
1157
        # same file.
1158
        distribution = packs.pack_distribution(150)
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1159
        pack_operations = packs.plan_autopack_combinations(existing_packs,
3711.4.2 by John Arbash Meinel
Change the logic to solve it in a different way.
1160
                                                           distribution)
1161
        self.assertEqual([[130, ['a', 'b', 'c', 'f', 'g']]], pack_operations)
3711.4.1 by John Arbash Meinel
Fix bug #242510, when determining the autopack sequence,
1162
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1163
    def test_all_packs_none(self):
1164
        format = self.get_format()
1165
        tree = self.make_branch_and_tree('.', format=format)
1166
        tree.lock_read()
1167
        self.addCleanup(tree.unlock)
2592.3.232 by Martin Pool
Disambiguate two member variables called _packs into _packs_by_name and _pack_collection
1168
        packs = tree.branch.repository._pack_collection
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1169
        packs.ensure_loaded()
1170
        self.assertEqual([], packs.all_packs())
1171
1172
    def test_all_packs_one(self):
1173
        format = self.get_format()
1174
        tree = self.make_branch_and_tree('.', format=format)
1175
        tree.commit('start')
1176
        tree.lock_read()
1177
        self.addCleanup(tree.unlock)
2592.3.232 by Martin Pool
Disambiguate two member variables called _packs into _packs_by_name and _pack_collection
1178
        packs = tree.branch.repository._pack_collection
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1179
        packs.ensure_loaded()
2592.3.176 by Robert Collins
Various pack refactorings.
1180
        self.assertEqual([
1181
            packs.get_pack_by_name(packs.names()[0])],
1182
            packs.all_packs())
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1183
1184
    def test_all_packs_two(self):
1185
        format = self.get_format()
1186
        tree = self.make_branch_and_tree('.', format=format)
1187
        tree.commit('start')
1188
        tree.commit('continue')
1189
        tree.lock_read()
1190
        self.addCleanup(tree.unlock)
2592.3.232 by Martin Pool
Disambiguate two member variables called _packs into _packs_by_name and _pack_collection
1191
        packs = tree.branch.repository._pack_collection
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1192
        packs.ensure_loaded()
1193
        self.assertEqual([
2592.3.176 by Robert Collins
Various pack refactorings.
1194
            packs.get_pack_by_name(packs.names()[0]),
1195
            packs.get_pack_by_name(packs.names()[1]),
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1196
            ], packs.all_packs())
1197
2592.3.176 by Robert Collins
Various pack refactorings.
1198
    def test_get_pack_by_name(self):
1199
        format = self.get_format()
1200
        tree = self.make_branch_and_tree('.', format=format)
1201
        tree.commit('start')
1202
        tree.lock_read()
1203
        self.addCleanup(tree.unlock)
2592.3.232 by Martin Pool
Disambiguate two member variables called _packs into _packs_by_name and _pack_collection
1204
        packs = tree.branch.repository._pack_collection
4145.1.6 by Robert Collins
More test fallout, but all caught now.
1205
        packs.reset()
2592.3.176 by Robert Collins
Various pack refactorings.
1206
        packs.ensure_loaded()
1207
        name = packs.names()[0]
1208
        pack_1 = packs.get_pack_by_name(name)
1209
        # the pack should be correctly initialised
3517.4.5 by Martin Pool
Correct use of packs._names in test_get_pack_by_name
1210
        sizes = packs._names[name]
3221.12.4 by Robert Collins
Implement basic repository supporting external references.
1211
        rev_index = GraphIndex(packs._index_transport, name + '.rix', sizes[0])
1212
        inv_index = GraphIndex(packs._index_transport, name + '.iix', sizes[1])
1213
        txt_index = GraphIndex(packs._index_transport, name + '.tix', sizes[2])
1214
        sig_index = GraphIndex(packs._index_transport, name + '.six', sizes[3])
2592.3.191 by Robert Collins
Give Pack responsibility for index naming, and two concrete classes - NewPack for new packs and ExistingPack for packs we read from disk.
1215
        self.assertEqual(pack_repo.ExistingPack(packs._pack_transport,
2592.3.219 by Robert Collins
Review feedback.
1216
            name, rev_index, inv_index, txt_index, sig_index), pack_1)
2592.3.176 by Robert Collins
Various pack refactorings.
1217
        # and the same instance should be returned on successive calls.
1218
        self.assertTrue(pack_1 is packs.get_pack_by_name(name))
1219
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1220
    def test_reload_pack_names_new_entry(self):
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1221
        tree, r, packs, revs = self.make_packs_and_alt_repo()
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1222
        names = packs.names()
1223
        # Add a new pack file into the repository
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1224
        rev4 = tree.commit('four')
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1225
        new_names = tree.branch.repository._pack_collection.names()
1226
        new_name = set(new_names).difference(names)
1227
        self.assertEqual(1, len(new_name))
1228
        new_name = new_name.pop()
1229
        # The old collection hasn't noticed yet
1230
        self.assertEqual(names, packs.names())
3789.1.8 by John Arbash Meinel
Change the api of reload_pack_names().
1231
        self.assertTrue(packs.reload_pack_names())
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1232
        self.assertEqual(new_names, packs.names())
1233
        # And the repository can access the new revision
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1234
        self.assertEqual({rev4:(revs[-1],)}, r.get_parent_map([rev4]))
3789.1.8 by John Arbash Meinel
Change the api of reload_pack_names().
1235
        self.assertFalse(packs.reload_pack_names())
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1236
1237
    def test_reload_pack_names_added_and_removed(self):
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1238
        tree, r, packs, revs = self.make_packs_and_alt_repo()
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1239
        names = packs.names()
1240
        # Now repack the whole thing
1241
        tree.branch.repository.pack()
1242
        new_names = tree.branch.repository._pack_collection.names()
1243
        # The other collection hasn't noticed yet
1244
        self.assertEqual(names, packs.names())
3789.1.8 by John Arbash Meinel
Change the api of reload_pack_names().
1245
        self.assertTrue(packs.reload_pack_names())
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1246
        self.assertEqual(new_names, packs.names())
3789.2.19 by John Arbash Meinel
Refactor to make the tests a bit simpler
1247
        self.assertEqual({revs[-1]:(revs[-2],)}, r.get_parent_map([revs[-1]]))
3789.1.8 by John Arbash Meinel
Change the api of reload_pack_names().
1248
        self.assertFalse(packs.reload_pack_names())
3789.1.2 by John Arbash Meinel
Add RepositoryPackCollection.reload_pack_names()
1249
3789.2.20 by John Arbash Meinel
The autopack code can now trigger itself to retry when _copy_revision_texts fails.
1250
    def test_autopack_reloads_and_stops(self):
1251
        tree, r, packs, revs = self.make_packs_and_alt_repo(write_lock=True)
1252
        # After we have determined what needs to be autopacked, trigger a
1253
        # full-pack via the other repo which will cause us to re-evaluate and
1254
        # decide we don't need to do anything
1255
        orig_execute = packs._execute_pack_operations
1256
        def _munged_execute_pack_ops(*args, **kwargs):
1257
            tree.branch.repository.pack()
1258
            return orig_execute(*args, **kwargs)
1259
        packs._execute_pack_operations = _munged_execute_pack_ops
1260
        packs._max_pack_count = lambda x: 1
1261
        packs.pack_distribution = lambda x: [10]
1262
        self.assertFalse(packs.autopack())
1263
        self.assertEqual(1, len(packs.names()))
1264
        self.assertEqual(tree.branch.repository._pack_collection.names(),
1265
                         packs.names())
1266
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1267
1268
class TestPack(TestCaseWithTransport):
1269
    """Tests for the Pack object."""
1270
1271
    def assertCurrentlyEqual(self, left, right):
1272
        self.assertTrue(left == right)
1273
        self.assertTrue(right == left)
1274
        self.assertFalse(left != right)
1275
        self.assertFalse(right != left)
1276
1277
    def assertCurrentlyNotEqual(self, left, right):
1278
        self.assertFalse(left == right)
1279
        self.assertFalse(right == left)
1280
        self.assertTrue(left != right)
1281
        self.assertTrue(right != left)
1282
1283
    def test___eq____ne__(self):
2592.3.191 by Robert Collins
Give Pack responsibility for index naming, and two concrete classes - NewPack for new packs and ExistingPack for packs we read from disk.
1284
        left = pack_repo.ExistingPack('', '', '', '', '', '')
1285
        right = pack_repo.ExistingPack('', '', '', '', '', '')
2592.3.173 by Robert Collins
Basic implementation of all_packs.
1286
        self.assertCurrentlyEqual(left, right)
1287
        # change all attributes and ensure equality changes as we do.
1288
        left.revision_index = 'a'
1289
        self.assertCurrentlyNotEqual(left, right)
1290
        right.revision_index = 'a'
1291
        self.assertCurrentlyEqual(left, right)
1292
        left.inventory_index = 'a'
1293
        self.assertCurrentlyNotEqual(left, right)
1294
        right.inventory_index = 'a'
1295
        self.assertCurrentlyEqual(left, right)
1296
        left.text_index = 'a'
1297
        self.assertCurrentlyNotEqual(left, right)
1298
        right.text_index = 'a'
1299
        self.assertCurrentlyEqual(left, right)
1300
        left.signature_index = 'a'
1301
        self.assertCurrentlyNotEqual(left, right)
1302
        right.signature_index = 'a'
1303
        self.assertCurrentlyEqual(left, right)
1304
        left.name = 'a'
1305
        self.assertCurrentlyNotEqual(left, right)
1306
        right.name = 'a'
1307
        self.assertCurrentlyEqual(left, right)
1308
        left.transport = 'a'
1309
        self.assertCurrentlyNotEqual(left, right)
1310
        right.transport = 'a'
1311
        self.assertCurrentlyEqual(left, right)
2592.3.179 by Robert Collins
Generate the revision_index_map for packing during the core operation, from the pack objects.
1312
1313
    def test_file_name(self):
2592.3.191 by Robert Collins
Give Pack responsibility for index naming, and two concrete classes - NewPack for new packs and ExistingPack for packs we read from disk.
1314
        pack = pack_repo.ExistingPack('', 'a_name', '', '', '', '')
2592.3.179 by Robert Collins
Generate the revision_index_map for packing during the core operation, from the pack objects.
1315
        self.assertEqual('a_name.pack', pack.file_name())
2592.3.192 by Robert Collins
Move new revision index management to NewPack.
1316
1317
1318
class TestNewPack(TestCaseWithTransport):
1319
    """Tests for pack_repo.NewPack."""
1320
2592.3.193 by Robert Collins
Move hash tracking of new packs into NewPack.
1321
    def test_new_instance_attributes(self):
2592.3.194 by Robert Collins
Output the revision index from NewPack.finish
1322
        upload_transport = self.get_transport('upload')
1323
        pack_transport = self.get_transport('pack')
1324
        index_transport = self.get_transport('index')
1325
        upload_transport.mkdir('.')
4241.6.8 by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)
1326
        collection = pack_repo.RepositoryPackCollection(
1327
            repo=None,
3830.3.1 by Martin Pool
NewPack should be constructed from the PackCollection, rather than attributes of it
1328
            transport=self.get_transport('.'),
1329
            index_transport=index_transport,
1330
            upload_transport=upload_transport,
1331
            pack_transport=pack_transport,
1332
            index_builder_class=BTreeBuilder,
4241.6.8 by Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil
Add --development6-rich-root, disabling the legacy and unneeded development2 format, and activating the tests for CHK features disabled pending this format. (Robert Collins, John Arbash Meinel, Ian Clatworthy, Vincent Ladeuil)
1333
            index_class=BTreeGraphIndex,
1334
            use_chk_index=False)
3830.3.1 by Martin Pool
NewPack should be constructed from the PackCollection, rather than attributes of it
1335
        pack = pack_repo.NewPack(collection)
3735.1.1 by Robert Collins
Add development2 formats using BTree indices.
1336
        self.assertIsInstance(pack.revision_index, BTreeBuilder)
1337
        self.assertIsInstance(pack.inventory_index, BTreeBuilder)
2929.3.5 by Vincent Ladeuil
New files, same warnings, same fixes.
1338
        self.assertIsInstance(pack._hash, type(osutils.md5()))
2592.3.194 by Robert Collins
Output the revision index from NewPack.finish
1339
        self.assertTrue(pack.upload_transport is upload_transport)
1340
        self.assertTrue(pack.index_transport is index_transport)
1341
        self.assertTrue(pack.pack_transport is pack_transport)
1342
        self.assertEqual(None, pack.index_sizes)
1343
        self.assertEqual(20, len(pack.random_name))
1344
        self.assertIsInstance(pack.random_name, str)
1345
        self.assertIsInstance(pack.start_time, float)
2951.1.2 by Robert Collins
Partial refactoring of pack_repo to create a Packer object for packing.
1346
1347
1348
class TestPacker(TestCaseWithTransport):
1349
    """Tests for the packs repository Packer class."""
2951.1.10 by Robert Collins
Peer review feedback with Ian.
1350
3824.2.4 by John Arbash Meinel
Add a test that ensures the pack ordering changes as part of calling .pack()
1351
    def test_pack_optimizes_pack_order(self):
1352
        builder = self.make_branch_builder('.')
1353
        builder.start_series()
1354
        builder.build_snapshot('A', None, [
1355
            ('add', ('', 'root-id', 'directory', None)),
1356
            ('add', ('f', 'f-id', 'file', 'content\n'))])
1357
        builder.build_snapshot('B', ['A'],
1358
            [('modify', ('f-id', 'new-content\n'))])
1359
        builder.build_snapshot('C', ['B'],
1360
            [('modify', ('f-id', 'third-content\n'))])
1361
        builder.build_snapshot('D', ['C'],
1362
            [('modify', ('f-id', 'fourth-content\n'))])
1363
        b = builder.get_branch()
1364
        b.lock_read()
1365
        builder.finish_series()
1366
        self.addCleanup(b.unlock)
1367
        # At this point, we should have 4 pack files available
1368
        # Because of how they were built, they correspond to
1369
        # ['D', 'C', 'B', 'A']
1370
        packs = b.repository._pack_collection.packs
1371
        packer = pack_repo.Packer(b.repository._pack_collection,
1372
                                  packs, 'testing',
1373
                                  revision_ids=['B', 'C'])
1374
        # Now, when we are copying the B & C revisions, their pack files should
1375
        # be moved to the front of the stack
3824.2.5 by Andrew Bennetts
Minor tweaks to comments etc.
1376
        # The new ordering moves B & C to the front of the .packs attribute,
1377
        # and leaves the others in the original order.
3824.2.4 by John Arbash Meinel
Add a test that ensures the pack ordering changes as part of calling .pack()
1378
        new_packs = [packs[1], packs[2], packs[0], packs[3]]
1379
        new_pack = packer.pack()
1380
        self.assertEqual(new_packs, packer.packs)
3146.6.1 by Aaron Bentley
InterDifferingSerializer shows a progress bar
1381
1382
3777.5.4 by John Arbash Meinel
OptimisingPacker now sets the optimize flags for the indexes being built.
1383
class TestOptimisingPacker(TestCaseWithTransport):
1384
    """Tests for the OptimisingPacker class."""
1385
1386
    def get_pack_collection(self):
1387
        repo = self.make_repository('.')
1388
        return repo._pack_collection
1389
1390
    def test_open_pack_will_optimise(self):
1391
        packer = pack_repo.OptimisingPacker(self.get_pack_collection(),
1392
                                            [], '.test')
1393
        new_pack = packer.open_pack()
1394
        self.assertIsInstance(new_pack, pack_repo.NewPack)
1395
        self.assertTrue(new_pack.revision_index._optimize_for_size)
1396
        self.assertTrue(new_pack.inventory_index._optimize_for_size)
1397
        self.assertTrue(new_pack.text_index._optimize_for_size)
1398
        self.assertTrue(new_pack.signature_index._optimize_for_size)
4462.2.6 by Robert Collins
Cause StreamSink to partially pack repositories after cross format fetches when beneficial.
1399
1400
4431.3.7 by Jonathan Lange
Cherrypick bzr.dev 4470, resolving conflicts.
1401
class TestCrossFormatPacks(TestCaseWithTransport):
1402
1403
    def log_pack(self, hint=None):
1404
        self.calls.append(('pack', hint))
1405
        self.orig_pack(hint=hint)
1406
        if self.expect_hint:
1407
            self.assertTrue(hint)
1408
1409
    def run_stream(self, src_fmt, target_fmt, expect_pack_called):
1410
        self.expect_hint = expect_pack_called
1411
        self.calls = []
1412
        source_tree = self.make_branch_and_tree('src', format=src_fmt)
1413
        source_tree.lock_write()
1414
        self.addCleanup(source_tree.unlock)
1415
        tip = source_tree.commit('foo')
1416
        target = self.make_repository('target', format=target_fmt)
1417
        target.lock_write()
1418
        self.addCleanup(target.unlock)
1419
        source = source_tree.branch.repository._get_source(target._format)
1420
        self.orig_pack = target.pack
1421
        target.pack = self.log_pack
1422
        search = target.search_missing_revision_ids(
1423
            source_tree.branch.repository, tip)
1424
        stream = source.get_stream(search)
1425
        from_format = source_tree.branch.repository._format
1426
        sink = target._get_sink()
1427
        sink.insert_stream(stream, from_format, [])
1428
        if expect_pack_called:
1429
            self.assertLength(1, self.calls)
1430
        else:
1431
            self.assertLength(0, self.calls)
1432
1433
    def run_fetch(self, src_fmt, target_fmt, expect_pack_called):
1434
        self.expect_hint = expect_pack_called
1435
        self.calls = []
1436
        source_tree = self.make_branch_and_tree('src', format=src_fmt)
1437
        source_tree.lock_write()
1438
        self.addCleanup(source_tree.unlock)
1439
        tip = source_tree.commit('foo')
1440
        target = self.make_repository('target', format=target_fmt)
1441
        target.lock_write()
1442
        self.addCleanup(target.unlock)
1443
        source = source_tree.branch.repository
1444
        self.orig_pack = target.pack
1445
        target.pack = self.log_pack
1446
        target.fetch(source)
1447
        if expect_pack_called:
1448
            self.assertLength(1, self.calls)
1449
        else:
1450
            self.assertLength(0, self.calls)
1451
1452
    def test_sink_format_hint_no(self):
1453
        # When the target format says packing makes no difference, pack is not
1454
        # called.
1455
        self.run_stream('1.9', 'rich-root-pack', False)
1456
1457
    def test_sink_format_hint_yes(self):
1458
        # When the target format says packing makes a difference, pack is
1459
        # called.
1460
        self.run_stream('1.9', '2a', True)
1461
1462
    def test_sink_format_same_no(self):
1463
        # When the formats are the same, pack is not called.
1464
        self.run_stream('2a', '2a', False)
1465
1466
    def test_IDS_format_hint_no(self):
1467
        # When the target format says packing makes no difference, pack is not
1468
        # called.
1469
        self.run_fetch('1.9', 'rich-root-pack', False)
1470
1471
    def test_IDS_format_hint_yes(self):
1472
        # When the target format says packing makes a difference, pack is
1473
        # called.
1474
        self.run_fetch('1.9', '2a', True)
1475
1476
    def test_IDS_format_same_no(self):
1477
        # When the formats are the same, pack is not called.
1478
        self.run_fetch('2a', '2a', False)