Skip to content

Commit 1102b45

Browse files
Fix github actions test failure (#198)
* Fix: Ensure sqlite logger is enabled in CI tests Co-authored-by: derek <derek@fireworks.ai> * formatting --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
1 parent 57f259f commit 1102b45

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

tests/pytest/test_pytest_ensure_logging.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,15 @@ 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+
13+
# Reset the cached underlying logger so it re-initializes with the current env
14+
try:
15+
monkeypatch.setattr(_dl, "_logger", None, raising=False)
16+
except Exception:
17+
pass
918
# Mock the SqliteEvaluationRowStore to track calls
1019
mock_store = Mock()
1120
mock_store.upsert_row = Mock()

0 commit comments

Comments
 (0)