Merge lp:~allenap/launchpad/bugs-with-blueprints-bug-707103 into lp:launchpad

[Gavin Panella]

[r=henninge][ui=none][bug=707103] Bugs with or without linked Blueprints can be selected via searchTasks(). This is available to the internal and web-service (devel) APIs.

Make it possible to search for bugs with or without linked Blueprints. I will update the advanced search in a subsequent branch; this branch just deals with the internal and webservice API.

[Henning Eggers (henninge)]

[Sorry, somehow my mail client messed up. I hope you can figure out which lines my comments are referring to.]

Hi Gavin,
thanks for addressing this issue. Webservice API is not my strong side
so please forgive if I make the wrong assumptions.
The real problem in this branch that I see is
test_linked_blueprints_in_devel which I am sure needs fixing. Please
see my comments below.

 review needs-fixing

Cheers,
Henning

> > === modified file 'lib/lp/bugs/interfaces/bugtarget.py' ---
> > lib/lp/bugs/interfaces/bugtarget.py 2010-12-02 16:14:30 +0000
> > +++ lib/lp/bugs/interfaces/bugtarget.py 2011-01-31 14:57:17
> > +0000
[...]
> > @@ -203,16 +215,21 @@ has_bugtasks = Attribute( "True if a BugTask
> > has ever been reported for this target.")
> >
> > + # searchTasks devel API declaration.
> > @call_with(search_params=None, user=REQUEST_USER)
> > @operation_parameters(**search_tasks_params_for_api_devel)
> > @operation_returns_collection_of(IBugTask)
> > @export_read_operation() + + # Pop the *default* version
> > (decorators are run last to first).
> > @operation_removed_in_version('devel')
> >
> > + # searchTasks default API declaration.
> > @call_with(search_params=None, user=REQUEST_USER) -
> > @operation_parameters(**search_tasks_params_for_api_1_0) +
> > @operation_parameters(**search_tasks_params_for_api_default)
> > @operation_returns_collection_of(IBugTask)
> > @export_read_operation() + def searchTasks(search_params,
> > user=None, order_by=None, search_text=None, status=None,
> > importance=None,
I think it is due to my lack of webservice foo but I don't understand
how this works. Why are the decorators repeating themselves? Is that
the proper way to handle that? I am just asking so you can double
check, I don't expect a full explanation atm. ;-) Thanks for the added
comments, though.

> > === modified file 'lib/lp/bugs/interfaces/bugtask.py' ---
> > lib/lp/bugs/interfaces/bugtask.py 2010-12-17 04:26:11 +0000 +++
> > lib/lp/bugs/interfaces/bugtask.py 2011-01-31 14:57:17 +0000 @@
> > -9,6 +9,7 @@
> >
> > __all__ = [ 'BUG_SUPERVISOR_BUGTASK_STATUSES', +
> > 'BugBlueprintSearch', 'BugBranchSearch',
> > 'BugTagsSearchCombinator', 'BugTaskImportance', @@ -28,8 +29,6 @@
> > 'IDistroBugTask', 'IDistroSeriesBugTask',
> > 'IFrontPageBugTaskSearch', - 'IllegalRelatedBugTasksParams', -
> > 'IllegalTarget',
Hm, did you re-sort this manually or did some tool do it? For the
imports fixing script I implemented case insensitive sorting. But
that does not touch __all__.

> > 'INominationsReviewTableBatchNavigator', 'INullBugTask',
> > 'IPersonBugTaskSearch', @@ -37,13 +36,16 @@
> > 'IRemoveQuestionFromBugTaskForm', 'IUpstreamBugTask',
> > 'IUpstreamProductBugTaskSearch', +
> > 'IllegalRelatedBugTasksParams', + 'IllegalTarget',
> > 'RESOLVED_BUGTASK_STATUSES', 'UNRESOLVED_BUGTASK_STATUSES',
> > 'UserCannotEditBugTaskAssignee',
> > 'UserCannotEditBugTaskImportance',
> > 'UserCannotEditBugTaskMilestone', 'UserCannotEditBugTaskStatus', -
> > 'valid_remote_bug_url'] + 'valid_remote_bug_url', + ]
> >
> > from lazr.enum import ( DBEnumeratedType, @@ -340,7 +342,7 @@
> > """Bug branch search option.
> >
> > The possible values to search for bugs having branche...

review: Needs Fixing

[Gavin Panella (allenap)]

> [Sorry, somehow my mail client messed up. I hope you can figure out which
> lines my comments are referring to.]

Wow, it really nuked it :)

>
> Hi Gavin,
> thanks for addressing this issue. Webservice API is not my strong side
> so please forgive if I make the wrong assumptions.
> The real problem in this branch that I see is
> test_linked_blueprints_in_devel which I am sure needs fixing. Please
> see my comments below.
>
> review needs-fixing
>
> Cheers,
> Henning
>
>> === modified file 'lib/lp/bugs/interfaces/bugtarget.py' ---
>> lib/lp/bugs/interfaces/bugtarget.py 2010-12-02 16:14:30 +0000
>> +++ lib/lp/bugs/interfaces/bugtarget.py 2011-01-31 14:57:17
>> +0000
> [...]
>> @@ -203,16 +215,21 @@ has_bugtasks = Attribute( "True if a BugTask
>> has ever been reported for this target.")
>>
>> + # searchTasks devel API declaration.
>> @call_with(search_params=None, user=REQUEST_USER)
>> @operation_parameters(**search_tasks_params_for_api_devel)
>> @operation_returns_collection_of(IBugTask)
>> @export_read_operation() + + # Pop the *default* version
>> (decorators are run last to first).
>> @operation_removed_in_version('devel')
>>
>> + # searchTasks default API declaration.
>> @call_with(search_params=None, user=REQUEST_USER) -
>> @operation_parameters(**search_tasks_params_for_api_1_0) +
>> @operation_parameters(**search_tasks_params_for_api_default)
>> @operation_returns_collection_of(IBugTask)
>> @export_read_operation() + def searchTasks(search_params,
>> user=None, order_by=None, search_text=None, status=None,
>> importance=None,
>
> I think it is due to my lack of webservice foo but I don't understand
> how this works. Why are the decorators repeating themselves? Is that
> the proper way to handle that? I am just asking so you can double
> check, I don't expect a full explanation atm. ;-) Thanks for the added
> comments, though.

It's a bit crazy. While fixing this branch I filed a couple of bugs
against lazr.restful because of it, but here goes...

Everything has to be read in reverse, then we have to realise that
annotations are assembled onto a stack.

* This block declares the default API for searchTasks.

    export_read_operation()
    operation_returns_collection_of(IBugTask)
    operation_parameters(**search_tasks_params_for_api_default)
    call_with(search_params=None, user=REQUEST_USER)

* This is a weird operation. It declares that the operation above is
  not available in the devel API version by pushing a new version onto
  the annotations stack and setting its type to "removed_operation".

    operation_removed_in_version('devel')

* This block now declares the devel API for searchTasks. The
  annotation on the stack is already for the devel version by virtue
  of operation_removed_in_version. The export_read_operation below
  changes the annotation type to "read_operation".

    export_read_operation()
    operation_returns_collection_of(IBugTask)
    operation_parameters(**search_tasks_params_for_api_devel)
    call_with(search_params=None, user=REQUEST_USER)

>
>> === modified file 'lib/lp/bugs/interfaces/bugtask.py' ---
>> lib/lp/bugs/interfaces/bugtask.py 2010-12-17 04:26:11 +0000 +++
>> lib/lp/bugs/interfaces/...

[Henning Eggers (henninge)]

Thanks for your reply an the explanations.

>> Hm, did you re-sort this manually or did some tool do it? For the
>> imports fixing script I implemented case insensitive sorting. But
>> that does not touch __all__.
>
>A tool did it for me: Emacs :)
>
>(I set sort-fold-case before using sort-lines.)
>
>Is it wrong?

Since we don't have a written-down convention on it, it is not. ;-) I was merely wondering because for the import formatting I had used case-insensitive sorting, after a little discussion, because it is easier for humans to parse. But we never mentioned to use the same for __all__ although that would make sense to me. I was just wondering if you did this consciously (and have an opinion) or not. I guess the latter. No big deal, though.

You missed a major comment, though, which I am sure is due to the wacky formatting of my reply. I apologize for that again. Here it is:

> === modified file 'lib/lp/bugs/tests/test_searchtasks_webservice.py'
> @@ -35,3 +38,51 @@
> response = self.webservice.named_get('/mebuntu/inkanyamba',
> 'searchTasks', api_version='devel').jsonBody()
> self.assertEqual(response['total_size'], 1)
> +
> + def test_linked_blueprints_in_devel(self):
> + response = self.webservice.named_get('/mebuntu/inkanyamba',
> + 'searchTasks', api_version='devel').jsonBody()
> + self.assertEqual(response['total_size'], 1)
> +

I am not really sure I understand what this test does. I don't see
where the code refers to the new search parameter. Actually, it looks
identical to the previous test. Something missing here?

I won't block you on this because of my currently long feedback loop and I trust you will be able to either explain or fix it.

Thanks again!

review: Approve (code)

[Gavin Panella (allenap)]

> Thanks for your reply an the explanations.
>
> >> Hm, did you re-sort this manually or did some tool do it? For the
> >> imports fixing script I implemented case insensitive sorting. But
> >> that does not touch __all__.
> >
> >A tool did it for me: Emacs :)
> >
> >(I set sort-fold-case before using sort-lines.)
> >
> >Is it wrong?
>
> Since we don't have a written-down convention on it, it is not. ;-) I was
> merely wondering because for the import formatting I had used case-insensitive
> sorting, after a little discussion, because it is easier for humans to parse.
> But we never mentioned to use the same for __all__ although that would make
> sense to me. I was just wondering if you did this consciously (and have an
> opinion) or not. I guess the latter. No big deal, though.

Ah, I thought we had agreed to sort case-insensitively. My preference
is actually for case-sensitive sorting since it is the default in
Emacs, and because I, strangely, find it easier to read.

> You missed a major comment, though, which I am sure is due to the wacky
> formatting of my reply. I apologize for that again. Here it is:

Oops, thanks.

> > === modified file 'lib/lp/bugs/tests/test_searchtasks_webservice.py'
> > @@ -35,3 +38,51 @@
> > response = self.webservice.named_get('/mebuntu/inkanyamba',
> > 'searchTasks', api_version='devel').jsonBody()
> > self.assertEqual(response['total_size'], 1)
> > +
> > + def test_linked_blueprints_in_devel(self):
> > + response = self.webservice.named_get('/mebuntu/inkanyamba',
> > + 'searchTasks', api_version='devel').jsonBody()
> > + self.assertEqual(response['total_size'], 1)
> > +
>
> I am not really sure I understand what this test does. I don't see
> where the code refers to the new search parameter. Actually, it looks
> identical to the previous test. Something missing here?

It was a copy and paste error! Thanks for spotting it.

=== modified file 'lib/lp/bugs/interfaces/bugtarget.py'
--- lib/lp/bugs/interfaces/bugtarget.py	2010-12-02 16:14:30 +0000
+++ lib/lp/bugs/interfaces/bugtarget.py	2011-02-01 16:35:45 +0000
@@ -35,7 +35,6 @@
     )
 from lazr.restful.fields import Reference
 from lazr.restful.interface import copy_field
-
 from zope.interface import (
     Attribute,
     Interface,
@@ -52,6 +51,7 @@
 
 from canonical.launchpad import _
 from lp.bugs.interfaces.bugtask import (
+    BugBlueprintSearch,
     BugBranchSearch,
     BugTagsSearchCombinator,
     IBugTask,
@@ -59,7 +59,8 @@
     )
 from lp.services.fields import Tag
 
-search_tasks_params_for_api_1_0 = {
+
+search_tasks_params_common = {
     "order_by": List(
         title=_('List of fields by which the results are ordered.'),
         value_type=Text(),
@@ -80,7 +81,6 @@
     "tags": copy_field(IBugTaskSearch['tag']),
     "tags_combinator": copy_field(IBugTaskSearch['tags_combinator']),
     "omit_duplicates": copy_field(IBugTaskSearch['omit_dupes']),
-    "omit_targeted": copy_field(IBugTaskSearch['omit_targeted']),
     "status_upstream": copy_field(IBugTaskSearch['status_upstream']),
     "milestone_assignment": copy_field(
         IBugTaskSearch['milestone_assignment']),
@@ -172,9 +172,21 @@
             "date."),
         required=False),
     }
-search_tasks_params_for_api_devel = search_tasks_params_for_api_1_0.copy()
-search_tasks_params_for_api_devel["omit_targeted"] = copy_field(
-    IBugTaskSearch['omit_targeted'], default=False)
+
+search_tasks_params_for_api_default = dict(
+    search_tasks_params_common,
+    omit_targeted=copy_field(
+        IBugTaskSearch['omit_targeted']))
+
+search_tasks_params_for_api_devel = dict(
+    search_tasks_params_common,
+    omit_targeted=copy_field(
+        IBugTaskSearch['omit_targeted'], default=False),
+    linked_blueprints=Choice(
+        title=_(
+            u"Search for bugs that are linked to blueprints or for "
+            u"bugs that are not linked to blueprints."),
+        vocabulary=BugBlueprintSearch, required=False))
 
 
 class IHasBugs(Interface):
@@ -203,16 +215,21 @@
     has_bugtasks = Attribute(
         "True if a BugTask has ever been reported for this target.")
 
+    # searchTasks devel API declaration.
     @call_with(search_params=None, user=REQUEST_USER)
     @operation_parameters(**search_tasks_params_for_api_devel)
     @operation_returns_collection_of(IBugTask)
     @export_read_operation()
+
+    # Pop the *default* version (decorators are run last to first).
     @operation_removed_in_version('devel')
 
+    # searchTasks default API declaration.
     @call_with(search_params=None, user=REQUEST_USER)
-    @operation_parameters(**search_tasks_params_for_api_1_0)
+    @operation_parameters(**search_tasks_params_for_api_default)
     @operation_returns_collection_of(IBugTask)
     @export_read_operation()
+
     def searchTasks(search_params, user=None,
                     order_by=None, search_text=None,
                     status=None, importance=None,
@@ -232,8 +249,8 @@
                     hardware_owner_is_affected_by_bug=False,
                     hardware_owner_is_subscribed_to_bug=False,
                     hardware_is_linked_to_bug=False, linked_branches=None,
-                    structural_subscriber=None, modified_since=None,
-                    created_since=None, prejoins=[]):
+                    linked_blueprints=None, structural_subscriber=None,
+                    modified_since=None, created_since=None, prejoins=[]):
         """Search the IBugTasks reported on this entity.
 
         :search_params: a BugTaskSearchParams object

=== modified file 'lib/lp/bugs/interfaces/bugtask.py'
--- lib/lp/bugs/interfaces/bugtask.py	2010-12-17 04:26:11 +0000
+++ lib/lp/bugs/interfaces/bugtask.py	2011-02-01 16:35:45 +0000
@@ -9,6 +9,7 @@
 
 __all__ = [
     'BUG_SUPERVISOR_BUGTASK_STATUSES',
+    'BugBlueprintSearch',
     'BugBranchSearch',
     'BugTagsSearchCombinator',
     'BugTaskImportance',
@@ -28,8 +29,6 @@
     'IDistroBugTask',
     'IDistroSeriesBugTask',
     'IFrontPageBugTaskSearch',
-    'IllegalRelatedBugTasksParams',
-    'IllegalTarget',
     'INominationsReviewTableBatchNavigator',
     'INullBugTask',
     'IPersonBugTaskSearch',
@@ -37,13 +36,16 @@
     'IRemoveQuestionFromBugTaskForm',
     'IUpstreamBugTask',
     'IUpstreamProductBugTaskSearch',
+    'IllegalRelatedBugTasksParams',
+    'IllegalTarget',
     'RESOLVED_BUGTASK_STATUSES',
     'UNRESOLVED_BUGTASK_STATUSES',
     'UserCannotEditBugTaskAssignee',
     'UserCannotEditBugTaskImportance',
     'UserCannotEditBugTaskMilestone',
     'UserCannotEditBugTaskStatus',
-    'valid_remote_bug_url']
+    'valid_remote_bug_url',
+    ]
 
 from lazr.enum import (
     DBEnumeratedType,
@@ -340,7 +342,7 @@
     """Bug branch search option.
 
     The possible values to search for bugs having branches attached
-    or not having branches attched.
+    or not having branches attached.
     """
 
     ALL = Item("Show all bugs")
@@ -350,6 +352,20 @@
     BUGS_WITHOUT_BRANCHES = Item("Show only Bugs without linked Branches")
 
 
+class BugBlueprintSearch(EnumeratedType):
+    """Bug blueprint search option.
+
+    The possible values to search for bugs having blueprints attached
+    or not having blueprints attached.
+    """
+
+    ALL = Item("Show all bugs")
+
+    BUGS_WITH_BLUEPRINTS = Item("Show only Bugs with linked Blueprints")
+
+    BUGS_WITHOUT_BLUEPRINTS = Item("Show only Bugs without linked Blueprints")
+
+
 # XXX: Brad Bollenbach 2005-12-02 bugs=5320:
 # In theory, INCOMPLETE belongs in UNRESOLVED_BUGTASK_STATUSES, but the
 # semantics of our current reports would break if it were added to the
@@ -1144,9 +1160,9 @@
                  hardware_owner_is_affected_by_bug=False,
                  hardware_owner_is_subscribed_to_bug=False,
                  hardware_is_linked_to_bug=False,
-                 linked_branches=None, structural_subscriber=None,
-                 modified_since=None, created_since=None,
-                 exclude_conjoined_tasks=False):
+                 linked_branches=None, linked_blueprints=None,
+                 structural_subscriber=None, modified_since=None,
+                 created_since=None, exclude_conjoined_tasks=False):
 
         self.bug = bug
         self.searchtext = searchtext
@@ -1189,6 +1205,7 @@
             hardware_owner_is_subscribed_to_bug)
         self.hardware_is_linked_to_bug = hardware_is_linked_to_bug
         self.linked_branches = linked_branches
+        self.linked_blueprints = linked_blueprints
         self.structural_subscriber = structural_subscriber
         self.modified_since = modified_since
         self.created_since = created_since
@@ -1265,8 +1282,8 @@
                        hardware_owner_is_affected_by_bug=False,
                        hardware_owner_is_subscribed_to_bug=False,
                        hardware_is_linked_to_bug=False, linked_branches=None,
-                       structural_subscriber=None, modified_since=None,
-                       created_since=None):
+                       linked_blueprints=None, structural_subscriber=None,
+                       modified_since=None, created_since=None):
         """Create and return a new instance using the parameter list."""
         search_params = cls(user=user, orderby=order_by)
 
@@ -1332,7 +1349,8 @@
             hardware_owner_is_subscribed_to_bug)
         search_params.hardware_is_linked_to_bug = (
             hardware_is_linked_to_bug)
-        search_params.linked_branches=linked_branches
+        search_params.linked_branches = linked_branches
+        search_params.linked_blueprints = linked_blueprints
         search_params.structural_subscriber = structural_subscriber
         search_params.modified_since = modified_since
         search_params.created_since = created_since

=== modified file 'lib/lp/bugs/model/bugtarget.py'
--- lib/lp/bugs/model/bugtarget.py	2010-11-30 21:39:25 +0000
+++ lib/lp/bugs/model/bugtarget.py	2011-02-01 16:35:45 +0000
@@ -94,7 +94,8 @@
                     hardware_owner_is_affected_by_bug=False,
                     hardware_owner_is_subscribed_to_bug=False,
                     hardware_is_linked_to_bug=False, linked_branches=None,
-                    modified_since=None, created_since=None, prejoins=[]):
+                    linked_blueprints=None, modified_since=None,
+                    created_since=None, prejoins=[]):
         """See `IHasBugs`."""
         if status is None:
             # If no statuses are supplied, default to the

=== modified file 'lib/lp/bugs/model/bugtask.py'
--- lib/lp/bugs/model/bugtask.py	2011-01-20 04:50:35 +0000
+++ lib/lp/bugs/model/bugtask.py	2011-02-01 16:35:45 +0000
@@ -103,6 +103,7 @@
 from lp.bugs.interfaces.bugtask import (
     BUG_SUPERVISOR_BUGTASK_STATUSES,
     BugBranchSearch,
+    BugBlueprintSearch,
     BugTaskImportance,
     BugTaskSearchParams,
     BugTaskStatus,
@@ -2066,6 +2067,10 @@
             # we don't need to add any clause.
             pass
 
+        linked_blueprints_clause = self._buildBlueprintRelatedClause(params)
+        if linked_blueprints_clause is not None:
+            extra_clauses.append(linked_blueprints_clause)
+
         if params.modified_since:
             extra_clauses.append(
                 "Bug.date_last_updated > %s" % (
@@ -2318,6 +2323,20 @@
             ', '.join(tables), ' AND '.join(clauses))
         return clause
 
+    def _buildBlueprintRelatedClause(self, params):
+        """Find bugs related to Blueprints, or not."""
+        linked_blueprints = params.linked_blueprints
+        if linked_blueprints == BugBlueprintSearch.BUGS_WITH_BLUEPRINTS:
+            return "EXISTS (%s)" % (
+                "SELECT 1 FROM SpecificationBug"
+                " WHERE SpecificationBug.bug = Bug.id")
+        elif linked_blueprints == BugBlueprintSearch.BUGS_WITHOUT_BLUEPRINTS:
+            return "NOT EXISTS (%s)" % (
+                "SELECT 1 FROM SpecificationBug"
+                " WHERE SpecificationBug.bug = Bug.id")
+        else:
+            return None
+
     def buildOrigin(self, join_tables, prejoin_tables, clauseTables):
         """Build the parameter list for Store.using().
 

=== modified file 'lib/lp/bugs/tests/test_bugtask_search.py'
--- lib/lp/bugs/tests/test_bugtask_search.py	2011-01-13 17:00:41 +0000
+++ lib/lp/bugs/tests/test_bugtask_search.py	2011-02-01 16:35:45 +0000
@@ -31,6 +31,7 @@
 from lp.bugs.interfaces.bugattachment import BugAttachmentType
 from lp.bugs.interfaces.bugtask import (
     BugBranchSearch,
+    BugBlueprintSearch,
     BugTaskImportance,
     BugTaskSearchParams,
     BugTaskStatus,
@@ -445,6 +446,21 @@
             user=None, linked_branches=BugBranchSearch.BUGS_WITHOUT_BRANCHES)
         self.assertSearchFinds(params, self.bugtasks[1:])
 
+    def test_blueprints_linked(self):
+        # Search results can be limited to bugs with or without linked
+        # blueprints.
+        with person_logged_in(self.owner):
+            blueprint = self.factory.makeSpecification()
+            blueprint.linkBug(self.bugtasks[0].bug)
+        params = self.getBugTaskSearchParams(
+            user=None, linked_blueprints=(
+                BugBlueprintSearch.BUGS_WITH_BLUEPRINTS))
+        self.assertSearchFinds(params, self.bugtasks[:1])
+        params = self.getBugTaskSearchParams(
+            user=None, linked_blueprints=(
+                BugBlueprintSearch.BUGS_WITHOUT_BLUEPRINTS))
+        self.assertSearchFinds(params, self.bugtasks[1:])
+
     def test_limit_search_to_one_bug(self):
         # Search results can be limited to a given bug.
         params = self.getBugTaskSearchParams(

=== modified file 'lib/lp/bugs/tests/test_searchtasks_webservice.py'
--- lib/lp/bugs/tests/test_searchtasks_webservice.py	2010-10-04 19:50:45 +0000
+++ lib/lp/bugs/tests/test_searchtasks_webservice.py	2011-02-01 16:35:45 +0000
@@ -5,26 +5,29 @@
 
 __metaclass__ = type
 
-from canonical.launchpad.ftests import login
-from lp.testing import TestCaseWithFactory
 from canonical.launchpad.testing.pages import LaunchpadWebServiceCaller
 from canonical.testing.layers import DatabaseFunctionalLayer
+from lp.testing import (
+    person_logged_in,
+    TestCaseWithFactory,
+    )
 
 
 class TestOmitTargetedParameter(TestCaseWithFactory):
     """Test all values for the omit_targeted search parameter."""
+
     layer = DatabaseFunctionalLayer
 
     def setUp(self):
-        TestCaseWithFactory.setUp(self)
-        login('[e-mail]')
-        self.distro = self.factory.makeDistribution(name='mebuntu')
-        self.release = self.factory.makeDistroRelease(name='inkanyamba',
-            distribution=self.distro)
+        super(TestOmitTargetedParameter, self).setUp()
+        self.owner = self.factory.makePerson()
+        with person_logged_in(self.owner):
+            self.distro = self.factory.makeDistribution(name='mebuntu')
+        self.release = self.factory.makeDistroRelease(
+            name='inkanyamba', distribution=self.distro)
         self.bug = self.factory.makeBugTask(target=self.release)
-
-        self.webservice = LaunchpadWebServiceCaller('launchpad-library',
-            'salgado-change-anything')
+        self.webservice = LaunchpadWebServiceCaller(
+            'launchpad-library', 'salgado-change-anything')
 
     def test_omit_targeted_old_default_true(self):
         response = self.webservice.named_get('/mebuntu/inkanyamba',
@@ -35,3 +38,46 @@
         response = self.webservice.named_get('/mebuntu/inkanyamba',
             'searchTasks', api_version='devel').jsonBody()
         self.assertEqual(response['total_size'], 1)
+
+
+class TestLinkedBlueprintsParameter(TestCaseWithFactory):
+    """Tests for the linked_blueprints parameter."""
+
+    layer = DatabaseFunctionalLayer
+
+    def setUp(self):
+        super(TestLinkedBlueprintsParameter, self).setUp()
+        self.owner = self.factory.makePerson()
+        with person_logged_in(self.owner):
+            self.product = self.factory.makeProduct()
+        self.bug = self.factory.makeBugTask(target=self.product)
+        self.webservice = LaunchpadWebServiceCaller(
+            'launchpad-library', 'salgado-change-anything')
+
+    def search(self, api_version, **kwargs):
+        return self.webservice.named_get(
+            '/%s' % self.product.name, 'searchTasks',
+            api_version=api_version, **kwargs).jsonBody()
+
+    def test_linked_blueprints_in_devel(self):
+        # Searching for linked Blueprints works in the devel API.
+        self.search("devel", linked_blueprints="Show all bugs")
+
+    def test_linked_blueprints_in_devel_2(self):
+        # The linked_blueprints is considered. An error is returned if its
+        # value is not a member of BugBlueprintSearch.
+        self.assertRaises(
+            ValueError, self.search, "devel",
+            linked_blueprints="Teabags!")
+
+    def test_linked_blueprints_not_in_1_0(self):
+        # Searching for linked Blueprints does not work in the 1.0 API. No
+        # validation is performed for the linked_blueprints parameter, and
+        # thus no error is returned when we pass rubbish.
+        self.search("1.0", linked_blueprints="Teabags!")
+
+    def test_linked_blueprints_not_in_beta(self):
+        # Searching for linked Blueprints does not work in the beta API. No
+        # validation is performed for the linked_blueprints parameter, and
+        # thus no error is returned when we pass rubbish.
+        self.search("beta", linked_blueprints="Teabags!")