fix to notifications entry layout

Author: jreiberkyle

planet/order_request.py

@@ -143,7 +143,21 @@ def notifications(email: Optional[bool] = None,
         webhook_per_order: Request a single webhook call per order instead
             of one call per each delivered item.
     '''
-    return dict((k, v) for k, v in locals().items() if v)
+    notifications_dict = {}
+
+    if webhook_url:
+        webhook_dict = {
+            'url': webhook_url
+            }
+        if webhook_per_order is not None:
+            webhook_dict['per_order'] = webhook_per_order
+
+        notifications_dict['webhook'] = webhook_dict
+
+    if email is not None:
+        notifications_dict['email'] = email
+
+    return notifications_dict
 
 
 def delivery(archive_type: Optional[str] = None,

tests/unit/test_order_request.py

@@ -46,9 +46,9 @@ def test_build_request():
         }
     }
     notifications = {
-        'email': 'email',
-        'webhook_url': 'webhookurl',
-        'webhook_per_order': True
+        'email': 'email', 'webhook': {
+            'url': 'webhookurl', 'per_order': True
+        }
     }
     order_type = 'partial'
     tool = {'bandmath': 'jsonstring'}
@@ -121,9 +121,9 @@ def test_notifications():
     notifications_config = order_request.notifications(
         email='email', webhook_url='webhookurl', webhook_per_order=True)
     expected = {
-        'email': 'email',
-        'webhook_url': 'webhookurl',
-        'webhook_per_order': True
+        'email': 'email', 'webhook': {
+            'url': 'webhookurl', 'per_order': True
+        }
     }
     assert notifications_config == expected