/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: 2017-06-08 23:30:31 UTC
  • mto: This revision was merged to the branch mainline in revision 6690.
  • Revision ID: jelmer@jelmer.uk-20170608233031-3qavls2o7a1pqllj
Update imports.

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/test_branchbuilder.py
40
40
        branch = builder.get_branch()
41
41
        self.assertIsInstance(branch, _mod_branch.Branch)
42
42
        self.assertEqual(self.get_transport().clone('foo').base,
43
 
                         branch.base)
 
43
            branch.base)
44
44
        self.assertEqual(
45
45
            (0, _mod_revision.NULL_REVISION),
46
46
            branch.last_revision_info())
93
93
        """build_commit() takes a parent_ids argument."""
94
94
        builder = BranchBuilder(self.get_transport().clone('foo'))
95
95
        rev_id1 = builder.build_commit(
96
 
            parent_ids=[b"ghost"], allow_leftmost_as_ghost=True)
 
96
            parent_ids=["ghost"], allow_leftmost_as_ghost=True)
97
97
        rev_id2 = builder.build_commit(parent_ids=[])
98
98
        branch = builder.get_branch()
99
99
        self.assertEqual((1, rev_id2), branch.last_revision_info())
100
100
        self.assertEqual(
101
 
            [b"ghost"],
 
101
            ["ghost"],
102
102
            branch.repository.get_revision(rev_id1).parent_ids)
103
103
 
104
104
 
116
116
 
117
117
    def build_a_rev(self):
118
118
        builder = BranchBuilder(self.get_transport().clone('foo'))
119
 
        rev_id1 = builder.build_snapshot(None,
120
 
                                         [('add', ('', b'a-root-id', 'directory', None)),
121
 
                                          ('add', ('a', b'a-id', 'file', b'contents'))],
122
 
                                         revision_id=b'A-id')
123
 
        self.assertEqual(b'A-id', rev_id1)
 
119
        rev_id1 = builder.build_snapshot('A-id', None,
 
120
            [('add', ('', 'a-root-id', 'directory', None)),
 
121
             ('add', ('a', 'a-id', 'file', 'contents'))])
 
122
        self.assertEqual('A-id', rev_id1)
124
123
        return builder
125
124
 
126
125
    def test_add_one_file(self):
127
126
        builder = self.build_a_rev()
128
127
        branch = builder.get_branch()
129
 
        self.assertEqual((1, b'A-id'), branch.last_revision_info())
130
 
        rev_tree = branch.repository.revision_tree(b'A-id')
 
128
        self.assertEqual((1, 'A-id'), branch.last_revision_info())
 
129
        rev_tree = branch.repository.revision_tree('A-id')
131
130
        rev_tree.lock_read()
132
131
        self.addCleanup(rev_tree.unlock)
133
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
134
 
                              (u'a', b'a-id', 'file')], rev_tree)
135
 
        self.assertEqual(b'contents', rev_tree.get_file_text('a'))
 
132
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
133
                              (u'a', 'a-id', 'file')], rev_tree)
 
134
        self.assertEqual('contents', rev_tree.get_file_text('a-id'))
136
135
 
137
136
    def test_add_second_file(self):
138
137
        builder = self.build_a_rev()
139
 
        rev_id2 = builder.build_snapshot(None,
140
 
                                         [('add', ('b', b'b-id', 'file', b'content_b'))],
141
 
                                         revision_id=b'B-id')
142
 
        self.assertEqual(b'B-id', rev_id2)
 
138
        rev_id2 = builder.build_snapshot('B-id', None,
 
139
            [('add', ('b', 'b-id', 'file', 'content_b'))])
 
140
        self.assertEqual('B-id', rev_id2)
143
141
        branch = builder.get_branch()
144
142
        self.assertEqual((2, rev_id2), branch.last_revision_info())
145
143
        rev_tree = branch.repository.revision_tree(rev_id2)
146
144
        rev_tree.lock_read()
147
145
        self.addCleanup(rev_tree.unlock)
148
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
149
 
                              (u'a', b'a-id', 'file'),
150
 
                              (u'b', b'b-id', 'file')], rev_tree)
151
 
        self.assertEqual(b'content_b', rev_tree.get_file_text('b'))
 
146
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
147
                              (u'a', 'a-id', 'file'),
 
148
                              (u'b', 'b-id', 'file')], rev_tree)
 
149
        self.assertEqual('content_b', rev_tree.get_file_text('b-id'))
152
150
 
153
151
    def test_add_empty_dir(self):
154
152
        builder = self.build_a_rev()
155
 
        rev_id2 = builder.build_snapshot(None,
156
 
                                         [('add', ('b', b'b-id', 'directory', None))],
157
 
                                         revision_id=b'B-id')
158
 
        rev_tree = builder.get_branch().repository.revision_tree(b'B-id')
159
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
160
 
                              (u'a', b'a-id', 'file'),
161
 
                              (u'b', b'b-id', 'directory'),
162
 
                              ], rev_tree)
 
153
        rev_id2 = builder.build_snapshot('B-id', None,
 
154
            [('add', ('b', 'b-id', 'directory', None))])
 
155
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
156
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
157
                              (u'a', 'a-id', 'file'),
 
158
                              (u'b', 'b-id', 'directory'),
 
159
                             ], rev_tree)
163
160
 
164
161
    def test_commit_timestamp(self):
165
162
        builder = self.make_branch_builder('foo')
166
 
        rev_id = builder.build_snapshot(None,
167
 
                                        [('add', (u'', None, 'directory', None))],
168
 
                                        timestamp=1234567890)
 
163
        rev_id = builder.build_snapshot(None, None,
 
164
            [('add', (u'', None, 'directory', None))],
 
165
            timestamp=1234567890)
169
166
        rev = builder.get_branch().repository.get_revision(rev_id)
170
167
        self.assertEqual(
171
168
            1234567890,
173
170
 
174
171
    def test_commit_message_default(self):
175
172
        builder = BranchBuilder(self.get_transport().clone('foo'))
176
 
        rev_id = builder.build_snapshot(None,
177
 
                                        [('add', (u'', None, 'directory', None))])
 
173
        rev_id = builder.build_snapshot(None, None,
 
174
            [('add', (u'', None, 'directory', None))])
178
175
        branch = builder.get_branch()
179
176
        rev = branch.repository.get_revision(rev_id)
180
177
        self.assertEqual(u'commit 1', rev.message)
181
178
 
182
179
    def test_commit_message_supplied(self):
183
180
        builder = BranchBuilder(self.get_transport().clone('foo'))
184
 
        rev_id = builder.build_snapshot(None,
185
 
                                        [('add', (u'', None, 'directory', None))],
186
 
                                        message=u'Foo')
 
181
        rev_id = builder.build_snapshot(None, None,
 
182
            [('add', (u'', None, 'directory', None))],
 
183
            message=u'Foo')
187
184
        branch = builder.get_branch()
188
185
        rev = branch.repository.get_revision(rev_id)
189
186
        self.assertEqual(u'Foo', rev.message)
190
187
 
191
188
    def test_commit_message_callback(self):
192
189
        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')
 
190
        rev_id = builder.build_snapshot(None, None,
 
191
            [('add', (u'', None, 'directory', None))],
 
192
            message_callback=lambda x:u'Foo')
196
193
        branch = builder.get_branch()
197
194
        rev = branch.repository.get_revision(rev_id)
198
195
        self.assertEqual(u'Foo', rev.message)
199
196
 
200
197
    def test_modify_file(self):
201
198
        builder = self.build_a_rev()
202
 
        rev_id2 = builder.build_snapshot(None,
203
 
                                         [('modify', ('a', b'new\ncontent\n'))],
204
 
                                         revision_id=b'B-id')
205
 
        self.assertEqual(b'B-id', rev_id2)
 
199
        rev_id2 = builder.build_snapshot('B-id', None,
 
200
            [('modify', ('a-id', 'new\ncontent\n'))])
 
201
        self.assertEqual('B-id', rev_id2)
206
202
        branch = builder.get_branch()
207
203
        rev_tree = branch.repository.revision_tree(rev_id2)
208
204
        rev_tree.lock_read()
209
205
        self.addCleanup(rev_tree.unlock)
210
 
        self.assertEqual(b'new\ncontent\n',
211
 
                         rev_tree.get_file_text('a'))
 
206
        self.assertEqual('new\ncontent\n', rev_tree.get_file_text('a-id'))
212
207
 
213
208
    def test_delete_file(self):
214
209
        builder = self.build_a_rev()
215
 
        rev_id2 = builder.build_snapshot(None,
216
 
                                         [('unversion', 'a')], revision_id=b'B-id')
217
 
        self.assertEqual(b'B-id', rev_id2)
 
210
        rev_id2 = builder.build_snapshot('B-id', None,
 
211
            [('unversion', 'a-id')])
 
212
        self.assertEqual('B-id', rev_id2)
218
213
        branch = builder.get_branch()
219
214
        rev_tree = branch.repository.revision_tree(rev_id2)
220
215
        rev_tree.lock_read()
221
216
        self.addCleanup(rev_tree.unlock)
222
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory')], rev_tree)
 
217
        self.assertTreeShape([(u'', 'a-root-id', 'directory')], rev_tree)
223
218
 
224
219
    def test_delete_directory(self):
225
220
        builder = self.build_a_rev()
226
 
        rev_id2 = builder.build_snapshot(None,
227
 
                                         [('add', ('b', b'b-id', 'directory', None)),
228
 
                                          ('add', ('b/c', b'c-id', 'file', b'foo\n')),
229
 
                                             ('add', ('b/d', b'd-id',
230
 
                                                      'directory', None)),
231
 
                                             ('add', ('b/d/e', b'e-id',
232
 
                                                      'file', b'eff\n')),
233
 
                                          ], revision_id=b'B-id')
234
 
        rev_tree = builder.get_branch().repository.revision_tree(b'B-id')
235
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
236
 
                              (u'a', b'a-id', 'file'),
237
 
                              (u'b', b'b-id', 'directory'),
238
 
                              (u'b/c', b'c-id', 'file'),
239
 
                              (u'b/d', b'd-id', 'directory'),
240
 
                              (u'b/d/e', b'e-id', 'file')], rev_tree)
 
221
        rev_id2 = builder.build_snapshot('B-id', None,
 
222
            [('add', ('b', 'b-id', 'directory', None)),
 
223
             ('add', ('b/c', 'c-id', 'file', 'foo\n')),
 
224
             ('add', ('b/d', 'd-id', 'directory', None)),
 
225
             ('add', ('b/d/e', 'e-id', 'file', 'eff\n')),
 
226
            ])
 
227
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
228
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
229
                              (u'a', 'a-id', 'file'),
 
230
                              (u'b', 'b-id', 'directory'),
 
231
                              (u'b/c', 'c-id', 'file'),
 
232
                              (u'b/d', 'd-id', 'directory'),
 
233
                              (u'b/d/e', 'e-id', 'file')], rev_tree)
241
234
        # Removing a directory removes all child dirs
242
 
        builder.build_snapshot(
243
 
            None, [('unversion', 'b')],
244
 
            revision_id=b'C-id')
245
 
        rev_tree = builder.get_branch().repository.revision_tree(b'C-id')
246
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
247
 
                              (u'a', b'a-id', 'file'),
248
 
                              ], rev_tree)
 
235
        builder.build_snapshot('C-id', None, [('unversion', 'b-id')])
 
236
        rev_tree = builder.get_branch().repository.revision_tree('C-id')
 
237
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
238
                              (u'a', 'a-id', 'file'),
 
239
                             ], rev_tree)
249
240
 
250
241
    def test_unknown_action(self):
251
242
        builder = self.build_a_rev()
252
243
        e = self.assertRaises(ValueError,
253
 
                              builder.build_snapshot, None, [
254
 
                                  ('weirdo', ('foo',))],
255
 
                              revision_id=b'B-id')
 
244
            builder.build_snapshot, 'B-id', None, [('weirdo', ('foo',))])
256
245
        self.assertEqual('Unknown build action: "weirdo"', str(e))
257
246
 
258
247
    def test_rename(self):
259
248
        builder = self.build_a_rev()
260
 
        builder.build_snapshot(None,
261
 
                               [('rename', ('a', 'b'))], revision_id=b'B-id')
262
 
        rev_tree = builder.get_branch().repository.revision_tree(b'B-id')
263
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
264
 
                              (u'b', b'a-id', 'file')], rev_tree)
 
249
        builder.build_snapshot('B-id', None,
 
250
            [('rename', ('a', 'b'))])
 
251
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
252
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
253
                              (u'b', 'a-id', 'file')], rev_tree)
265
254
 
266
255
    def test_rename_into_subdir(self):
267
256
        builder = self.build_a_rev()
268
 
        builder.build_snapshot(None,
269
 
                               [('add', ('dir', b'dir-id', 'directory', None)),
270
 
                                ('rename', ('a', 'dir/a'))], revision_id=b'B-id')
271
 
        rev_tree = builder.get_branch().repository.revision_tree(b'B-id')
272
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
273
 
                              (u'dir', b'dir-id', 'directory'),
274
 
                              (u'dir/a', b'a-id', 'file')], rev_tree)
 
257
        builder.build_snapshot('B-id', None,
 
258
            [('add', ('dir', 'dir-id', 'directory', None)),
 
259
             ('rename', ('a', 'dir/a'))])
 
260
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
261
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
262
                              (u'dir', 'dir-id', 'directory'),
 
263
                              (u'dir/a', 'a-id', 'file')], rev_tree)
275
264
 
276
265
    def test_rename_out_of_unversioned_subdir(self):
277
266
        builder = self.build_a_rev()
278
 
        builder.build_snapshot(None,
279
 
                               [('add', ('dir', b'dir-id', 'directory', None)),
280
 
                                ('rename', ('a', 'dir/a'))],
281
 
                               revision_id=b'B-id')
282
 
        builder.build_snapshot(None,
283
 
                               [('rename', ('dir/a', 'a')),
284
 
                                ('unversion', 'dir')], revision_id=b'C-id')
285
 
        rev_tree = builder.get_branch().repository.revision_tree(b'C-id')
286
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
287
 
                              (u'a', b'a-id', 'file')], rev_tree)
 
267
        builder.build_snapshot('B-id', None,
 
268
            [('add', ('dir', 'dir-id', 'directory', None)),
 
269
             ('rename', ('a', 'dir/a'))])
 
270
        builder.build_snapshot('C-id', None,
 
271
            [('rename', ('dir/a', 'a')),
 
272
             ('unversion', 'dir-id')])
 
273
        rev_tree = builder.get_branch().repository.revision_tree('C-id')
 
274
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
275
                              (u'a', 'a-id', 'file')], rev_tree)
288
276
 
289
277
    def test_set_parent(self):
290
278
        builder = self.build_a_rev()
291
279
        builder.start_series()
292
280
        self.addCleanup(builder.finish_series)
293
 
        builder.build_snapshot([b'A-id'],
294
 
                               [('modify', ('a', b'new\ncontent\n'))],
295
 
                               revision_id=b'B-id')
296
 
        builder.build_snapshot([b'A-id'],
297
 
                               [('add', ('c', b'c-id', 'file', b'alt\ncontent\n'))],
298
 
                               revision_id=b'C-id')
 
281
        builder.build_snapshot('B-id', ['A-id'],
 
282
            [('modify', ('a-id', 'new\ncontent\n'))])
 
283
        builder.build_snapshot('C-id', ['A-id'],
 
284
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))])
299
285
        # We should now have a graph:
300
286
        #   A
301
287
        #   |\
302
288
        #   C B
303
289
        # And not A => B => C
304
290
        repo = builder.get_branch().repository
305
 
        self.assertEqual({b'B-id': (b'A-id',), b'C-id': (b'A-id',)},
306
 
                         repo.get_parent_map([b'B-id', b'C-id']))
307
 
        b_tree = repo.revision_tree(b'B-id')
308
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
309
 
                              (u'a', b'a-id', 'file'),
310
 
                              ], b_tree)
311
 
        self.assertEqual(b'new\ncontent\n', b_tree.get_file_text('a'))
 
291
        self.assertEqual({'B-id': ('A-id',), 'C-id': ('A-id',)},
 
292
                         repo.get_parent_map(['B-id', 'C-id']))
 
293
        b_tree = repo.revision_tree('B-id')
 
294
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
295
                              (u'a', 'a-id', 'file'),
 
296
                             ], b_tree)
 
297
        self.assertEqual('new\ncontent\n', b_tree.get_file_text('a-id'))
312
298
 
313
299
        # We should still be using the content from A in C, not from B
314
 
        c_tree = repo.revision_tree(b'C-id')
315
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
316
 
                              (u'a', b'a-id', 'file'),
317
 
                              (u'c', b'c-id', 'file'),
318
 
                              ], c_tree)
319
 
        self.assertEqual(b'contents', c_tree.get_file_text('a'))
320
 
        self.assertEqual(b'alt\ncontent\n', c_tree.get_file_text('c'))
 
300
        c_tree = repo.revision_tree('C-id')
 
301
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
302
                              (u'a', 'a-id', 'file'),
 
303
                              (u'c', 'c-id', 'file'),
 
304
                             ], c_tree)
 
305
        self.assertEqual('contents', c_tree.get_file_text('a-id'))
 
306
        self.assertEqual('alt\ncontent\n', c_tree.get_file_text('c-id'))
321
307
 
322
308
    def test_set_merge_parent(self):
323
309
        builder = self.build_a_rev()
324
310
        builder.start_series()
325
311
        self.addCleanup(builder.finish_series)
326
 
        builder.build_snapshot([b'A-id'],
327
 
                               [('add', ('b', b'b-id', 'file', b'b\ncontent\n'))],
328
 
                               revision_id=b'B-id')
329
 
        builder.build_snapshot([b'A-id'],
330
 
                               [('add', ('c', b'c-id', 'file', b'alt\ncontent\n'))],
331
 
                               revision_id=b'C-id')
332
 
        builder.build_snapshot([b'B-id', b'C-id'], [], revision_id=b'D-id')
 
312
        builder.build_snapshot('B-id', ['A-id'],
 
313
            [('add', ('b', 'b-id', 'file', 'b\ncontent\n'))])
 
314
        builder.build_snapshot('C-id', ['A-id'],
 
315
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))])
 
316
        builder.build_snapshot('D-id', ['B-id', 'C-id'], [])
333
317
        repo = builder.get_branch().repository
334
 
        self.assertEqual({b'B-id': (b'A-id',), b'C-id': (b'A-id',),
335
 
                          b'D-id': (b'B-id', b'C-id')},
336
 
                         repo.get_parent_map([b'B-id', b'C-id', b'D-id']))
337
 
        d_tree = repo.revision_tree(b'D-id')
 
318
        self.assertEqual({'B-id': ('A-id',), 'C-id': ('A-id',),
 
319
                          'D-id': ('B-id', 'C-id')},
 
320
                         repo.get_parent_map(['B-id', 'C-id', 'D-id']))
 
321
        d_tree = repo.revision_tree('D-id')
338
322
        # Note: by default a merge node does *not* pull in the changes from the
339
323
        #       merged tree, you have to supply it yourself.
340
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
341
 
                              (u'a', b'a-id', 'file'),
342
 
                              (u'b', b'b-id', 'file'),
343
 
                              ], d_tree)
 
324
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
325
                              (u'a', 'a-id', 'file'),
 
326
                              (u'b', 'b-id', 'file'),
 
327
                             ], d_tree)
344
328
 
345
329
    def test_set_merge_parent_and_contents(self):
346
330
        builder = self.build_a_rev()
347
331
        builder.start_series()
348
332
        self.addCleanup(builder.finish_series)
349
 
        builder.build_snapshot([b'A-id'],
350
 
                               [('add', ('b', b'b-id', 'file', b'b\ncontent\n'))],
351
 
                               revision_id=b'B-id')
352
 
        builder.build_snapshot([b'A-id'],
353
 
                               [('add', ('c', b'c-id', 'file', b'alt\ncontent\n'))],
354
 
                               revision_id=b'C-id')
355
 
        builder.build_snapshot([b'B-id', b'C-id'],
356
 
                               [('add', ('c', b'c-id', 'file', b'alt\ncontent\n'))],
357
 
                               revision_id=b'D-id')
 
333
        builder.build_snapshot('B-id', ['A-id'],
 
334
            [('add', ('b', 'b-id', 'file', 'b\ncontent\n'))])
 
335
        builder.build_snapshot('C-id', ['A-id'],
 
336
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))])
 
337
        builder.build_snapshot('D-id', ['B-id', 'C-id'],
 
338
            [('add', ('c', 'c-id', 'file', 'alt\ncontent\n'))])
358
339
        repo = builder.get_branch().repository
359
 
        self.assertEqual({b'B-id': (b'A-id',), b'C-id': (b'A-id',),
360
 
                          b'D-id': (b'B-id', b'C-id')},
361
 
                         repo.get_parent_map([b'B-id', b'C-id', b'D-id']))
362
 
        d_tree = repo.revision_tree(b'D-id')
363
 
        self.assertTreeShape([(u'', b'a-root-id', 'directory'),
364
 
                              (u'a', b'a-id', 'file'),
365
 
                              (u'b', b'b-id', 'file'),
366
 
                              (u'c', b'c-id', 'file'),
367
 
                              ], d_tree)
 
340
        self.assertEqual({'B-id': ('A-id',), 'C-id': ('A-id',),
 
341
                          'D-id': ('B-id', 'C-id')},
 
342
                         repo.get_parent_map(['B-id', 'C-id', 'D-id']))
 
343
        d_tree = repo.revision_tree('D-id')
 
344
        self.assertTreeShape([(u'', 'a-root-id', 'directory'),
 
345
                              (u'a', 'a-id', 'file'),
 
346
                              (u'b', 'b-id', 'file'),
 
347
                              (u'c', 'c-id', 'file'),
 
348
                             ], d_tree)
368
349
        # Because we copied the exact text into *this* tree, the 'c' file
369
350
        # should look like it was not modified in the merge
370
 
        self.assertEqual(b'C-id', d_tree.get_file_revision('c'))
 
351
        self.assertEqual('C-id', d_tree.get_file_revision('c-id'))
371
352
 
372
353
    def test_set_parent_to_null(self):
373
354
        builder = self.build_a_rev()
374
355
        builder.start_series()
375
356
        self.addCleanup(builder.finish_series)
376
 
        builder.build_snapshot([],
377
 
                               [('add', ('', None, 'directory', None))],
378
 
                               revision_id=b'B-id')
 
357
        builder.build_snapshot('B-id', [],
 
358
            [('add', ('', None, 'directory', None))])
379
359
        # We should now have a graph:
380
360
        #   A B
381
361
        # And not A => B
382
362
        repo = builder.get_branch().repository
383
 
        self.assertEqual({b'A-id': (_mod_revision.NULL_REVISION,),
384
 
                          b'B-id': (_mod_revision.NULL_REVISION,), },
385
 
                         repo.get_parent_map([b'A-id', b'B-id']))
 
363
        self.assertEqual({'A-id': (_mod_revision.NULL_REVISION,),
 
364
                          'B-id': (_mod_revision.NULL_REVISION,),},
 
365
                         repo.get_parent_map(['A-id', 'B-id']))
386
366
 
 
367
    
387
368
    def test_start_finish_series(self):
388
369
        builder = BranchBuilder(self.get_transport().clone('foo'))
389
370
        builder.start_series()
400
381
        builder = BranchBuilder(self.get_transport().clone('foo'))
401
382
        builder.start_series()
402
383
        try:
403
 
            builder.build_snapshot([b'ghost'],
404
 
                                   [('add', ('', b'ROOT_ID', 'directory', ''))],
405
 
                                   allow_leftmost_as_ghost=True, revision_id=b'tip')
 
384
            builder.build_snapshot('tip', ['ghost'],
 
385
                [('add', ('', 'ROOT_ID', 'directory', ''))],
 
386
                allow_leftmost_as_ghost=True)
406
387
        finally:
407
388
            builder.finish_series()
408
389
        b = builder.get_branch()
409
390
        b.lock_read()
410
391
        self.addCleanup(b.unlock)
411
 
        self.assertEqual((b'ghost',),
412
 
                         b.repository.get_graph().get_parent_map([b'tip'])[b'tip'])
 
392
        self.assertEqual(('ghost',),
 
393
            b.repository.get_graph().get_parent_map(['tip'])['tip'])
413
394
 
414
395
    def test_unversion_root_add_new_root(self):
415
396
        builder = BranchBuilder(self.get_transport().clone('foo'))
416
397
        builder.start_series()
417
 
        builder.build_snapshot(None,
418
 
                               [('add', ('', b'TREE_ROOT', 'directory', ''))],
419
 
                               revision_id=b'rev-1')
420
 
        builder.build_snapshot(None,
421
 
                               [('unversion', ''),
422
 
                                ('add', ('', b'my-root', 'directory', ''))],
423
 
                               revision_id=b'rev-2')
 
398
        builder.build_snapshot('rev-1', None,
 
399
            [('add', ('', 'TREE_ROOT', 'directory', ''))])
 
400
        builder.build_snapshot('rev-2', None,
 
401
            [('unversion', 'TREE_ROOT'),
 
402
             ('add', ('', 'my-root', 'directory', ''))])
424
403
        builder.finish_series()
425
 
        rev_tree = builder.get_branch().repository.revision_tree(b'rev-2')
426
 
        self.assertTreeShape([(u'', b'my-root', 'directory')], rev_tree)
 
404
        rev_tree = builder.get_branch().repository.revision_tree('rev-2')
 
405
        self.assertTreeShape([(u'', 'my-root', 'directory')], rev_tree)
427
406
 
428
407
    def test_empty_flush(self):
429
408
        """A flush with no actions before it is a no-op."""
430
409
        builder = BranchBuilder(self.get_transport().clone('foo'))
431
410
        builder.start_series()
432
 
        builder.build_snapshot(None,
433
 
                               [('add', ('', b'TREE_ROOT', 'directory', ''))],
434
 
                               revision_id=b'rev-1')
435
 
        builder.build_snapshot(None, [('flush', None)], revision_id=b'rev-2')
 
411
        builder.build_snapshot('rev-1', None,
 
412
            [('add', ('', 'TREE_ROOT', 'directory', ''))])
 
413
        builder.build_snapshot('rev-2', None, [('flush', None)])
436
414
        builder.finish_series()
437
 
        rev_tree = builder.get_branch().repository.revision_tree(b'rev-2')
438
 
        self.assertTreeShape([(u'', b'TREE_ROOT', 'directory')], rev_tree)
 
415
        rev_tree = builder.get_branch().repository.revision_tree('rev-2')
 
416
        self.assertTreeShape([(u'', 'TREE_ROOT', 'directory')], rev_tree)
439
417
 
440
418
    def test_kind_change(self):
441
419
        """It's possible to change the kind of an entry in a single snapshot
443
421
        """
444
422
        builder = BranchBuilder(self.get_transport().clone('foo'))
445
423
        builder.start_series()
446
 
        builder.build_snapshot(None,
447
 
                               [('add', (u'', b'a-root-id', 'directory', None)),
448
 
                                ('add', (u'a', b'a-id', 'file', b'content\n'))],
449
 
                               revision_id=b'A-id')
450
 
        builder.build_snapshot(None,
451
 
                               [('unversion', 'a'),
452
 
                                ('flush', None),
453
 
                                   ('add', (u'a', b'a-id', 'directory', None))],
454
 
                               revision_id=b'B-id')
 
424
        builder.build_snapshot('A-id', None,
 
425
            [('add', (u'', 'a-root-id', 'directory', None)),
 
426
             ('add', (u'a', 'a-id', 'file', 'content\n'))])
 
427
        builder.build_snapshot('B-id', None,
 
428
            [('unversion', 'a-id'),
 
429
             ('flush', None),
 
430
             ('add', (u'a', 'a-id', 'directory', None))])
455
431
        builder.finish_series()
456
 
        rev_tree = builder.get_branch().repository.revision_tree(b'B-id')
 
432
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
457
433
        self.assertTreeShape(
458
 
            [(u'', b'a-root-id', 'directory'), (u'a', b'a-id', 'directory')],
 
434
            [(u'', 'a-root-id', 'directory'), (u'a', 'a-id', 'directory')],
459
435
            rev_tree)
460
436
 
461
437
    def test_pivot_root(self):
464
440
        """
465
441
        builder = BranchBuilder(self.get_transport().clone('foo'))
466
442
        builder.start_series()
467
 
        builder.build_snapshot(None,
468
 
                               [('add', (u'', b'orig-root', 'directory', None)),
469
 
                                ('add', (u'dir', b'dir-id', 'directory', None))],
470
 
                               revision_id=b'A-id')
471
 
        builder.build_snapshot(None,
472
 
                               [('unversion', ''),  # implicitly unversions all children
473
 
                                ('flush', None),
474
 
                                   ('add', (u'', b'dir-id', 'directory', None))],
475
 
                               revision_id=b'B-id')
 
443
        builder.build_snapshot('A-id', None,
 
444
            [('add', (u'', 'orig-root', 'directory', None)),
 
445
             ('add', (u'dir', 'dir-id', 'directory', None))])
 
446
        builder.build_snapshot('B-id', None,
 
447
            [('unversion', 'orig-root'),  # implicitly unversions all children
 
448
             ('flush', None),
 
449
             ('add', (u'', 'dir-id', 'directory', None))])
476
450
        builder.finish_series()
477
 
        rev_tree = builder.get_branch().repository.revision_tree(b'B-id')
478
 
        self.assertTreeShape([(u'', b'dir-id', 'directory')], rev_tree)
 
451
        rev_tree = builder.get_branch().repository.revision_tree('B-id')
 
452
        self.assertTreeShape([(u'', 'dir-id', 'directory')], rev_tree)
 
453