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

[Vigilance Area] Rendre les données de test plus consistantes #1632

Merged
merged 3 commits into from
Aug 19, 2024

Conversation

maximeperraultdev
Copy link
Collaborator

@maximeperraultdev maximeperraultdev commented Aug 13, 2024

Point d'attention si on est autour du 31/12 -> 01/01 les tests peuvent ne pas passer mais je n'ai pas de meilleurs solutions à proposer


  • Tests E2E (Cypress)

@maximeperraultdev maximeperraultdev changed the title fix: update vigilance area e2e test [Vigilance Area] Rendre les données de test plus consistantes Aug 14, 2024
@maximeperraultdev maximeperraultdev force-pushed the fix/e2e/vigilance_area_period branch 2 times, most recently from 88abc13 to f00b97f Compare August 14, 2024 13:11
THEN date_trunc('year', CURRENT_DATE) + INTERVAL '6 month' -- Pour éviter Q2 et les 3 premiers mois
WHEN EXTRACT(MONTH FROM CURRENT_DATE) <= 9
THEN date_trunc('year', CURRENT_DATE) + INTERVAL '3 month' -- Pour éviter Q3 et les 3 premiers mois
ELSE date_trunc('year', CURRENT_DATE) -- Pour éviter Q4
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dans ce cas (mois courant entre octobre et décembre) la date date_within_year_not_in_quarter_nor_3_months sera le 1er janvier de l'année en cours, ce qui ne correspond pas à une "Date dans l'année courante, mais pas dans le trimestre actuel ni dans les trois premiers mois de l'année" a priori ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est pas les trois premiers mois de l'année mais les trois prochains mois

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai noté juste un probleme quand on sera le 31 decembre / 1 er janvier mais on ne devrait rien merger à ce moment là de l'année normalement ^^

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay !
Peut-être plus simplement on pourrait ajouter 6 mois et boucler quand ça dépasse 12 avec un modulo % :

date_within_year_not_in_quarter_nor_3_months = SELECT make_date(
    EXTRACT(year FROM CURRENT_DATE)::INTEGER,
    (EXTRACT(month FROM CURRENT_DATE) + 6 - 1)::INTEGER % 12 + 1,
    1
)

@maximeperraultdev maximeperraultdev merged commit f44e731 into main Aug 19, 2024
21 checks passed
@maximeperraultdev maximeperraultdev deleted the fix/e2e/vigilance_area_period branch August 19, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants