retour commentaire

anct-cnum · Sep 20, 2024 · d9b1616 · d9b1616
1 parent ad6ff11
commit d9b1616
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 26 deletions.
diff --git a/...irmation-email-candidature-conseiller/PageConfirmationEmailCandidatureConseiller.test.jsx b/...irmation-email-candidature-conseiller/PageConfirmationEmailCandidatureConseiller.test.jsx
@@ -10,7 +10,6 @@ describe('confirmation Email', () => {
   it('quand j’affiche la page de confirmation de l’email validée alors le titre et les informations de la page s’affichent', () => {
     // WHEN
     render(<ConfirmationEmailCandidature />);
-    vi.spyOn(window, 'scrollTo').mockImplementation();
 
     // THEN
     const titre = screen.getByRole('heading', { level: 1, name: 'Confirmation de l’enregistrement de votre candidature' });
@@ -25,22 +24,21 @@ describe('confirmation Email', () => {
     expect(envoyer).toBeInTheDocument();
   });
 
-  it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message success s’affiche', async () => {
+  it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message de succès s’affiche', async () => {
     // WHEN
-    vi.spyOn(window, 'scrollTo').mockImplementation();
-    render(<ConfirmationEmailCandidature />);
     vi.stubGlobal('fetch', vi.fn(
       () => ({ status: 200, json: async () => Promise.resolve({}) }))
     );
+    render(<ConfirmationEmailCandidature />);
 
     const envoyer = screen.getByRole('button', { name: 'Confirmer' });
     expect(envoyer).toBeInTheDocument();
     // eslint-disable-next-line testing-library/no-unnecessary-act
     await act(() => {
       fireEvent.click(envoyer);
     });
-    expect(envoyer).not.toBeInTheDocument();
     // THEN
+    expect(envoyer).not.toBeInTheDocument();
     const titre = screen.getByRole('heading', { level: 1, name: 'Confirmation de l’enregistrement de votre candidature' });
     expect(titre).toBeInTheDocument();
 
@@ -53,14 +51,13 @@ describe('confirmation Email', () => {
     const messageError403 = screen.queryByText('Impossible de valider l’email, le lien a expiré ou est invalide.', { selector: 'p' });
     expect(messageError403).not.toBeInTheDocument();
 
-    const messageErrorGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
-    expect(messageErrorGenerale).not.toBeInTheDocument();
+    const messageErreurGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
+    expect(messageErreurGenerale).not.toBeInTheDocument();
   });
 
   it('quand l’utilisateur clique sur le bouton et que le lien est invalide alors un message d’erreur s’affiche', async () => {
 
     // GIVEN
-    vi.spyOn(window, 'scrollTo').mockImplementation();
     vi.stubGlobal('fetch', vi.fn(
       () => ({ status: 403, json: async () => Promise.resolve({}) }))
     );
@@ -87,12 +84,12 @@ describe('confirmation Email', () => {
     const messageSuccess = screen.queryByText('Votre email a été confirmé et votre inscription est maintenant active.' +
     ' Vous serez contacté par mail ou par téléphone si une structure est intéressée par votre profil.', { selector: 'p' });
     expect(messageSuccess).not.toBeInTheDocument();
-    const messageErrorGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
-    expect(messageErrorGenerale).not.toBeInTheDocument();
+    const messageErreurGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
+    expect(messageErreurGenerale).not.toBeInTheDocument();
   });
 
 
-  it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendu se produit alors un message d’erreur s’affiche', async () => {
+  it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendue se produit alors un message d’erreur s’affiche', async () => {
     // GIVEN
     vi.stubGlobal('fetch', vi.fn(
       () => ({ status: 500, json: async () => Promise.resolve({}) }))

diff --git a/src/views/confirmation-email-candidature-structure/ConfirmationEmailCandidatureStructure.jsx b/src/views/confirmation-email-candidature-structure/ConfirmationEmailCandidatureStructure.jsx
@@ -2,7 +2,6 @@ import React, { useEffect, useState } from 'react';
 import { useParams } from 'react-router-dom';
 import { useApiConfirmationEmailCandidatureStructure } from './useApiConfirmationEmailCandidatureStructure';
 
-
 export default function ConfirmationEmailCandidatureStructure() {
   const [reponseStatusConfirmation, setReponseStatusConfirmation] = useState(null);
   const { actionConfirmationEmailCandidatureStructure } = useApiConfirmationEmailCandidatureStructure();
@@ -11,7 +10,7 @@ export default function ConfirmationEmailCandidatureStructure() {
   useEffect(() => {
     window.scrollTo({ top: 0, behavior: 'smooth' });
   }, []);
-  const ClickBoutonConfirmer = async () => {
+  const confirmerEmailCandidatureStructure = async () => {
     const response = await actionConfirmationEmailCandidatureStructure(token);
     setReponseStatusConfirmation(response.status);
   };
@@ -32,7 +31,7 @@ export default function ConfirmationEmailCandidatureStructure() {
         <p>
           Appuyez sur le bouton pour confirmer votre email
         </p>
-        <button onClick={ClickBoutonConfirmer} className="fr-btn fr-btn--secondary">
+        <button onClick={confirmerEmailCandidatureStructure} className="fr-btn fr-btn--secondary">
           Confirmer
         </button>
       </>}

diff --git a/...nfirmation-email-candidature-structure/PageConfirmationEmailCandidatureStructure.test.jsx b/...nfirmation-email-candidature-structure/PageConfirmationEmailCandidatureStructure.test.jsx
@@ -10,7 +10,6 @@ describe('confirmation Email', () => {
   it('quand j’affiche la page de confirmation de l’email validée alors le titre et les informations de la page s’affichent', () => {
     // WHEN
     render(<ConfirmationEmailCandidatureStructure />);
-    vi.spyOn(window, 'scrollTo').mockImplementation();
 
     // THEN
     const titre = screen.getByRole('heading', { level: 1, name: 'Confirmation de l’enregistrement de votre candidature' });
@@ -25,13 +24,12 @@ describe('confirmation Email', () => {
     expect(envoyer).toBeInTheDocument();
   });
 
-  it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message success s’affiche', async () => {
+  it('quand l’utilisateur clique sur le bouton et que le lien est valide alors un message de succès s’affiche', async () => {
     // WHEN
-    vi.spyOn(window, 'scrollTo').mockImplementation();
-    render(<ConfirmationEmailCandidatureStructure />);
     vi.stubGlobal('fetch', vi.fn(
       () => ({ status: 200, json: async () => Promise.resolve({}) }))
     );
+    render(<ConfirmationEmailCandidatureStructure />);
 
     const envoyer = screen.getByRole('button', { name: 'Confirmer' });
     expect(envoyer).toBeInTheDocument();
@@ -53,14 +51,13 @@ describe('confirmation Email', () => {
     const messageError403 = screen.queryByText('Impossible de valider l’email, le lien a expiré ou est invalide.', { selector: 'p' });
     expect(messageError403).not.toBeInTheDocument();
 
-    const messageErrorGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
-    expect(messageErrorGenerale).not.toBeInTheDocument();
+    const messageErreurGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
+    expect(messageErreurGenerale).not.toBeInTheDocument();
   });
 
   it('quand l’utilisateur clique sur le bouton et que le lien est invalide alors un message d’erreur s’affiche', async () => {
 
     // GIVEN
-    vi.spyOn(window, 'scrollTo').mockImplementation();
     vi.stubGlobal('fetch', vi.fn(
       () => ({ status: 403, json: async () => Promise.resolve({}) }))
     );
@@ -87,12 +84,12 @@ describe('confirmation Email', () => {
     const messageSuccess = screen.queryByText('Votre email a été confirmé et votre inscription est maintenant active.' +
     ' Vous recevrez un mail d’activation de votre espace structure lorsque votre candidature aura été validée.', { selector: 'p' });
     expect(messageSuccess).not.toBeInTheDocument();
-    const messageErrorGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
-    expect(messageErrorGenerale).not.toBeInTheDocument();
+    const messageErreurGenerale = screen.queryByText('Une erreur s’est produite veuillez réessayer plus tard.', { selector: 'p' });
+    expect(messageErreurGenerale).not.toBeInTheDocument();
   });
 
 
-  it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendu se produit alors un message d’erreur s’affiche', async () => {
+  it('quand l’utilisateur clique sur le bouton et qu’une erreur innatendue se produit alors un message d’erreur s’affiche', async () => {
     // GIVEN
     vi.stubGlobal('fetch', vi.fn(
       () => ({ status: 500, json: async () => Promise.resolve({}) }))

diff --git a/...s/confirmation-email-candidature-structure/useApiConfirmationEmailCandidatureStructure.js b/...s/confirmation-email-candidature-structure/useApiConfirmationEmailCandidatureStructure.js
@@ -1,7 +1,5 @@
-
 export const useApiConfirmationEmailCandidatureStructure = () => {
 
-
   const actionConfirmationEmailCandidatureStructure = async token => {
     const baseUrl = import.meta.env.VITE_APP_API_URL;
     const requestOptions = {