Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(hel-524) fix wording problem #361

Merged
merged 2 commits into from
Sep 6, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions src/frontend/configuration/wording/Wording.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -167,6 +167,7 @@ export interface Wording {

// Bloc Autorisation
readonly TITRE_BLOC_AUTORISATION_ET_CAPACITÉ: string;
readonly CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE: string;
readonly AUTORISATIONS_SANITAIRE: string;
readonly AUTORISATIONS_MS: string;
readonly AUTORISATIONS_ACTIVITES: string;
Expand Down
1 change: 1 addition & 0 deletions src/frontend/configuration/wording/WordingFr.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -216,6 +216,7 @@ export class WordingFr implements Wording {
// Bloc Autorisation Sanitaire
readonly AUTRES_ACTIVITÉS: string = "Autres activités";
readonly AUTRES_ACTIVITÉS_SAN: string = "Autres activités de soins";
readonly CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE: string = "Capacité installée par activités sanitaires";
readonly RECONNAISSANCES_CONTRACTUELLES: string = "Reconnaissances contractuelles";
readonly ÉQUIPEMENTS_MATÉRIELS_LOURDS: string = "Équipements matériels lourds";
readonly DATE_DE_FIN: string = "Date de fin";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ export const BlocAutorisationsCapacites = ({ entitéJuridiqueAutorisationsCapaci
<ul className="indicateurs">
<GraphiqueCapacitésParActivité
estEntitéJuridique
estSanitaire={false}
graphiqueCapacitésParActivitéViewModel={entitéJuridiqueAutorisationsCapacitesViewModel.graphiqueCapacitesParActivitesViewModel}
/>
{!entitéJuridiqueAutorisationsCapacitesViewModel.lesAutorisationsActivitesNeSontPasRenseignées() && (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@ describe("GraphiqueCapacitésParActivité", () => {

it('affiche les informations de l’indicateur "Capacité par activités"', () => {
// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);

// THEN
expect(graphiqueTest.titre(wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS)).toBeInTheDocument();
expect(graphiqueTest.titre(wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE)).toBeInTheDocument();
expect(graphiqueTest.dateMiseAJour("SAE", "02/09/2022")[0]).toBeInTheDocument();
expect(graphiqueTest.boutonAfficherTranscription).toHaveAttribute("aria-expanded", "false");
expect(graphiqueTest.boutonAfficherTranscription).not.toBeDisabled();
Expand All @@ -38,14 +38,14 @@ describe("GraphiqueCapacitésParActivité", () => {
describe("Info bulle", () => {
it('affiche le contenu de l’info bulle après avoir cliqué sur le bouton "détails" (Capacité par activités)', () => {
// GIVEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);

// WHEN
graphiqueTest.ouvreDétail();

// THEN
expect(graphiqueTest.détail).toHaveAttribute("data-fr-opened", "true");
const infoBulle = screen.getByRole("dialog", { name: wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS });
const infoBulle = screen.getByRole("dialog", { name: wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE });
expect(graphiqueTest.boutonFermerDétail).toBeInTheDocument();
expect(graphiqueTest.abréviationFichierSource("SAE")).toHaveAttribute("title", wording.SAE_TITLE);
const élémentsDeCompréhension = within(infoBulle).getByRole("region", { name: wording.ÉLÉMENTS_DE_COMPRÉHENSION });
Expand All @@ -62,7 +62,7 @@ describe("GraphiqueCapacitésParActivité", () => {

it('ferme l’info bulle après avoir cliqué sur le bouton "Fermer" (Capacité par activités)', () => {
// GIVEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);
graphiqueTest.ouvreDétail();

// WHEN
Expand All @@ -76,12 +76,12 @@ describe("GraphiqueCapacitésParActivité", () => {
describe("Transcription textuelle", () => {
it("affiche un tableau descriptif avec les toutes les activités", () => {
// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={graphiqueCapacitésViewModel} />);

// THEN
const tableau = graphiqueTest.transcriptionTable;

const libellésLigneDEnTête = [wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS, wording.LITS, wording.PLACES];
const libellésLigneDEnTête = [wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE, wording.LITS, wording.PLACES];
const indicateursLigneDEnTête = within(tableau).getAllByRole("columnheader");
libellésLigneDEnTête.forEach((libellé, index) => {
expect(indicateursLigneDEnTête[index].textContent).toBe(libellé);
Expand Down Expand Up @@ -149,7 +149,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);

// THEN
const tableau = graphiqueTest.transcriptionTable;
Expand Down Expand Up @@ -182,7 +182,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);

// THEN
const tableau = graphiqueTest.transcriptionTable;
Expand All @@ -195,7 +195,7 @@ describe("GraphiqueCapacitésParActivité", () => {
// GIVEN
const autorisationsSansActivité = new GraphiqueCapacitésParActivitéViewModel([], wording);
// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);
// THEN
const transcription = graphiqueTest.boutonAfficherTranscription;
expect(transcription).toBeDisabled();
Expand All @@ -207,7 +207,7 @@ describe("GraphiqueCapacitésParActivité", () => {
// GIVEN
const autorisationsSansActivité = new GraphiqueCapacitésParActivitéViewModel([], wording);
// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);

// THEN
const exergue = screen.getByText(
Expand All @@ -224,7 +224,7 @@ describe("GraphiqueCapacitésParActivité", () => {
wording
);
// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);

// THEN
const exergue = screen.getByText(`${wording.AUCUNE_DONNÉE_RENSEIGNÉE} ${annéeEnCours - 5}, ${annéeEnCours - 3}, ${annéeEnCours - 1}`, { selector: "p" });
Expand All @@ -244,7 +244,7 @@ describe("GraphiqueCapacitésParActivité", () => {
wording
);
// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);

// THEN
const exergue = screen.queryByText(`${wording.AUCUNE_DONNÉE_RENSEIGNÉE}`, {
Expand Down Expand Up @@ -277,7 +277,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsSansActivité} />);

// THEN
const exergue = screen.getByText(`${wording.AUCUNE_DONNÉE_RENSEIGNÉE} ${annéeEnCours - 5}`, {
Expand All @@ -299,7 +299,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
// THEN
const indicateursAutorisationsEtCapacités = screen.getAllByRole("option");
expect(indicateursAutorisationsEtCapacités).toHaveLength(2);
Expand All @@ -315,7 +315,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
// THEN
const années: HTMLOptionElement[] = screen.getAllByRole("option");
expect(années[0].selected).toBe(true);
Expand Down Expand Up @@ -348,7 +348,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
// THEN
const indicateursAutorisationsEtCapacités = screen.getAllByRole("option");
expect(indicateursAutorisationsEtCapacités).toHaveLength(1);
Expand All @@ -371,7 +371,7 @@ describe("GraphiqueCapacitésParActivité", () => {
);

// WHEN
renderFakeComponent(<GraphiqueCapacitésParActivité graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
renderFakeComponent(<GraphiqueCapacitésParActivité estSanitaire graphiqueCapacitésParActivitéViewModel={autorisationsViewModel} />);
const select = screen.getByRole("combobox");
fireEvent.change(select, { target: { value: "2021" } });

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,10 @@ import { GraphiqueCapacitésParActivitéViewModel } from "./GraphiqueCapacitésP
type GraphiqueCapacitésParActivitéProps = Readonly<{
graphiqueCapacitésParActivitéViewModel: GraphiqueCapacitésParActivitéViewModel;
estEntitéJuridique?: boolean;
estSanitaire: boolean;
}>;

export const GraphiqueCapacitésParActivité = ({ graphiqueCapacitésParActivitéViewModel, estEntitéJuridique = false }: GraphiqueCapacitésParActivitéProps) => {
export const GraphiqueCapacitésParActivité = ({ graphiqueCapacitésParActivitéViewModel, estEntitéJuridique = false, estSanitaire }: GraphiqueCapacitésParActivitéProps) => {
const { wording } = useDependencies();
const [annéeEnCours, setAnnéeEnCours] = useState<number>(graphiqueCapacitésParActivitéViewModel.annéeInitiale);

Expand All @@ -29,12 +30,12 @@ export const GraphiqueCapacitésParActivité = ({ graphiqueCapacitésParActivit
}
dateDeMiseÀJour={graphiqueCapacitésParActivitéViewModel.dateDeMiseÀJourDeLaCapacitéInstalléeParActivités}
identifiant="capacite-sanitaire"
nomDeLIndicateur={wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS}
nomDeLIndicateur={estSanitaire ? wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE : wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS}
source={wording.SAE}
>
<HistogrammesHorizontaux
annéesManquantes={graphiqueCapacitésParActivitéViewModel.annéesManquantes()}
nom={wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS}
nom={estSanitaire ? wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE : wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS}
nombreDAnnéeTotale={graphiqueCapacitésParActivitéViewModel.NOMBRE_ANNEES}
valeursDesHistogrammes={[
graphiqueCapacitésParActivitéViewModel.valeursLits(annéeEnCours),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ describe("La page établissement territorial sanitaire - bloc autorisation et ca
}),
},
{
indicateurAffiché: wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS,
indicateurAffiché: wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE,
viewModel: ÉtablissementTerritorialSanitaireViewModelTestBuilder.créeAvecAutorisationsEtCapacités(wording, paths, {
autorisations: {
activités: [],
Expand Down Expand Up @@ -518,7 +518,7 @@ describe("La page établissement territorial sanitaire - bloc autorisation et ca
// THEN
const autorisationEtCapacité = screen.getByRole("region", { name: wording.TITRE_BLOC_AUTORISATION_ET_CAPACITÉ });
const indicateurs = within(autorisationEtCapacité).getAllByRole("listitem");
const itemCapacitéParActivités = sélectionneLIndicateur(wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS, indicateurs);
const itemCapacitéParActivités = sélectionneLIndicateur(wording.CAPACITÉ_INSTALLÉE_PAR_ACTIVITÉS_SANITAIRE, indicateurs);
const indexPartieCapacitéParActivités = indicateurs.indexOf(itemCapacitéParActivités);
const itemAutorisations = sélectionneLIndicateur(wording.AUTORISATIONS_SANITAIRE, indicateurs);
const indexPartieAutorisations = indicateurs.indexOf(itemAutorisations);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export const BlocAutorisationEtCapacitéSanitaire = ({ établissementTerritorial
<Bloc isMain={false} titre={wording.TITRE_BLOC_AUTORISATION_ET_CAPACITÉ}>
<ul className={`indicateurs ${styles["liste-indicateurs"]}`}>
<GraphiqueCapacitésParActivité
graphiqueCapacitésParActivitéViewModel={établissementTerritorialSanitaireAutorisationsViewModel.graphiqueCapacitésParActivitéViewModel}
estSanitaire={true} graphiqueCapacitésParActivitéViewModel={établissementTerritorialSanitaireAutorisationsViewModel.graphiqueCapacitésParActivitéViewModel}
/>
{établissementTerritorialSanitaireAutorisationsViewModel.lesAutorisationsSontEllesRenseignées && (
<IndicateurGraphique
Expand Down