/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/tests/test_branchbuilder.py

  • Committer: Jelmer Vernooij
  • Date: 2018-05-19 13:16:11 UTC
  • mto: (6968.4.3 git-archive)
  • mto: This revision was merged to the branch mainline in revision 6972.
  • Revision ID: jelmer@jelmer.uk-20180519131611-l9h9ud41j7qg1m03
Move tar/zip to breezy.archive.

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2007-2011 Canonical Ltd
 
2
#
 
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.
 
7
#
 
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.
 
12
#
 
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
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
"""Tests for the BranchBuilder class."""
 
18
 
 
19
from .. import (
 
20
    branch as _mod_branch,
 
21
    revision as _mod_revision,
 
22
    tests,
 
23
    )
 
24
from ..bzr import (
 
25
    branch as _mod_bzrbranch,
 
26
    )
 
27
from ..branchbuilder import BranchBuilder
 
28
 
 
29
 
 
30
class TestBranchBuilder(tests.TestCaseWithMemoryTransport):
 
31
 
 
32
    def test_create(self):
 
33
        """Test the constructor api."""
 
34
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
35
        # we dont care if the branch has been built or not at this point.
 
36
 
 
37
    def test_get_branch(self):
 
38
        """get_branch returns the created branch."""
 
39
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
40
        branch = builder.get_branch()
 
41
        self.assertIsInstance(branch, _mod_branch.Branch)
 
42
        self.assertEqual(self.get_transport().clone('foo').base,
 
43
            branch.base)
 
44
        self.assertEqual(
 
45
            (0, _mod_revision.NULL_REVISION),
 
46
            branch.last_revision_info())
 
47
 
 
48
    def test_format(self):
 
49
        """Making a BranchBuilder with a format option sets the branch type."""
 
50
        builder = BranchBuilder(self.get_transport(), format='dirstate-tags')
 
51
        branch = builder.get_branch()
 
52
        self.assertIsInstance(branch, _mod_bzrbranch.BzrBranch6)
 
53
 
 
54
    def test_build_one_commit(self):
 
55
        """doing build_commit causes a commit to happen."""
 
56
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
57
        rev_id = builder.build_commit()
 
58
        branch = builder.get_branch()
 
59
        self.assertEqual((1, rev_id), branch.last_revision_info())
 
60
        self.assertEqual(
 
61
            'commit 1',
 
62
            branch.repository.get_revision(branch.last_revision()).message)
 
63
 
 
64
    def test_build_commit_timestamp(self):
 
65
        """You can set a date when committing."""
 
66
        builder = self.make_branch_builder('foo')
 
67
        rev_id = builder.build_commit(timestamp=1236043340)
 
68
        branch = builder.get_branch()
 
69
        self.assertEqual((1, rev_id), branch.last_revision_info())
 
70
        rev = branch.repository.get_revision(branch.last_revision())
 
71
        self.assertEqual(
 
72
            'commit 1',
 
73
            rev.message)
 
74
        self.assertEqual(
 
75
            1236043340,
 
76
            int(rev.timestamp))
 
77
 
 
78
    def test_build_two_commits(self):
 
79
        """The second commit has the right parents and message."""
 
80
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
81
        rev_id1 = builder.build_commit()
 
82
        rev_id2 = builder.build_commit()
 
83
        branch = builder.get_branch()
 
84
        self.assertEqual((2, rev_id2), branch.last_revision_info())
 
85
        self.assertEqual(
 
86
            'commit 2',
 
87
            branch.repository.get_revision(branch.last_revision()).message)
 
88
        self.assertEqual(
 
89
            [rev_id1],
 
90
            branch.repository.get_revision(branch.last_revision()).parent_ids)
 
91
 
 
92
    def test_build_commit_parent_ids(self):
 
93
        """build_commit() takes a parent_ids argument."""
 
94
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
95
        rev_id1 = builder.build_commit(
 
96
            parent_ids=["ghost"], allow_leftmost_as_ghost=True)
 
97
        rev_id2 = builder.build_commit(parent_ids=[])
 
98
        branch = builder.get_branch()
 
99
        self.assertEqual((1, rev_id2), branch.last_revision_info())
 
100
        self.assertEqual(
 
101
            ["ghost"],
 
102
            branch.repository.get_revision(rev_id1).parent_ids)
 
103
 
 
104
 
 
105
class TestBranchBuilderBuildSnapshot(tests.TestCaseWithMemoryTransport):
 
106
 
 
107
    def assertTreeShape(self, expected_shape, tree):
 
108
        """Check that the tree shape matches expectations."""
 
109
        tree.lock_read()
 
110
        try:
 
111
            entries = [(path, ie.file_id, ie.kind)
 
112
                       for path, ie in tree.iter_entries_by_dir()]
 
113
        finally:
 
114
            tree.unlock()
 
115
        self.assertEqual(expected_shape, entries)
 
116
 
 
117
    def build_a_rev(self):
 
118
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
119
        rev_id1 = builder.build_snapshot(None,
 
120
            [('add', ('', 'a-root-id', 'directory', None)),
 
121
             ('add', ('a', 'a-id', 'file', 'contents'))],
 
122
            revision_id='A-id')
 
123
        self.assertEqual('A-id', rev_id1)
 
124
        return builder
 
125
 
 
126
    def test_add_one_file(self):
 
127
        builder = self.build_a_rev()
 
128
        branch = builder.get_branch()
 
129
        self.assertEqual((1, 'A-id'), branch.last_revision_info())
 
130
        rev_tree = branch.repository.revision_tree('A-id')
 
131
        rev_tree.lock_read()
 
132
        self.addCleanup(rev_tree.unlock)
 
133
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
134
                              (u'a', 'a-id', 'file')], rev_tree)
 
135
        self.assertEqual('contents', rev_tree.get_file_text('a'))
 
136
 
 
137
    def test_add_second_file(self):
 
138
        builder = self.build_a_rev()
 
139
        rev_id2 = builder.build_snapshot(None,
 
140
            [('add', ('b', 'b-id', 'file', 'content_b'))],
 
141
            revision_id='B-id')
 
142
        self.assertEqual('B-id', rev_id2)
 
143
        branch = builder.get_branch()
 
144
        self.assertEqual((2, rev_id2), branch.last_revision_info())
 
145
        rev_tree = branch.repository.revision_tree(rev_id2)
 
146
        rev_tree.lock_read()
 
147
        self.addCleanup(rev_tree.unlock)
 
148
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
149
                              (u'a', 'a-id', 'file'),
 
150
                              (u'b', 'b-id', 'file')], rev_tree)
 
151
        self.assertEqual('content_b', rev_tree.get_file_text('b'))
 
152
 
 
153
    def test_add_empty_dir(self):
 
154
        builder = self.build_a_rev()
 
155
        rev_id2 = builder.build_snapshot(None,
 
156
            [('add', ('b', 'b-id', 'directory', None))],
 
157
            revision_id='B-id')
 
158
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
159
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
160
                              (u'a', 'a-id', 'file'),
 
161
                              (u'b', 'b-id', 'directory'),
 
162
                             ], rev_tree)
 
163
 
 
164
    def test_commit_timestamp(self):
 
165
        builder = self.make_branch_builder('foo')
 
166
        rev_id = builder.build_snapshot(None,
 
167
            [('add', (u'', None, 'directory', None))],
 
168
            timestamp=1234567890)
 
169
        rev = builder.get_branch().repository.get_revision(rev_id)
 
170
        self.assertEqual(
 
171
            1234567890,
 
172
            int(rev.timestamp))
 
173
 
 
174
    def test_commit_message_default(self):
 
175
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
176
        rev_id = builder.build_snapshot(None,
 
177
            [('add', (u'', None, 'directory', None))])
 
178
        branch = builder.get_branch()
 
179
        rev = branch.repository.get_revision(rev_id)
 
180
        self.assertEqual(u'commit 1', rev.message)
 
181
 
 
182
    def test_commit_message_supplied(self):
 
183
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
184
        rev_id = builder.build_snapshot(None,
 
185
            [('add', (u'', None, 'directory', None))],
 
186
            message=u'Foo')
 
187
        branch = builder.get_branch()
 
188
        rev = branch.repository.get_revision(rev_id)
 
189
        self.assertEqual(u'Foo', rev.message)
 
190
 
 
191
    def test_commit_message_callback(self):
 
192
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
193
        rev_id = builder.build_snapshot(None,
 
194
            [('add', (u'', None, 'directory', None))],
 
195
            message_callback=lambda x:u'Foo')
 
196
        branch = builder.get_branch()
 
197
        rev = branch.repository.get_revision(rev_id)
 
198
        self.assertEqual(u'Foo', rev.message)
 
199
 
 
200
    def test_modify_file(self):
 
201
        builder = self.build_a_rev()
 
202
        rev_id2 = builder.build_snapshot(None,
 
203
            [('modify', ('a', 'new\ncontent\n'))],
 
204
            revision_id='B-id')
 
205
        self.assertEqual('B-id', rev_id2)
 
206
        branch = builder.get_branch()
 
207
        rev_tree = branch.repository.revision_tree(rev_id2)
 
208
        rev_tree.lock_read()
 
209
        self.addCleanup(rev_tree.unlock)
 
210
        self.assertEqual('new\ncontent\n',
 
211
                         rev_tree.get_file_text(rev_tree.id2path(b'a-id')))
 
212
 
 
213
    def test_delete_file(self):
 
214
        builder = self.build_a_rev()
 
215
        rev_id2 = builder.build_snapshot(None,
 
216
            [('unversion', 'a')], revision_id='B-id')
 
217
        self.assertEqual('B-id', rev_id2)
 
218
        branch = builder.get_branch()
 
219
        rev_tree = branch.repository.revision_tree(rev_id2)
 
220
        rev_tree.lock_read()
 
221
        self.addCleanup(rev_tree.unlock)
 
222
        self.assertTreeShape([(u'', 'a-root-id', 'directory')], rev_tree)
 
223
 
 
224
    def test_delete_directory(self):
 
225
        builder = self.build_a_rev()
 
226
        rev_id2 = builder.build_snapshot(None,
 
227
            [('add', ('b', 'b-id', 'directory', None)),
 
228
             ('add', ('b/c', 'c-id', 'file', 'foo\n')),
 
229
             ('add', ('b/d', 'd-id', 'directory', None)),
 
230
             ('add', ('b/d/e', 'e-id', 'file', 'eff\n')),
 
231
            ], revision_id='B-id')
 
232
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
233
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
234
                              (u'a', 'a-id', 'file'),
 
235
                              (u'b', 'b-id', 'directory'),
 
236
                              (u'b/c', 'c-id', 'file'),
 
237
                              (u'b/d', 'd-id', 'directory'),
 
238
                              (u'b/d/e', 'e-id', 'file')], rev_tree)
 
239
        # Removing a directory removes all child dirs
 
240
        builder.build_snapshot(
 
241
                None, [('unversion', 'b')],
 
242
                revision_id='C-id')
 
243
        rev_tree = builder.get_branch().repository.revision_tree('C-id')
 
244
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
245
                              (u'a', 'a-id', 'file'),
 
246
                             ], rev_tree)
 
247
 
 
248
    def test_unknown_action(self):
 
249
        builder = self.build_a_rev()
 
250
        e = self.assertRaises(ValueError,
 
251
            builder.build_snapshot, None, [('weirdo', ('foo',))],
 
252
            revision_id='B-id')
 
253
        self.assertEqual('Unknown build action: "weirdo"', str(e))
 
254
 
 
255
    def test_rename(self):
 
256
        builder = self.build_a_rev()
 
257
        builder.build_snapshot(None,
 
258
            [('rename', ('a', 'b'))], revision_id='B-id')
 
259
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
260
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
261
                              (u'b', 'a-id', 'file')], rev_tree)
 
262
 
 
263
    def test_rename_into_subdir(self):
 
264
        builder = self.build_a_rev()
 
265
        builder.build_snapshot(None,
 
266
            [('add', ('dir', 'dir-id', 'directory', None)),
 
267
             ('rename', ('a', 'dir/a'))], revision_id='B-id')
 
268
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
269
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
270
                              (u'dir', 'dir-id', 'directory'),
 
271
                              (u'dir/a', 'a-id', 'file')], rev_tree)
 
272
 
 
273
    def test_rename_out_of_unversioned_subdir(self):
 
274
        builder = self.build_a_rev()
 
275
        builder.build_snapshot(None,
 
276
            [('add', ('dir', 'dir-id', 'directory', None)),
 
277
             ('rename', ('a', 'dir/a'))],
 
278
            revision_id='B-id')
 
279
        builder.build_snapshot(None,
 
280
            [('rename', ('dir/a', 'a')),
 
281
             ('unversion', 'dir')], revision_id='C-id')
 
282
        rev_tree = builder.get_branch().repository.revision_tree('C-id')
 
283
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
284
                              (u'a', 'a-id', 'file')], rev_tree)
 
285
 
 
286
    def test_set_parent(self):
 
287
        builder = self.build_a_rev()
 
288
        builder.start_series()
 
289
        self.addCleanup(builder.finish_series)
 
290
        builder.build_snapshot(['A-id'],
 
291
            [('modify', ('a', 'new\ncontent\n'))],
 
292
            revision_id='B-id')
 
293
        builder.build_snapshot(['A-id'],
 
294
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))],
 
295
            revision_id='C-id')
 
296
        # We should now have a graph:
 
297
        #   A
 
298
        #   |\
 
299
        #   C B
 
300
        # And not A => B => C
 
301
        repo = builder.get_branch().repository
 
302
        self.assertEqual({'B-id': ('A-id',), 'C-id': ('A-id',)},
 
303
                         repo.get_parent_map(['B-id', 'C-id']))
 
304
        b_tree = repo.revision_tree('B-id')
 
305
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
306
                              (u'a', 'a-id', 'file'),
 
307
                             ], b_tree)
 
308
        self.assertEqual('new\ncontent\n', b_tree.get_file_text('a'))
 
309
 
 
310
        # We should still be using the content from A in C, not from B
 
311
        c_tree = repo.revision_tree('C-id')
 
312
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
313
                              (u'a', 'a-id', 'file'),
 
314
                              (u'c', 'c-id', 'file'),
 
315
                             ], c_tree)
 
316
        self.assertEqual('contents', c_tree.get_file_text('a'))
 
317
        self.assertEqual('alt\ncontent\n', c_tree.get_file_text('c'))
 
318
 
 
319
    def test_set_merge_parent(self):
 
320
        builder = self.build_a_rev()
 
321
        builder.start_series()
 
322
        self.addCleanup(builder.finish_series)
 
323
        builder.build_snapshot(['A-id'],
 
324
            [('add', ('b', 'b-id', 'file', 'b\ncontent\n'))],
 
325
            revision_id='B-id')
 
326
        builder.build_snapshot(['A-id'],
 
327
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))],
 
328
            revision_id='C-id')
 
329
        builder.build_snapshot(['B-id', 'C-id'], [], revision_id='D-id')
 
330
        repo = builder.get_branch().repository
 
331
        self.assertEqual({'B-id': ('A-id',), 'C-id': ('A-id',),
 
332
                          'D-id': ('B-id', 'C-id')},
 
333
                         repo.get_parent_map(['B-id', 'C-id', 'D-id']))
 
334
        d_tree = repo.revision_tree('D-id')
 
335
        # Note: by default a merge node does *not* pull in the changes from the
 
336
        #       merged tree, you have to supply it yourself.
 
337
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
338
                              (u'a', 'a-id', 'file'),
 
339
                              (u'b', 'b-id', 'file'),
 
340
                             ], d_tree)
 
341
 
 
342
    def test_set_merge_parent_and_contents(self):
 
343
        builder = self.build_a_rev()
 
344
        builder.start_series()
 
345
        self.addCleanup(builder.finish_series)
 
346
        builder.build_snapshot(['A-id'],
 
347
            [('add', ('b', 'b-id', 'file', 'b\ncontent\n'))],
 
348
            revision_id='B-id')
 
349
        builder.build_snapshot(['A-id'],
 
350
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))],
 
351
            revision_id='C-id')
 
352
        builder.build_snapshot(['B-id', 'C-id'],
 
353
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))],
 
354
            revision_id='D-id')
 
355
        repo = builder.get_branch().repository
 
356
        self.assertEqual({'B-id': ('A-id',), 'C-id': ('A-id',),
 
357
                          'D-id': ('B-id', 'C-id')},
 
358
                         repo.get_parent_map(['B-id', 'C-id', 'D-id']))
 
359
        d_tree = repo.revision_tree('D-id')
 
360
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
361
                              (u'a', 'a-id', 'file'),
 
362
                              (u'b', 'b-id', 'file'),
 
363
                              (u'c', 'c-id', 'file'),
 
364
                             ], d_tree)
 
365
        # Because we copied the exact text into *this* tree, the 'c' file
 
366
        # should look like it was not modified in the merge
 
367
        self.assertEqual('C-id', d_tree.get_file_revision('c'))
 
368
 
 
369
    def test_set_parent_to_null(self):
 
370
        builder = self.build_a_rev()
 
371
        builder.start_series()
 
372
        self.addCleanup(builder.finish_series)
 
373
        builder.build_snapshot([],
 
374
            [('add', ('', None, 'directory', None))],
 
375
            revision_id='B-id')
 
376
        # We should now have a graph:
 
377
        #   A B
 
378
        # And not A => B
 
379
        repo = builder.get_branch().repository
 
380
        self.assertEqual({'A-id': (_mod_revision.NULL_REVISION,),
 
381
                          'B-id': (_mod_revision.NULL_REVISION,),},
 
382
                         repo.get_parent_map(['A-id', 'B-id']))
 
383
 
 
384
    
 
385
    def test_start_finish_series(self):
 
386
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
387
        builder.start_series()
 
388
        try:
 
389
            self.assertIsNot(None, builder._tree)
 
390
            self.assertEqual('w', builder._tree._lock_mode)
 
391
            self.assertTrue(builder._branch.is_locked())
 
392
        finally:
 
393
            builder.finish_series()
 
394
        self.assertIs(None, builder._tree)
 
395
        self.assertFalse(builder._branch.is_locked())
 
396
 
 
397
    def test_ghost_mainline_history(self):
 
398
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
399
        builder.start_series()
 
400
        try:
 
401
            builder.build_snapshot(['ghost'],
 
402
                [('add', ('', 'ROOT_ID', 'directory', ''))],
 
403
                allow_leftmost_as_ghost=True, revision_id='tip')
 
404
        finally:
 
405
            builder.finish_series()
 
406
        b = builder.get_branch()
 
407
        b.lock_read()
 
408
        self.addCleanup(b.unlock)
 
409
        self.assertEqual(('ghost',),
 
410
            b.repository.get_graph().get_parent_map(['tip'])['tip'])
 
411
 
 
412
    def test_unversion_root_add_new_root(self):
 
413
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
414
        builder.start_series()
 
415
        builder.build_snapshot(None,
 
416
            [('add', ('', 'TREE_ROOT', 'directory', ''))],
 
417
            revision_id='rev-1')
 
418
        builder.build_snapshot(None,
 
419
            [('unversion', ''),
 
420
             ('add', ('', 'my-root', 'directory', ''))],
 
421
            revision_id='rev-2')
 
422
        builder.finish_series()
 
423
        rev_tree = builder.get_branch().repository.revision_tree('rev-2')
 
424
        self.assertTreeShape([(u'', 'my-root', 'directory')], rev_tree)
 
425
 
 
426
    def test_empty_flush(self):
 
427
        """A flush with no actions before it is a no-op."""
 
428
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
429
        builder.start_series()
 
430
        builder.build_snapshot(None,
 
431
            [('add', ('', 'TREE_ROOT', 'directory', ''))],
 
432
            revision_id='rev-1')
 
433
        builder.build_snapshot(None, [('flush', None)], revision_id='rev-2')
 
434
        builder.finish_series()
 
435
        rev_tree = builder.get_branch().repository.revision_tree('rev-2')
 
436
        self.assertTreeShape([(u'', 'TREE_ROOT', 'directory')], rev_tree)
 
437
 
 
438
    def test_kind_change(self):
 
439
        """It's possible to change the kind of an entry in a single snapshot
 
440
        with a bit of help from the 'flush' action.
 
441
        """
 
442
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
443
        builder.start_series()
 
444
        builder.build_snapshot(None,
 
445
            [('add', (u'', 'a-root-id', 'directory', None)),
 
446
             ('add', (u'a', 'a-id', 'file', 'content\n'))],
 
447
            revision_id='A-id')
 
448
        builder.build_snapshot(None,
 
449
            [('unversion', 'a'),
 
450
             ('flush', None),
 
451
             ('add', (u'a', 'a-id', 'directory', None))],
 
452
            revision_id='B-id')
 
453
        builder.finish_series()
 
454
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
455
        self.assertTreeShape(
 
456
            [(u'', 'a-root-id', 'directory'), (u'a', 'a-id', 'directory')],
 
457
            rev_tree)
 
458
 
 
459
    def test_pivot_root(self):
 
460
        """It's possible (albeit awkward) to move an existing dir to the root
 
461
        in a single snapshot by using unversion then flush then add.
 
462
        """
 
463
        builder = BranchBuilder(self.get_transport().clone('foo'))
 
464
        builder.start_series()
 
465
        builder.build_snapshot(None,
 
466
            [('add', (u'', 'orig-root', 'directory', None)),
 
467
             ('add', (u'dir', 'dir-id', 'directory', None))],
 
468
            revision_id='A-id')
 
469
        builder.build_snapshot(None,
 
470
            [('unversion', ''),  # implicitly unversions all children
 
471
             ('flush', None),
 
472
             ('add', (u'', 'dir-id', 'directory', None))],
 
473
            revision_id='B-id')
 
474
        builder.finish_series()
 
475
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
476
        self.assertTreeShape([(u'', 'dir-id', 'directory')], rev_tree)
 
477