~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

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

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2007-04-23 07:50:15 UTC
  • mfrom: (2376.4.43 bug-support)
  • Revision ID: pqm@pqm.ubuntu.com-20070423075015-340ajk1vo3pzxheu
(robertc) bzr --fixes support to allow recording of bugs that are fixed by commits. (Jonathan Lange, James Henstridge, Robert Collins)

Show diffs side-by-side

added added

removed removed

Lines of Context:
358
358
        # --no-strict overrides --strict
359
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_fixes_bug_sets_property(self):
 
373
        """commit --fixes=lp:234 sets the lp:234 revprop to 'fixed'."""
 
374
        tree = self.make_branch_and_tree('tree')
 
375
        self.build_tree(['tree/hello.txt'])
 
376
        tree.add('hello.txt')
 
377
        self.run_bzr(
 
378
            'commit', '-m', 'hello', '--fixes=lp:234', 'tree/hello.txt')
 
379
 
 
380
        # Get the revision properties, ignoring the branch-nick property, which
 
381
        # we don't care about for this test.
 
382
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
383
        properties = dict(last_rev.properties)
 
384
        del properties['branch-nick']
 
385
 
 
386
        self.assertEqual({'bugs': 'https://launchpad.net/bugs/234 fixed'},
 
387
                         properties)
 
388
 
 
389
    def test_fixes_multiple_bugs_sets_properties(self):
 
390
        """--fixes can be used more than once to show that bugs are fixed."""
 
391
        tree = self.make_branch_and_tree('tree')
 
392
        self.build_tree(['tree/hello.txt'])
 
393
        tree.add('hello.txt')
 
394
        self.run_bzr(
 
395
            'commit', '-m', 'hello', '--fixes=lp:123', '--fixes=lp:235',
 
396
            'tree/hello.txt')
 
397
 
 
398
        # Get the revision properties, ignoring the branch-nick property, which
 
399
        # we don't care about for this test.
 
400
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
401
        properties = dict(last_rev.properties)
 
402
        del properties['branch-nick']
 
403
 
 
404
        self.assertEqual(
 
405
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
406
                     'https://launchpad.net/bugs/235 fixed'},
 
407
            properties)
 
408
 
 
409
    def test_fixes_bug_with_alternate_trackers(self):
 
410
        """--fixes can be used on a properly configured branch to mark bug
 
411
        fixes on multiple trackers.
 
412
        """
 
413
        tree = self.make_branch_and_tree('tree')
 
414
        tree.branch.get_config().set_user_option(
 
415
            'trac_twisted_url', 'http://twistedmatrix.com/trac')
 
416
        self.build_tree(['tree/hello.txt'])
 
417
        tree.add('hello.txt')
 
418
        self.run_bzr(
 
419
            'commit', '-m', 'hello', '--fixes=lp:123',
 
420
            '--fixes=twisted:235', 'tree/')
 
421
 
 
422
        # Get the revision properties, ignoring the branch-nick property, which
 
423
        # we don't care about for this test.
 
424
        last_rev = tree.branch.repository.get_revision(tree.last_revision())
 
425
        properties = dict(last_rev.properties)
 
426
        del properties['branch-nick']
 
427
 
 
428
        self.assertEqual(
 
429
            {'bugs': 'https://launchpad.net/bugs/123 fixed\n'
 
430
                     'http://twistedmatrix.com/trac/ticket/235 fixed'},
 
431
            properties)
 
432
 
 
433
    def test_fixes_unknown_bug_prefix(self):
 
434
        tree = self.make_branch_and_tree('tree')
 
435
        self.build_tree(['tree/hello.txt'])
 
436
        tree.add('hello.txt')
 
437
        self.run_bzr_error(
 
438
            ["Unrecognized bug %s. Commit refused." % 'xxx:123'],
 
439
            'commit', '-m', 'add b', '--fixes=xxx:123',
 
440
            working_dir='tree')
 
441
 
 
442
    def test_fixes_invalid_bug_number(self):
 
443
        tree = self.make_branch_and_tree('tree')
 
444
        self.build_tree(['tree/hello.txt'])
 
445
        tree.add('hello.txt')
 
446
        self.run_bzr_error(
 
447
            ["Invalid bug identifier for %s. Commit refused." % 'lp:orange'],
 
448
            'commit', '-m', 'add b', '--fixes=lp:orange',
 
449
            working_dir='tree')
 
450
 
 
451
    def test_fixes_invalid_argument(self):
 
452
        """Raise an appropriate error when the fixes argument isn't tag:id."""
 
453
        tree = self.make_branch_and_tree('tree')
 
454
        self.build_tree(['tree/hello.txt'])
 
455
        tree.add('hello.txt')
 
456
        self.run_bzr_error(
 
457
            [r"Invalid bug orange. Must be in the form of 'tag:id'\. "
 
458
             r"Commit refused\."],
 
459
            'commit', '-m', 'add b', '--fixes=orange',
 
460
            working_dir='tree')