Merge pull request #909 from planetlabs/search-update-fix-908

Match search_update interface to update-search cli interface

Author: jreiberkyle

planet/clients/data.py

@@ -219,17 +219,17 @@ async def create_search(self,
 
     async def update_search(self,
                             search_id: str,
-                            name: str,
                             item_types: List[str],
                             search_filter: dict,
+                            name: str,
                             enable_email: bool = False) -> dict:
         """Update an existing saved search.
 
         Parameters:
             search_id: Saved search identifier.
-            name: The name of the saved search.
             item_types: The item types to include in the search.
             search_filter: Structured search criteria.
+            name: The name of the saved search.
             enable_email: Send a daily email when new results are added.
 
         Returns:

tests/integration/test_data_api.py

@@ -352,8 +352,9 @@ async def test_update_search_basic(search_filter, session):
 
     cl = DataClient(session, base_url=TEST_URL)
     search = await cl.update_search(VALID_SEARCH_ID,
-                                    'test', ['PSScene'],
-                                    search_filter)
+                                    item_types=['PSScene'],
+                                    search_filter=search_filter,
+                                    name='test')
 
     # check that request is correct
     expected_request = {