From 7107af61811a6ac1504c563766208c776c6f8f2f Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Mon, 9 Oct 2023 23:34:03 +0200 Subject: [PATCH] Fix unit tests --- app/scodoc/sco_abs_notification.py | 7 ++++++- sco_version.py | 2 +- tests/api/test_api_assiduites.py | 4 ++-- tests/unit/test_assiduites.py | 2 +- 4 files changed, 10 insertions(+), 5 deletions(-) diff --git a/app/scodoc/sco_abs_notification.py b/app/scodoc/sco_abs_notification.py index cb51e7be5..057d71913 100644 --- a/app/scodoc/sco_abs_notification.py +++ b/app/scodoc/sco_abs_notification.py @@ -34,7 +34,7 @@ Il suffit d'appeler abs_notify() après chaque ajout d'absence. import datetime from typing import Optional -from flask import g, url_for +from flask import current_app, g, url_for from flask_mail import Message from app import db @@ -55,9 +55,14 @@ def abs_notify(etudid: int, date: str | datetime.datetime): Considère le nombre d'absence dans le semestre courant (s'il n'y a pas de semestre courant, ne fait rien, car l'etudiant n'est pas inscrit au moment de l'absence!). + + NE FAIT RIEN EN MODE DEBUG. """ from app.scodoc import sco_assiduites + if current_app and current_app.config["DEBUG"]: + return + formsemestre = retreive_current_formsemestre(etudid, date) if not formsemestre: return # non inscrit a la date, pas de notification diff --git a/sco_version.py b/sco_version.py index 45da20ff0..2fb32fb90 100644 --- a/sco_version.py +++ b/sco_version.py @@ -1,7 +1,7 @@ # -*- mode: python -*- # -*- coding: utf-8 -*- -SCOVERSION = "9.6.42" +SCOVERSION = "9.6.43" SCONAME = "ScoDoc" diff --git a/tests/api/test_api_assiduites.py b/tests/api/test_api_assiduites.py index 59b82d38a..9c361c231 100644 --- a/tests/api/test_api_assiduites.py +++ b/tests/api/test_api_assiduites.py @@ -226,7 +226,7 @@ def test_route_create(api_admin_headers): assert len(res["errors"]) == 1 assert ( res["errors"][0]["message"] - == "Duplication des assiduités (la période rentrée rentre en conflit avec une assiduité enregistrée)" + == "Duplication: la période rentre en conflit avec une plage enregistrée" ) res = POST_JSON( @@ -269,7 +269,7 @@ def test_route_create(api_admin_headers): assert ( res["errors"][0]["message"] - == "Duplication des assiduités (la période rentrée rentre en conflit avec une assiduité enregistrée)" + == "Duplication: la période rentre en conflit avec une plage enregistrée" ) assert res["errors"][1]["message"] == "param 'moduleimpl_id': invalide" assert res["errors"][2]["message"] == "param 'etat': invalide" diff --git a/tests/unit/test_assiduites.py b/tests/unit/test_assiduites.py index 1fceaa0d8..3bc5c51a3 100644 --- a/tests/unit/test_assiduites.py +++ b/tests/unit/test_assiduites.py @@ -877,7 +877,7 @@ def ajouter_assiduites( except ScoValueError as excp: assert ( excp.args[0] - == "Duplication des assiduités (la période rentrée rentre en conflit avec une assiduité enregistrée)" + == "Duplication: la période rentre en conflit avec une plage enregistrée" ) try: Assiduite.create_assiduite(