Merge lp:~al-maisan/launchpad/class-provides-508292 into lp:launchpad/db-devel

Proposed by Muharem Hrnjadovic
Status: Merged
Approved by: Jonathan Lange
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~al-maisan/launchpad/class-provides-508292
Merge into: lp:launchpad/db-devel
Diff against target: 59 lines (+6/-4)
4 files modified
lib/lp/buildmaster/model/buildfarmjob.py (+1/-2)
lib/lp/buildmaster/tests/test_manager.py (+2/-0)
lib/lp/soyuz/configure.zcml (+2/-0)
lib/lp/soyuz/model/buildqueue.py (+1/-2)
To merge this branch: bzr merge lp:~al-maisan/launchpad/class-provides-508292
Reviewer Review Type Date Requested Status
Jonathan Lange (community) Approve
Review via email: mp+17527@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Muharem Hrnjadovic (al-maisan) wrote :

Fixes a few errors and missing things in the integration branch submitted earlier.

Revision history for this message
Jonathan Lange (jml) wrote :

Looks good, thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/buildmaster/model/buildfarmjob.py'
--- lib/lp/buildmaster/model/buildfarmjob.py 2010-01-16 05:54:16 +0000
+++ lib/lp/buildmaster/model/buildfarmjob.py 2010-01-16 23:48:17 +0000
@@ -19,8 +19,7 @@
19class BuildFarmJob:19class BuildFarmJob:
20 """Mix-in class for `IBuildFarmJob` implementations."""20 """Mix-in class for `IBuildFarmJob` implementations."""
21 implements(IBuildFarmJob)21 implements(IBuildFarmJob)
22 classProvides(IBuildFarmCandidateJobSelection)22 classProvides(IBuildFarmCandidateJobSelection, ISpecificBuildFarmJobClass)
23 classProvides(ISpecificBuildFarmJobClass)
2423
25 def score(self):24 def score(self):
26 """See `IBuildFarmJob`."""25 """See `IBuildFarmJob`."""
2726
=== modified file 'lib/lp/buildmaster/tests/test_manager.py'
--- lib/lp/buildmaster/tests/test_manager.py 2010-01-11 23:43:59 +0000
+++ lib/lp/buildmaster/tests/test_manager.py 2010-01-16 23:48:17 +0000
@@ -494,12 +494,14 @@
494494
495 def assertBuildingJob(self, job, builder, logtail=None):495 def assertBuildingJob(self, job, builder, logtail=None):
496 """Assert the given job is building on the given builder."""496 """Assert the given job is building on the given builder."""
497 from lp.services.job.interfaces.job import JobStatus
497 if logtail is None:498 if logtail is None:
498 logtail = 'Dummy sampledata entry, not processing'499 logtail = 'Dummy sampledata entry, not processing'
499500
500 self.assertTrue(job is not None)501 self.assertTrue(job is not None)
501 self.assertEqual(job.builder, builder)502 self.assertEqual(job.builder, builder)
502 self.assertTrue(job.date_started is not None)503 self.assertTrue(job.date_started is not None)
504 self.assertEqual(job.job.status, JobStatus.RUNNING)
503 build = getUtility(IBuildSet).getByQueueEntry(job)505 build = getUtility(IBuildSet).getByQueueEntry(job)
504 self.assertEqual(build.buildstate, BuildStatus.BUILDING)506 self.assertEqual(build.buildstate, BuildStatus.BUILDING)
505 self.assertEqual(job.logtail, logtail)507 self.assertEqual(job.logtail, logtail)
506508
=== modified file 'lib/lp/soyuz/configure.zcml'
--- lib/lp/soyuz/configure.zcml 2010-01-16 05:54:16 +0000
+++ lib/lp/soyuz/configure.zcml 2010-01-16 23:48:17 +0000
@@ -895,6 +895,8 @@
895 <class895 <class
896 class="lp.soyuz.model.buildpackagejob.BuildPackageJob">896 class="lp.soyuz.model.buildpackagejob.BuildPackageJob">
897 <allow897 <allow
898 interface="lp.buildmaster.interfaces.buildfarmjob.IBuildFarmJob"/>
899 <allow
898 interface="lp.soyuz.interfaces.buildpackagejob.IBuildPackageJob"/>900 interface="lp.soyuz.interfaces.buildpackagejob.IBuildPackageJob"/>
899 </class>901 </class>
900 <!-- 902 <!--
901903
=== modified file 'lib/lp/soyuz/model/buildqueue.py'
--- lib/lp/soyuz/model/buildqueue.py 2010-01-16 05:54:16 +0000
+++ lib/lp/soyuz/model/buildqueue.py 2010-01-16 23:48:17 +0000
@@ -68,8 +68,7 @@
68 lastscore = IntCol(dbName='lastscore', default=0)68 lastscore = IntCol(dbName='lastscore', default=0)
69 manual = BoolCol(dbName='manual', default=False)69 manual = BoolCol(dbName='manual', default=False)
70 estimated_duration = IntervalCol()70 estimated_duration = IntervalCol()
71 processor = ForeignKey(71 processor = ForeignKey(dbName='processor', foreignKey='Processor')
72 dbName='processor', foreignKey='Processor', notNull=True)
73 virtualized = BoolCol(dbName='virtualized')72 virtualized = BoolCol(dbName='virtualized')
7473
75 @property74 @property

Subscribers

People subscribed via source and target branches

to status/vote changes: