← Back to branch summary

~dpb/tarmac/ls-fixes

« back to all changes in this revision

Viewing changes to tarmac/tests/test_commands.py

  • Committer: Jonathan Lange
  • Date: 2012-12-20 13:01:05 UTC
  • mfrom: (414.1.9 break-up-do-merges)
  • Revision ID: jml@canonical.com-20121220130105-hif52d79o2eaft03
Merge James's branch to break up _do_merges, plus changes from my review.

Show diffs side-by-side

added added

removed removed

Lines of Context:
tarmac/tests/test_commands.py
198
198
 
199
199
    def test_get_reviews(self):
200
200
        """Test that the _get_reviews method gives the right lists."""
201
 
        self.assertEqual(self.command._get_reviews(self.proposals[0]),
202
 
                         [u'Reviewer;Needs Fixing'])
203
 
        self.assertEqual(self.command._get_reviews(self.proposals[1]),
204
 
                         [u'Reviewer;Approve', u'Reviewer2;Abstain'])
 
201
        self.assertEqual(
 
202
            commands._get_reviews(self.proposals[0]),
 
203
            [u'Reviewer;Needs Fixing'])
 
204
        self.assertEqual(
 
205
            commands._get_reviews(self.proposals[1]),
 
206
            [u'Reviewer;Approve', u'Reviewer2;Abstain'])
205
207
 
206
208
    def test_run_merge_url_substitution(self):
207
209
        """Test that the merge urls get substituted correctly."""