feat: 6189 ajout des tests UI pour le simu preavis de licenciement #6191

Viczei · 2024-10-07T14:15:21Z

No description provided.

Viczei · 2024-10-07T14:52:08Z

...s/code-du-travail-modeles/src/simulators/preavis-licenciement/preavis-licenciement.data.json

+      "question": "Le salarié était-il cadre dans l'ancienne convention collective de la métallurgie (avant le 01/01/2024) ?",
+      "note": "Si le salarié dépendait précédemment de la convention collective des ingénieurs et cadres de la métallurgie (IDCC 650) et qu’il est désormais classé non-cadre en raison de la nouvelle classification des emplois (depuis le 01/01/2024), il bénéficie quand même de la durée de préavis qui s’applique aux cadres."


question en doublon

Viczei · 2024-10-07T14:52:58Z

...s/code-du-travail-modeles/src/simulators/preavis-licenciement/preavis-licenciement.data.json

@@ -1353,7 +1349,7 @@
      "type": "préavis de Licenciement",
      "idcc": 843,
      "typeRupture": null,
-      "answer": "1 mois et demi",
+      "answer": "1.5 mois",


renommage sous forme numérique pour faciliter la génération

mais du coup ça veut dire que dans le front maintenant on affiche 1.5 mois, c'ets pas top non ?

Je suis d'accord, on doit revenir à 1 mois et demi ici.

Viczei · 2024-10-07T14:53:44Z

...s/code-du-travail-modeles/src/simulators/preavis-licenciement/preavis-licenciement.data.json

-    {
-      "criteria": {
-        "catégorie professionnelle": "77| Négociateur VRP"
-      },
-      "type": "préavis de Licenciement",
-      "idcc": 1527,
-      "typeRupture": null,
-      "answer": "2 mois",
-      "answer2": null,
-      "answer3": "60",
-      "ref": "Article 32",
-      "refUrl": "https://www.legifrance.gouv.fr/affichIDCCArticle.do?idArticle=KALIARTI000023759231&cidTexte=KALITEXT000023759095"
-    },


Nettoyage du json, les Négociateur VRP sont déjà défini au dessus. Cet objet me faisais généré un test non valide

carolineBda · 2024-10-10T10:07:38Z

...s/code-du-travail-modeles/src/simulators/preavis-licenciement/preavis-licenciement.data.json

@@ -5125,7 +5108,7 @@
      "type": "préavis de Licenciement",
      "idcc": 3239,
      "typeRupture": null,
-      "answer": "8 jours calendaires",


est-ce que t'as validé avec le métier que c'était ok d'enlever ça ?

carolineBda · 2024-10-10T10:24:06Z

Je mets mes commentaires ici pour qu'on en parle tous ensemble :

Ajout d'un test sans CC (pour tester le code du travail) ?
Je me souviens plus ce qu'on avait dit pour les tests spécifiques à la 3239 (tester qu'on a bien désactivé le légal)
On a pas testé tous les cas particuliers. Pour moi il faut le faire pour être sur qu'on ne les perds pas. Je remet la liste ici pour s'en souvenir :

'1 jour ouvré'
'1 mois'
'1 mois (de date à date)'
'1 mois et demi'
'1 semaine'
'1 semaine (de date à date)'
'14 jours calendaires'
'15 jours'
'15 jours calendaires'
'2 jours'
'2 jours ouvrés'
'2 mois'
'2 mois (de date à date)'
'2 semaines'
'3 mois'
'3 mois (sauf stipulation contractuelle différente)'
'3 mois (de date à date)'
'4 mois'
'4 mois (de date à date)'
'6 mois'
'7 jours calendaires'
'8 jours'
'8 jours calendaires'
'La convention collective ne prévoit pas de préavis pour ce cas de figure'
'Pas de préavis'
'la convention collective ne prévoit pas de préavis pour ce cas de figure'
'pour les salariés hors classification, la durée du préavis est fixée par le contrat de travail dans la limite de 3 mois'

carolineBda · 2024-10-10T10:28:28Z

...e-du-travail-frontend/src/outils/DureePreavisLicenciement/__tests__/agreements/1351.test.tsx

+
+
+    it("should display expected answer", () => {
+      expect(screen.queryAllByText(/1 jour/g)[0]).toBeInTheDocument();


Suggested change

expect(screen.queryAllByText(/1 jour/g)[0]).toBeInTheDocument();

expect(screen.queryAllByText(/1 jour ouvré/g)[0]).toBeInTheDocument();

carolineBda · 2024-10-10T10:30:06Z

...e-du-travail-frontend/src/outils/DureePreavisLicenciement/__tests__/agreements/2941.test.tsx

+
+
+    it("should display expected answer", () => {
+      expect(screen.queryAllByText(/1 mois/g)[0]).toBeInTheDocument();


Suggested change

expect(screen.queryAllByText(/1 mois/g)[0]).toBeInTheDocument();

expect(screen.queryAllByText(/1 mois (de date à date)/g)[0]).toBeInTheDocument();

carolineBda · 2024-10-10T14:23:46Z

...de-du-travail-frontend/src/outils/DureePreavisLicenciement/__tests__/agreements/843.test.tsx

+
+
+    it("should display expected answer", () => {
+      expect(screen.queryAllByText(/5 mois/g)[0]).toBeInTheDocument();


Faux. Ici ça devrait être "1 mois et demi"

…e-licenciement

github-actions · 2024-10-23T07:29:31Z

🎉 Deployment for commit 3403dd0 :

Ingresses

🚀 https://code-du-travail-numerique-6189-ajout-des-tests-ui-pou-tf4bse84.ovh.fabrique.social.gouv.fr/

Docker images

📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/code-du-travail-numerique/app:sha-3403dd039987c9a3da44f0188c273d652d595bf9

Debug

m-maillot · 2024-10-23T07:07:19Z

...s/code-du-travail-modeles/src/simulators/preavis-licenciement/preavis-licenciement.data.json

@@ -1353,7 +1349,7 @@
      "type": "préavis de Licenciement",
      "idcc": 843,
      "typeRupture": null,
-      "answer": "1 mois et demi",
+      "answer": "1.5 mois",


Je suis d'accord, on doit revenir à 1 mois et demi ici.

feat: add TU preavis licenciement

6fe57cc

Viczei self-assigned this Oct 7, 2024

Viczei linked an issue Oct 7, 2024 that may be closed by this pull request

Ajout des tests ui pour le simu Préavis de licenciement #6189

Closed

Viczei changed the title ~~6189 ajout des tests UI pour le simu preavis de licenciement~~ feat: 6189 ajout des tests UI pour le simu preavis de licenciement Oct 7, 2024

chore: update TU

aa10bb8

Viczei temporarily deployed to review-auto October 7, 2024 14:37 — with GitHub Actions Inactive

chore: json updates

6527f10

Viczei requested review from m-maillot, carolineBda and maxgfr October 7, 2024 14:46

Viczei commented Oct 7, 2024

View reviewed changes

maxgfr approved these changes Oct 7, 2024

View reviewed changes

Viczei temporarily deployed to review-auto October 9, 2024 09:13 — with GitHub Actions Inactive

chore: update TU

65a3ace

carolineBda reviewed Oct 10, 2024

View reviewed changes

Viczei temporarily deployed to review-auto October 15, 2024 12:40 — with GitHub Actions Inactive

victor added 2 commits October 15, 2024 14:40

chore: format

b76dccd

Merge branch 'dev' into 6189-ajout-des-tests-ui-pour-le-simu-pravis-d…

d2925ce

…e-licenciement

Viczei had a problem deploying to review-auto October 17, 2024 13:44 — with GitHub Actions Failure

Viczei temporarily deployed to review-auto October 17, 2024 14:02 — with GitHub Actions Inactive

victor and others added 2 commits October 17, 2024 16:02

feat: revert calendaire

4a2c350

test: ajout d'un test sur la 3239

531af54

m-maillot had a problem deploying to review-auto October 23, 2024 07:15 — with GitHub Actions Error

fix: retrait du résultat avec virgule sur la 843

3403dd0

m-maillot temporarily deployed to review-auto October 23, 2024 07:19 — with GitHub Actions Inactive

m-maillot approved these changes Oct 23, 2024

View reviewed changes

m-maillot merged commit 7f6a995 into dev Oct 23, 2024
23 checks passed

m-maillot deleted the 6189-ajout-des-tests-ui-pour-le-simu-pravis-de-licenciement branch October 23, 2024 07:37

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 6189 ajout des tests UI pour le simu preavis de licenciement #6191

feat: 6189 ajout des tests UI pour le simu preavis de licenciement #6191

Viczei commented Oct 7, 2024

Viczei Oct 7, 2024

Viczei Oct 7, 2024

carolineBda Oct 10, 2024

m-maillot Oct 23, 2024

Viczei Oct 7, 2024 •

edited

Loading

carolineBda Oct 10, 2024 •

edited

Loading

carolineBda commented Oct 10, 2024 •

edited

Loading

carolineBda Oct 10, 2024 •

edited

Loading

carolineBda Oct 10, 2024

carolineBda Oct 10, 2024

github-actions bot commented Oct 23, 2024

m-maillot Oct 23, 2024

		"question": "Le salarié était-il cadre dans l'ancienne convention collective de la métallurgie (avant le 01/01/2024) ?",
		"note": "Si le salarié dépendait précédemment de la convention collective des ingénieurs et cadres de la métallurgie (IDCC 650) et qu’il est désormais classé non-cadre en raison de la nouvelle classification des emplois (depuis le 01/01/2024), il bénéficie quand même de la durée de préavis qui s’applique aux cadres."



		it("should display expected answer", () => {
		expect(screen.queryAllByText(/1 jour/g)[0]).toBeInTheDocument();

	expect(screen.queryAllByText(/1 jour/g)[0]).toBeInTheDocument();
	expect(screen.queryAllByText(/1 jour ouvré/g)[0]).toBeInTheDocument();



		it("should display expected answer", () => {
		expect(screen.queryAllByText(/1 mois/g)[0]).toBeInTheDocument();

	expect(screen.queryAllByText(/1 mois/g)[0]).toBeInTheDocument();
	expect(screen.queryAllByText(/1 mois (de date à date)/g)[0]).toBeInTheDocument();



		it("should display expected answer", () => {
		expect(screen.queryAllByText(/5 mois/g)[0]).toBeInTheDocument();

feat: 6189 ajout des tests UI pour le simu preavis de licenciement #6191

feat: 6189 ajout des tests UI pour le simu preavis de licenciement #6191

Conversation

Viczei commented Oct 7, 2024

Viczei Oct 7, 2024

Choose a reason for hiding this comment

Viczei Oct 7, 2024

Choose a reason for hiding this comment

carolineBda Oct 10, 2024

Choose a reason for hiding this comment

m-maillot Oct 23, 2024

Choose a reason for hiding this comment

Viczei Oct 7, 2024 • edited Loading

Choose a reason for hiding this comment

carolineBda Oct 10, 2024 • edited Loading

Choose a reason for hiding this comment

carolineBda commented Oct 10, 2024 • edited Loading

carolineBda Oct 10, 2024 • edited Loading

Choose a reason for hiding this comment

carolineBda Oct 10, 2024

Choose a reason for hiding this comment

carolineBda Oct 10, 2024

Choose a reason for hiding this comment

github-actions bot commented Oct 23, 2024

m-maillot Oct 23, 2024

Choose a reason for hiding this comment

Viczei Oct 7, 2024 •

edited

Loading

carolineBda Oct 10, 2024 •

edited

Loading

carolineBda commented Oct 10, 2024 •

edited

Loading

carolineBda Oct 10, 2024 •

edited

Loading