diff --git a/app/scodoc/sco_formsemestre.py b/app/scodoc/sco_formsemestre.py index fc49f4330..38b0e9cb9 100644 --- a/app/scodoc/sco_formsemestre.py +++ b/app/scodoc/sco_formsemestre.py @@ -493,7 +493,7 @@ def sem_in_semestre_scolaire( jour_pivot_periode, mois_pivot_periode, ) - if periode == 0 or None: + if periode is None or periode == 0: return sem_annee == year else: return sem_annee == year and sem_periode == periode diff --git a/tests/unit/test_periode.py b/tests/unit/test_periode.py index 91c60c916..fbc90d3c3 100644 --- a/tests/unit/test_periode.py +++ b/tests/unit/test_periode.py @@ -7,7 +7,7 @@ Utiliser comme: """ import datetime -from app.scodoc.sco_formsemestre import get_periode +from app.scodoc.sco_formsemestre import get_periode, sem_in_semestre_scolaire """ Calcule la session associée à un formsemestre sous la forme (année, période) année: première année de l'année scolaire @@ -32,30 +32,86 @@ Début FormSemestre pivot_année pivot_periode Résultat """ -def test_general(): - # A with default +def test_default(): + # with default assert (2021, 2) == get_periode(datetime.datetime(2022, 1, 1)) - # A assert (2021, 2) == get_periode(datetime.datetime(2022, 1, 1), 1, 8, 1, 12) - # automne nord + + +def test_automne_nord(): assert (2022, 1) == get_periode(datetime.datetime(2022, 9, 1)) - # noel nord + + +def test_noel_nord(): assert (2022, 2) == get_periode(datetime.datetime(2022, 12, 15)) - # été nord + + +def test_étét_nord(): assert (2021, 2) == get_periode(datetime.datetime(2022, 7, 30)) - # printemps sud + + +def test_printemps_sud(): assert (2022, 1) == get_periode(datetime.datetime(2022, 1, 1), 1, 1, 1, 8) - # automne sud + + +def test_automne_sud(): assert (2022, 2) == get_periode(datetime.datetime(2022, 8, 2), 1, 1, 1, 8) - # noel sud + + +def test_noel_sud(): assert (2022, 2) == get_periode(datetime.datetime(2022, 12, 30), 1, 1, 1, 8) - # été sud + + +def test_été_sud(): assert (2022, 1) == get_periode(datetime.datetime(2022, 7, 30), 1, 1, 1, 8) - # nouvel an sud + + +def test_nouvel_an_sud(): assert (2021, 2) == get_periode(datetime.datetime(2022, 1, 1), 3, 1, 1, 8) - # nouvel an pp < pa + + +def test_nouvel_an_special_pp_before_pa(): assert (2023, 1) == get_periode(datetime.datetime(2024, 1, 10), 1, 8, 1, 2) - # été pivot pp