2022-06-03 08:09:47 -06:00
|
|
|
import pytest
|
|
|
|
|
from django.urls import reverse
|
|
|
|
|
from django.utils import timezone
|
|
|
|
|
from rest_framework import status
|
|
|
|
|
from rest_framework.test import APIClient
|
|
|
|
|
|
|
|
|
|
from apps.base.models import UserNotificationPolicy
|
|
|
|
|
from apps.base.models.user_notification_policy import NotificationChannelPublicAPIOptions
|
|
|
|
|
|
|
|
|
|
TYPE_WAIT = "wait"
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture()
|
|
|
|
|
def personal_notification_rule_public_api_setup(
|
|
|
|
|
make_organization_and_user_with_token,
|
|
|
|
|
make_user_notification_policy,
|
|
|
|
|
):
|
|
|
|
|
organization, user, token = make_organization_and_user_with_token()
|
|
|
|
|
notification_rule_wait = make_user_notification_policy(
|
|
|
|
|
user, wait_delay=UserNotificationPolicy.FIVE_MINUTES, step=UserNotificationPolicy.Step.WAIT
|
|
|
|
|
)
|
|
|
|
|
notification_rule_phone_call = make_user_notification_policy(
|
|
|
|
|
user, notify_by=UserNotificationPolicy.NotificationChannel.PHONE_CALL, step=UserNotificationPolicy.Step.NOTIFY
|
|
|
|
|
)
|
|
|
|
|
notification_rule_important = make_user_notification_policy(
|
|
|
|
|
user,
|
|
|
|
|
notify_by=UserNotificationPolicy.NotificationChannel.PHONE_CALL,
|
|
|
|
|
step=UserNotificationPolicy.Step.NOTIFY,
|
|
|
|
|
important=True,
|
|
|
|
|
)
|
|
|
|
|
return organization, user, token, notification_rule_wait, notification_rule_phone_call, notification_rule_important
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_get_personal_notification_rule(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, notification_rule_phone_call, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse(
|
|
|
|
|
"api-public:personal_notification_rules-detail", kwargs={"pk": notification_rule_phone_call.public_primary_key}
|
|
|
|
|
)
|
|
|
|
|
response = client.get(url, format="json", HTTP_AUTHORIZATION=token)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"id": notification_rule_phone_call.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[notification_rule_phone_call.notify_by],
|
|
|
|
|
"position": notification_rule_phone_call.order,
|
|
|
|
|
"important": False,
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_200_OK
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_get_personal_notification_rules_list(personal_notification_rule_public_api_setup):
|
|
|
|
|
(
|
|
|
|
|
_,
|
|
|
|
|
user,
|
|
|
|
|
token,
|
|
|
|
|
notification_rule_wait,
|
|
|
|
|
notification_rule_phone_call,
|
|
|
|
|
notification_rule_important,
|
|
|
|
|
) = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
response = client.get(url, format="json", HTTP_AUTHORIZATION=token)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"count": 3,
|
|
|
|
|
"next": None,
|
|
|
|
|
"previous": None,
|
|
|
|
|
"results": [
|
|
|
|
|
{
|
|
|
|
|
"id": notification_rule_wait.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": TYPE_WAIT,
|
|
|
|
|
"duration": timezone.timedelta(seconds=300).seconds,
|
|
|
|
|
"position": notification_rule_wait.order,
|
|
|
|
|
"important": False,
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"id": notification_rule_phone_call.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[notification_rule_phone_call.notify_by],
|
|
|
|
|
"position": notification_rule_phone_call.order,
|
|
|
|
|
"important": False,
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"id": notification_rule_important.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[notification_rule_important.notify_by],
|
|
|
|
|
"position": notification_rule_important.order,
|
|
|
|
|
"important": True,
|
|
|
|
|
},
|
|
|
|
|
],
|
augment API response pagination attributes (#2471)
# What this PR does
This PR:
- adds a few attributes to paginated API responses
- removes channel filter "send demo alert" internal API endpoint + tests
(this endpoint was marked as deprecated + not consumed by the web UI)
With the new paginated API response schema, the web UI will no longer
need to:
- hardcode `ITEMS_PER_PAGE` for each table
- manually calculate total number of pages
(these two things ☝️ will be done in
https://github.com/grafana/oncall/issues/2476)
For `GET /api/internal/v1/alertgroups` the response will now look like
this:
```diff
{
"next": <url> | None,
"previous": <url> | None,
"results": [],
++ "page_size": <int>
}
```
For all other paginated API responses, the response will now look like:
```diff
{
"count": <int>,
"next": <url> | None,
"previous": <url> | None,
"results": [],
++ "page_size": <int>,
++ "current_page_number": <int>,
++ "total_pages": <int>
}
```
## TODO
- [x] update public API docs to include these new attributes
## Checklist
- [x] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
2023-07-14 17:19:40 +02:00
|
|
|
"current_page_number": 1,
|
|
|
|
|
"page_size": 50,
|
|
|
|
|
"total_pages": 1,
|
2022-06-03 08:09:47 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_200_OK
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_get_personal_notification_rules_list_important(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, _, notification_rule_important = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
response = client.get(url + "?important=true", format="json", HTTP_AUTHORIZATION=token)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"count": 1,
|
|
|
|
|
"next": None,
|
|
|
|
|
"previous": None,
|
|
|
|
|
"results": [
|
|
|
|
|
{
|
|
|
|
|
"id": notification_rule_important.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[notification_rule_important.notify_by],
|
|
|
|
|
"position": notification_rule_important.order,
|
|
|
|
|
"important": True,
|
|
|
|
|
}
|
|
|
|
|
],
|
augment API response pagination attributes (#2471)
# What this PR does
This PR:
- adds a few attributes to paginated API responses
- removes channel filter "send demo alert" internal API endpoint + tests
(this endpoint was marked as deprecated + not consumed by the web UI)
With the new paginated API response schema, the web UI will no longer
need to:
- hardcode `ITEMS_PER_PAGE` for each table
- manually calculate total number of pages
(these two things ☝️ will be done in
https://github.com/grafana/oncall/issues/2476)
For `GET /api/internal/v1/alertgroups` the response will now look like
this:
```diff
{
"next": <url> | None,
"previous": <url> | None,
"results": [],
++ "page_size": <int>
}
```
For all other paginated API responses, the response will now look like:
```diff
{
"count": <int>,
"next": <url> | None,
"previous": <url> | None,
"results": [],
++ "page_size": <int>,
++ "current_page_number": <int>,
++ "total_pages": <int>
}
```
## TODO
- [x] update public API docs to include these new attributes
## Checklist
- [x] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
2023-07-14 17:19:40 +02:00
|
|
|
"current_page_number": 1,
|
|
|
|
|
"page_size": 50,
|
|
|
|
|
"total_pages": 1,
|
2022-06-03 08:09:47 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_200_OK
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_get_personal_notification_rules_list_non_important(personal_notification_rule_public_api_setup):
|
|
|
|
|
(
|
|
|
|
|
_,
|
|
|
|
|
user,
|
|
|
|
|
token,
|
|
|
|
|
notification_rule_wait,
|
|
|
|
|
notification_rule_phone_call,
|
|
|
|
|
_,
|
|
|
|
|
) = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
response = client.get(url + "?important=false", format="json", HTTP_AUTHORIZATION=token)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"count": 2,
|
|
|
|
|
"next": None,
|
|
|
|
|
"previous": None,
|
|
|
|
|
"results": [
|
|
|
|
|
{
|
|
|
|
|
"id": notification_rule_wait.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": TYPE_WAIT,
|
|
|
|
|
"duration": timezone.timedelta(seconds=300).seconds,
|
|
|
|
|
"position": notification_rule_wait.order,
|
|
|
|
|
"important": False,
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"id": notification_rule_phone_call.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[notification_rule_phone_call.notify_by],
|
|
|
|
|
"position": notification_rule_phone_call.order,
|
|
|
|
|
"important": False,
|
|
|
|
|
},
|
|
|
|
|
],
|
augment API response pagination attributes (#2471)
# What this PR does
This PR:
- adds a few attributes to paginated API responses
- removes channel filter "send demo alert" internal API endpoint + tests
(this endpoint was marked as deprecated + not consumed by the web UI)
With the new paginated API response schema, the web UI will no longer
need to:
- hardcode `ITEMS_PER_PAGE` for each table
- manually calculate total number of pages
(these two things ☝️ will be done in
https://github.com/grafana/oncall/issues/2476)
For `GET /api/internal/v1/alertgroups` the response will now look like
this:
```diff
{
"next": <url> | None,
"previous": <url> | None,
"results": [],
++ "page_size": <int>
}
```
For all other paginated API responses, the response will now look like:
```diff
{
"count": <int>,
"next": <url> | None,
"previous": <url> | None,
"results": [],
++ "page_size": <int>,
++ "current_page_number": <int>,
++ "total_pages": <int>
}
```
## TODO
- [x] update public API docs to include these new attributes
## Checklist
- [x] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
2023-07-14 17:19:40 +02:00
|
|
|
"current_page_number": 1,
|
|
|
|
|
"page_size": 50,
|
|
|
|
|
"total_pages": 1,
|
2022-06-03 08:09:47 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_200_OK
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_update_personal_notification_rule(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, notification_rule_phone_call, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse(
|
|
|
|
|
"api-public:personal_notification_rules-detail", kwargs={"pk": notification_rule_phone_call.public_primary_key}
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
data_to_update = {
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[UserNotificationPolicy.NotificationChannel.SMS]
|
|
|
|
|
}
|
|
|
|
|
assert notification_rule_phone_call.notify_by != UserNotificationPolicy.NotificationChannel.SMS
|
|
|
|
|
response = client.put(url, format="json", HTTP_AUTHORIZATION=token, data=data_to_update)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"id": notification_rule_phone_call.public_primary_key,
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": data_to_update["type"],
|
|
|
|
|
"position": notification_rule_phone_call.order,
|
|
|
|
|
"important": False,
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_200_OK
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_create_personal_notification_rule_wait(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, _, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
data_for_create = {
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": TYPE_WAIT,
|
|
|
|
|
"position": 1,
|
|
|
|
|
"duration": timezone.timedelta(seconds=300).seconds,
|
|
|
|
|
}
|
|
|
|
|
response = client.post(url, format="json", HTTP_AUTHORIZATION=token, data=data_for_create)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"id": response.data["id"],
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": TYPE_WAIT,
|
|
|
|
|
"duration": data_for_create["duration"],
|
|
|
|
|
"position": data_for_create["position"],
|
|
|
|
|
"important": False,
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_201_CREATED
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
notification_rule = UserNotificationPolicy.objects.get(public_primary_key=response.data["id"])
|
|
|
|
|
assert notification_rule.step == UserNotificationPolicy.Step.WAIT
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_create_personal_notification_rule_notify_by_sms(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, _, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
data_for_create = {
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[UserNotificationPolicy.NotificationChannel.SMS],
|
|
|
|
|
"position": 1,
|
|
|
|
|
}
|
|
|
|
|
response = client.post(url, format="json", HTTP_AUTHORIZATION=token, data=data_for_create)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"id": response.data["id"],
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[UserNotificationPolicy.NotificationChannel.SMS],
|
|
|
|
|
"position": data_for_create["position"],
|
|
|
|
|
"important": False,
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_201_CREATED
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
notification_rule = UserNotificationPolicy.objects.get(public_primary_key=response.data["id"])
|
|
|
|
|
assert notification_rule.step == UserNotificationPolicy.Step.NOTIFY
|
|
|
|
|
assert notification_rule.notify_by == UserNotificationPolicy.NotificationChannel.SMS
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_create_personal_notification_rule_notify_by_sms_important(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, _, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
data_for_create = {
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[UserNotificationPolicy.NotificationChannel.SMS],
|
|
|
|
|
"position": 1,
|
|
|
|
|
"important": True,
|
|
|
|
|
}
|
|
|
|
|
response = client.post(url, format="json", HTTP_AUTHORIZATION=token, data=data_for_create)
|
|
|
|
|
|
|
|
|
|
expected_response = {
|
|
|
|
|
"id": response.data["id"],
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": NotificationChannelPublicAPIOptions.LABELS[UserNotificationPolicy.NotificationChannel.SMS],
|
|
|
|
|
"position": data_for_create["position"],
|
|
|
|
|
"important": data_for_create["important"],
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_201_CREATED
|
|
|
|
|
assert response.json() == expected_response
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_create_personal_notification_rule_invalid_data(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, _, _, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse("api-public:personal_notification_rules-list")
|
|
|
|
|
data_for_create = {
|
|
|
|
|
"user_id": user.public_primary_key,
|
|
|
|
|
"type": "invalid_type",
|
|
|
|
|
"position": 1,
|
|
|
|
|
}
|
|
|
|
|
response = client.post(url, format="json", HTTP_AUTHORIZATION=token, data=data_for_create)
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_400_BAD_REQUEST
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.django_db
|
|
|
|
|
def test_delete_personal_notification_rule(personal_notification_rule_public_api_setup):
|
|
|
|
|
_, user, token, notification_rule_wait, _, _ = personal_notification_rule_public_api_setup
|
|
|
|
|
|
|
|
|
|
client = APIClient()
|
|
|
|
|
|
|
|
|
|
url = reverse(
|
|
|
|
|
"api-public:personal_notification_rules-detail", kwargs={"pk": notification_rule_wait.public_primary_key}
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
response = client.delete(url, format="json", HTTP_AUTHORIZATION=token)
|
|
|
|
|
|
|
|
|
|
assert response.status_code == status.HTTP_204_NO_CONTENT
|