From ddfd94b0bab41fd01590b32c915c094dea81f42f Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Tue, 25 Jun 2024 22:25:00 +0200 Subject: [PATCH] =?UTF-8?q?Revert=20"Merge=20pull=20request=20'Assiduit?= =?UTF-8?q?=C3=A9=20:=20routes=20api=20assiduites=20evaluations'=20(#941)?= =?UTF-8?q?=20from=20iziram/ScoDoc:assiduites=5Fevals=20into=20master"?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit 3cfe3c51743f243ea89f6a2e1ad93dd724ee0f07, reversing changes made to b40d33feaad1f25e084060011081a5301238e446. --- app/api/assiduites.py | 89 --------------- app/scodoc/sco_assiduites.py | 88 --------------- tests/api/test_api_assiduites.py | 42 ------- tests/unit/test_assiduites.py | 106 ------------------ .../fakedatabase/create_test_api_database.py | 5 +- 5 files changed, 2 insertions(+), 328 deletions(-) diff --git a/app/api/assiduites.py b/app/api/assiduites.py index 503130eda..7d46ddfe4 100644 --- a/app/api/assiduites.py +++ b/app/api/assiduites.py @@ -12,7 +12,6 @@ from flask_json import as_json from flask_login import current_user, login_required from flask_sqlalchemy.query import Query from sqlalchemy.orm.exc import ObjectDeletedError -from werkzeug.exceptions import HTTPException from app import db, log, set_sco_dept import app.scodoc.sco_assiduites as scass @@ -22,7 +21,6 @@ from app.api import api_web_bp, get_model_api_object, tools from app.decorators import permission_required, scodoc from app.models import ( Assiduite, - Evaluation, FormSemestre, Identite, ModuleImpl, @@ -284,7 +282,6 @@ def assiduites(etudid: int = None, nip=None, ine=None, with_query: bool = False) 404, message="étudiant inconnu", ) - # Récupération des assiduités de l'étudiant assiduites_query: Query = etud.assiduites @@ -307,92 +304,6 @@ def assiduites(etudid: int = None, nip=None, ine=None, with_query: bool = False) return data_set -@bp.route("/assiduites//evaluations") -@api_web_bp.route("/assiduites//evaluations") -# etudid -@bp.route("/assiduites/etudid//evaluations") -@api_web_bp.route("/assiduites/etudid//evaluations") -# ine -@bp.route("/assiduites/ine//evaluations") -@api_web_bp.route("/assiduites/ine//evaluations") -# nip -@bp.route("/assiduites/nip//evaluations") -@api_web_bp.route("/assiduites/nip//evaluations") -@login_required -@scodoc -@as_json -@permission_required(Permission.ScoView) -def assiduites_evaluations(etudid: int = None, nip=None, ine=None): - """ - Retourne la liste de toutes les évaluations de l'étudiant - Pour chaque évaluation, retourne la liste des objets assiduités - sur la plage de l'évaluation - - Présentation du retour : - [ - { - "evaluation_id": 1234, - "assiduites": [ - { - "assiduite_id":1234, - ... - }, - ] - } - ] - - """ - # Récupération de l'étudiant - etud: Identite = tools.get_etud(etudid, nip, ine) - if etud is None: - return json_error( - 404, - message="étudiant inconnu", - ) - - # Récupération des évaluations et des assidiutés - etud_evaluations_assiduites: list[dict] = scass.get_etud_evaluations_assiduites( - etud - ) - - return etud_evaluations_assiduites - - -@api_web_bp.route("/evaluation//assiduites") -@bp.route("/evaluation//assiduites") -@login_required -@scodoc -@as_json -@permission_required(Permission.ScoView) -def evaluation_assiduites(evaluation_id): - """ - Retourne les objets assiduités de chaque étudiant sur la plage de l'évaluation - Présentation du retour : - { - "" : [ - { - "assiduite_id":1234, - ... - }, - ] - } - """ - # Récupération de l'évaluation - try: - evaluation: Evaluation = Evaluation.get_evaluation(evaluation_id) - except HTTPException: - return json_error(404, "L'évaluation n'existe pas") - - evaluation_assiduites_par_etudid: dict[int, list[Assiduite]] = {} - for assi in scass.get_evaluation_assiduites(evaluation): - etudid: str = str(assi.etudid) - etud_assiduites = evaluation_assiduites_par_etudid.get(etudid, []) - etud_assiduites.append(assi.to_dict(format_api=True)) - evaluation_assiduites_par_etudid[etudid] = etud_assiduites - - return evaluation_assiduites_par_etudid - - @bp.route("/assiduites/group/query", defaults={"with_query": True}) @api_web_bp.route("/assiduites/group/query", defaults={"with_query": True}) @login_required diff --git a/app/scodoc/sco_assiduites.py b/app/scodoc/sco_assiduites.py index 6626c9037..ff2088fc1 100644 --- a/app/scodoc/sco_assiduites.py +++ b/app/scodoc/sco_assiduites.py @@ -10,7 +10,6 @@ from flask_sqlalchemy.query import Query from app import log, db, set_sco_dept from app.models import ( - Evaluation, Identite, FormSemestre, FormSemestreInscription, @@ -732,93 +731,6 @@ def create_absence_billet( return calculator.to_dict()["demi"] -def get_evaluation_assiduites(evaluation: Evaluation) -> Query: - """ - Renvoie une query d'assiduité en fonction des étudiants inscrits à l'évaluation - et de la date de l'évaluation. - - Attention : Si l'évaluation n'a pas de date, renvoie une liste vide - """ - - # Evaluation sans date - if evaluation.date_debut is None: - return [] - - # Récupération des étudiants inscrits à l'évaluation - etuds: Query = Identite.query.join( - ModuleImplInscription, Identite.id == ModuleImplInscription.etudid - ).filter(ModuleImplInscription.moduleimpl_id == evaluation.moduleimpl_id) - - etudids: list[int] = [etud.id for etud in etuds] - - # Récupération des assiduités des étudiants inscrits à l'évaluation - date_debut: datetime = evaluation.date_debut - date_fin: datetime - - if evaluation.date_fin is not None: - date_fin = evaluation.date_fin - else: - # On met à la fin de la journée de date_debut - date_fin = datetime.combine(date_debut.date(), time.max) - - # Filtrage par rapport à la plage de l'évaluation - assiduites: Query = Assiduite.query.filter( - Assiduite.date_debut >= date_debut, - Assiduite.date_fin <= date_fin, - Assiduite.etudid.in_(etudids), - ) - - return assiduites - - -def get_etud_evaluations_assiduites(etud: Identite) -> list[dict]: - """ - Retourne la liste des évaluations d'un étudiant. Pour chaque évaluation, - retourne la liste des assiduités concernant la plage de l'évaluation. - """ - - etud_evaluations_assiduites: list[dict] = [] - - # On récupère les moduleimpls puis les évaluations liés aux moduleimpls - modsimpl_ids: list[int] = [ - modimp_inscr.moduleimpl_id - for modimp_inscr in ModuleImplInscription.query.filter_by(etudid=etud.id) - ] - evaluations: Query = Evaluation.query.filter( - Evaluation.moduleimpl_id.in_(modsimpl_ids) - ) - # Pour chaque évaluation, on récupère l'assiduité de l'étudiant sur la plage - # de l'évaluation - - for evaluation in evaluations: - eval_assis: dict = {"evaluation_id": evaluation.id, "assiduites": []} - # Pas d'assiduités si pas de date - if evaluation.date_debut is not None: - date_debut: datetime = evaluation.date_debut - date_fin: datetime - - if evaluation.date_fin is not None: - date_fin = evaluation.date_fin - else: - # On met à la fin de la journée de date_debut - date_fin = datetime.combine(date_debut.date(), time.max) - - # Filtrage par rapport à la plage de l'évaluation - assiduites: Query = etud.assiduites.filter( - Assiduite.date_debut >= date_debut, - Assiduite.date_fin <= date_fin, - ) - # On récupère les assiduités et on met à jour le dictionnaire - eval_assis["assiduites"] = [ - assi.to_dict(format_api=True) for assi in assiduites - ] - - # On ajoute le dictionnaire à la liste des évaluations - etud_evaluations_assiduites.append(eval_assis) - - return etud_evaluations_assiduites - - # Gestion du cache def get_assiduites_count(etudid: int, sem: dict) -> tuple[int, int, int]: """Les comptes d'absences de cet étudiant dans ce semestre: diff --git a/tests/api/test_api_assiduites.py b/tests/api/test_api_assiduites.py index b8f3f7e41..824fb51a7 100644 --- a/tests/api/test_api_assiduites.py +++ b/tests/api/test_api_assiduites.py @@ -43,8 +43,6 @@ ASSIDUITES_FIELDS = { "external_data": dict, } -ASSIDUITES_EVALUATIONS_FIELDS = {"evaluation_id": int, "assiduites": list} - CREATE_FIELD = {"assiduite_id": int} BATCH_FIELD = {"errors": list, "success": list} @@ -141,46 +139,6 @@ def test_route_assiduites(api_headers): check_failure_get(f"/assiduites/{FAUX}/query?", api_headers) -def test_route_assiduites_evaluations(api_headers): - """test de la route /assiduites//evaluations""" - - # Bon fonctionnement - - data = GET( - path=f"/assiduites/{ETUDID}/evaluations", headers=api_headers, dept=DEPT_ACRONYM - ) - assert isinstance(data, list) - for evals in data: - check_fields(evals, ASSIDUITES_FIELDS) - for assi in evals["assiduites"]: - check_fields(assi, ASSIDUITES_FIELDS) - - # Mauvais fonctionnement - check_failure_get(f"/assiduites/{FAUX}/evaluations", api_headers) - - -def test_route_evaluations_assiduites(api_headers): - """test de la route /evaluation//assiduites""" - - # Bon fonctionnement - evaluation_id = 1 - data = GET( - path=f"/evaluation/{evaluation_id}/assiduites", - headers=api_headers, - dept=DEPT_ACRONYM, - ) - assert isinstance(data, dict) - for key, val in data.items(): - assert isinstance(key, str), "Erreur les clés ne sont pas des strings" - assert isinstance(val, list), "Erreur, les valeurs ne sont pas des listes" - - for assi in val: - check_fields(assi, ASSIDUITES_FIELDS) - - # Mauvais fonctionnement - check_failure_get(f"/evaluation/{FAUX}/assiduites", api_headers) - - def test_route_formsemestre_assiduites(api_headers): """test de la route /assiduites/formsemestre/""" diff --git a/tests/unit/test_assiduites.py b/tests/unit/test_assiduites.py index 1ca1579cf..e6e057c87 100644 --- a/tests/unit/test_assiduites.py +++ b/tests/unit/test_assiduites.py @@ -7,17 +7,13 @@ ses fonctions liées Ecrit par HARTMANN Matthias (en s'inspirant de tests.unit.test_abs_count.py par Fares Amer ) """ - import pytest -from flask_sqlalchemy.query import Query - import app.scodoc.sco_assiduites as scass import app.scodoc.sco_utils as scu from app import db, log from app.models import ( Assiduite, - Evaluation, FormSemestre, Identite, Justificatif, @@ -1119,7 +1115,6 @@ def _setup_fake_db( "formsemestres": formsemestres, "etuds": etuds, "etud_faux": etud_faux, - "g_fake": g_fake, } @@ -1735,104 +1730,3 @@ def test_cache_assiduites(test_client): ) == (1, 1, 2) # Deuxième semestre 2nj / 1j / 3t (Identique car cache et non modifié) assert scass.get_assiduites_count(etud.id, formsemestre2.to_dict()) == (2, 1, 3) - - -def test_recuperation_evaluations(test_client): - """ - Vérification du bon fonctionnement de la récupération des assiduités d'une évaluation - et de la récupération de l'assiduité aux évaluations d'un étudiant - """ - - data = _setup_fake_db( - [("2024-01-01", "2024-06-30")], - 1, - 1, - ) - moduleimpl: ModuleImpl = data["moduleimpls"][0] - etud: Identite = data["etuds"][0] - - # Création d'assiduités pour tester les évaluations - assiduites_dates = [ - "2024-01-01", - "2024-01-02", - "2024-01-03", - ] - assiduites = [] - for assi in assiduites_dates: - assiduites.append( - Assiduite.create_assiduite( - etud=etud, - date_debut=scu.is_iso_formated(assi + "T10:00", True), - date_fin=scu.is_iso_formated(assi + "T12:00", True), - etat=scu.EtatAssiduite.ABSENT, - ) - ) - - # On génère une évaluation sans date - # elle devrait renvoyer une liste vide - evaluation_1: Evaluation = Evaluation.create( - moduleimpl=moduleimpl, - ) - assert scass.get_evaluation_assiduites(evaluation_1) == [] - - # On génère une évaluation le 01/01/24 de 10h à 12h - evaluation_2: Evaluation = Evaluation.create( - moduleimpl=moduleimpl, - date_debut=scu.is_iso_formated("2024-01-01T10:00", True), - date_fin=scu.is_iso_formated("2024-01-01T12:00", True), - ) - query: Query = scass.get_evaluation_assiduites(evaluation_2) - assert isinstance(query, Query), "Erreur, la fonction ne renvoie pas une Query" - - # On vérifie le contenu de la query - # Cette query devrait contenir que la première assiduité - assert ( - query.count() == 1 - ), "Erreur, la query ne contient pas le bon nombre d'assiduités" - assert ( - query.first() == assiduites[0] - ), "Erreur, la query ne contient pas la bonne assiduité" - - # On génère une évaluation du 02/01/24 au 03/01/24 - evaluation_3: Evaluation = Evaluation.create( - moduleimpl=moduleimpl, - date_debut=scu.is_iso_formated("2024-01-02T10:00", True), - date_fin=scu.is_iso_formated("2024-01-03T12:00", True), - ) - - query: Query = scass.get_evaluation_assiduites(evaluation_3) - assert isinstance(query, Query), "Erreur, la fonction ne renvoie pas une Query" - - # On vérifie le contenu de la query - # On devrait avoir les deux dernières assiduités - assert ( - query.count() == 2 - ), "Erreur, la query ne contient pas le bon nombre d'assiduités" - assert ( - query.all() == assiduites[1:] - ), "Erreur, la query ne contient pas les bonnes assiduités" - - # Test de la récupération des assiduités aux évaluations - evaluations_assiduites = scass.get_etud_evaluations_assiduites(etud) - - assert isinstance( - evaluations_assiduites, list - ), "Erreur, la fonction ne renvoie pas une liste" - assert len(evaluations_assiduites) == 3, "Erreur, le nombre d'évaluations est faux" - assert all( - isinstance(e, dict) for e in evaluations_assiduites - ), "Erreur, les éléments de la liste ne sont pas des dictionnaires" - assert all( - "evaluation_id" in e and "assiduites" in e for e in evaluations_assiduites - ), "Erreur, les dictionnaires ne contiennent pas les bonnes clés" - assert ( - evaluations_assiduites[0]["assiduites"] == [] - ), "Erreur, la première évaluation ne devrait pas contenir d'assiduités" - - assert evaluations_assiduites[1]["assiduites"][0] == assiduites[0].to_dict( - format_api=True - ), "Erreur, la deuxième évaluation n'est pas bonne" - - assert evaluations_assiduites[2]["assiduites"] == [ - assi.to_dict(format_api=True) for assi in assiduites[1:] - ], "Erreur, la troisième évaluation n'est pas bonne" diff --git a/tools/fakedatabase/create_test_api_database.py b/tools/fakedatabase/create_test_api_database.py index adeaa7c33..cd5182bee 100644 --- a/tools/fakedatabase/create_test_api_database.py +++ b/tools/fakedatabase/create_test_api_database.py @@ -395,13 +395,12 @@ def ajouter_assiduites_justificatifs(formsemestre: FormSemestre): MODS.append(None) for etud in formsemestre.etuds: - # Se base sur la date des évaluations base_date = datetime.datetime( - 2022, 3, [1, 8, 15, 22, 29][random.randint(0, 4)], 8, 0, 0 + 2021, 9, [6, 13, 20, 27][random.randint(0, 3)], 8, 0, 0 ) base_date = localize_datetime(base_date) - for i in range(random.randint(1, 4)): + for i in range(random.randint(1, 5)): etat = random.randint(0, 2) moduleimpl = random.choice(MODS) deb_date = base_date + datetime.timedelta(days=i)