Launchpad itself

Merge lp://qastaging/~salgado/launchpad/expose-blueprint-dependencies into lp://qastaging/launchpad

  1. expose-blueprint-dependencies
  2. Merge into devel
Proposed by Guilherme Salgado
Status: Merged
Approved by: Aaron Bentley
Approved revision: 12001
Merged at revision: 12001
Proposed branch: lp://qastaging/~salgado/launchpad/expose-blueprint-dependencies
Merge into: lp://qastaging/launchpad
Diff against target: 109 lines (+29/-8)
4 files modified
lib/canonical/launchpad/interfaces/_schema_circular_imports.py (+3/-0)
lib/lp/blueprints/interfaces/specification.py (+11/-2)
lib/lp/blueprints/tests/test_hasspecifications.py (+1/-1)
lib/lp/blueprints/tests/test_webservice.py (+14/-5)
To merge this branch: bzr merge lp://qastaging/~salgado/launchpad/expose-blueprint-dependencies
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Review via email: mp+42145@code.qastaging.launchpad.net

Commit message

[r=abentley][ui=none][bug=682757]

Description of the change

Expose ISpecification.dependencies on the webservice API.

To post a comment you must log in.
lp://qastaging/~salgado/launchpad/expose-blueprint-dependencies updated
11999. By Guilherme Salgado

merge test refactoring from other branch

12000. By Guilherme Salgado

Merge trunk

12001. By Guilherme Salgado

Tweak some tests to use assertContentEqual instead of sorting their lists manually

Revision history for this message
Aaron Bentley (abentley) wrote :

At 23, the import should be multi-line. Also, you should consider using lp.testing.ws_object rather than getSpecOnWebservice, because it is more general.

review: Approve
lp://qastaging/~salgado/launchpad/expose-blueprint-dependencies updated
12002. By Guilherme Salgado

Use multi-line imports when importing multiple names

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.