From fb3e3b6fad07320e65bb805b7aa5d05c6029f9f3 Mon Sep 17 00:00:00 2001 From: Ornella Ourfi Date: Wed, 2 Oct 2024 15:53:23 +0200 Subject: [PATCH 1/4] fix trim --- src/components/commun/Input.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/components/commun/Input.jsx b/src/components/commun/Input.jsx index 2db9e1c..a889626 100644 --- a/src/components/commun/Input.jsx +++ b/src/components/commun/Input.jsx @@ -17,7 +17,7 @@ export default function Input({ children, id, isRequired = true, type = 'text', min={min} readOnly={readOnly} name={id} - value={value} + value={value?.trim()} aria-busy={ariaBusy} data-testid={testId} /> From 7f2ce053d7b8d456797aff6b9d8986748b5ebd8a Mon Sep 17 00:00:00 2001 From: Ornella Ourfi Date: Wed, 2 Oct 2024 17:52:08 +0200 Subject: [PATCH 2/4] aucompletion off pour le form conseiller --- src/components/commun/Input.jsx | 5 ++++- src/views/candidature-conseiller/AddressChooser.jsx | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/components/commun/Input.jsx b/src/components/commun/Input.jsx index a889626..c08fadc 100644 --- a/src/components/commun/Input.jsx +++ b/src/components/commun/Input.jsx @@ -1,7 +1,8 @@ import React from 'react'; import PropTypes from 'prop-types'; -export default function Input({ children, id, isRequired = true, type = 'text', pattern, onChange, list, min, readOnly, isLoading, ariaBusy, value, testId = '' }) { +export default function Input({ children, id, isRequired = true, autoComplete = 'on', testId = '', type = 'text', + pattern, onChange, list, min, readOnly, isLoading, ariaBusy, value }) { return (
@@ -11,6 +12,7 @@ export default function Input({ children, id, isRequired = true, type = 'text', type={type} id={id} required={isRequired} + autoComplete={autoComplete} pattern={pattern} onChange={onChange} list={list} @@ -35,6 +37,7 @@ Input.propTypes = { children: PropTypes.node, id: PropTypes.string, isRequired: PropTypes.bool, + autoComplete: PropTypes.string, type: PropTypes.string, pattern: PropTypes.string, onChange: PropTypes.func, diff --git a/src/views/candidature-conseiller/AddressChooser.jsx b/src/views/candidature-conseiller/AddressChooser.jsx index 7d112f8..194357a 100644 --- a/src/views/candidature-conseiller/AddressChooser.jsx +++ b/src/views/candidature-conseiller/AddressChooser.jsx @@ -10,6 +10,7 @@ export default function AddressChooser() { return ( <> { From 46f0e7c61581254e222ebe65d09ec047fd907988 Mon Sep 17 00:00:00 2001 From: Ornella Ourfi Date: Wed, 2 Oct 2024 19:22:54 +0200 Subject: [PATCH 3/4] gestion dans le cas failed to fetch --- .../CandidatureConseiller.jsx | 5 +- .../CandidatureConseiller.test.jsx | 49 ++++++++++++++- .../CandidatureCoordinateur.jsx | 5 +- .../CandidatureCoordinateur.test.jsx | 60 ++++++++++++++++++- .../CandidatureStructure.jsx | 5 +- .../CandidatureStructure.test.jsx | 60 ++++++++++++++++++- 6 files changed, 176 insertions(+), 8 deletions(-) diff --git a/src/views/candidature-conseiller/CandidatureConseiller.jsx b/src/views/candidature-conseiller/CandidatureConseiller.jsx index e1ca6e7..231e2e0 100644 --- a/src/views/candidature-conseiller/CandidatureConseiller.jsx +++ b/src/views/candidature-conseiller/CandidatureConseiller.jsx @@ -54,10 +54,13 @@ export default function CandidatureConseiller() { } else { const conseillerData = await buildConseillerData(formData); const resultatCreation = await creerCandidatureConseiller(conseillerData); - if (resultatCreation.status >= 400) { + if (resultatCreation.status && resultatCreation.status >= 400) { const error = await resultatCreation.json(); setValidationError(error.message); window.scrollTo({ top: 0, behavior: 'smooth' }); + } else if (!resultatCreation.status) { + setValidationError(resultatCreation.message); + window.scrollTo({ top: 0, behavior: 'smooth' }); } else { navigate('/candidature-validee-conseiller'); } diff --git a/src/views/candidature-conseiller/CandidatureConseiller.test.jsx b/src/views/candidature-conseiller/CandidatureConseiller.test.jsx index 961eaf5..e5157ca 100644 --- a/src/views/candidature-conseiller/CandidatureConseiller.test.jsx +++ b/src/views/candidature-conseiller/CandidatureConseiller.test.jsx @@ -522,7 +522,7 @@ describe('candidature conseiller', () => { vi.useRealTimers(); }); - it('quand je remplis complètementle formulaire avec un numéro téléphone valide, alors je suis redirigé vers la page de candidature validée', async () => { + it('quand je remplis complètement le formulaire avec un numéro téléphone valide, alors je suis redirigé vers la page de candidature validée', async () => { // GIVEN vi.useFakeTimers(); vi.setSystemTime(new Date(2023, 11, 12, 13)); @@ -661,4 +661,51 @@ describe('candidature conseiller', () => { vi.useRealTimers(); }); + + it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => { + // GIVEN + vi.useFakeTimers(); + vi.setSystemTime(new Date(2023, 11, 12, 13)); + + vi.spyOn(useApiAdmin, 'useApiAdmin').mockImplementation(() => ({ + creerCandidatureConseiller: vi.fn().mockReturnValue({ message: 'Failed to fetch' }), + buildConseillerData: vi.fn(), + })); + + render(); + const prenom = screen.getByLabelText('Prénom *'); + fireEvent.change(prenom, { target: { value: 'Jean' } }); + const nom = screen.getByLabelText('Nom *'); + fireEvent.change(nom, { target: { value: 'Dupont' } }); + const email = screen.getByLabelText('Adresse e-mail * Format attendu : nom@domaine.fr'); + fireEvent.change(email, { target: { value: 'jean.dupont@example.com' } }); + const adresse = screen.getByLabelText('Votre lieu d’habitation * Saississez le nom ou le code postal de votre commune.'); + fireEvent.change(adresse, { target: { value: '93100 Montreuil' } }); + const telephone = screen.getByLabelText('Téléphone Format attendu : +33122334455'); + fireEvent.change(telephone, { target: { value: '+33159590730' } }); + const enEmploi = screen.getByRole('checkbox', { name: 'En emploi' }); + fireEvent.click(enEmploi); + const oui = screen.getByRole('radio', { name: 'Oui' }); + fireEvent.click(oui); + const date = screen.getByLabelText('Choisir une date'); + fireEvent.change(date, { target: { value: dateDujour() } }); + const _5km = screen.getByRole('radio', { name: '5 km' }); + fireEvent.click(_5km); + const descriptionMotivation = screen.getByLabelText('Votre message *'); + fireEvent.change(descriptionMotivation, { target: { value: 'je suis motivé !' } }); + + // WHEN + const envoyer = screen.getByRole('button', { name: 'Envoyer votre candidature' }); + + // eslint-disable-next-line testing-library/no-unnecessary-act + await act(() => { + fireEvent.click(envoyer); + }); + + // THEN + const contenuErreurValidation = screen.getByText('Failed to fetch', { selector: 'p' }); + expect(contenuErreurValidation).toBeInTheDocument(); + + vi.useRealTimers(); + }); }); diff --git a/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx b/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx index b754f52..77d8304 100644 --- a/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx +++ b/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx @@ -39,10 +39,13 @@ export default function CandidatureCoordinateur() { const formData = new FormData(event.currentTarget); const coordinateurData = await buildCoordinateurData(formData, geoLocation, codeCommune); const resultatCreation = await creerCandidatureCoordinateur(coordinateurData); - if (resultatCreation.status >= 400) { + if (resultatCreation.status && resultatCreation.status >= 400) { const error = await resultatCreation.json(); setValidationError(error.message); window.scrollTo({ top: 0, behavior: 'smooth' }); + } else if (!resultatCreation.status) { + setValidationError(resultatCreation.message); + window.scrollTo({ top: 0, behavior: 'smooth' }); } else { navigate('/candidature-validee-structure'); } diff --git a/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx b/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx index 358db4a..8747042 100644 --- a/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx +++ b/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx @@ -3,7 +3,7 @@ import { describe, expect, it, vi } from 'vitest'; import CandidatureCoordinateur from './CandidatureCoordinateur'; import { textMatcher, dateDujour } from '../../../test/test-utils'; import * as ReactRouterDom from 'react-router-dom'; -import { useApiAdmin } from '../candidature-conseiller/useApiAdmin'; +import * as useApiAdmin from '../candidature-conseiller/useApiAdmin'; import { useEntrepriseFinder } from '../candidature-structure/useEntrepriseFinder'; vi.mock('react-router-dom', () => ({ @@ -427,7 +427,7 @@ describe('candidature coordinateur', () => { ] ]; - const { buildCoordinateurData } = renderHook(() => useApiAdmin()).result.current; + const { buildCoordinateurData } = renderHook(() => useApiAdmin.useApiAdmin()).result.current; const { getGeoLocationFromAddress } = renderHook(() => useEntrepriseFinder()).result.current; // //WHEN @@ -462,5 +462,61 @@ describe('candidature coordinateur', () => { vi.useRealTimers(); }); + + it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => { + // GIVEN + vi.useFakeTimers(); + vi.setSystemTime(new Date(2023, 11, 12, 13)); + + vi.spyOn(useApiAdmin, 'useApiAdmin').mockImplementation(() => ({ + creerCandidatureCoordinateur: vi.fn().mockReturnValue({ message: 'Failed to fetch' }), + buildCoordinateurData: vi.fn(), + })); + + render(); + const siret = screen.getByLabelText('SIRET / RIDET *'); + fireEvent.change(siret, { target: { value: '1234567890123' } }); + const denomination = screen.getByLabelText('Dénomination *'); + fireEvent.change(denomination, { target: { value: 'Entreprise' } }); + const adresse = screen.getByLabelText('Adresse *'); + fireEvent.change(adresse, { target: { value: '75007 Paris' } }); + const typeStructure = screen.getByRole('radio', { name: 'Une commune' }); + fireEvent.click(typeStructure); + const prenom = screen.getByLabelText('Prénom *'); + fireEvent.change(prenom, { target: { value: 'Jean' } }); + const nom = screen.getByLabelText('Nom *'); + fireEvent.change(nom, { target: { value: 'Dupont' } }); + const fonction = screen.getByLabelText('Fonction *'); + fireEvent.change(fonction, { target: { value: 'Test' } }); + const email = screen.getByLabelText('Adresse e-mail *'); + fireEvent.change(email, { target: { value: 'jean.dupont@example.com' } }); + const telephone = screen.getByLabelText('Téléphone *'); + fireEvent.change(telephone, { target: { value: '+33123456789' } }); + const identificationCandidat = screen.getByRole('radio', { name: 'Oui' }); + fireEvent.click(identificationCandidat); + const typeMission = screen.getByRole('radio', { name: 'Accompagnera également des publics' }); + fireEvent.click(typeMission); + const date = screen.getByLabelText('Choisir une date'); + fireEvent.change(date, { target: { value: dateDujour() } }); + const descriptionMotivation = screen.getByLabelText('Votre message *'); + fireEvent.change(descriptionMotivation, { target: { value: 'je suis motivé !' } }); + const confirmation = screen.getByRole('checkbox', { name: 'Je confirme avoir lu et pris connaissance des conditions d’engagement. *' }); + fireEvent.click(confirmation); + + // WHEN + const envoyer = screen.getByRole('button', { name: 'Envoyer votre candidature' }); + + // eslint-disable-next-line testing-library/no-unnecessary-act + await act(() => { + fireEvent.click(envoyer); + }); + + + // THEN + const contenuErreurValidation = screen.getByText('Failed to fetch', { selector: 'p' }); + expect(contenuErreurValidation).toBeInTheDocument(); + + vi.useRealTimers(); + }); }); diff --git a/src/views/candidature-structure/CandidatureStructure.jsx b/src/views/candidature-structure/CandidatureStructure.jsx index 3695c26..c4d2f90 100644 --- a/src/views/candidature-structure/CandidatureStructure.jsx +++ b/src/views/candidature-structure/CandidatureStructure.jsx @@ -40,10 +40,13 @@ export default function CandidatureStructure() { const formData = new FormData(event.currentTarget); const structureData = await buildStructureData(formData, geoLocation, codeCommune); const resultatCreation = await creerCandidatureStructure(structureData); - if (resultatCreation.status >= 400) { + if (resultatCreation.status && resultatCreation.status >= 400) { const error = await resultatCreation.json(); setValidationError(error.message); window.scrollTo({ top: 0, behavior: 'smooth' }); + } else if (!resultatCreation.status) { + setValidationError(resultatCreation.message); + window.scrollTo({ top: 0, behavior: 'smooth' }); } else { navigate('/candidature-validee-structure'); } diff --git a/src/views/candidature-structure/CandidatureStructure.test.jsx b/src/views/candidature-structure/CandidatureStructure.test.jsx index c890545..9153acf 100644 --- a/src/views/candidature-structure/CandidatureStructure.test.jsx +++ b/src/views/candidature-structure/CandidatureStructure.test.jsx @@ -3,7 +3,7 @@ import { describe, expect, it, vi } from 'vitest'; import CandidatureStructure from './CandidatureStructure'; import { textMatcher, dateDujour } from '../../../test/test-utils'; import * as ReactRouterDom from 'react-router-dom'; -import { useApiAdmin } from '../candidature-conseiller/useApiAdmin'; +import * as useApiAdmin from '../candidature-conseiller/useApiAdmin'; import { useEntrepriseFinder } from './useEntrepriseFinder'; vi.mock('react-router-dom', () => ({ @@ -519,7 +519,7 @@ describe('candidature structure', () => { ] ]; - const { buildStructureData } = renderHook(() => useApiAdmin()).result.current; + const { buildStructureData } = renderHook(() => useApiAdmin.useApiAdmin()).result.current; const { getGeoLocationFromAddress } = renderHook(() => useEntrepriseFinder()).result.current; // //WHEN @@ -555,5 +555,61 @@ describe('candidature structure', () => { vi.useRealTimers(); }); + + it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => { + // GIVEN + vi.useFakeTimers(); + vi.setSystemTime(new Date(2023, 11, 12, 13)); + + vi.spyOn(useApiAdmin, 'useApiAdmin').mockImplementation(() => ({ + creerCandidatureStructure: vi.fn().mockReturnValue({ message: 'Failed to fetch' }), + buildStructureData: vi.fn(), + })); + + render(); + const siret = screen.getByLabelText('SIRET / RIDET *'); + fireEvent.change(siret, { target: { value: '1234567890123' } }); + const denomination = screen.getByLabelText('Dénomination *'); + fireEvent.change(denomination, { target: { value: 'Entreprise' } }); + const adresse = screen.getByLabelText('Adresse *'); + fireEvent.change(adresse, { target: { value: '75007 Paris' } }); + const typeStructure = screen.getByRole('radio', { name: 'Une commune' }); + fireEvent.click(typeStructure); + const prenom = screen.getByLabelText('Prénom *'); + fireEvent.change(prenom, { target: { value: 'Jean' } }); + const nom = screen.getByLabelText('Nom *'); + fireEvent.change(nom, { target: { value: 'Dupont' } }); + const fonction = screen.getByLabelText('Fonction *'); + fireEvent.change(fonction, { target: { value: 'Test' } }); + const email = screen.getByLabelText('Adresse e-mail *'); + fireEvent.change(email, { target: { value: 'jean.dupont@example.com' } }); + const telephone = screen.getByLabelText('Téléphone *'); + fireEvent.change(telephone, { target: { value: '+33123456789' } }); + const nombre = screen.getByLabelText('Combien de conseillers numériques souhaitez-vous accueillir ?*'); + fireEvent.change(nombre, { target: { value: 1 } }); + const identificationCandidat = screen.getByRole('radio', { name: 'Oui' }); + fireEvent.click(identificationCandidat); + const date = screen.getByLabelText('Choisir une date'); + fireEvent.change(date, { target: { value: dateDujour() } }); + const descriptionMotivation = screen.getByLabelText('Votre message *'); + fireEvent.change(descriptionMotivation, { target: { value: 'je suis motivé !' } }); + const confirmation = screen.getByRole('checkbox', { name: 'Je confirme avoir lu et pris connaissance des conditions d’engagement. *' }); + fireEvent.click(confirmation); + + // WHEN + const envoyer = screen.getByRole('button', { name: 'Envoyer votre candidature' }); + + // eslint-disable-next-line testing-library/no-unnecessary-act + await act(() => { + fireEvent.click(envoyer); + }); + + + // THEN + const contenuErreurValidation = screen.getByText('Failed to fetch', { selector: 'p' }); + expect(contenuErreurValidation).toBeInTheDocument(); + + vi.useRealTimers(); + }); }); From 066305c4d9f3507bae208fa21f49f91c42491303 Mon Sep 17 00:00:00 2001 From: Ornella Ourfi Date: Thu, 3 Oct 2024 14:48:26 +0200 Subject: [PATCH 4/4] retour commentaire --- src/views/candidature-conseiller/CandidatureConseiller.jsx | 2 +- src/views/candidature-conseiller/CandidatureConseiller.test.jsx | 2 +- src/views/candidature-coordinateur/CandidatureCoordinateur.jsx | 2 +- .../candidature-coordinateur/CandidatureCoordinateur.test.jsx | 2 +- src/views/candidature-structure/CandidatureStructure.jsx | 2 +- src/views/candidature-structure/CandidatureStructure.test.jsx | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/views/candidature-conseiller/CandidatureConseiller.jsx b/src/views/candidature-conseiller/CandidatureConseiller.jsx index 231e2e0..ed0f3a6 100644 --- a/src/views/candidature-conseiller/CandidatureConseiller.jsx +++ b/src/views/candidature-conseiller/CandidatureConseiller.jsx @@ -54,7 +54,7 @@ export default function CandidatureConseiller() { } else { const conseillerData = await buildConseillerData(formData); const resultatCreation = await creerCandidatureConseiller(conseillerData); - if (resultatCreation.status && resultatCreation.status >= 400) { + if (resultatCreation?.status >= 400) { const error = await resultatCreation.json(); setValidationError(error.message); window.scrollTo({ top: 0, behavior: 'smooth' }); diff --git a/src/views/candidature-conseiller/CandidatureConseiller.test.jsx b/src/views/candidature-conseiller/CandidatureConseiller.test.jsx index e5157ca..ee73c21 100644 --- a/src/views/candidature-conseiller/CandidatureConseiller.test.jsx +++ b/src/views/candidature-conseiller/CandidatureConseiller.test.jsx @@ -662,7 +662,7 @@ describe('candidature conseiller', () => { vi.useRealTimers(); }); - it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => { + it('quand je remplis le formulaire et qu’une erreur se produit alors un message d’erreur s’affiche', async () => { // GIVEN vi.useFakeTimers(); vi.setSystemTime(new Date(2023, 11, 12, 13)); diff --git a/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx b/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx index 77d8304..b1a72b6 100644 --- a/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx +++ b/src/views/candidature-coordinateur/CandidatureCoordinateur.jsx @@ -39,7 +39,7 @@ export default function CandidatureCoordinateur() { const formData = new FormData(event.currentTarget); const coordinateurData = await buildCoordinateurData(formData, geoLocation, codeCommune); const resultatCreation = await creerCandidatureCoordinateur(coordinateurData); - if (resultatCreation.status && resultatCreation.status >= 400) { + if (resultatCreation?.status >= 400) { const error = await resultatCreation.json(); setValidationError(error.message); window.scrollTo({ top: 0, behavior: 'smooth' }); diff --git a/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx b/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx index 8747042..f166a49 100644 --- a/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx +++ b/src/views/candidature-coordinateur/CandidatureCoordinateur.test.jsx @@ -463,7 +463,7 @@ describe('candidature coordinateur', () => { vi.useRealTimers(); }); - it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => { + it('quand je remplis le formulaire et qu’une erreur se produit alors un message d’erreur s’affiche', async () => { // GIVEN vi.useFakeTimers(); vi.setSystemTime(new Date(2023, 11, 12, 13)); diff --git a/src/views/candidature-structure/CandidatureStructure.jsx b/src/views/candidature-structure/CandidatureStructure.jsx index c4d2f90..9208e5c 100644 --- a/src/views/candidature-structure/CandidatureStructure.jsx +++ b/src/views/candidature-structure/CandidatureStructure.jsx @@ -40,7 +40,7 @@ export default function CandidatureStructure() { const formData = new FormData(event.currentTarget); const structureData = await buildStructureData(formData, geoLocation, codeCommune); const resultatCreation = await creerCandidatureStructure(structureData); - if (resultatCreation.status && resultatCreation.status >= 400) { + if (resultatCreation?.status >= 400) { const error = await resultatCreation.json(); setValidationError(error.message); window.scrollTo({ top: 0, behavior: 'smooth' }); diff --git a/src/views/candidature-structure/CandidatureStructure.test.jsx b/src/views/candidature-structure/CandidatureStructure.test.jsx index 9153acf..de05cdd 100644 --- a/src/views/candidature-structure/CandidatureStructure.test.jsx +++ b/src/views/candidature-structure/CandidatureStructure.test.jsx @@ -556,7 +556,7 @@ describe('candidature structure', () => { vi.useRealTimers(); }); - it('quand je remplis le formulaire et q’une erreur se produit alors un message d’erreur s’affiche', async () => { + it('quand je remplis le formulaire et qu’une erreur se produit alors un message d’erreur s’affiche', async () => { // GIVEN vi.useFakeTimers(); vi.setSystemTime(new Date(2023, 11, 12, 13));