From fa2709b38c6291f25b4fdc683e10f211424d4e41 Mon Sep 17 00:00:00 2001 From: Colin Maudry Date: Fri, 12 Jun 2026 13:50:57 +0200 Subject: [PATCH] =?UTF-8?q?fix:=20re-v=C3=A9rifier=20db=5Fpath.exists()=20?= =?UTF-8?q?dans=20le=20handler=20et=20assertion=20return=20value=20(#78)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Sonnet 4.6 --- src/db.py | 2 +- tests/test_db.py | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/db.py b/src/db.py index 9e3f24f..c4d0892 100644 --- a/src/db.py +++ b/src/db.py @@ -128,7 +128,7 @@ def _ensure_database() -> Path: else: logger.debug("Base de données déjà disponible et à jour.") except Exception as e: - if db_exists: + if db_exists and db_path.exists(): logger.error( f"Bootstrap données KO ({e}). " f"Réutilisation du DuckDB existant : {db_path}" diff --git a/tests/test_db.py b/tests/test_db.py index 865da05..56dee54 100644 --- a/tests/test_db.py +++ b/tests/test_db.py @@ -340,7 +340,8 @@ def test_ensure_database_reuses_db_when_build_raises(tmp_path, monkeypatch): monkeypatch.setenv("DATA_FILE_PARQUET_PATH", "http://unreachable") monkeypatch.setattr(db, "should_rebuild", lambda *a, **k: True) monkeypatch.setattr(db, "build_database", _raise) - db._ensure_database() # ne doit pas lever + result = db._ensure_database() # ne doit pas lever + assert result == dbf assert dbf.read_bytes() == b"existing"