add legifrance credentials to ci pytest runs (#87)

* add legifrance credentials to ci pytest runs

* improve _get_legifrance_credentials

.github/workflows/check.yml

@@ -31,3 +31,5 @@ jobs:
         run: tox r --skip-pkg-install
         env:
           PYTEST_ADDOPTS: "-vv --durations=10"
+          CATLEG_LF_CLIENT_ID: ${{ secrets.CATLEG_LF_CLIENT_ID }}
+          CATLEG_LF_CLIENT_SECRET: ${{ secrets.CATLEG_LF_CLIENT_SECRET }}

src/catleg/query.py

@@ -7,6 +7,7 @@
 
 import functools
 import logging
+import re
 from collections.abc import Iterable
 from datetime import date, datetime, timedelta, timezone
 from typing import Protocol
@@ -28,6 +29,10 @@ def _lf_timestamp_to_datetime(ts):
 # Legifrance uses 2999-01-01 to mark a non-expired or non-expiring text
 END_OF_TIME = _lf_timestamp_to_datetime(32472144000000)
 
+LF_TOKEN_REGEX = re.compile(
+    r"[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}"
+)
+
 
 class Backend(Protocol):
     async def article(self, id_or_url: str) -> Article | None:
@@ -208,6 +213,13 @@ def to_markdown(self) -> str:
 def _get_legifrance_credentials(*, raise_if_missing=True):
     client_id = settings.get("lf_client_id")
     client_secret = settings.get("lf_client_secret")
+
+    if not LF_TOKEN_REGEX.match(client_id):
+        client_id = None
+
+    if not LF_TOKEN_REGEX.match(client_secret):
+        client_secret = None
+
     if raise_if_missing:
         if (client_id is None) or (client_secret is None):
             raise ValueError(

tox.ini

@@ -11,6 +11,9 @@ deps =
     pytest-sugar
 commands =
     pytest {posargs:tests}
+passenv =
+    CATLEG_LF_CLIENT_SECRET
+    CATLEG_LF_CLIENT_ID
 
 [testenv:type]
 description = run type checks