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

chore(api): poc eligibility add tests #4317

Merged
merged 16 commits into from
Oct 14, 2024
Merged

chore(api): poc eligibility add tests #4317

merged 16 commits into from
Oct 14, 2024

Conversation

achorein
Copy link
Collaborator

@achorein achorein commented Sep 13, 2024

Description

Améliore la maintenabilité et ajout des tests d'intégration sur les différents cas du POC d'eligibilité par département #4291

Todo

  • Ajout TI sur /cohort-session/eligibility/2023
  • Passage de utils/cohort en TS

Cas spécifique

  • résidant+scolarisé dans dép X : si le dép X a un séjour, vérifier que le jeune peut candidater
  • résidant+scolarisé dans dép X : si le dép X n’a PAS de séjour, vérifier que le jeune peut PAS candidater
  • résidant dans dép Y + scolarisé dans dép X : si le dép X a un séjour, vérifier que le jeune peut candidater
  • résidant dans dép Y + scolarisé dans dép X : si le dép X n’a PAS de séjour, alors vérifier que le jeune ne peut PAS candidaté
  • ⚠️résidant dans dép Y + scolarisé dans dép X + demande d’affectation intradépartementale : si le dép Y a un séjour + si le dép X n’a PAS de séjour, alors le jeune peut candidater
  • ⚠️résidant dans dép Y + scolarisé dans dép X + demande d’affectation intradépartementale : si le dép Y n’a PAS de séjour + si le dép X a un séjour, alors le jeune peut ne peut PAS candidater (<<< ça c’est dans l’idéal MAIS avec la façon dont les écrans d’instruction sont agencés, un jeune dont le département de scolarité n’offre pas de séjour mais qui souhaite être affecté en intradép et son dép de résidence a un séjour, ALORS il est coincé et verra pas l’option)

Dates de naissance

  • 15 ans à J-1 (❌)
  • 15 ans le jour J (✅)
  • 18 ans au cours du jour du séjour (❌) => pas géré côté api ?
  • 18 ans le surlendemain du séjour (✅)
  • 18 au dernier jour du séjour (❌) => pas géré côté api ?
  • 16 ans (✅)

Niveaux scolaires

  • CAP 1ère année ✅
  • CAP 2ème année ✅
  • 3ème✅
  • 4ème✅
  • Seconde GT✅
  • Seconde pro✅
  • Première GT✅
  • Première Pro✅
  • Terminale GT✅
  • Terminale Pro✅
  • Non scolarisés✅
  • Scolarisé autre✅

Eligibilité - Zones géographiques

  • Cas d’une région qui ne participe pas à 1 séjour /4
  • Cas d’une région qui ne participe pas en totalité à un séjour (seul un département participe)
  • Cas d’une région qui participe à tous les séjours

Paramètres dynamiques

  • Ouverture des inscriptions → Vérifier que les dates et heures sont bien correctes sur toutes les cohortes
  • Modification du dossier par le volontaire

Ticket / Issue

https://www.notion.so/jeveuxaider/Tests-op-rations-Eligibilit-ge-niveau-scolaire-et-zone-g-ographique-99285155b36f4295b86282ced93356b0

Testing instructions

@achorein achorein self-assigned this Sep 13, 2024
@achorein achorein force-pushed the chore/test-eligibility branch 2 times, most recently from 96762fd to ddbd9b0 Compare September 13, 2024 14:11
@achorein achorein added WIP Work In Progress R&R labels Sep 13, 2024
Base automatically changed from back-eligibility to main September 16, 2024 09:08
@achorein achorein force-pushed the chore/test-eligibility branch 4 times, most recently from 3ca2194 to 7eb6ddc Compare September 16, 2024 15:15
@achorein achorein marked this pull request as ready for review September 16, 2024 15:43
@achorein achorein force-pushed the chore/test-eligibility branch 7 times, most recently from 52e6336 to 6581c91 Compare September 20, 2024 07:09
api/src/__tests__/cohort-session.test.ts Show resolved Hide resolved
api/src/__tests__/cohort-session.test.ts Show resolved Hide resolved
api/src/utils/cohort.ts Show resolved Hide resolved
api/src/utils/cohort.ts Show resolved Hide resolved
api/src/__tests__/cohort-session.test.ts Show resolved Hide resolved
api/src/controllers/cohort-session.ts Outdated Show resolved Hide resolved
api/src/utils/cohort.ts Show resolved Hide resolved
@achorein
Copy link
Collaborator Author

@hlecourt ok pour toi ?

Copy link

codeclimate bot commented Oct 14, 2024

Code Climate has analyzed commit ed90d09 and detected 8 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2
Duplication 6

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 45.6% (0.3% change).

View more on Code Climate.

@achorein achorein merged commit 9489d02 into main Oct 14, 2024
12 checks passed
@achorein achorein deleted the chore/test-eligibility branch October 14, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R&R WIP Work In Progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants