BPL-250 Fudge for coverage bug

See nedbat/coveragepy#1012 & nedbat/coveragepy#1082
zyntrixio · Nov 12, 2021 · 89222a8 · 89222a8
1 parent d526581
commit 89222a8
Show file tree

Hide file tree

Showing 10 changed files with 15 additions and 11 deletions.
diff --git a/.coveragerc b/.coveragerc
@@ -10,4 +10,6 @@ branch = True
 
 [report]
 fail_under = 85
-exclude_lines = pragma: no cover
+exclude_lines =
+        pragma: no cover
+        pragma: coverage bug 1012
diff --git a/app/api/deps.py b/app/api/deps.py
@@ -9,7 +9,7 @@
 from app.db.session import AsyncSessionMaker
 from app.enums import HttpErrors
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from app.models import RetailerConfig
 
 

diff --git a/app/api/endpoints/voucher.py b/app/api/endpoints/voucher.py
@@ -31,10 +31,12 @@ async def update_voucher_status(
     update_data = payload.dict(exclude_unset=True)
     voucher = await crud.get_voucher(db_session, voucher_id, retailer.id)
     incorrect_response_payload = {"display_message": "Voucher status incorrect.", "error": "STATUS_NOT_CHANGED"}
-    if voucher.status != AccountHolderVoucherStatuses.ISSUED:
+    if voucher.status != AccountHolderVoucherStatuses.ISSUED:  # pragma: coverage bug 1012
         response.status_code = status.HTTP_202_ACCEPTED
         return incorrect_response_payload
-    if voucher.expiry_date < datetime.utcnow() and voucher.expiry_date < update_data["date"]:
+    if (
+        voucher.expiry_date < datetime.utcnow() and voucher.expiry_date < update_data["date"]
+    ):  # pragma: coverage bug 1012
         if update_data["status"] == AccountHolderVoucherStatuses.CANCELLED:
             response.status_code = status.HTTP_202_ACCEPTED
             return incorrect_response_payload

diff --git a/app/core/config.py b/app/core/config.py
@@ -11,7 +11,7 @@
 
 from app.core.key_vault import KeyVault
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from pydantic.typing import CallableGenerator
 
 BASE_DIR = os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))

diff --git a/app/core/utils.py b/app/core/utils.py
@@ -9,7 +9,7 @@
 
 from app.enums import HttpErrors
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from pydantic.typing import CallableGenerator  # pragma: no cover
 
 

diff --git a/app/models/account_holder.py b/app/models/account_holder.py
@@ -9,7 +9,7 @@
 from app.db.base_class import Base, TimestampMixin, utc_timestamp_sql
 from app.enums.account_holder import AccountHolderStatuses, AccountHolderVoucherStatuses
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from .retailer import RetailerConfig  # noqa 640 pragma: no cover
 
 

diff --git a/app/models/retailer.py b/app/models/retailer.py
@@ -5,7 +5,7 @@
 
 from app.db.base_class import Base, TimestampMixin
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from .account_holder import AccountHolder  # noqa 640 pragma: no cover
 
 

diff --git a/app/schemas/retailer.py b/app/schemas/retailer.py
@@ -3,7 +3,7 @@
 
 from pydantic import EmailStr, Field, constr, create_model
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from pydantic import BaseModel  # pragma: no cover
 
 

diff --git a/app/tasks/error_handlers.py b/app/tasks/error_handlers.py
@@ -12,7 +12,7 @@
 logger = logging.getLogger("tasks")
 
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from inspect import Traceback
 
 

diff --git a/app/tasks/prometheus.py b/app/tasks/prometheus.py
@@ -2,7 +2,7 @@
 
 from prometheus_client import Counter
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from requests import RequestException, Response  # pragma: no cover