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
1=== modified file 'lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py'
2--- lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py 2013-09-02 08:11:58 +0000
3+++ lib/lp/soyuz/tests/test_binarypackagebuildbehavior.py 2013-09-03 09:33:30 +0000
4@@ -27,7 +27,6 @@
5 IBuildFarmJobBehavior,
6 )
7 from lp.buildmaster.tests.mock_slaves import (
8- AbortedSlave,
9 AbortingSlave,
10 BuildingSlave,
11 OkSlave,
12@@ -411,17 +410,6 @@
13 d = self.interactor.updateBuild(self.candidate)
14 return d.addCallback(got_update)
15
16- def test_aborted_collection(self):
17- # The builder aborted the job.
18- self.patch(BuilderSlave, 'makeBuilderSlave',
19- FakeMethod(AbortedSlave()))
20-
21- def got_update(ignored):
22- self.assertEqual(BuildStatus.NEEDSBUILD, self.build.status)
23-
24- d = self.interactor.updateBuild(self.candidate)
25- return d.addCallback(got_update)
26-
27 def test_aborting_collection(self):
28 # The builder is in the process of aborting.
29 self.patch(BuilderSlave, 'makeBuilderSlave',