From 1c3d434e926e3f2c0c0e813b2132e93e42d4a857 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=A9my=20PASTOURET?= <pastouret.jeremy@gmail.com>
Date: Thu, 7 Nov 2024 14:51:07 +0100
Subject: [PATCH] Nettoyage AB testing Recap email (#4702)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* feat: nettoyage de l'ab testing dans le recap mail

* feat: oublie d'un autre aspect d'ABtesting

* feat: supprime les imports inutiles

* feat: mise à jour des tests unitaires

* fix: prettier clean
---
 cypress/utils/results.js                      |  8 +++--
 lib/enums/event.ts                            |  2 --
 .../modals/recap-email-and-sms-modal.vue      |  4 +--
 src/components/recap-email-and-sms-form.vue   | 35 ++-----------------
 src/components/titre-chapitre.vue             |  6 +---
 src/plugins/ab-testing-service.ts             | 13 -------
 src/router.ts                                 |  2 --
 tools/views/index.html                        |  5 ++-
 8 files changed, 14 insertions(+), 61 deletions(-)

diff --git a/cypress/utils/results.js b/cypress/utils/results.js
index ebe49c0f5d..67aa8d2237 100644
--- a/cypress/utils/results.js
+++ b/cypress/utils/results.js
@@ -253,7 +253,9 @@ const receiveResultsEmail = () => {
 
   const email = "prenom.nom@beta.gouv.fr"
   cy.get("input#email").should("be.visible").type(email)
-  cy.get(".fr-btn:contains(J'accepte d'être recontacté ou recontactée)")
+  cy.get(
+    ".fr-btn:contains(Je reçois mon récapitulatif et je me fais accompagner par téléphone)"
+  )
     .should("be.visible")
     .click()
 
@@ -296,7 +298,9 @@ const receiveResultsSms = () => {
     .click()
   // scroll to input#phone
   cy.get("input#phone").scrollIntoView().should("be.visible").type(phone)
-  cy.get(".fr-btn:contains(J'accepte d'être recontacté ou recontactée)")
+  cy.get(
+    ".fr-btn:contains(Je reçois mon récapitulatif et je me fais accompagner par téléphone)"
+  )
     .should("be.visible")
     .click()
 
diff --git a/lib/enums/event.ts b/lib/enums/event.ts
index 4eef9cd60e..13378f9b38 100644
--- a/lib/enums/event.ts
+++ b/lib/enums/event.ts
@@ -38,8 +38,6 @@ export enum EventAction {
   FormulaireAffiche = "Formulaire affiché",
   FormulaireEmailInvalide = "Invalid email form",
   FormulaireTelephoneInvalide = "Invalid phone form",
-  FormulaireValideAvecRecontact = "Formulaire validé avec recontact",
-  FormulaireValideSansRecontact = "Formulaire validé sans recontact",
   Instructions = CTALabel.Instructions,
   JeDonneMonAvis = "Je donne mon avis",
   Link = "link",
diff --git a/src/components/modals/recap-email-and-sms-modal.vue b/src/components/modals/recap-email-and-sms-modal.vue
index 66e26457eb..8d0980f0da 100644
--- a/src/components/modals/recap-email-and-sms-modal.vue
+++ b/src/components/modals/recap-email-and-sms-modal.vue
@@ -1,11 +1,9 @@
 <script setup lang="ts">
 import RecapEmailAndSmsForm from "@/components/recap-email-and-sms-form.vue"
 import { computed } from "vue"
-import ABTestingService from "@/plugins/ab-testing-service.js"
 
 const headerTitle = computed(() => {
-  return process.env.VITE_SHOW_SMS_TAB &&
-    ABTestingService.getValues().Followup_SMS === "show"
+  return process.env.VITE_SHOW_SMS_TAB
     ? "Recevoir les résultats par email/SMS"
     : "Recevoir les résultats par email"
 })
diff --git a/src/components/recap-email-and-sms-form.vue b/src/components/recap-email-and-sms-form.vue
index 8e26847ab6..75e8dbd646 100644
--- a/src/components/recap-email-and-sms-form.vue
+++ b/src/components/recap-email-and-sms-form.vue
@@ -7,7 +7,6 @@ import { useRouter } from "vue-router"
 import * as Sentry from "@sentry/vue"
 import StatisticsMixin from "@/mixins/statistics.js"
 import { EventCategory, EventAction } from "@lib/enums/event.js"
-import ABTestingService from "@/plugins/ab-testing-service.js"
 
 const router = useRouter()
 const store = useStore()
@@ -36,15 +35,7 @@ const inputPhonePattern = computed(() => {
   return `^(((\\+?|00)(${diallingCodes})\\s?|0)[67])([\\s\\.\\-]?\\d{2}){4}`
 })
 
-const showSms =
-  process.env.VITE_SHOW_SMS_TAB &&
-  ABTestingService.getValues().Followup_SMS === "show"
-
-StatisticsMixin.methods.sendEventToMatomo(
-  EventCategory.Followup,
-  EventAction.FormulaireAffiche,
-  ABTestingService.getValues().CTA_EmailRecontact
-)
+const showSms = process.env.VITE_SHOW_SMS_TAB
 
 const sendRecap = async (surveyOptin) => {
   try {
@@ -66,21 +57,6 @@ const sendRecap = async (surveyOptin) => {
       phoneInputErrorMessage.value = true
       emailInputErrorMessage.value = true
     }
-
-    const hasInputErrorMessage =
-      phoneInputErrorMessage.value || emailInputErrorMessage.value
-
-    if (!hasInputErrorMessage) {
-      const eventAction = surveyOptin
-        ? EventAction.FormulaireValideAvecRecontact
-        : EventAction.FormulaireValideSansRecontact
-
-      StatisticsMixin.methods.sendEventToMatomo(
-        EventCategory.Followup,
-        eventAction,
-        ABTestingService.getValues().CTA_EmailRecontact
-      )
-    }
   } catch (error: any) {
     const errorMessage = error?.response?.data?.toString() || ""
     if (!errorMessage.includes("Invalid destination address")) {
@@ -193,14 +169,7 @@ const sendRecapByEmail = async (surveyOptin) => {
 }
 
 function computeCtaText() {
-  const ctaVersion = ABTestingService.getValues().CTA_EmailRecontact
-  if (ctaVersion === "version_test_1") {
-    return "J’accepte qu’on me recontacte pour faire le point sur mes démarches"
-  } else if (ctaVersion === "version_test_2") {
-    return "Je reçois mon récapitulatif et je me fais accompagner par téléphone"
-  } else {
-    return "J'accepte d'être recontacté ou recontactée"
-  }
+  return "Je reçois mon récapitulatif et je me fais accompagner par téléphone"
 }
 
 const ctaText = ref(computeCtaText())
diff --git a/src/components/titre-chapitre.vue b/src/components/titre-chapitre.vue
index cb475ab23b..acbf907b54 100644
--- a/src/components/titre-chapitre.vue
+++ b/src/components/titre-chapitre.vue
@@ -45,7 +45,6 @@ import SendRecapButton from "@/components/buttons/send-recap-button.vue"
 import { useStore } from "@/stores/index.js"
 import { useRouter } from "vue-router"
 import { useResultsStore } from "@/stores/results.js"
-import ABTestingService from "@/plugins/ab-testing-service.js"
 
 export default {
   name: "TitreChapitre",
@@ -65,10 +64,7 @@ export default {
       return this.resultsStore.shouldDisplayResults
     },
     showSMS() {
-      return (
-        process.env.VITE_SHOW_SMS_TAB &&
-        ABTestingService.getValues().Followup_SMS === "show"
-      )
+      return process.env.VITE_SHOW_SMS_TAB
     },
     title() {
       return this.getTitleByRoute(this.$route)
diff --git a/src/plugins/ab-testing-service.ts b/src/plugins/ab-testing-service.ts
index 76f545e888..1456524fd4 100644
--- a/src/plugins/ab-testing-service.ts
+++ b/src/plugins/ab-testing-service.ts
@@ -59,19 +59,6 @@ function getEnvironment() {
     ABTestingEnvironment.plans_to_ask_question.value ||
     (Math.random() > 0.5 ? "show" : "hidden")
 
-  const versions = ["version_actuelle", "version_test_1", "version_test_2"]
-  const ctaEmailRecontact = ABTestingEnvironment.CTA_EmailRecontact || {}
-  ctaEmailRecontact.index ||= 5
-  ctaEmailRecontact.value ||=
-    versions[Math.floor(Math.random() * versions.length)]
-  ABTestingEnvironment.CTA_EmailRecontact = ctaEmailRecontact
-
-  ABTestingEnvironment.Followup_SMS = ABTestingEnvironment.Followup_SMS || {}
-  ABTestingEnvironment.Followup_SMS.index = 2
-  ABTestingEnvironment.Followup_SMS.value =
-    ABTestingEnvironment.Followup_SMS.value ||
-    (Math.random() > 0.5 ? "show" : "hide")
-
   storageService.local.setItem("ABTesting", ABTestingEnvironment)
   return ABTestingEnvironment
 }
diff --git a/src/router.ts b/src/router.ts
index 938921378d..c1e10ca83f 100644
--- a/src/router.ts
+++ b/src/router.ts
@@ -279,12 +279,10 @@ const router = createRouter({
       path: "/init-ci",
       name: "init-ci",
       redirect: () => {
-        ABTestingService.setVariant("CTA_EmailRecontact", "version_actuelle")
         ABTestingService.setVariant(
           "aides_bafa",
           "aides_bafa_fusionnees_conserve_position"
         )
-        ABTestingService.setVariant("Followup_SMS", "show")
         return "/"
       },
     },
diff --git a/tools/views/index.html b/tools/views/index.html
index d548ebbefa..c1782acf36 100644
--- a/tools/views/index.html
+++ b/tools/views/index.html
@@ -11,7 +11,10 @@ <h1>Service d'email local</h1>
         <p>
           Pour ajouter un suivi, effectuer une simulation jusqu'au bout, cliquer
           sur le bouton <b>"Recevoir les résultats par email"</b> et enfin sur
-          <b>"j'accepte d'être recontacté ou recontactée par email"</b></p
+          <b
+            >"Je reçois mon récapitulatif et je me fais accompagner par
+            téléphone"</b
+          ></p
         >
       </span>
       <ul v-if="followups && typeKeys">