Skip to content

Commit

Permalink
Test that the hubspot cookie collision doesn't raise an error
Browse files Browse the repository at this point in the history
  • Loading branch information
zachaysan committed Dec 9, 2024
1 parent 987fabc commit ced5902
Showing 1 changed file with 34 additions and 0 deletions.
34 changes: 34 additions & 0 deletions api/tests/unit/organisations/test_unit_organisations_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,40 @@ def test_non_superuser_can_create_new_organisation_by_default(
assert HubspotTracker.objects.filter(user=staff_user).exists()


def test_colliding_hubspot_cookies_are_ignored(
staff_client: APIClient,
staff_user: FFAdminUser,
admin_user: FFAdminUser,
) -> None:
# Given
org_name = "Test create org"
webhook_notification_email = "test@email.com"
url = reverse("api-v1:organisations:organisation-list")
colliding_cookie = "test_cookie_tracker"
HubspotTracker.objects.create(
user=admin_user,
hubspot_cookie=colliding_cookie,
)
data = {
"name": org_name,
"webhook_notification_email": webhook_notification_email,
HUBSPOT_COOKIE_NAME: colliding_cookie,
}

assert not HubspotTracker.objects.filter(user=staff_user).exists()

# When
response = staff_client.post(url, data=data)

# Then
assert response.status_code == status.HTTP_201_CREATED
assert (
Organisation.objects.get(name=org_name).webhook_notification_email
== webhook_notification_email
)
assert not HubspotTracker.objects.filter(user=staff_user).exists()


@override_settings(RESTRICT_ORG_CREATE_TO_SUPERUSERS=True)
def test_create_new_orgnisation_returns_403_with_non_superuser(
staff_client: APIClient,
Expand Down

0 comments on commit ced5902

Please sign in to comment.