Merge lp:~maxb/bzr-builddeb/testfix2 into lp:bzr-builddeb

Proposed by Max Bowsher
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: 529
Merged at revision: 530
Proposed branch: lp:~maxb/bzr-builddeb/testfix2
Merge into: lp:bzr-builddeb
Prerequisite: lp:~maxb/bzr-builddeb/python-debian-compat
Diff against target: 34 lines (+4/-4)
1 file modified
tests/test_upstream.py (+4/-4)
To merge this branch: bzr merge lp:~maxb/bzr-builddeb/testfix2
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+51436@code.launchpad.net

Commit message

Fix some tests which erroneously expect a Version object instead of a string.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/test_upstream.py'
2--- tests/test_upstream.py 2011-02-21 16:49:06 +0000
3+++ tests/test_upstream.py 2011-02-26 21:12:28 +0000
4@@ -375,18 +375,18 @@
5
6 def test_snapshot_none_existing(self):
7 self.revhistory = ["somerevid"]
8- self.assertEquals(Version("1.2+bzr1"),
9+ self.assertEquals("1.2+bzr1",
10 _upstream_branch_version(self.revhistory, {}, "bla", "1.2", self.get_suffix))
11
12 def test_snapshot_nothing_new(self):
13 self.revhistory = []
14- self.assertEquals(Version("1.2"),
15+ self.assertEquals("1.2",
16 _upstream_branch_version(self.revhistory, {}, "bla", "1.2", self.get_suffix))
17
18 def test_new_tagged_release(self):
19 """Last revision is tagged - use as upstream version."""
20 self.revhistory = ["somerevid"]
21- self.assertEquals(Version("1.3"),
22+ self.assertEquals("1.3",
23 _upstream_branch_version(self.revhistory, {"somerevid": [u"1.3"]}, "bla", "1.2", self.get_suffix))
24
25 def test_refresh_snapshot_pre(self):
26@@ -411,7 +411,7 @@
27 class TestUpstreamTagToVersion(TestCase):
28
29 def test_prefix(self):
30- self.assertEquals(Version("5.0"), upstream_tag_to_version(u"release-5.0"))
31+ self.assertEquals("5.0", upstream_tag_to_version(u"release-5.0"))
32
33 def test_gibberish(self):
34 self.assertIs(None, upstream_tag_to_version(u"blabla"))

Subscribers

People subscribed via source and target branches