fix: validation question defi (#907)

betagouv · Feb 7, 2025 · 612ad41 · 612ad41
1 parent 3b7536b
commit 612ad41
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 31 deletions.
diff --git a/src/components/pages/PageQuestionDefi.vue b/src/components/pages/PageQuestionDefi.vue
@@ -29,7 +29,7 @@
           <form v-if="!reponseAEteDonnee" @submit.prevent="validerLaReponse">
             <BoutonRadio
               v-model="reponse"
-              :default-value="reponse ? reponse.toString() : undefined"
+              :default-value="reponse"
               :description="defiViewModel.description"
               :legende="defiViewModel.libelle"
               :options="
@@ -56,14 +56,6 @@
               <textarea id="explication" v-model="explication" class="fr-input fr-mb-4w" name="explication" />
             </div>
 
-            <Alert
-              v-if="alerte.isActive && !isReponseInitialeDifferente"
-              :type="alerte.type"
-              :titre="alerte.titre"
-              :message="alerte.message"
-              class="fr-mb-4w"
-            />
-
             <button class="fr-btn fr-btn--lg fr-mb-2w" title="Valider">Valider</button>
           </form>
 
@@ -104,13 +96,11 @@
 <script lang="ts" setup>
   import { computed, onMounted, ref } from 'vue';
   import { useRoute } from 'vue-router';
-  import Alert from '@/components/custom/Alert.vue';
   import BoutonRadio from '@/components/custom/BoutonRadio.vue';
   import CarteInfo from '@/components/custom/CarteInfo.vue';
   import DefiFin from '@/components/custom/Defi/DefiFin.vue';
   import ThematiqueTag from '@/components/custom/Thematiques/ThematiqueTag.vue';
   import FilDAriane from '@/components/dsfr/FilDAriane.vue';
-  import { useAlerte } from '@/composables/useAlerte';
   import { DefiPresenterImpl, DefiViewModel, ReponsePossible } from '@/domaines/defi/adapters/defi.presenter.impl';
   import { DefiRepositoryAxios } from '@/domaines/defi/adapters/defi.repository.axios';
   import { EnvoyerReponseDefiUsecase } from '@/domaines/defi/envoyerReponseDefi.usecase';
@@ -153,7 +143,6 @@
     'Christophe',
   ];
   const route = useRoute();
-  const { alerte, afficherAlerte } = useAlerte();
   const questionId = Array.isArray(route.params.id) ? route.params.id[0] : route.params.id;
 
   const isLoading = ref<boolean>(true);
@@ -165,9 +154,6 @@
   const reponseInitiale = ref<string>('');
 
   const utilisateurId = utilisateurStore().utilisateur.id;
-  const isReponseInitialeDifferente = computed(() => {
-    return reponse.value !== reponseInitiale.value;
-  });
 
   const obtenirPrenomsAleatoires = (count, max) => {
     const indices: number[] = [];
@@ -193,26 +179,21 @@
       utilisateurId,
       new DefiPresenterImpl((viewModel: DefiViewModel) => {
         defiViewModel.value = viewModel;
-        reponse.value = viewModel.reponse;
+        reponse.value = defiViewModel.value.reponse;
         explication.value = viewModel.explicationRefus || '';
       }),
     );
+
     isLoading.value = false;
     reponseInitiale.value = reponse.value;
   });
 
   const validerLaReponse = async () => {
-    if (!isReponseInitialeDifferente.value) {
-      afficherAlerte('error', 'Erreur', 'Veuillez sélectionner une réponse pour continuer');
-      return;
-    }
-
     const envoyerReponseUsecase = new EnvoyerReponseDefiUsecase(
       new DefiRepositoryAxios(),
       ToDoListEventBusImpl.getInstance(),
     );
     await envoyerReponseUsecase.execute(utilisateurId, questionId, reponse.value, explication.value);
-    alerte.value.isActive = false;
 
     if (defiViewModel.value?.reponse) {
       aDejaRepondu.value = true;

diff --git a/src/domaines/defi/adapters/defi.presenter.impl.ts b/src/domaines/defi/adapters/defi.presenter.impl.ts
@@ -27,17 +27,18 @@ export class DefiPresenterImpl implements DefiPresenter {
   constructor(private readonly questionViewModel: (viewModel: DefiViewModel) => void) {}
 
   presente(defi: Defi) {
+    const reponsePossibles = this.determinerReponsesPossible(defi.status);
     this.questionViewModel({
       id: defi.id,
       libelle: defi.libelle,
       points: defi.points.toLocaleString(),
-      reponses_possibles: this.determinerReponsesPossible(defi.status),
+      reponses_possibles: reponsePossibles,
       thematiqueTag: {
         label: MenuThematiques.getThematiqueData(defi.thematique).labelDansLeMenu,
         style: TagThematique.getTagThematiqueUtilitaire(defi.thematique),
       },
       description: defi.description,
-      reponse: defi.status,
+      reponse: reponsePossibles.filter(r => r.id === defi.status).length > 0 ? defi.status : reponsePossibles[0].id,
       astuces: this.determinerParagrapheVide(defi.astuces),
       pourquoi: this.determinerParagrapheVide(defi.pourquoi),
       explicationRefus: defi.explicationRefus,

diff --git a/tests/defi/recupererDefi.usecase.spec.ts b/tests/defi/recupererDefi.usecase.spec.ts
@@ -32,7 +32,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
         libelle: 'Defi libelle',
         points: '10',
         pourquoi: 'Défi pourquoi',
-        reponse: 'todo',
+        reponse: 'en_cours',
         explicationRefus: undefined,
         reponses_possibles: [
           {
@@ -85,7 +85,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
         libelle: 'Defi libelle',
         points: '10',
         pourquoi: 'Défi pourquoi',
-        reponse: 'en_cours',
+        reponse: 'fait',
         explicationRefus: undefined,
         reponses_possibles: [
           {
@@ -136,7 +136,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
         points: '10',
         pourquoi: 'Défi pourquoi',
         explicationRefus: undefined,
-        reponse: 'fait',
+        reponse: 'deja_fait',
         reponses_possibles: [{ id: 'deja_fait', label: '<span aria-hidden="true">✅</span> Déjà fait' }],
         thematiqueTag: {
           label: 'Me déplacer',
@@ -181,7 +181,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
           points: '10',
           pourquoi: 'Défi pourquoi',
           explicationRefus: undefined,
-          reponse: 'fait',
+          reponse: 'deja_fait',
           reponses_possibles: [{ id: 'deja_fait', label: '<span aria-hidden="true">✅</span> Déjà fait' }],
           thematiqueTag: {
             label: 'Me déplacer',
@@ -224,7 +224,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
           points: '10',
           pourquoi: 'Défi pourquoi',
           explicationRefus: undefined,
-          reponse: 'fait',
+          reponse: 'deja_fait',
           reponses_possibles: [{ id: 'deja_fait', label: '<span aria-hidden="true">✅</span> Déjà fait' }],
           thematiqueTag: {
             label: 'Me déplacer',
@@ -267,7 +267,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
           points: '10',
           pourquoi: 'Défi pourquoi',
           explicationRefus: undefined,
-          reponse: 'fait',
+          reponse: 'deja_fait',
           reponses_possibles: [{ id: 'deja_fait', label: '<span aria-hidden="true">✅</span> Déjà fait' }],
           thematiqueTag: {
             label: 'Me déplacer',
@@ -310,7 +310,7 @@ describe('Fichier de tests pour récuperer un défi', () => {
           points: '10',
           pourquoi: 'Défi pourquoi',
           explicationRefus: undefined,
-          reponse: 'fait',
+          reponse: 'deja_fait',
           reponses_possibles: [{ id: 'deja_fait', label: '<span aria-hidden="true">✅</span> Déjà fait' }],
           thematiqueTag: {
             label: 'Me déplacer',