← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/blackbox/test_commit.py

  • Committer: John Arbash Meinel
  • Date: 2007-03-14 20:47:17 UTC
  • mto: (2353.4.2 locking)
  • mto: This revision was merged to the branch mainline in revision 2360.
  • Revision ID: john@arbash-meinel.com-20070314204717-htynwogv97fqr22a
Cleanup errors, and change ReadOnlyLockError to pass around more details.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
38
38
        # If forced, it should succeed, but this is not tested here.
39
39
        self.run_bzr("init")
40
40
        self.build_tree(['hello.txt'])
41
 
        out,err = self.run_bzr('commit -m empty', retcode=3)
 
41
        out,err = self.run_bzr("commit", "-m", "empty", retcode=3)
42
42
        self.assertEqual('', out)
43
43
        self.assertStartsWith(err, 'bzr: ERROR: no changes to commit.'
44
44
                                  ' use --unchanged to commit anyhow\n')
46
46
    def test_commit_success(self):
47
47
        """Successful commit should not leave behind a bzr-commit-* file"""
48
48
        self.run_bzr("init")
49
 
        self.run_bzr('commit --unchanged -m message')
50
 
        self.assertEqual('', self.run_bzr('unknowns')[0])
 
49
        self.run_bzr("commit", "--unchanged", "-m", "message")
 
50
        self.assertEqual('', self.capture('unknowns'))
51
51
 
52
52
        # same for unicode messages
53
 
        self.run_bzr(["commit", "--unchanged", "-m", u'foo\xb5'])
54
 
        self.assertEqual('', self.run_bzr('unknowns')[0])
 
53
        self.run_bzr("commit", "--unchanged", "-m", u'foo\xb5')
 
54
        self.assertEqual('', self.capture('unknowns'))
55
55
 
56
56
    def test_commit_with_path(self):
57
57
        """Commit tree with path of root specified"""
58
 
        self.run_bzr('init a')
 
58
        self.run_bzr('init', 'a')
59
59
        self.build_tree(['a/a_file'])
60
 
        self.run_bzr('add a/a_file')
61
 
        self.run_bzr(['commit', '-m', 'first commit', 'a'])
 
60
        self.run_bzr('add', 'a/a_file')
 
61
        self.run_bzr('commit', '-m', 'first commit', 'a')
62
62
 
63
 
        self.run_bzr('branch a b')
 
63
        self.run_bzr('branch', 'a', 'b')
64
64
        self.build_tree_contents([('b/a_file', 'changes in b')])
65
 
        self.run_bzr(['commit', '-m', 'first commit in b', 'b'])
 
65
        self.run_bzr('commit', '-m', 'first commit in b', 'b')
66
66
 
67
67
        self.build_tree_contents([('a/a_file', 'new contents')])
68
 
        self.run_bzr(['commit', '-m', 'change in a', 'a'])
 
68
        self.run_bzr('commit', '-m', 'change in a', 'a')
69
69
 
70
70
        os.chdir('b')
71
 
        self.run_bzr('merge ../a', retcode=1) # will conflict
 
71
        self.run_bzr('merge', '../a', retcode=1) # will conflict
72
72
        os.chdir('..')
73
 
        self.run_bzr('resolved b/a_file')
74
 
        self.run_bzr(['commit', '-m', 'merge into b', 'b'])
 
73
        self.run_bzr('resolved', 'b/a_file')
 
74
        self.run_bzr('commit', '-m', 'merge into b', 'b')
75
75
 
76
76
 
77
77
    def test_10_verbose_commit(self):
78
78
        """Add one file and examine verbose commit output"""
79
 
        self.run_bzr("init")
 
79
        self.runbzr("init")
80
80
        self.build_tree(['hello.txt'])
81
 
        self.run_bzr("add hello.txt")
82
 
        out,err = self.run_bzr('commit -m added')
 
81
        self.runbzr("add hello.txt")
 
82
        out,err = self.run_bzr("commit", "-m", "added")
83
83
        self.assertEqual('', out)
84
84
        self.assertEqual('added hello.txt\n'
85
85
                         'Committed revision 1.\n',
98
98
        # Verbose commit of modified file should say so
99
99
        wt = self.prepare_simple_history()
100
100
        self.build_tree_contents([('hello.txt', 'new contents')])
101
 
        out, err = self.run_bzr('commit -m modified')
 
101
        out, err = self.run_bzr("commit", "-m", "modified")
102
102
        self.assertEqual('', out)
103
103
        self.assertEqual('modified hello.txt\n'
104
104
                         'Committed revision 2.\n',
108
108
        # Verbose commit of renamed file should say so
109
109
        wt = self.prepare_simple_history()
110
110
        wt.rename_one('hello.txt', 'gutentag.txt')
111
 
        out, err = self.run_bzr('commit -m renamed')
 
111
        out, err = self.run_bzr("commit", "-m", "renamed")
112
112
        self.assertEqual('', out)
113
113
        self.assertEqual('renamed hello.txt => gutentag.txt\n'
114
114
                         'Committed revision 2.\n',
120
120
        os.mkdir('subdir')
121
121
        wt.add(['subdir'])
122
122
        wt.rename_one('hello.txt', 'subdir/hello.txt')
123
 
        out, err = self.run_bzr('commit -m renamed')
 
123
        out, err = self.run_bzr("commit", "-m", "renamed")
124
124
        self.assertEqual('', out)
125
125
        self.assertEqualDiff('added subdir\n'
126
126
                             'renamed hello.txt => subdir/hello.txt\n'
133
133
        wt = BzrDir.create_standalone_workingtree('.')
134
134
        self.build_tree(['hello.txt', 'extra.txt'])
135
135
        wt.add(['hello.txt'])
136
 
        out,err = self.run_bzr('commit -m added')
 
136
        out,err = self.run_bzr("commit", "-m", "added")
137
137
        self.assertEqual('', out)
138
138
        self.assertEqual('added hello.txt\n'
139
139
                         'Committed revision 1.\n',
141
141
 
142
142
    def test_verbose_commit_with_unchanged(self):
143
143
        """Unchanged files should not be listed by default in verbose output"""
144
 
        self.run_bzr("init")
 
144
        self.runbzr("init")
145
145
        self.build_tree(['hello.txt', 'unchanged.txt'])
146
 
        self.run_bzr('add unchanged.txt')
147
 
        self.run_bzr('commit -m unchanged unchanged.txt')
148
 
        self.run_bzr("add hello.txt")
149
 
        out,err = self.run_bzr('commit -m added')
 
146
        self.runbzr('add unchanged.txt')
 
147
        self.runbzr('commit -m unchanged unchanged.txt')
 
148
        self.runbzr("add hello.txt")
 
149
        out,err = self.run_bzr("commit", "-m", "added")
150
150
        self.assertEqual('', out)
151
151
        self.assertEqual('added hello.txt\n'
152
152
                         'Committed revision 2.\n',
202
202
            other_tree.unlock()
203
203
        this_tree.merge_from_branch(other_tree.branch)
204
204
        os.chdir('this')
205
 
        out,err = self.run_bzr('commit -m added')
 
205
        out,err = self.run_bzr("commit", "-m", "added")
206
206
        os.chdir('..')
207
207
        self.assertEqual('', out)
208
208
        self.assertEqualDiff(
219
219
            err)
220
220
 
221
221
    def test_empty_commit_message(self):
222
 
        self.run_bzr("init")
 
222
        self.runbzr("init")
223
223
        file('foo.c', 'wt').write('int main() {}')
224
 
        self.run_bzr('add foo.c')
225
 
        self.run_bzr('commit -m ""', retcode=3)
 
224
        self.runbzr(['add', 'foo.c'])
 
225
        self.runbzr(["commit", "-m", ""] , retcode=3)
226
226
 
227
227
    def test_other_branch_commit(self):
228
228
        # this branch is to ensure consistent behaviour, whether we're run
229
229
        # inside a branch, or not.
230
230
        os.mkdir('empty_branch')
231
231
        os.chdir('empty_branch')
232
 
        self.run_bzr('init')
 
232
        self.runbzr('init')
233
233
        os.mkdir('branch')
234
234
        os.chdir('branch')
235
 
        self.run_bzr('init')
 
235
        self.runbzr('init')
236
236
        file('foo.c', 'wt').write('int main() {}')
237
237
        file('bar.c', 'wt').write('int main() {}')
238
238
        os.chdir('..')
239
 
        self.run_bzr('add branch/foo.c')
240
 
        self.run_bzr('add branch')
 
239
        self.runbzr(['add', 'branch/foo.c'])
 
240
        self.runbzr(['add', 'branch'])
241
241
        # can't commit files in different trees; sane error
242
 
        self.run_bzr('commit -m newstuff branch/foo.c .', retcode=3)
243
 
        self.run_bzr('commit -m newstuff branch/foo.c')
244
 
        self.run_bzr('commit -m newstuff branch')
245
 
        self.run_bzr('commit -m newstuff branch', retcode=3)
 
242
        self.runbzr('commit -m newstuff branch/foo.c .', retcode=3)
 
243
        self.runbzr('commit -m newstuff branch/foo.c')
 
244
        self.runbzr('commit -m newstuff branch')
 
245
        self.runbzr('commit -m newstuff branch', retcode=3)
246
246
 
247
247
    def test_out_of_date_tree_commit(self):
248
248
        # check we get an error code and a clear message committing with an out
249
249
        # of date checkout
250
250
        self.make_branch_and_tree('branch')
251
251
        # make a checkout
252
 
        self.run_bzr('checkout --lightweight branch checkout')
 
252
        self.runbzr('checkout --lightweight branch checkout')
253
253
        # commit to the original branch to make the checkout out of date
254
 
        self.run_bzr('commit --unchanged -m message branch')
 
254
        self.runbzr('commit --unchanged -m message branch')
255
255
        # now commit to the checkout should emit
256
256
        # ERROR: Out of date with the branch, 'bzr update' is suggested
257
 
        output = self.run_bzr('commit --unchanged -m checkout_message '
 
257
        output = self.runbzr('commit --unchanged -m checkout_message '
258
258
                             'checkout', retcode=3)
259
259
        self.assertEqual(output,
260
260
                         ('',
264
264
    def test_local_commit_unbound(self):
265
265
        # a --local commit on an unbound branch is an error
266
266
        self.make_branch_and_tree('.')
267
 
        out, err = self.run_bzr('commit --local', retcode=3)
 
267
        out, err = self.run_bzr('commit', '--local', retcode=3)
268
268
        self.assertEqualDiff('', out)
269
269
        self.assertEqualDiff('bzr: ERROR: Cannot perform local-only commits '
270
270
                             'on unbound branches.\n', err)
275
275
        # past. This is a user story reported to fail in bug #43959 where 
276
276
        # a merge done in a checkout (using the update command) failed to
277
277
        # commit correctly.
278
 
        self.run_bzr('init trunk')
 
278
        self.run_bzr('init', 'trunk')
279
279
 
280
 
        self.run_bzr('checkout trunk u1')
 
280
        self.run_bzr('checkout', 'trunk', 'u1')
281
281
        self.build_tree_contents([('u1/hosts', 'initial contents')])
282
 
        self.run_bzr('add u1/hosts')
283
 
        self.run_bzr('commit -m add-hosts u1')
 
282
        self.run_bzr('add', 'u1/hosts')
 
283
        self.run_bzr('commit', '-m', 'add hosts', 'u1')
284
284
 
285
 
        self.run_bzr('checkout trunk u2')
 
285
        self.run_bzr('checkout', 'trunk', 'u2')
286
286
        self.build_tree_contents([('u2/hosts', 'altered in u2')])
287
 
        self.run_bzr('commit -m checkin-from-u2 u2')
 
287
        self.run_bzr('commit', '-m', 'checkin from u2', 'u2')
288
288
 
289
289
        # make an offline commits
290
290
        self.build_tree_contents([('u1/hosts', 'first offline change in u1')])
291
 
        self.run_bzr('commit -m checkin-offline --local u1')
 
291
        self.run_bzr('commit', '-m', 'checkin offline', '--local', 'u1')
292
292
 
293
293
        # now try to pull in online work from u2, and then commit our offline
294
294
        # work as a merge
295
295
        # retcode 1 as we expect a text conflict
296
 
        self.run_bzr('update u1', retcode=1)
297
 
        self.run_bzr('resolved u1/hosts')
 
296
        self.run_bzr('update', 'u1', retcode=1)
 
297
        self.run_bzr('resolved', 'u1/hosts')
298
298
        # add a text change here to represent resolving the merge conflicts in
299
299
        # favour of a new version of the file not identical to either the u1
300
300
        # version or the u2 version.
301
301
        self.build_tree_contents([('u1/hosts', 'merge resolution\n')])
302
 
        self.run_bzr('commit -m checkin-merge-of-the-offline-work-from-u1 u1')
 
302
        self.run_bzr('commit', '-m', 'checkin merge of the offline work from u1', 'u1')
303
303
 
304
304
    def test_commit_respects_spec_for_removals(self):
305
305
        """Commit with a file spec should only commit removals that match"""
309
309
        t.commit('Create')
310
310
        t.remove(['file-a', 'dir-a/file-b'])
311
311
        os.chdir('dir-a')
312
 
        result = self.run_bzr('commit . -m removed-file-b')[1]
 
312
        result = self.run_bzr('commit', '.', '-m' 'removed file-b')[1]
313
313
        self.assertNotContainsRe(result, 'file-a')
314
314
        result = self.run_bzr('status')[0]
315
315
        self.assertContainsRe(result, 'removed:\n  file-a')
321
321
        self.build_tree(['tree/a'])
322
322
        tree.add('a')
323
323
        # A simple change should just work
324
 
        self.run_bzr('commit --strict -m adding-a',
 
324
        self.run_bzr('commit', '--strict', '-m', 'adding a',
325
325
                     working_dir='tree')
326
326
 
327
327
    def test_strict_commit_no_changes(self):
334
334
        # With no changes, it should just be 'no changes'
335
335
        # Make sure that commit is failing because there is nothing to do
336
336
        self.run_bzr_error(['no changes to commit'],
337
 
                           'commit --strict -m no-changes',
 
337
                           'commit', '--strict', '-m', 'no changes',
338
338
                           working_dir='tree')
339
339
 
340
340
        # But --strict doesn't care if you supply --unchanged
341
 
        self.run_bzr('commit --strict --unchanged -m no-changes',
 
341
        self.run_bzr('commit', '--strict', '--unchanged', '-m', 'no changes',
342
342
                     working_dir='tree')
343
343
 
344
344
    def test_strict_commit_unknown(self):
352
352
        self.build_tree(['tree/b', 'tree/c'])
353
353
        tree.add('b')
354
354
        self.run_bzr_error(['Commit refused because there are unknown files'],
355
 
                           'commit --strict -m add-b',
 
355
                           'commit', '--strict', '-m', 'add b',
356
356
                           working_dir='tree')
357
357
 
358
358
        # --no-strict overrides --strict
359
 
        self.run_bzr('commit --strict -m add-b --no-strict',
 
359
        self.run_bzr('commit', '--strict', '-m', 'add b', '--no-strict',
360
360
                     working_dir='tree')
361
 
 
362
 
    def test_fixes_bug_output(self):
363
 
        """commit --fixes=lp:23452 succeeds without output."""
364
 
        tree = self.make_branch_and_tree('tree')
365
 
        self.build_tree(['tree/hello.txt'])
366
 
        tree.add('hello.txt')
367
 
        output, err = self.run_bzr(
368
 
            'commit -m hello --fixes=lp:23452 tree/hello.txt')
369
 
        self.assertEqual('', output)
370
 
        self.assertEqual('added hello.txt\nCommitted revision 1.\n', err)
371
 
 
372
 
    def test_no_bugs_no_properties(self):
373
 
        """If no bugs are fixed, the bugs property is not set.
374
 
 
375
 
        see https://beta.launchpad.net/bzr/+bug/109613
376
 
        """
377
 
        tree = self.make_branch_and_tree('tree')
378
 
        self.build_tree(['tree/hello.txt'])
379
 
        tree.add('hello.txt')
380
 
        self.run_bzr( 'commit -m hello tree/hello.txt')
381
 
        # Get the revision properties, ignoring the branch-nick property, which
382
 
        # we don't care about for this test.
383
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
384
 
        properties = dict(last_rev.properties)
385
 
        del properties['branch-nick']
386
 
        self.assertFalse('bugs' in properties)
387
 
 
388
 
    def test_fixes_bug_sets_property(self):
389
 
        """commit --fixes=lp:234 sets the lp:234 revprop to 'fixed'."""
390
 
        tree = self.make_branch_and_tree('tree')
391
 
        self.build_tree(['tree/hello.txt'])
392
 
        tree.add('hello.txt')
393
 
        self.run_bzr('commit -m hello --fixes=lp:234 tree/hello.txt')
394
 
 
395
 
        # Get the revision properties, ignoring the branch-nick property, which
396
 
        # we don't care about for this test.
397
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
398
 
        properties = dict(last_rev.properties)
399
 
        del properties['branch-nick']
400
 
 
401
 
        self.assertEqual({'bugs': 'https://launchpad.net/bugs/234 fixed'},
402
 
                         properties)
403
 
 
404
 
    def test_fixes_multiple_bugs_sets_properties(self):
405
 
        """--fixes can be used more than once to show that bugs are fixed."""
406
 
        tree = self.make_branch_and_tree('tree')
407
 
        self.build_tree(['tree/hello.txt'])
408
 
        tree.add('hello.txt')
409
 
        self.run_bzr('commit -m hello --fixes=lp:123 --fixes=lp:235'
410
 
                     ' tree/hello.txt')
411
 
 
412
 
        # Get the revision properties, ignoring the branch-nick property, which
413
 
        # we don't care about for this test.
414
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
415
 
        properties = dict(last_rev.properties)
416
 
        del properties['branch-nick']
417
 
 
418
 
        self.assertEqual(
419
 
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
420
 
                     'https://launchpad.net/bugs/235 fixed'},
421
 
            properties)
422
 
 
423
 
    def test_fixes_bug_with_alternate_trackers(self):
424
 
        """--fixes can be used on a properly configured branch to mark bug
425
 
        fixes on multiple trackers.
426
 
        """
427
 
        tree = self.make_branch_and_tree('tree')
428
 
        tree.branch.get_config().set_user_option(
429
 
            'trac_twisted_url', 'http://twistedmatrix.com/trac')
430
 
        self.build_tree(['tree/hello.txt'])
431
 
        tree.add('hello.txt')
432
 
        self.run_bzr('commit -m hello --fixes=lp:123 --fixes=twisted:235 tree/')
433
 
 
434
 
        # Get the revision properties, ignoring the branch-nick property, which
435
 
        # we don't care about for this test.
436
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
437
 
        properties = dict(last_rev.properties)
438
 
        del properties['branch-nick']
439
 
 
440
 
        self.assertEqual(
441
 
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
442
 
                     'http://twistedmatrix.com/trac/ticket/235 fixed'},
443
 
            properties)
444
 
 
445
 
    def test_fixes_unknown_bug_prefix(self):
446
 
        tree = self.make_branch_and_tree('tree')
447
 
        self.build_tree(['tree/hello.txt'])
448
 
        tree.add('hello.txt')
449
 
        self.run_bzr_error(
450
 
            ["Unrecognized bug %s. Commit refused." % 'xxx:123'],
451
 
            'commit -m add-b --fixes=xxx:123',
452
 
            working_dir='tree')
453
 
 
454
 
    def test_fixes_invalid_bug_number(self):
455
 
        tree = self.make_branch_and_tree('tree')
456
 
        self.build_tree(['tree/hello.txt'])
457
 
        tree.add('hello.txt')
458
 
        self.run_bzr_error(
459
 
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
460
 
            'commit -m add-b --fixes=lp:orange',
461
 
            working_dir='tree')
462
 
 
463
 
    def test_fixes_invalid_argument(self):
464
 
        """Raise an appropriate error when the fixes argument isn't tag:id."""
465
 
        tree = self.make_branch_and_tree('tree')
466
 
        self.build_tree(['tree/hello.txt'])
467
 
        tree.add('hello.txt')
468
 
        self.run_bzr_error(
469
 
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
470
 
             r"Commit refused\."],
471
 
            'commit -m add-b --fixes=orange',
472
 
            working_dir='tree')