fix(hubspot): create hubspot company with domain (#3844)

api/integrations/lead_tracking/hubspot/lead_tracker.py

@@ -119,7 +119,10 @@ def update_company_active_subscription(
     def _get_or_create_company_by_domain(self, domain: str) -> dict:
         company = self.client.get_company_by_domain(domain)
         if not company:
-            company = self.client.create_company(name=domain)
+            # Since we don't know the company's name, we pass the domain as
+            # both the name and the domain. This can then be manually
+            # updated in Hubspot if needed.
+            company = self.client.create_company(name=domain, domain=domain)
 
         return company
 

api/tests/unit/integrations/lead_tracking/hubspot/test_unit_hubspot_tasks.py

@@ -36,10 +36,12 @@ def test_track_hubspot_lead_without_organisation(
     # Given
     hubspot_company_id = "company-id"
     hubspot_contact_id = "contact-id"
+    domain = "example.com"
+    email = f"test@{domain}"
 
     settings.ENABLE_HUBSPOT_LEAD_TRACKING = True
 
-    user = FFAdminUser.objects.create(email="[email protected]")
+    user = FFAdminUser.objects.create(email=email)
 
     mock_hubspot_client = mocker.MagicMock(spec=HubspotClient)
     mocker.patch(
@@ -56,5 +58,8 @@ def test_track_hubspot_lead_without_organisation(
     track_hubspot_lead_without_organisation(user_id=user.id)
 
     # Then
+    mock_hubspot_client.create_company.assert_called_once_with(
+        name=domain, domain=domain
+    )
     mock_hubspot_client.create_contact.assert_called_once_with(user, hubspot_company_id)
     assert HubspotLead.objects.filter(user=user).exists()