Launchpad itself

Merge lp:~abentley/launchpad/no-default-powerpc into lp:launchpad

  1. no-default-powerpc
  2. Merge into devel
Proposed by Aaron Bentley
Status: Merged
Merged at revision: 10833
Proposed branch: lp:~abentley/launchpad/no-default-powerpc
Merge into: lp:launchpad
Prerequisite: lp:~abentley/launchpad/recipe-build-index
Diff against target: 53 lines (+5/-7)
3 files modified
lib/lp/code/browser/tests/test_sourcepackagerecipe.py (+2/-2)
lib/lp/code/interfaces/sourcepackagerecipebuild.py (+1/-2)
lib/lp/testing/factory.py (+2/-3)
To merge this branch: bzr merge lp:~abentley/launchpad/no-default-powerpc
Reviewer Review Type Date Requested Status
Paul Hummer (community) code Approve
Review via email: mp+24855@code.launchpad.net

Commit message

Cleanups from recipe-build-index.

Description of the change

= Summary =
Fixes for some issues I encountered while hacking on recipe build index:
 - test_sourcepackagerecipebuild should be in code/model/tests, not code/tests.
 - test_recipebuilder should be in code/model/tests, not code/tests.
 - LaunchpadObjectFactory.makeDistroArchSeries should default to None, not
   'powerpc'

== Proposed fix ==
See above.

== Pre-implementation notes ==
None

== Implementation details ==
Fixed the one callsite for makeDistroArchSeries that was specifying None.

== Tests ==
bin/test -t test_sourcepackagerecipebuild -t test_recipebuilder

== Demo and Q/A ==
None

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/lp/code/model/tests/test_sourcepackagerecipebuild.py
  lib/lp/code/interfaces/sourcepackagerecipebuild.py
  configs/development/build-from-branch.zcml
  lib/lp/testing/factory.py
  lib/lp/code/templates/sourcepackagerecipebuild-index.pt
  lib/lp/testing/__init__.py
  lib/lp/code/model/sourcepackagerecipebuild.py
  lib/lp/code/templates/sourcepackagerecipe-index.pt
  lib/canonical/launchpad/webapp/configure.zcml
  lib/lp/code/browser/sourcepackagerecipe.py
  lib/canonical/launchpad/webapp/tales.py
  lib/lp/code/browser/tests/test_sourcepackagerecipe.py

To post a comment you must log in.
Revision history for this message
Paul Hummer (rockstar) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/code/browser/tests/test_sourcepackagerecipe.py'
--- lib/lp/code/browser/tests/test_sourcepackagerecipe.py 2010-05-07 13:47:27 +0000
+++ lib/lp/code/browser/tests/test_sourcepackagerecipe.py 2010-05-07 13:47:29 +0000
@@ -429,8 +429,8 @@
429 my-recipe429 my-recipe
430 Build status430 Build status
431 Successfully built431 Successfully built
432 Started on 2008-12-31432 Started on .*
433 Finished on 2009-01-01433 Finished on .*
434 \(took 1 minute, 0.0 seconds\)434 \(took 1 minute, 0.0 seconds\)
435 buildlog \(8 bytes\)435 buildlog \(8 bytes\)
436 uploadlog \(10 bytes\)436 uploadlog \(10 bytes\)
437437
=== modified file 'lib/lp/code/interfaces/sourcepackagerecipebuild.py'
--- lib/lp/code/interfaces/sourcepackagerecipebuild.py 2010-05-07 13:47:27 +0000
+++ lib/lp/code/interfaces/sourcepackagerecipebuild.py 2010-05-07 13:47:29 +0000
@@ -40,8 +40,7 @@
4040
41 binary_builds = CollectionField(41 binary_builds = CollectionField(
42 Reference(IBinaryPackageBuild),42 Reference(IBinaryPackageBuild),
43 title=_("The binary builds that resulted from this."),43 title=_("The binary builds that resulted from this."), readonly=True)
44 readonly=True)
4544
46 datestarted = Datetime(title=u'The time the build started.')45 datestarted = Datetime(title=u'The time the build started.')
4746
4847
=== renamed file 'lib/lp/code/tests/test_recipebuilder.py' => 'lib/lp/code/model/tests/test_recipebuilder.py'
=== renamed file 'lib/lp/code/tests/test_sourcepackagerecipebuild.py' => 'lib/lp/code/model/tests/test_sourcepackagerecipebuild.py'
=== modified file 'lib/lp/testing/factory.py'
--- lib/lp/testing/factory.py 2010-05-07 13:47:27 +0000
+++ lib/lp/testing/factory.py 2010-05-07 13:47:29 +0000
@@ -1626,7 +1626,7 @@
1626 makeDistroSeries = makeDistroRelease1626 makeDistroSeries = makeDistroRelease
16271627
1628 def makeDistroArchSeries(self, distroseries=None,1628 def makeDistroArchSeries(self, distroseries=None,
1629 architecturetag='powerpc', processorfamily=None,1629 architecturetag=None, processorfamily=None,
1630 official=True, owner=None,1630 official=True, owner=None,
1631 supports_virtualized=False):1631 supports_virtualized=False):
1632 """Create a new distroarchseries"""1632 """Create a new distroarchseries"""
@@ -2171,8 +2171,7 @@
2171 if distroarchseries is None:2171 if distroarchseries is None:
2172 distroarchseries = self.makeDistroArchSeries(2172 distroarchseries = self.makeDistroArchSeries(
2173 distroseries=source_package_release.upload_distroseries,2173 distroseries=source_package_release.upload_distroseries,
2174 processorfamily=processor.family,2174 processorfamily=processor.family)
2175 architecturetag=None)
2176 binary_package_build = BinaryPackageBuild(2175 binary_package_build = BinaryPackageBuild(
2177 sourcepackagerelease=source_package_release,2176 sourcepackagerelease=source_package_release,
2178 processor=processor,2177 processor=processor,