feat(démarche): ajoute une description optionnelle sur les démarches (#…

…900)

src/api/graphql/schemas/titres-demarches.graphql

@@ -19,6 +19,9 @@ type Demarche {
   """
   id: ID!
 
+  "Description optionnelle de la démarche"
+  description: String
+
   """
   Slug composé de la concaténation:
   - de l'id du titre
@@ -69,10 +72,12 @@ type Phase {
 input InputDemarcheCreation {
   typeId: ID!
   titreId: ID!
+  description: String
 }
 
 input InputDemarcheModification {
   id: ID!
   typeId: ID!
   titreId: ID!
+  description: String
 }

src/api/rest/format/titres-demarches.ts

@@ -51,6 +51,7 @@ const titresDemarchesFormatTable = (titresDemarches: ITitreDemarche[]) =>
       titre_statut: titre.statut!.nom,
       type: titreDemarche.type!.nom,
       statut: titreDemarche.statut!.nom,
+      description: titreDemarche.description,
       titre_references: titre.references
         ?.map(r => `${r.type?.nom} : ${r.nom}`)
         .join(';'),

src/database/models/titres-demarches.ts

@@ -24,7 +24,8 @@ class TitresDemarches extends Model {
       titreId: { type: 'string', maxLength: 128 },
       typeId: { type: 'string', maxLength: 8 },
       statutId: { type: 'string', maxLength: 3 },
-      ordre: { type: 'integer' }
+      ordre: { type: 'integer' },
+      description: { type: ['string', 'null'] }
     }
   }
 

src/database/queries/permissions/titres-demarches.ts

@@ -112,34 +112,22 @@ const titreDemarcheModificationQuery = (
 ): void => {
   let modificationQuery = raw('false')
   if (permissionCheck(user?.permissionId, ['super'])) {
-    modificationQuery = raw('not exists(?)', [
-      titreDemarcheEtapesQuery(demarcheAlias)
-    ])
+    modificationQuery = raw('true')
   } else if (
     permissionCheck(user?.permissionId, ['admin', 'editeur']) &&
     user?.administrations?.length
   ) {
     modificationQuery = titresDemarchesAdministrationsModificationQuery(
       user.administrations,
       'type'
-    )
-      .whereRaw('?? = ??', ['titresModification.id', 'titresDemarches.titreId'])
-      .whereNotExists(titreDemarcheEtapesQuery(demarcheAlias))
+    ).whereRaw('?? = ??', ['titresModification.id', 'titresDemarches.titreId'])
 
     q.groupBy(`${demarcheAlias}.id`, 'type.travaux')
   }
 
   q.select(modificationQuery.as('modification'))
 }
 
-const titreDemarcheEtapesQuery = (demarcheAlias: string) =>
-  TitresEtapes.query()
-    .alias('titresDemarchesEtapes')
-    .whereRaw('?? = ??', [
-      'titresDemarchesEtapes.titreDemarcheId',
-      `${demarcheAlias}.id`
-    ])
-
 const titreEtapesCreationQuery = (
   demarcheAlias: string,
   user: IUtilisateur | null | undefined

src/knex/migrations-data/20220127141738_titres_demarches_description_add.js

@@ -0,0 +1,6 @@
+exports.up = knex =>
+  knex.schema.alterTable('titres_demarches', function (table) {
+    table.string('description').nullable()
+  })
+
+exports.down = () => ({})

src/types.ts

@@ -773,6 +773,7 @@ interface ITitreEtapeJustificatif {
 
 interface ITitreDemarche {
   id: string
+  description?: string
   slug?: string
   titreId: string
   titre?: ITitre | null

tests/administrations-permissions-demarches.test.ts

@@ -138,8 +138,8 @@ describe('Création des démarches', () => {
 describe('Modification des démarches', () => {
   test.each`
     administrationId       | modifier
-    ${'ope-onf-973-01'}    | ${false}
-    ${'min-mtes-dgaln-01'} | ${false}
+    ${'ope-onf-973-01'}    | ${true}
+    ${'min-mtes-dgaln-01'} | ${true}
     ${'min-dajb-01'}       | ${false}
   `(
     "un utilisateur admin de l’administration $administrationId peut modifier des démarches d'un titre ARM : $modifier",
@@ -150,8 +150,8 @@ describe('Modification des démarches', () => {
   test.each`
     administrationId       | modifier
     ${'ope-onf-973-01'}    | ${false}
-    ${'dea-guyane-01'}     | ${false}
-    ${'min-mtes-dgaln-01'} | ${false}
+    ${'dea-guyane-01'}     | ${true}
+    ${'min-mtes-dgaln-01'} | ${true}
     ${'min-dajb-01'}       | ${false}
   `(
     "un utilisateur admin de l’administration $administrationId peut modifier des démarches d'un titre AXM : $modifier",
@@ -162,7 +162,7 @@ describe('Modification des démarches', () => {
   test.each`
     administrationId       | modifier
     ${'min-mtes-dgec-01'}  | ${false}
-    ${'min-mtes-dgaln-01'} | ${false}
+    ${'min-mtes-dgaln-01'} | ${true}
     ${'min-dajb-01'}       | ${false}
   `(
     "un utilisateur admin de l’administration $administrationId peut modifier des démarches d'un titre CXM : $modifier",
@@ -173,7 +173,7 @@ describe('Modification des démarches', () => {
   test.each`
     administrationId       | modifier
     ${'min-mtes-dgec-01'}  | ${false}
-    ${'min-mtes-dgaln-01'} | ${false}
+    ${'min-mtes-dgaln-01'} | ${true}
     ${'min-dajb-01'}       | ${false}
   `(
     "un utilisateur admin de l’administration $administrationId peut modifier des démarches d'un titre PRM : $modifier",
@@ -184,7 +184,7 @@ describe('Modification des démarches', () => {
   test.each`
     administrationId       | modifier
     ${'min-mtes-dgec-01'}  | ${false}
-    ${'min-mtes-dgaln-01'} | ${false}
+    ${'min-mtes-dgaln-01'} | ${true}
     ${'min-dajb-01'}       | ${false}
   `(
     "un utilisateur admin de l’administration $administrationId peut modifier des démarches d'un titre PXM : $modifier",

tests/titres-demarches.test.ts

@@ -251,7 +251,9 @@ describe('demarcheModifier', () => {
     )
 
     expect(res.body.errors).toHaveLength(1)
-    expect(res.body.errors[0].message).toBe('droits insuffisants')
+    expect(res.body.errors[0].message).toBe(
+      'impossible de modifier le type d’une démarche si celle-ci a déjà une ou plusieurs étapes'
+    )
   })
 })