← 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: Wouter van Heyst
  • Date: 2006-06-07 16:05:27 UTC
  • mto: This revision was merged to the branch mainline in revision 1752.
  • Revision ID: larstiq@larstiq.dyndns.org-20060607160527-2b3649154d0e2e84
more code cleanup

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_commit.py
1
 
# Copyright (C) 2005, 2006 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2005, 2006 by Canonical Ltd
 
2
 
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
6
6
# (at your option) any later version.
7
 
#
 
7
 
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
12
 
#
 
12
 
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
21
21
import re
22
22
import sys
23
23
 
24
 
from bzrlib import (
25
 
    ignores,
26
 
    )
27
24
from bzrlib.branch import Branch
28
25
from bzrlib.bzrdir import BzrDir
29
26
from bzrlib.errors import BzrCommandError
36
33
    def test_05_empty_commit(self):
37
34
        """Commit of tree with no versioned files should fail"""
38
35
        # If forced, it should succeed, but this is not tested here.
39
 
        self.run_bzr("init")
 
36
        self.runbzr("init")
40
37
        self.build_tree(['hello.txt'])
41
 
        out,err = self.run_bzr("commit", "-m", "empty", retcode=3)
42
 
        self.assertEqual('', out)
43
 
        self.assertStartsWith(err, 'bzr: ERROR: no changes to commit.'
44
 
                                  ' use --unchanged to commit anyhow\n')
45
 
 
46
 
    def test_commit_success(self):
47
 
        """Successful commit should not leave behind a bzr-commit-* file"""
48
 
        self.run_bzr("init")
49
 
        self.run_bzr("commit", "--unchanged", "-m", "message")
50
 
        self.assertEqual('', self.run_bzr(['unknowns'])[0])
51
 
 
52
 
        # same for unicode messages
53
 
        self.run_bzr("commit", "--unchanged", "-m", u'foo\xb5')
54
 
        self.assertEqual('', self.run_bzr(['unknowns'])[0])
 
38
        self.runbzr("commit -m empty", retcode=3)
55
39
 
56
40
    def test_commit_with_path(self):
57
41
        """Commit tree with path of root specified"""
76
60
 
77
61
    def test_10_verbose_commit(self):
78
62
        """Add one file and examine verbose commit output"""
79
 
        self.run_bzr("init")
 
63
        self.runbzr("init")
80
64
        self.build_tree(['hello.txt'])
81
 
        self.run_bzr("add hello.txt")
 
65
        self.runbzr("add hello.txt")
82
66
        out,err = self.run_bzr("commit", "-m", "added")
83
67
        self.assertEqual('', out)
84
68
        self.assertEqual('added hello.txt\n'
141
125
 
142
126
    def test_verbose_commit_with_unchanged(self):
143
127
        """Unchanged files should not be listed by default in verbose output"""
144
 
        self.run_bzr("init")
 
128
        self.runbzr("init")
145
129
        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")
 
130
        self.runbzr('add unchanged.txt')
 
131
        self.runbzr('commit -m unchanged unchanged.txt')
 
132
        self.runbzr("add hello.txt")
149
133
        out,err = self.run_bzr("commit", "-m", "added")
150
134
        self.assertEqual('', out)
151
135
        self.assertEqual('added hello.txt\n'
200
184
            other_tree.commit('modify all sample files and dirs.')
201
185
        finally:
202
186
            other_tree.unlock()
203
 
        this_tree.merge_from_branch(other_tree.branch)
 
187
        self.merge(other_tree.branch, this_tree)
204
188
        os.chdir('this')
205
189
        out,err = self.run_bzr("commit", "-m", "added")
206
190
        os.chdir('..')
219
203
            err)
220
204
 
221
205
    def test_empty_commit_message(self):
222
 
        self.run_bzr("init")
 
206
        self.runbzr("init")
223
207
        file('foo.c', 'wt').write('int main() {}')
224
 
        self.run_bzr('add foo.c')
225
 
        self.run_bzr('commit -m ""', retcode=3)
 
208
        self.runbzr(['add', 'foo.c'])
 
209
        self.runbzr(["commit", "-m", ""] , retcode=3)
226
210
 
227
211
    def test_other_branch_commit(self):
228
212
        # this branch is to ensure consistent behaviour, whether we're run
229
213
        # inside a branch, or not.
230
214
        os.mkdir('empty_branch')
231
215
        os.chdir('empty_branch')
232
 
        self.run_bzr('init')
 
216
        self.runbzr('init')
233
217
        os.mkdir('branch')
234
218
        os.chdir('branch')
235
 
        self.run_bzr('init')
 
219
        self.runbzr('init')
236
220
        file('foo.c', 'wt').write('int main() {}')
237
221
        file('bar.c', 'wt').write('int main() {}')
238
222
        os.chdir('..')
239
 
        self.run_bzr(['add', 'branch/foo.c'])
240
 
        self.run_bzr(['add', 'branch'])
 
223
        self.runbzr(['add', 'branch/foo.c'])
 
224
        self.runbzr(['add', 'branch'])
241
225
        # 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)
 
226
        self.runbzr('commit -m newstuff branch/foo.c .', retcode=3)
 
227
        self.runbzr('commit -m newstuff branch/foo.c')
 
228
        self.runbzr('commit -m newstuff branch')
 
229
        self.runbzr('commit -m newstuff branch', retcode=3)
246
230
 
247
231
    def test_out_of_date_tree_commit(self):
248
232
        # check we get an error code and a clear message committing with an out
249
233
        # of date checkout
250
234
        self.make_branch_and_tree('branch')
251
235
        # make a checkout
252
 
        self.run_bzr('checkout --lightweight branch checkout')
 
236
        self.runbzr('checkout --lightweight branch checkout')
253
237
        # commit to the original branch to make the checkout out of date
254
 
        self.run_bzr('commit --unchanged -m message branch')
 
238
        self.runbzr('commit --unchanged -m message branch')
255
239
        # now commit to the checkout should emit
256
240
        # ERROR: Out of date with the branch, 'bzr update' is suggested
257
 
        output = self.run_bzr('commit --unchanged -m checkout_message '
 
241
        output = self.runbzr('commit --unchanged -m checkout_message '
258
242
                             'checkout', retcode=3)
259
243
        self.assertEqual(output,
260
244
                         ('',
300
284
        # version or the u2 version.
301
285
        self.build_tree_contents([('u1/hosts', 'merge resolution\n')])
302
286
        self.run_bzr('commit', '-m', 'checkin merge of the offline work from u1', 'u1')
303
 
 
304
 
    def test_commit_respects_spec_for_removals(self):
305
 
        """Commit with a file spec should only commit removals that match"""
306
 
        t = self.make_branch_and_tree('.')
307
 
        self.build_tree(['file-a', 'dir-a/', 'dir-a/file-b'])
308
 
        t.add(['file-a', 'dir-a', 'dir-a/file-b'])
309
 
        t.commit('Create')
310
 
        t.remove(['file-a', 'dir-a/file-b'])
311
 
        os.chdir('dir-a')
312
 
        result = self.run_bzr('commit', '.', '-m' 'removed file-b')[1]
313
 
        self.assertNotContainsRe(result, 'file-a')
314
 
        result = self.run_bzr('status')[0]
315
 
        self.assertContainsRe(result, 'removed:\n  file-a')
316
 
 
317
 
    def test_strict_commit(self):
318
 
        """Commit with --strict works if everything is known"""
319
 
        ignores._set_user_ignores([])
320
 
        tree = self.make_branch_and_tree('tree')
321
 
        self.build_tree(['tree/a'])
322
 
        tree.add('a')
323
 
        # A simple change should just work
324
 
        self.run_bzr('commit', '--strict', '-m', 'adding a',
325
 
                     working_dir='tree')
326
 
 
327
 
    def test_strict_commit_no_changes(self):
328
 
        """commit --strict gives "no changes" if there is nothing to commit"""
329
 
        tree = self.make_branch_and_tree('tree')
330
 
        self.build_tree(['tree/a'])
331
 
        tree.add('a')
332
 
        tree.commit('adding a')
333
 
 
334
 
        # With no changes, it should just be 'no changes'
335
 
        # Make sure that commit is failing because there is nothing to do
336
 
        self.run_bzr_error(['no changes to commit'],
337
 
                           'commit', '--strict', '-m', 'no changes',
338
 
                           working_dir='tree')
339
 
 
340
 
        # But --strict doesn't care if you supply --unchanged
341
 
        self.run_bzr('commit', '--strict', '--unchanged', '-m', 'no changes',
342
 
                     working_dir='tree')
343
 
 
344
 
    def test_strict_commit_unknown(self):
345
 
        """commit --strict fails if a file is unknown"""
346
 
        tree = self.make_branch_and_tree('tree')
347
 
        self.build_tree(['tree/a'])
348
 
        tree.add('a')
349
 
        tree.commit('adding a')
350
 
 
351
 
        # Add one file so there is a change, but forget the other
352
 
        self.build_tree(['tree/b', 'tree/c'])
353
 
        tree.add('b')
354
 
        self.run_bzr_error(['Commit refused because there are unknown files'],
355
 
                           'commit', '--strict', '-m', 'add b',
356
 
                           working_dir='tree')
357
 
 
358
 
        # --no-strict overrides --strict
359
 
        self.run_bzr('commit', '--strict', '-m', 'add b', '--no-strict',
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(
394
 
            'commit', '-m', 'hello', '--fixes=lp:234', 'tree/hello.txt')
395
 
 
396
 
        # Get the revision properties, ignoring the branch-nick property, which
397
 
        # we don't care about for this test.
398
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
399
 
        properties = dict(last_rev.properties)
400
 
        del properties['branch-nick']
401
 
 
402
 
        self.assertEqual({'bugs': 'https://launchpad.net/bugs/234 fixed'},
403
 
                         properties)
404
 
 
405
 
    def test_fixes_multiple_bugs_sets_properties(self):
406
 
        """--fixes can be used more than once to show that bugs are fixed."""
407
 
        tree = self.make_branch_and_tree('tree')
408
 
        self.build_tree(['tree/hello.txt'])
409
 
        tree.add('hello.txt')
410
 
        self.run_bzr(
411
 
            'commit', '-m', 'hello', '--fixes=lp:123', '--fixes=lp:235',
412
 
            'tree/hello.txt')
413
 
 
414
 
        # Get the revision properties, ignoring the branch-nick property, which
415
 
        # we don't care about for this test.
416
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
417
 
        properties = dict(last_rev.properties)
418
 
        del properties['branch-nick']
419
 
 
420
 
        self.assertEqual(
421
 
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
422
 
                     'https://launchpad.net/bugs/235 fixed'},
423
 
            properties)
424
 
 
425
 
    def test_fixes_bug_with_alternate_trackers(self):
426
 
        """--fixes can be used on a properly configured branch to mark bug
427
 
        fixes on multiple trackers.
428
 
        """
429
 
        tree = self.make_branch_and_tree('tree')
430
 
        tree.branch.get_config().set_user_option(
431
 
            'trac_twisted_url', 'http://twistedmatrix.com/trac')
432
 
        self.build_tree(['tree/hello.txt'])
433
 
        tree.add('hello.txt')
434
 
        self.run_bzr(
435
 
            'commit', '-m', 'hello', '--fixes=lp:123',
436
 
            '--fixes=twisted:235', 'tree/')
437
 
 
438
 
        # Get the revision properties, ignoring the branch-nick property, which
439
 
        # we don't care about for this test.
440
 
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
441
 
        properties = dict(last_rev.properties)
442
 
        del properties['branch-nick']
443
 
 
444
 
        self.assertEqual(
445
 
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
446
 
                     'http://twistedmatrix.com/trac/ticket/235 fixed'},
447
 
            properties)
448
 
 
449
 
    def test_fixes_unknown_bug_prefix(self):
450
 
        tree = self.make_branch_and_tree('tree')
451
 
        self.build_tree(['tree/hello.txt'])
452
 
        tree.add('hello.txt')
453
 
        self.run_bzr_error(
454
 
            ["Unrecognized bug %s. Commit refused." % 'xxx:123'],
455
 
            'commit', '-m', 'add b', '--fixes=xxx:123',
456
 
            working_dir='tree')
457
 
 
458
 
    def test_fixes_invalid_bug_number(self):
459
 
        tree = self.make_branch_and_tree('tree')
460
 
        self.build_tree(['tree/hello.txt'])
461
 
        tree.add('hello.txt')
462
 
        self.run_bzr_error(
463
 
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
464
 
            'commit', '-m', 'add b', '--fixes=lp:orange',
465
 
            working_dir='tree')
466
 
 
467
 
    def test_fixes_invalid_argument(self):
468
 
        """Raise an appropriate error when the fixes argument isn't tag:id."""
469
 
        tree = self.make_branch_and_tree('tree')
470
 
        self.build_tree(['tree/hello.txt'])
471
 
        tree.add('hello.txt')
472
 
        self.run_bzr_error(
473
 
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
474
 
             r"Commit refused\."],
475
 
            'commit', '-m', 'add b', '--fixes=orange',
476
 
            working_dir='tree')