Merge lp:~cjwatson/launchpad/testfix-destroy-builder-aborted into lp:launchpad

Proposed by Colin Watson
Status: Merged
Approved by: William Grant
Approved revision: no longer in the source branch.
Merged at revision: 16756
Proposed branch: lp:~cjwatson/launchpad/testfix-destroy-builder-aborted
Merge into: lp:launchpad
Diff against target: 29 lines (+0/-12)
1 file modified
lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py (+0/-12)
To merge this branch: bzr merge lp:~cjwatson/launchpad/testfix-destroy-builder-aborted
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+183598@code.launchpad.net

Commit message

Remove another reference to AbortedSlave.

Description of the change

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py'
--- lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py 2013-09-02 08:11:58 +0000
+++ lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py 2013-09-03 09:33:30 +0000
@@ -27,7 +27,6 @@
27 IBuildFarmJobBehavior,27 IBuildFarmJobBehavior,
28 )28 )
29from lp.buildmaster.tests.mock_slaves import (29from lp.buildmaster.tests.mock_slaves import (
30 AbortedSlave,
31 AbortingSlave,30 AbortingSlave,
32 BuildingSlave,31 BuildingSlave,
33 OkSlave,32 OkSlave,
@@ -411,17 +410,6 @@
411 d = self.interactor.updateBuild(self.candidate)410 d = self.interactor.updateBuild(self.candidate)
412 return d.addCallback(got_update)411 return d.addCallback(got_update)
413412
414 def test_aborted_collection(self):
415 # The builder aborted the job.
416 self.patch(BuilderSlave, 'makeBuilderSlave',
417 FakeMethod(AbortedSlave()))
418
419 def got_update(ignored):
420 self.assertEqual(BuildStatus.NEEDSBUILD, self.build.status)
421
422 d = self.interactor.updateBuild(self.candidate)
423 return d.addCallback(got_update)
424
425 def test_aborting_collection(self):413 def test_aborting_collection(self):
426 # The builder is in the process of aborting.414 # The builder is in the process of aborting.
427 self.patch(BuilderSlave, 'makeBuilderSlave',415 self.patch(BuilderSlave, 'makeBuilderSlave',