Skip to content
Snippets Groups Projects

Draft: Ajoute des informations sur les réformes chargées dans le /status de l'API web

Le modèle chargé comprend : openfisca-france, openfisca-france-reforms et leximpact-survey-scenario

Les versions de ces 3 librairies seraient donc utiles. Ceci est traité par la MR !80 (merged)

?) À joindre avec une indication concernant la requête ? Par exemple pour préciser si la base est avec ou sans openfisca-france-reforms ? Indiquer ce qu'on a de chargé et sur quel commit pour les requêtes appelant le plf ?

- - - -

Question élucidée) D'un point de vue version, étrangement pour openfisca-france-reforms, la dernière version sur Pypi est inférieure à celle obtenue avec make install (1.0.288) :arrow_right: on ne publie plus openfisca-france-reforms sur pypi parce qu'il dépend de la branche openfisca-france wip-leximpact et que pypi refuse une telle dépendance.

Edited by sandcha

Merge request reports

Pipeline #10804 failed

Pipeline failed for a701c411 on add-reform-in-status

Approval is optional
Test summary results are being parsed
Merge blocked: 3 checks failed
Merge request must not be draft.
Pipeline must succeed.
Merge conflicts must be resolved.

Merge details

  • The source branch is 615 commits behind the target branch.
  • 3 commits and 1 merge commit will be added to master.
  • Source branch will be deleted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1 import pytest
2 from fastapi.testclient import TestClient # needs httpx library
3
4 from leximpact_socio_fisca_simu_etat_api.server import app
5
6
7 @pytest.fixture()
8 def test_client():
9 return TestClient(app) # set raise_server_exceptions=False ?
  • sandcha
    sandcha @sandcha started a thread on the diff
  • 5
    6
    7 @pytest.fixture()
    8 def test_client():
    9 return TestClient(app) # set raise_server_exceptions=False ?
    10
    11
    12 def test_alive(test_client):
    13 response = test_client.get("/")
    14 assert response.status_code == 200
    15 assert response.json() == {"message": "please go to /docs"}
    16
    17
    18 def test_get_status(test_client):
    19 response = test_client.get("/status")
    20 assert response.status_code == 503
  • sandcha changed the description

    changed the description

  • sandcha changed the description

    changed the description

  • Rebase en cours pour récupérer le fix des tests (erreur résolue : FAILED tests/test_irpp.py::TestIRPP::test_irpp_plf - KeyError: 'amendement').

    Edited by sandcha
  • sandcha added 21 commits

    added 21 commits

    • 9f5d74bf...44851a90 - 18 commits from branch master
    • 62e9a7b8 - Init /status web api endpoint test
    • 15ed9969 - Add missing AGREGATS_PATH is env.example as required by any call to Status object
    • a701c411 - Add version_reforms to status endpoint schema

    Compare with previous version

  • @sandcha est ce toujours d'actualité ?

  • sandcha changed the description

    changed the description

  • assigned to @sandcha

  • Please register or sign in to reply
    Loading