From e6f82d1ed94cd1cf4282972671819b0df934efa2 Mon Sep 17 00:00:00 2001 From: eric-intuitem <71850047+eric-intuitem@users.noreply.github.com> Date: Thu, 18 Apr 2024 22:09:52 +0200 Subject: [PATCH] finish replacing --- backend/app_tests/test_vars.py | 2 +- backend/core/apps.py | 14 +++++++------- backend/core/tests/test_helpers.py | 8 ++++---- backend/core/views.py | 4 ++-- frontend/messages/en.json | 2 +- frontend/messages/fr.json | 2 +- frontend/src/lib/utils/locales.ts | 2 +- frontend/tests/utils/test-data.ts | 2 +- frontend/tests/utils/test-utils.ts | 2 +- 9 files changed, 19 insertions(+), 19 deletions(-) diff --git a/backend/app_tests/test_vars.py b/backend/app_tests/test_vars.py index bdda0ef8d..fe9384a92 100644 --- a/backend/app_tests/test_vars.py +++ b/backend/app_tests/test_vars.py @@ -44,7 +44,7 @@ }, "BI-UG-GAD": { "folder": "Global", - "name": "Global_auditor", + "name": "Global_reader", "perms": READER_PERMISSIONS_LIST, }, "BI-UG-GAP": { diff --git a/backend/core/apps.py b/backend/core/apps.py index 3541f25af..9d49751c9 100644 --- a/backend/core/apps.py +++ b/backend/core/apps.py @@ -251,7 +251,7 @@ def startup(**kwargs): print("startup handler: initialize database") - auditor_permissions = Permission.objects.filter( + reader_permissions = Permission.objects.filter( codename__in=READER_PERMISSIONS_LIST ) @@ -277,8 +277,8 @@ def startup(**kwargs): name="Global", content_type=Folder.ContentType.ROOT, builtin=True ) # update builtin roles to facilitate migrations - auditor, created = Role.objects.get_or_create(name="BI-RL-AUD", builtin=True) - auditor.permissions.set(auditor_permissions) + reader, created = Role.objects.get_or_create(name="BI-RL-AUD", builtin=True) + reader.permissions.set(reader_permissions) approver, created = Role.objects.get_or_create(name="BI-RL-APP", builtin=True) approver.permissions.set(approver_permissions) analyst, created = Role.objects.get_or_create(name="BI-RL-ANA", builtin=True) @@ -302,23 +302,23 @@ def startup(**kwargs): folder=Folder.get_root_folder(), ) ra1.perimeter_folders.add(administrators.folder) - # if global auditors user group does not exist, then create it + # if global readers user group does not exist, then create it if not UserGroup.objects.filter( name="BI-UG-GAD", folder=Folder.get_root_folder() ).exists(): - global_auditors = UserGroup.objects.create( + global_readers = UserGroup.objects.create( name="BI-UG-GAD", folder=Folder.objects.get(content_type=Folder.ContentType.ROOT), builtin=True, ) ra2 = RoleAssignment.objects.create( - user_group=global_auditors, + user_group=global_readers, role=Role.objects.get(name="BI-RL-AUD"), is_recursive=True, builtin=True, folder=Folder.get_root_folder(), ) - ra2.perimeter_folders.add(global_auditors.folder) + ra2.perimeter_folders.add(global_readers.folder) # if global approvers user group does not exist, then create it if not UserGroup.objects.filter( name="BI-UG-GAP", folder=Folder.get_root_folder() diff --git a/backend/core/tests/test_helpers.py b/backend/core/tests/test_helpers.py index 8068e4db2..ef5abe66b 100644 --- a/backend/core/tests/test_helpers.py +++ b/backend/core/tests/test_helpers.py @@ -44,7 +44,7 @@ def test_get_rating_options_perm_to_view_matrix(): ) RiskScenario.objects.create(name="test", risk_assessment=risk_assessment) role = Role.objects.create(name="test") - auditor_permissions = Permission.objects.filter( + reader_permissions = Permission.objects.filter( codename__in=[ "view_project", "view_riskassessment", @@ -58,7 +58,7 @@ def test_get_rating_options_perm_to_view_matrix(): "view_usergroup", ] ) - role.permissions.set(auditor_permissions) + role.permissions.set(reader_permissions) role.save() role_assignment = RoleAssignment.objects.create( user=user, @@ -109,7 +109,7 @@ def test_get_rating_options_abbr_perm_to_view_matrix(): ) RiskScenario.objects.create(name="test", risk_assessment=risk_assessment) role = Role.objects.create(name="test") - auditor_permissions = Permission.objects.filter( + reader_permissions = Permission.objects.filter( codename__in=[ "view_project", "view_riskassessment", @@ -123,7 +123,7 @@ def test_get_rating_options_abbr_perm_to_view_matrix(): "view_usergroup", ] ) - role.permissions.set(auditor_permissions) + role.permissions.set(reader_permissions) role.save() role_assignment = RoleAssignment.objects.create( user=user, diff --git a/backend/core/views.py b/backend/core/views.py index ad31e0bb2..da1613d8b 100644 --- a/backend/core/views.py +++ b/backend/core/views.py @@ -903,7 +903,7 @@ def perform_create(self, serializer): serializer.save() folder = Folder.objects.get(id=serializer.data["id"]) if folder.content_type == Folder.ContentType.DOMAIN: - auditors = UserGroup.objects.create( + readers = UserGroup.objects.create( name=UserGroupCodename.READER, folder=folder, builtin=True ) approvers = UserGroup.objects.create( @@ -916,7 +916,7 @@ def perform_create(self, serializer): name=UserGroupCodename.DOMAIN_MANAGER, folder=folder, builtin=True ) ra1 = RoleAssignment.objects.create( - user_group=auditors, + user_group=readers, role=Role.objects.get(name=RoleCodename.READER), builtin=True, folder=Folder.get_root_folder(), diff --git a/frontend/messages/en.json b/frontend/messages/en.json index 3b6896058..640a9ef50 100644 --- a/frontend/messages/en.json +++ b/frontend/messages/en.json @@ -367,7 +367,7 @@ "extraLarge": "Extra-large", "rid": "RID", "scope": "Scope", - "auditor": "Reader", + "reader": "Reader", "lastUpdate": "Last update", "riskScenarioAssetHelpText": "Assets impacted by this risk scenario", "riskScenarioMeasureHelpText": "The existing applied controls to manage this risk", diff --git a/frontend/messages/fr.json b/frontend/messages/fr.json index 1d605e939..bbac559fe 100644 --- a/frontend/messages/fr.json +++ b/frontend/messages/fr.json @@ -367,7 +367,7 @@ "extraLarge": "Très grand", "rid": "RID", "scope": "Périmètre", - "auditor": "Auditeur", + "reader": "Auditeur", "lastUpdate": "Dernière mise à jour", "riskScenarioAssetHelpText": "Biens sensibles impactés par ce scénario de risque", "riskScenarioMeasureHelpText": "Les mesures appliquées existantes pour gérer ce risque", diff --git a/frontend/src/lib/utils/locales.ts b/frontend/src/lib/utils/locales.ts index 22b921956..9f31a74fc 100644 --- a/frontend/src/lib/utils/locales.ts +++ b/frontend/src/lib/utils/locales.ts @@ -300,7 +300,7 @@ export function localItems(languageTag: string): LocalItems { notApplicable: m.notApplicable({ languageTag: languageTag }), administrator: m.administrator({ languageTag: languageTag }), analyst: m.analyst({ languageTag: languageTag }), - auditor: m.auditor({ languageTag: languageTag }), + reader: m.reader({ languageTag: languageTag }), domainManager: m.domainManager({ languageTag: languageTag }), authors: m.authors({ languageTag: languageTag }), reviewers: m.reviewers({ languageTag: languageTag }), diff --git a/frontend/tests/utils/test-data.ts b/frontend/tests/utils/test-data.ts index abe634393..687ccd051 100644 --- a/frontend/tests/utils/test-data.ts +++ b/frontend/tests/utils/test-data.ts @@ -76,7 +76,7 @@ export default { "view_user", ] }, - auditor: { + reader: { name: 'Reader', perms: [ "view_project", diff --git a/frontend/tests/utils/test-utils.ts b/frontend/tests/utils/test-utils.ts index d33a189a5..b826b5721 100644 --- a/frontend/tests/utils/test-utils.ts +++ b/frontend/tests/utils/test-utils.ts @@ -359,7 +359,7 @@ export class TestContent { last_name: vars.user.lastName, user_groups: [ `${vars.folderName} - ${vars.usergroups.analyst.name}`, - `${vars.folderName} - ${vars.usergroups.auditor.name}`, + `${vars.folderName} - ${vars.usergroups.reader.name}`, `${vars.folderName} - ${vars.usergroups.domainManager.name}`, `${vars.folderName} - ${vars.usergroups.approver.name}` ],