update manage_tempaltes

This commit is contained in:
Beverly Nguyen
2025-07-22 13:02:12 -07:00
parent afaae0a695
commit 1f3c3aba32
6 changed files with 14 additions and 14 deletions

View File

@@ -77,11 +77,11 @@ def user_json(
if permissions is None:
permissions = {
str(generate_uuid()): [
"view_activity",
"send_texts",
"send_emails",
"manage_users",
"manage_templates",
ServicePermission.VIEW_ACTIVITY,
ServicePermission.SEND_TEXTS,
ServicePermission.SEND_EMAILS,
ServicePermission.MANAGE_USERS,
ServicePermission.MANAGE_TEMPLATES,
ServicePermission.MANAGE_SETTINGS,
"manage_api_keys",
]

View File

@@ -5,7 +5,7 @@ from flask import url_for
from freezegun import freeze_time
import app
from app.enums import ServicePermission
from app.enums import AuthType, ServicePermission
from notifications_python_client.errors import HTTPError
from tests import service_json
from tests.conftest import (

View File

@@ -1179,7 +1179,7 @@ def test_get_users_report(client_request, platform_admin_user, mocker):
"permissions": {
"test service": [
ServicePermission.MANAGE_USERS,
"manage_templates",
ServicePermission.MANAGE_TEMPLATES,
ServicePermission.MANAGE_SETTINGS,
ServicePermission.SEND_TEXTS,
ServicePermission.SEND_EMAILS,

View File

@@ -608,7 +608,7 @@ def test_get_manage_folder_viewing_permissions_for_users_not_visible_when_no_man
active_user_with_permissions["permissions"][SERVICE_ONE_ID] = [
ServicePermission.SEND_TEXTS,
ServicePermission.SEND_EMAILS,
"manage_templates",
ServicePermission.MANAGE_TEMPLATES,
"manage_api_keys",
ServicePermission.VIEW_ACTIVITY,
]
@@ -862,7 +862,7 @@ def test_manage_folder_users_doesnt_change_permissions_current_user_cannot_manag
active_user_with_permissions["permissions"][SERVICE_ONE_ID] = [
ServicePermission.SEND_TEXTS,
ServicePermission.SEND_EMAILS,
"manage_templates",
ServicePermission.MANAGE_TEMPLATES,
"manage_api_keys",
ServicePermission.VIEW_ACTIVITY,
]

View File

@@ -36,7 +36,7 @@ def test_permissions(
request.view_args.update({"service_id": "foo"})
api_user_active["permissions"] = {
"foo": [ServicePermission.MANAGE_USERS, "manage_templates", ServicePermission.MANAGE_SETTINGS]
"foo": [ServicePermission.MANAGE_USERS, ServicePermission.MANAGE_TEMPLATES, ServicePermission.MANAGE_SETTINGS]
}
api_user_active["services"] = ["foo", "bar"]
@@ -66,7 +66,7 @@ def test_permissions_forbidden(
request.view_args.update({"service_id": "foo"})
api_user_active["permissions"] = {
"foo": [ServicePermission.MANAGE_USERS, "manage_templates", ServicePermission.MANAGE_SETTINGS]
"foo": [ServicePermission.MANAGE_USERS, ServicePermission.MANAGE_TEMPLATES, ServicePermission.MANAGE_SETTINGS]
}
api_user_active["services"] = ["foo", "bar"]
@@ -179,7 +179,7 @@ def test_user_with_no_permissions_to_service_goes_to_templates(
api_user_active,
):
api_user_active["permissions"] = {
"foo": [ServicePermission.MANAGE_USERS, "manage_templates", ServicePermission.MANAGE_SETTINGS]
"foo": [ServicePermission.MANAGE_USERS, ServicePermission.MANAGE_TEMPLATES, ServicePermission.MANAGE_SETTINGS]
}
api_user_active["services"] = ["foo", "bar"]
client_request.login(api_user_active)

View File

@@ -11,14 +11,14 @@ from app.utils.user_permissions import (
("db_permissions", "expected_ui_permissions"),
[
(
["manage_templates"],
[ServicePermission.MANAGE_TEMPLATES],
{ServicePermission.MANAGE_TEMPLATES},
),
(
[
ServicePermission.SEND_TEXTS,
ServicePermission.SEND_EMAILS,
"manage_templates",
ServicePermission.MANAGE_TEMPLATES,
"some_unknown_permission",
],
{