studio/people
Archived
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

✨(mailbox_manager) modify API to get maildomain

Browse Source 
Access to maildomain by slug name
This commit is contained in:
Sabrina Demagny
2024-06-03 16:59:55 +02:00
parent 23778fda0d
commit b4bafb6efb
12 changed files with 94 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/backend/mailbox_manager/tests/api/mail_domain/test_api_mail_domains_delete.py
View File

@@ -18,7 +18,7 @@ def test_api_mail_domains__delete_anonymous():
domain = factories.MailDomainFactory()
response = APIClient().delete(
f"/api/v1.0/mail-domains/{domain.id!s}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_401_UNAUTHORIZED
@@ -36,7 +36,7 @@ def test_api_mail_domains__delete_authenticated_unrelated():
client = APIClient()
client.force_login(identity.user)
response = client.delete(
f"/api/v1.0/mail-domains/{domain.id!s}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_404_NOT_FOUND
@@ -56,7 +56,7 @@ def test_api_mail_domains__delete_authenticated_member():
client = APIClient()
client.force_login(user)
response = client.delete(
f"/api/v1.0/mail-domains/{domain.id}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_403_FORBIDDEN
@@ -78,7 +78,7 @@ def test_api_mail_domains__delete_authenticated_administrator():
client = APIClient()
client.force_login(user)
response = client.delete(
f"/api/v1.0/mail-domains/{domain.id}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_403_FORBIDDEN
@@ -100,7 +100,7 @@ def test_api_mail_domains__delete_authenticated_owner():
client = APIClient()
client.force_login(user)
response = client.delete(
f"/api/v1.0/mail-domains/{domain.id}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_204_NO_CONTENT

6
src/backend/mailbox_manager/tests/api/mail_domain/test_api_mail_domains_retrieve.py
View File

@@ -17,7 +17,7 @@ def test_api_mail_domains__retrieve_anonymous():
"""Anonymous users should not be allowed to retrieve a domain."""
domain = factories.MailDomainFactory()
response = APIClient().get(f"/api/v1.0/mail-domains/{domain.id}/")
response = APIClient().get(f"/api/v1.0/mail-domains/{domain.slug}/")
assert response.status_code == status.HTTP_401_UNAUTHORIZED
assert response.json() == {
@@ -38,7 +38,7 @@ def test_api_mail_domains__retrieve_authenticated_unrelated():
domain = factories.MailDomainFactory()
response = client.get(
f"/api/v1.0/mail-domains/{domain.id!s}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_404_NOT_FOUND
assert response.json() == {"detail": "No MailDomain matches the given query."}
@@ -59,7 +59,7 @@ def test_api_mail_domains__retrieve_authenticated_related():
factories.MailDomainAccessFactory(domain=domain, user=user)
response = client.get(
f"/api/v1.0/mail-domains/{domain.id!s}/",
f"/api/v1.0/mail-domains/{domain.slug}/",
)
assert response.status_code == status.HTTP_200_OK

8
src/backend/mailbox_manager/tests/test_api_mailboxes_create.py
View File

@@ -18,7 +18,7 @@ def test_api_mailboxes__create_anonymous_forbidden():
mail_domain = factories.MailDomainFactory()
response = APIClient().post(
f"/api/v1.0/mail-domains/{mail_domain.id}/mailboxes/",
f"/api/v1.0/mail-domains/{mail_domain.slug}/mailboxes/",
{
"first_name": "jean",
"last_name": "doe",
@@ -44,7 +44,7 @@ def test_api_mailboxes__create_authenticated_missing_fields():
mail_domain = factories.MailDomainFactory()
response = client.post(
f"/api/v1.0/mail-domains/{mail_domain.id}/mailboxes/",
f"/api/v1.0/mail-domains/{mail_domain.slug}/mailboxes/",
{
"first_name": "jean",
"last_name": "doe",
@@ -58,7 +58,7 @@ def test_api_mailboxes__create_authenticated_missing_fields():
assert response.json() == {"local_part": ["This field is required."]}
response = client.post(
f"/api/v1.0/mail-domains/{mail_domain.id}/mailboxes/",
f"/api/v1.0/mail-domains/{mail_domain.slug}/mailboxes/",
{
"first_name": "jean",
"last_name": "doe",
@@ -82,7 +82,7 @@ def test_api_mailboxes__create_authenticated_successful():
mail_domain = factories.MailDomainFactory(name="saint-jean.collectivite.fr")
response = client.post(
f"/api/v1.0/mail-domains/{mail_domain.id}/mailboxes/",
f"/api/v1.0/mail-domains/{mail_domain.slug}/mailboxes/",
{
"first_name": "jean",
"last_name": "doe",

4
src/backend/mailbox_manager/tests/test_api_mailboxes_list.py
View File

@@ -18,7 +18,7 @@ def test_api_mailboxes__list_anonymous():
mail_domain = factories.MailDomainFactory()
factories.MailboxFactory.create_batch(2, domain=mail_domain)
response = APIClient().get(f"/api/v1.0/mail-domains/{mail_domain.id}/mailboxes/")
response = APIClient().get(f"/api/v1.0/mail-domains/{mail_domain.slug}/mailboxes/")
assert response.status_code == status.HTTP_401_UNAUTHORIZED
assert response.json() == {
"detail": "Authentication credentials were not provided."
@@ -37,7 +37,7 @@ def test_api_mailboxes__list_authenticated_no_query():
mailbox1 = factories.MailboxFactory(domain=mail_domain)
mailbox2 = factories.MailboxFactory(domain=mail_domain)
response = client.get(f"/api/v1.0/mail-domains/{mail_domain.id}/mailboxes/")
response = client.get(f"/api/v1.0/mail-domains/{mail_domain.slug}/mailboxes/")
assert response.status_code == status.HTTP_200_OK
assert response.json()["results"] == [
{