/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 bzrlib/tests/blackbox/test_update.py

  • Committer: Vincent Ladeuil
  • Date: 2010-01-25 15:55:48 UTC
  • mto: (4985.1.4 add-attr-cleanup)
  • mto: This revision was merged to the branch mainline in revision 4988.
  • Revision ID: v.ladeuil+lp@free.fr-20100125155548-0l352pujvt5bzl5e
Deploy addAttrCleanup on the whole test suite.

Several use case worth mentioning:

- setting a module or any other object attribute is the majority
by far. In some cases the setting itself is deferred but most of
the time we want to set at the same time we add the cleanup.

- there multiple occurrences of protecting hooks or ui factory
which are now useless (the test framework takes care of that now),

- there was some lambda uses that can now be avoided.

That first cleanup already simplifies things a lot.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_update.py
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006, 2009 Canonical Ltd
 
2
# -*- coding: utf-8 -*-
2
3
#
3
4
# This program is free software; you can redistribute it and/or modify
4
5
# it under the terms of the GNU General Public License as published by
170
171
        # get all three files and a pending merge.
171
172
        out, err = self.run_bzr('update checkout')
172
173
        self.assertEqual('', out)
173
 
        self.assertEqualDiff("""+N  file_b
 
174
        self.assertEqualDiff("""+N  file
174
175
All changes applied successfully.
175
 
+N  file
 
176
+N  file_b
176
177
All changes applied successfully.
177
178
Updated to revision 1 of branch %s
178
179
Your local commits will now show as pending merges with 'bzr status', and can be committed with 'bzr commit'.
195
196
 
196
197
        self.build_tree(['checkout1/'])
197
198
        checkout_dir = bzrdir.BzrDirMetaFormat1().initialize('checkout1')
198
 
        branch.BranchReferenceFormat().initialize(checkout_dir,
199
 
            target_branch=master.branch)
 
199
        branch.BranchReferenceFormat().initialize(checkout_dir, master.branch)
200
200
        checkout1 = checkout_dir.create_workingtree('m1')
201
201
 
202
202
        # Create a second branch, with an extra commit
241
241
        tree.commit('empty commit')
242
242
        self.run_bzr('update checkout')
243
243
 
244
 
    def test_update_with_merge_merged_to_master(self):
245
 
        # Test that 'bzr update' works correctly when you have
246
 
        # an update in the master tree, and a [lightweight or otherwise]
247
 
        # checkout which has merge a revision merged to master already.
248
 
        master = self.make_branch_and_tree('master')
249
 
        self.build_tree(['master/file'])
250
 
        master.add(['file'])
251
 
        master.commit('one', rev_id='m1')
252
 
 
253
 
        self.build_tree(['checkout1/'])
254
 
        checkout_dir = bzrdir.BzrDirMetaFormat1().initialize('checkout1')
255
 
        branch.BranchReferenceFormat().initialize(checkout_dir,
256
 
            target_branch=master.branch)
257
 
        checkout1 = checkout_dir.create_workingtree('m1')
258
 
 
259
 
        # Create a second branch, with an extra commit
260
 
        other = master.bzrdir.sprout('other').open_workingtree()
261
 
        self.build_tree(['other/file2'])
262
 
        other.add(['file2'])
263
 
        other.commit('other2', rev_id='o2')
264
 
 
265
 
        # Merge the other branch into checkout -  'start reviewing a patch'
266
 
        checkout1.merge_from_branch(other.branch)
267
 
        self.assertEqual(['o2'], checkout1.get_parent_ids()[1:])
268
 
 
269
 
        # Create a new commit in the master branch - 'someone else lands its'
270
 
        master.merge_from_branch(other.branch)
271
 
        master.commit('f3', rev_id='m2')
272
 
 
273
 
        # This should not report about local commits being pending
274
 
        # merges, because they were real merges (but are now gone).
275
 
        # It should perhaps report on them.
276
 
        out, err = self.run_bzr('update', working_dir='checkout1')
277
 
        self.assertEqual('', out)
278
 
        self.assertEqualDiff('''All changes applied successfully.
279
 
Updated to revision 2 of branch %s
280
 
''' % osutils.pathjoin(self.test_dir, 'master',),
281
 
                         err)
282
 
        # The pending merges should still be there
283
 
        self.assertEqual([], checkout1.get_parent_ids()[1:])
284
 
 
285
244
    def test_update_dash_r(self):
 
245
        # Test that 'bzr update' works correctly when you have
 
246
        # an update in the master tree, and a lightweight checkout
 
247
        # which has merged another branch
286
248
        master = self.make_branch_and_tree('master')
287
249
        os.chdir('master')
288
250
        self.build_tree(['./file1'])
304
266
        self.assertEquals(['m1'], master.get_parent_ids())
305
267
 
306
268
    def test_update_dash_r_outside_history(self):
307
 
        """Ensure that we can update -r to dotted revisions.
308
 
        """
 
269
        # Test that 'bzr update' works correctly when you have
 
270
        # an update in the master tree, and a lightweight checkout
 
271
        # which has merged another branch
309
272
        master = self.make_branch_and_tree('master')
310
273
        self.build_tree(['master/file1'])
311
274
        master.add(['file1'])
312
275
        master.commit('one', rev_id='m1')
313
276
 
314
 
        # Create a second branch, with extra commits
 
277
        # Create a second branch, with an extra commit
315
278
        other = master.bzrdir.sprout('other').open_workingtree()
316
 
        self.build_tree(['other/file2', 'other/file3'])
 
279
        self.build_tree(['other/file2'])
317
280
        other.add(['file2'])
318
281
        other.commit('other2', rev_id='o2')
319
 
        other.add(['file3'])
320
 
        other.commit('other3', rev_id='o3')
321
282
 
322
283
        os.chdir('master')
323
284
        self.run_bzr('merge ../other')
324
285
        master.commit('merge', rev_id='merge')
325
286
 
326
 
        # Switch to o2. file3 was added only in o3 and should be deleted.
327
 
        out, err = self.run_bzr('update -r revid:o2')
328
 
        self.assertContainsRe(err, '-D\s+file3')
329
 
        self.assertContainsRe(err, 'All changes applied successfully\.')
330
 
        self.assertContainsRe(err, 'Updated to revision 1.1.1 of branch .*')
331
 
 
332
 
        # Switch back to latest
333
 
        out, err = self.run_bzr('update')
334
 
        self.assertContainsRe(err, '\+N\s+file3')
335
 
        self.assertContainsRe(err, 'All changes applied successfully\.')
336
 
        self.assertContainsRe(err, 'Updated to revision 2 of branch .*')
 
287
        out, err = self.run_bzr('update -r revid:o2',
 
288
                                retcode=3)
 
289
        self.assertEqual('', out)
 
290
        self.assertEqual('bzr: ERROR: branch has no revision o2\n'
 
291
                         'bzr update --revision only works'
 
292
                         ' for a revision in the branch history\n',
 
293
                         err)
337
294
 
338
295
    def test_update_dash_r_in_master(self):
339
296
        # Test that 'bzr update' works correctly when you have
358
315
2>All changes applied successfully.
359
316
2>Updated to revision 2 of branch .../master
360
317
''')
361
 
 
362
 
    def test_update_checkout_prevent_double_merge(self):
363
 
        """"Launchpad bug 113809 in bzr "update performs two merges"
364
 
        https://launchpad.net/bugs/113809"""
365
 
        master = self.make_branch_and_tree('master')
366
 
        self.build_tree_contents([('master/file', 'initial contents\n')])
367
 
        master.add(['file'])
368
 
        master.commit('one', rev_id='m1')
369
 
 
370
 
        checkout = master.branch.create_checkout('checkout')
371
 
        lightweight = checkout.branch.create_checkout('lightweight',
372
 
                                                      lightweight=True)
373
 
 
374
 
        # time to create a mess
375
 
        # add a commit to the master
376
 
        self.build_tree_contents([('master/file', 'master\n')])
377
 
        master.commit('two', rev_id='m2')
378
 
        self.build_tree_contents([('master/file', 'master local changes\n')])
379
 
 
380
 
        # local commit on the checkout
381
 
        self.build_tree_contents([('checkout/file', 'checkout\n')])
382
 
        checkout.commit('tree', rev_id='c2', local=True)
383
 
        self.build_tree_contents([('checkout/file',
384
 
                                   'checkout local changes\n')])
385
 
 
386
 
        # lightweight 
387
 
        self.build_tree_contents([('lightweight/file',
388
 
                                   'lightweight local changes\n')])
389
 
 
390
 
        # now update (and get conflicts)
391
 
        out, err = self.run_bzr('update lightweight', retcode=1)
392
 
        self.assertEqual('', out)
393
 
        # NB: these conflicts are actually in the source code
394
 
        self.assertFileEqual('''\
395
 
<<<<<<< TREE
396
 
lightweight local changes
397
 
=======
398
 
checkout
399
 
>>>>>>> MERGE-SOURCE
400
 
''',
401
 
                             'lightweight/file')
402
 
 
403
 
        # resolve it
404
 
        self.build_tree_contents([('lightweight/file',
405
 
                                   'lightweight+checkout\n')])
406
 
        self.run_bzr('resolve lightweight/file')
407
 
 
408
 
        # check we get the second conflict
409
 
        out, err = self.run_bzr('update lightweight', retcode=1)
410
 
        self.assertEqual('', out)
411
 
        # NB: these conflicts are actually in the source code
412
 
        self.assertFileEqual('''\
413
 
<<<<<<< TREE
414
 
lightweight+checkout
415
 
=======
416
 
master
417
 
>>>>>>> MERGE-SOURCE
418
 
''',
419
 
                             'lightweight/file')