change data to recipients

Author: allanstone

facturapi/resources/invoices.py

@@ -194,7 +194,9 @@ def cancel(cls, invoice_id: str, motive: str) -> 'Invoice':
 
     @classmethod
     def send_by_email(
-        cls, invoice_id: str, data: Optional[Union[str, List[str]]] = None
+        cls,
+        invoice_id: str,
+        recipients: Optional[Union[str, List[str]]] = None,
     ) -> bool:
         """Send an invoice by email.
 
@@ -203,7 +205,7 @@ def send_by_email(
 
         Args:
             invoice_id: The ID of the invoice to send.
-            data: The email addresses to send the invoice to.
+            recipients: The email addresses to send the invoice to.
                    If not provided, the invoice will be sent to the customer's
                    registered email.
 
@@ -219,8 +221,8 @@ def send_by_email(
 
         endpoint = f"{cls._resource}/{invoice_id}/email"
         payload = {}
-        if isinstance(data, (str, list)):
-            payload["email"] = data
+        if isinstance(recipients, (str, list)):
+            payload["email"] = recipients
         response = client.post(endpoint, payload)
         return response.get("ok", False)
 

tests/resources/test_invoices.py

@@ -116,8 +116,10 @@ def test_send_invoice_by_email_without_email():
 def test_send_invoice_by_email_with_email():
     invoice_id = "67e59a55f4f823d3d978f3cb"
     email = "frida_kahlo@test.com"
-    data = [email]
-    resp = facturapi.Invoice.send_by_email(invoice_id=invoice_id, data=data)
+    recipients = [email]
+    resp = facturapi.Invoice.send_by_email(
+        invoice_id=invoice_id, recipients=recipients
+    )
 
     assert resp is True