From c5b6b0a26aec1fc7195a1aa4ce75b593b57fb5f8 Mon Sep 17 00:00:00 2001
From: Colin Goutte <cgte@bk.ru>
Date: Thu, 7 Oct 2021 14:10:13 +0200
Subject: [PATCH] cover 100% main and crud

---
 papi/main.py | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/papi/main.py b/papi/main.py
index 23e5356..6a49a2e 100644
--- a/papi/main.py
+++ b/papi/main.py
@@ -110,11 +110,11 @@ def list_notification_as_text(request: Request, idsonde: str):
     notifs = notifications[idsonde]
     try:
         content = notifs[-1]
-    except IndexError:
+    except IndexError:  # pragma: no cover
         return
     try:
         recipients = sondeid2notifsemails(idsonde)
-    except KeyError:
+    except KeyError:  # pragma: no cover
         recipients = ["mail1@xxx", "mail2@xxx"]
 
     changements = content["changes"]
@@ -125,7 +125,6 @@ def list_notification_as_text(request: Request, idsonde: str):
         for k, v in d.items():
             if message.endswith(k):
                 return "color: %s;" % v
-        return ""
 
     data = {"recipients": recipients, "changements": changements, "status": status}
     return templates.TemplateResponse(
@@ -143,7 +142,7 @@ def post_sonde_error(
     idsonde: str, body: dict = Body(...), db: Session = Depends(get_db)
 ):
     if not (sonde := crud.get_sonde(db, idsonde)):
-        return
+        return  # pragma: no cover
     Notifier.error_sonde(idsonde)
     # create fake sample
     last = list(crud.get_mesure(db, sonde.sonde_id, only_last=1))[0]
@@ -217,8 +216,8 @@ def get_rapport(idsonde):
 @app.get("/sonde/{idsonde}/historique")
 def historique(idsonde, db: Session = Depends(get_db)):
     if not (sonde := crud.get_sonde(db, idsonde)):
-        return
-    mesures = sonde.mesures
+        return  # pragma: no cover
+    mesures = crud.get_mesure(db, sonde.sonde_id)
 
     for previous, present in zip(mesures, mesures[1:]):
         date = present.content["date"]
@@ -238,15 +237,15 @@ def historique(idsonde, db: Session = Depends(get_db)):
 def list_notification_as_text(
     request: Request, idsonde: str, db: Session = Depends(get_db)
 ):
+    if not (sonde := crud.get_sonde(db, idsonde)):
+        return  # pragma: no cover
+
     def coloriser(message):
         d = {"error": "red", "warning": "orange", "pending": "yellow", "ok": "green"}
         for k, v in d.items():
             if message.endswith(k):
                 return "color: %s;" % v
-        return ""
 
-    if not (sonde := crud.get_sonde(db, idsonde)):
-        return
     mesures = [x for x in crud.get_mesure(db, sonde.sonde_id, only_last=100)]
 
     res = []