Merge lp:~alecu/ubuntuone-client/assertis-is-only-27-stable into lp:ubuntuone-client/stable-1-6

Proposed by Alejandro J. Cura
Status: Merged
Approved by: Alejandro J. Cura
Approved revision: 962
Merged at revision: 963
Proposed branch: lp:~alecu/ubuntuone-client/assertis-is-only-27-stable
Merge into: lp:ubuntuone-client/stable-1-6
Diff against target: 29 lines (+3/-3)
1 file modified
tests/status/test_aggregator.py (+3/-3)
To merge this branch: bzr merge lp:~alecu/ubuntuone-client/assertis-is-only-27-stable
Reviewer Review Type Date Requested Status
dobey (community) Approve
Natalia Bidart (community) Approve
Review via email: mp+58424@code.launchpad.net

Commit message

replace assertIs with assertIdentical so tests work on Maverick and earlier (LP: #766181)

Description of the change

replace assertIs with assertIdentical so tests work on Maverick and earlier (LP: #766181)

To post a comment you must log in.
Revision history for this message
Natalia Bidart (nataliabidart) :
review: Approve
Revision history for this message
dobey (dobey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/status/test_aggregator.py'
--- tests/status/test_aggregator.py 2011-04-12 20:41:55 +0000
+++ tests/status/test_aggregator.py 2011-04-20 02:52:29 +0000
@@ -1202,7 +1202,7 @@
1202 self.assertEqual(self.aggregator.done_counter, 0)1202 self.assertEqual(self.aggregator.done_counter, 0)
1203 self.assertEqual(len(self.aggregator.files_uploading), 0)1203 self.assertEqual(len(self.aggregator.files_uploading), 0)
1204 self.assertEqual(len(self.aggregator.files_downloading), 0)1204 self.assertEqual(len(self.aggregator.files_downloading), 0)
1205 self.assertIs(self.aggregator.queue_done_timer, None)1205 self.assertIdentical(self.aggregator.queue_done_timer, None)
12061206
1207 def assertMiscCommandQueued(self, fc):1207 def assertMiscCommandQueued(self, fc):
1208 """Assert that some command was queued."""1208 """Assert that some command was queued."""
@@ -1386,7 +1386,7 @@
1386 self.aggregator.clock.advance(self.aggregator.finished_delay / 2)1386 self.aggregator.clock.advance(self.aggregator.finished_delay / 2)
1387 fc2 = FakeCommand()1387 fc2 = FakeCommand()
1388 self.status_frontend.download_started(fc2)1388 self.status_frontend.download_started(fc2)
1389 self.assertIs(self.aggregator.queue_done_timer, None)1389 self.assertIdentical(self.aggregator.queue_done_timer, None)
1390 self.aggregator.clock.advance(self.aggregator.finished_delay)1390 self.aggregator.clock.advance(self.aggregator.finished_delay)
1391 self.status_frontend.download_finished(fc2)1391 self.status_frontend.download_finished(fc2)
13921392
@@ -1400,7 +1400,7 @@
1400 self.aggregator.clock.advance(self.aggregator.finished_delay / 2)1400 self.aggregator.clock.advance(self.aggregator.finished_delay / 2)
1401 fc2 = FakeCommand()1401 fc2 = FakeCommand()
1402 self.status_frontend.upload_started(fc2)1402 self.status_frontend.upload_started(fc2)
1403 self.assertIs(self.aggregator.queue_done_timer, None)1403 self.assertIdentical(self.aggregator.queue_done_timer, None)
1404 self.aggregator.clock.advance(self.aggregator.finished_delay)1404 self.aggregator.clock.advance(self.aggregator.finished_delay)
1405 self.status_frontend.upload_finished(fc2)1405 self.status_frontend.upload_finished(fc2)
14061406

Subscribers

People subscribed via source and target branches