Renomme des éléments liés aux droits

backend/src/auth/auth.module.ts

@@ -5,8 +5,8 @@ import { CollectivitesModule } from '../collectivites/collectivites.module';
 import { CommonModule } from '../common/common.module';
 import { ConfigurationModule } from '../config/configuration.module';
 import { AuthGuard } from './guards/auth.guard';
-import { PermissionService } from '../auth/gestion-des-droits/permission.service';
-import { RoleService } from '../auth/gestion-des-droits/roles/role.service';
+import { PermissionService } from '@/backend/auth/authorizations/permission.service';
+import { RoleService } from '@/backend/auth/authorizations/roles/role.service';
 
 @Global()
 @Module({

backend/src/auth/authorizations/permission-operation.enum.ts

@@ -0,0 +1,18 @@
+export enum PermissionOperation {
+  // Collectivités
+  COLLECTIVITES_VISITE = 'collectivites.visite',
+  COLLECTIVITES_LECTURE = 'collectivites.lecture',
+  // Référentiels
+  REFERENTIELS_LECTURE = 'referentiels.lecture',
+  REFERENTIELS_EDITION = 'referentiels.edition',
+  // Fiches actions
+  PLANS_FICHES_EDITION = 'plans.fiches.edition',
+  PLANS_FICHES_LECTURE = 'plans.fiches.lecture',
+  PLANS_FICHES_VISITE = 'plans.fiches.visite',
+  // Indicateurs
+  INDICATEURS_LECTURE = 'indicateurs.lecture',
+  INDICATEURS_VISITE = 'indicateurs.visite',
+  INDICATEURS_EDITION = 'indicateurs.edition',
+  INDICATEURS_TRAJECTOIRES_LECTURE = 'indicateurs.trajectoires.lecture',
+  INDICATEURS_TRAJECTOIRES_EDITION = 'indicateurs.trajectoires.edition',
+}

backend/src/auth/authorizations/permission.models.ts

@@ -0,0 +1,73 @@
+import { Role } from '@/backend/auth/authorizations/roles/role.enum';
+import { PermissionOperation } from '@/backend/auth/authorizations/permission-operation.enum';
+
+export const Permission: Record<Role, PermissionOperation[]> = {
+  [Role.CONNECTE]: [],
+  [Role.VERIFIE]: [PermissionOperation.COLLECTIVITES_VISITE, PermissionOperation.PLANS_FICHES_VISITE, PermissionOperation.INDICATEURS_VISITE],
+  [Role.SUPPORT]: [
+    PermissionOperation.COLLECTIVITES_VISITE,
+    PermissionOperation.COLLECTIVITES_LECTURE,
+    PermissionOperation.REFERENTIELS_LECTURE,
+    PermissionOperation.PLANS_FICHES_VISITE,
+    PermissionOperation.PLANS_FICHES_LECTURE,
+    PermissionOperation.INDICATEURS_VISITE,
+    PermissionOperation.INDICATEURS_LECTURE,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
+  ],
+  [Role.ADEME]: [
+    PermissionOperation.COLLECTIVITES_VISITE,
+    PermissionOperation.PLANS_FICHES_VISITE,
+    PermissionOperation.INDICATEURS_VISITE,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
+  ],
+  [Role.LECTURE]: [
+    PermissionOperation.COLLECTIVITES_VISITE,
+    PermissionOperation.COLLECTIVITES_LECTURE,
+    PermissionOperation.REFERENTIELS_LECTURE,
+    PermissionOperation.PLANS_FICHES_VISITE,
+    PermissionOperation.PLANS_FICHES_LECTURE,
+    PermissionOperation.INDICATEURS_VISITE,
+    PermissionOperation.INDICATEURS_LECTURE,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
+  ],
+  [Role.EDITION]: [
+    PermissionOperation.COLLECTIVITES_VISITE,
+    PermissionOperation.COLLECTIVITES_LECTURE,
+    PermissionOperation.REFERENTIELS_LECTURE,
+    PermissionOperation.REFERENTIELS_EDITION,
+    PermissionOperation.PLANS_FICHES_VISITE,
+    PermissionOperation.PLANS_FICHES_LECTURE,
+    PermissionOperation.PLANS_FICHES_EDITION,
+    PermissionOperation.INDICATEURS_VISITE,
+    PermissionOperation.INDICATEURS_LECTURE,
+    PermissionOperation.INDICATEURS_EDITION,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_EDITION,
+  ],
+  [Role.ADMIN]: [
+    PermissionOperation.COLLECTIVITES_VISITE,
+    PermissionOperation.COLLECTIVITES_LECTURE,
+    PermissionOperation.REFERENTIELS_LECTURE,
+    PermissionOperation.REFERENTIELS_EDITION,
+    PermissionOperation.PLANS_FICHES_VISITE,
+    PermissionOperation.PLANS_FICHES_LECTURE,
+    PermissionOperation.PLANS_FICHES_EDITION,
+    PermissionOperation.INDICATEURS_VISITE,
+    PermissionOperation.INDICATEURS_LECTURE,
+    PermissionOperation.INDICATEURS_EDITION,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_EDITION,
+  ],
+  [Role.AUDITEUR]: [
+    PermissionOperation.COLLECTIVITES_VISITE,
+    PermissionOperation.COLLECTIVITES_LECTURE,
+    PermissionOperation.REFERENTIELS_LECTURE,
+    PermissionOperation.REFERENTIELS_EDITION,
+    PermissionOperation.PLANS_FICHES_VISITE,
+    PermissionOperation.PLANS_FICHES_LECTURE,
+    PermissionOperation.INDICATEURS_VISITE,
+    PermissionOperation.INDICATEURS_LECTURE,
+    PermissionOperation.INDICATEURS_EDITION,
+    PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
+  ],
+};

backend/src/auth/gestion-des-droits/permission.service.e2e-spec.ts → backend/src/auth/authorizations/permission.service.e2e-spec.ts

@@ -1,13 +1,13 @@
 import { getAuthUser, getTestApp, getTestDatabase, YOULOU_DOUDOU } from '@/backend/test';
 import { INestApplication } from '@nestjs/common';
-import { PermissionService } from '@/backend/auth/gestion-des-droits/permission.service';
+import { PermissionService } from '@/backend/auth/authorizations/permission.service';
 import { AuthenticatedUser } from '@/backend/auth/models/auth.models';
-import { Authorization } from '@/backend/auth/gestion-des-droits/authorization.enum';
-import { ResourceType } from '@/backend/auth/gestion-des-droits/resource-type.enum';
+import { PermissionOperation } from '@/backend/auth/authorizations/permission-operation.enum';
+import { ResourceType } from '@/backend/auth/authorizations/resource-type.enum';
 import DatabaseService from '../../common/services/database.service';
 import { eq } from 'drizzle-orm';
-import { utilisateurSupportTable } from '@/backend/auth/gestion-des-droits/roles/utilisateur-support.table';
-import { utilisateurVerifieTable } from '@/backend/auth/gestion-des-droits/roles/utilisateur-verifie.table';
+import { utilisateurSupportTable } from '@/backend/auth/authorizations/roles/utilisateur-support.table';
+import { utilisateurVerifieTable } from '@/backend/auth/authorizations/roles/utilisateur-verifie.table';
 import { dcpTable } from '@/backend/auth';
 import { collectiviteTable } from '@/backend/collectivites/models/collectivite.table';
 
@@ -29,9 +29,9 @@ describe('Gestion des droits', () => {
   describe('Droit en visite sur une collectivité -> NOK', async () => {
     test('Utilisateur vérifié -> OK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_VISITE,
+          PermissionOperation.COLLECTIVITES_VISITE,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -44,9 +44,9 @@ describe('Gestion des droits', () => {
         .set({ verifie: false })
         .where(eq(utilisateurVerifieTable.userId, yoloDodoUser.id));
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_VISITE,
+          PermissionOperation.COLLECTIVITES_VISITE,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -70,9 +70,9 @@ describe('Gestion des droits', () => {
         .set({ accessRestreint: true })
         .where(eq(collectiviteTable.id, 20));
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_VISITE,
+          PermissionOperation.COLLECTIVITES_VISITE,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -94,9 +94,9 @@ describe('Gestion des droits', () => {
   describe('Droit en lecture sur une collectivité -> NOK', async () => {
     test('Utilisateur vérifié sur sa collectivité -> OK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_LECTURE,
+          PermissionOperation.COLLECTIVITES_LECTURE,
           ResourceType.COLLECTIVITE,
           1,
           true
@@ -110,9 +110,9 @@ describe('Gestion des droits', () => {
         .set({ verifie: false })
         .where(eq(utilisateurVerifieTable.userId, yoloDodoUser.id));
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_LECTURE,
+          PermissionOperation.COLLECTIVITES_LECTURE,
           ResourceType.COLLECTIVITE,
           1,
           true
@@ -133,9 +133,9 @@ describe('Gestion des droits', () => {
 
     test('Utilisateur vérifié sur une autre collectivité -> NOK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_LECTURE,
+          PermissionOperation.COLLECTIVITES_LECTURE,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -148,9 +148,9 @@ describe('Gestion des droits', () => {
         .set({ support: true })
         .where(eq(utilisateurSupportTable.userId, yoloDodoUser.id));
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.COLLECTIVITES_CONTENT_LECTURE,
+          PermissionOperation.COLLECTIVITES_LECTURE,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -170,9 +170,9 @@ describe('Gestion des droits', () => {
     });
     test('Auditeur sur sa collectivité audité -> OK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           youlouDoudouUser,
-          Authorization.COLLECTIVITES_CONTENT_LECTURE,
+          PermissionOperation.COLLECTIVITES_LECTURE,
           ResourceType.COLLECTIVITE,
           10,
           true
@@ -184,9 +184,9 @@ describe('Gestion des droits', () => {
   describe('Droit en edition sur une collectivité -> NOK', async () => {
     test('Sur sa collectivité -> OK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.FICHES_EDITION,
+          PermissionOperation.PLANS_FICHES_EDITION,
           ResourceType.COLLECTIVITE,
           1,
           true
@@ -195,9 +195,9 @@ describe('Gestion des droits', () => {
     });
     test('Sur une autre collectivité -> NOK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.FICHES_EDITION,
+          PermissionOperation.PLANS_FICHES_EDITION,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -209,9 +209,9 @@ describe('Gestion des droits', () => {
   describe("Droit en lecture sur la trajectoire d'une collectivité -> NOK", async () => {
     test('Sur sa collectivité -> OK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.INDICATEURS_TRAJECTOIRE_LECTURE,
+          PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
           ResourceType.COLLECTIVITE,
           1,
           true
@@ -221,9 +221,9 @@ describe('Gestion des droits', () => {
 
     test('Sur une autre collectivité -> NOK', async () => {
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.INDICATEURS_TRAJECTOIRE_LECTURE,
+          PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
           ResourceType.COLLECTIVITE,
           20,
           true
@@ -237,9 +237,9 @@ describe('Gestion des droits', () => {
         .set({ email: '[email protected]' })
         .where(eq(dcpTable.userId, yoloDodoUser.id));
       expect(
-        await permissionService.hasTheRightTo(
+        await permissionService.isAllowed(
           yoloDodoUser,
-          Authorization.INDICATEURS_TRAJECTOIRE_LECTURE,
+          PermissionOperation.INDICATEURS_TRAJECTOIRES_LECTURE,
           ResourceType.COLLECTIVITE,
           20,
           true

backend/src/auth/gestion-des-droits/permission.service.ts → backend/src/auth/authorizations/permission.service.ts

@@ -1,10 +1,10 @@
 import { Injectable, Logger, UnauthorizedException } from '@nestjs/common';
 import { AuthRole, AuthUser } from '../models/auth.models';
 import { RoleService } from './roles/role.service';
-import { Authorization } from '@/backend/auth/gestion-des-droits/authorization.enum';
-import { ResourceType } from '@/backend/auth/gestion-des-droits/resource-type.enum';
-import { Permission } from '@/backend/auth/gestion-des-droits/permission.models';
-import { Role } from '@/backend/auth/gestion-des-droits/roles/role.enum';
+import { PermissionOperation } from '@/backend/auth/authorizations/permission-operation.enum';
+import { ResourceType } from '@/backend/auth/authorizations/resource-type.enum';
+import { Permission } from '@/backend/auth/authorizations/permission.models';
+import { Role } from '@/backend/auth/authorizations/roles/role.enum';
 
 @Injectable()
 export class PermissionService {
@@ -15,28 +15,28 @@ export class PermissionService {
   /**
    * Vérifie l'autorisation de l'utilisateur sur une ressource
    * @param user
-   * @param authorization
+   * @param operation
    * @param resourceType type de la ressource
    * @param resourceId identifiant de la ressource, null si resourceType = PLATEFORME
    * @param doNotThrow vrai pour ne pas générer une erreur si l'utilisateur n'a pas l'autorisation
    */
-  async hasTheRightTo(
+  async isAllowed(
     user: AuthUser,
-    authorization: Authorization,
+    operation: PermissionOperation,
     resourceType: ResourceType,
     resourceId: number | null,
     doNotThrow?: boolean
   ): Promise<boolean> {
     this.logger.log(
-      `Vérification que l'utilisateur ${user.id} possède l'autorisation ${authorization} sur la ressource ${resourceType} ${resourceId}`
+      `Vérification que l'utilisateur ${user.id} possède l'autorisation ${operation} sur la ressource ${resourceType} ${resourceId}`
     );
     if (user.role === AuthRole.SERVICE_ROLE) {
       // Le service rôle à tous les droits
       return true;
     }
 
     // Récupère les rôles de l'utilisateur pour la ressource donnée
-    const roles = await this.roleService.getUserRolesForAResource(
+    const roles = await this.roleService.getUserRoles(
       user,
       resourceType,
       resourceId
@@ -53,33 +53,33 @@ export class PermissionService {
     }
 
     // Récupère les autorisations des rôles de l'utilisateur
-    const authorizations: Set<Authorization> = new Set();
+    const operations: Set<PermissionOperation> = new Set();
     for (const role of roles) {
       Permission[role as Role].forEach((permission) =>
-        authorizations.add(permission)
+        operations.add(permission)
       );
     }
 
     this.logger.log(
       `L'utilisateur ${user.id} possède les autorisations ${JSON.stringify([
-        ...authorizations,
+        ...operations,
       ])} sur la ressource ${resourceType} ${resourceId}`
     );
 
-    // Vérifie si l'autorisation demandée est dans la liste des autorisations de l'utilisateur
-    const hasTheRight = authorizations.has(authorization);
+    // Vérifie si l'opération demandée est dans la liste des autorisations de l'utilisateur
+    const hasTheRight = operations.has(operation);
     if (!hasTheRight) {
       this.logger.log(
-        `L'utilisateur ${user.id} ne possède pas l'autorisation ${authorization} sur la ressource ${resourceType} ${resourceId}`
+        `L'utilisateur ${user.id} ne possède pas l'autorisation ${operation} sur la ressource ${resourceType} ${resourceId}`
       );
       if(!doNotThrow){
         throw new UnauthorizedException(
-          `Droits insuffisants, l'utilisateur ${user.id} n'a pas l'autorisation ${authorization} sur la ressource ${resourceType} ${resourceId}`
+          `Droits insuffisants, l'utilisateur ${user.id} n'a pas l'autorisation ${operation} sur la ressource ${resourceType} ${resourceId}`
         );
       }
     }else{
       this.logger.log(
-        `L'utilisateur ${user.id} possède l'autorisation ${authorization} sur la ressource ${resourceType} ${resourceId}`
+        `L'utilisateur ${user.id} possède l'autorisation ${operation} sur la ressource ${resourceType} ${resourceId}`
       );
     }
 

backend/src/auth/gestion-des-droits/roles/role.service.ts → backend/src/auth/authorizations/roles/role.service.ts

@@ -9,8 +9,8 @@ import {
 import { NiveauAcces } from './niveau-acces.enum';
 import { utilisateurSupportTable } from './utilisateur-support.table';
 import { utilisateurVerifieTable } from './utilisateur-verifie.table';
-import { ResourceType } from '@/backend/auth/gestion-des-droits/resource-type.enum';
-import { Role } from '@/backend/auth/gestion-des-droits/roles/role.enum';
+import { ResourceType } from '@/backend/auth/authorizations/resource-type.enum';
+import { Role } from '@/backend/auth/authorizations/roles/role.enum';
 import { dcpTable } from '@/backend/auth';
 
 
@@ -20,7 +20,7 @@ export class RoleService {
 
   constructor(private readonly databaseService: DatabaseService) {}
 
-  async getUserRolesForAResource(
+  async getUserRoles(
     user: AuthUser,
     resourceType: ResourceType,
     resourceId: number | null