Skip to content

Commit fde105b

Browse files
cursoragentxzrderek
andcommitted
Fix: Ensure sqlite logger is enabled in CI tests
Co-authored-by: derek <derek@fireworks.ai>
1 parent 57f259f commit fde105b

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

tests/pytest/test_pytest_ensure_logging.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,14 @@ async def test_ensure_logging(monkeypatch):
66
"""
77
Ensure that default SQLITE logger gets called by mocking the storage and checking that the storage is called.
88
"""
9+
# Ensure default sqlite logger is enabled in CI environments and reset lazy cache
10+
monkeypatch.setenv("DISABLE_EP_SQLITE_LOG", "0")
11+
from eval_protocol.dataset_logger import default_logger as _dl
12+
# Reset the cached underlying logger so it re-initializes with the current env
13+
try:
14+
monkeypatch.setattr(_dl, "_logger", None, raising=False)
15+
except Exception:
16+
pass
917
# Mock the SqliteEvaluationRowStore to track calls
1018
mock_store = Mock()
1119
mock_store.upsert_row = Mock()

0 commit comments

Comments
 (0)