Fix impproper setting of header_to value

header_to was being set to an object when the recipient was not
a string. Fixes #118.

Author: Rich Leland

sparkpost/transmissions.py

@@ -61,6 +61,7 @@ def _translate_keys(self, **kwargs):
             model['recipients']['list_id'] = recipient_list
         else:
             recipients = kwargs.get('recipients', [])
+            recipients = self._extract_recipients(recipients)
             cc = kwargs.get('cc')
             bcc = kwargs.get('bcc')
 
@@ -72,7 +73,7 @@ def _translate_keys(self, **kwargs):
                 bcc_copies = self._format_copies(recipients, bcc)
                 recipients = recipients + bcc_copies
 
-            model['recipients'] = self._extract_recipients(recipients)
+            model['recipients'] = recipients
 
         attachments = kwargs.get('attachments', [])
         model['content']['attachments'] = self._extract_attachments(
@@ -90,9 +91,19 @@ def _format_copies(self, recipients, copies):
         if len(recipients) > 0:
             formatted_copies = self._extract_recipients(copies)
             for recipient in formatted_copies:
-                recipient['address'].update({'header_to': recipients[0]})
+                recipient['address'].update({
+                    'header_to': self._format_header_to(recipients[0])
+                })
         return formatted_copies
 
+    def _format_header_to(self, recipient):
+        if 'name' in recipient['address']:
+            return '"{name}" <{email}>'.format(
+                name=recipient['address']['name'],
+                email=recipient['address']['email']
+            )
+        return recipient['address']['email']
+
     def _extract_attachments(self, attachments):
         formatted_attachments = []
         for attachment in attachments:

test/test_transmissions.py

@@ -58,6 +58,42 @@ def test_translate_keys_for_from_email():
     }
 
 
+def test_format_header_to():
+    t = Transmissions('uri', 'key')
+    formatted = t._format_header_to(recipient={
+        'address': {'email': 'primary@example.com'}
+    })
+    assert formatted == 'primary@example.com'
+
+    formatted = t._format_header_to(recipient={
+        'address': {'name': 'Testing', 'email': 'primary@example.com'}
+    })
+    assert formatted == '"Testing" <primary@example.com>'
+
+
+def test_cc_with_sub_data():
+    t = Transmissions('uri', 'key')
+    results = t._translate_keys(
+        recipients=[{
+            'address': {'email': 'primary@example.com'},
+            'substitution_data': {'fake': 'data'}
+        }],
+        cc=['ccone@example.com']
+    )
+    assert results['recipients'] == [
+        {
+            'address': {'email': 'primary@example.com'},
+            'substitution_data': {'fake': 'data'}
+        },
+        {
+            'address': {
+                'email': 'ccone@example.com',
+                'header_to': 'primary@example.com'
+            }
+        }
+    ]
+
+
 def test_translate_keys_with_cc():
     t = Transmissions('uri', 'key')
     results = t._translate_keys(recipients=['primary@example.com'],