Skip to content

Commit f7bdbb5

Browse files
committed
Fix performance tests: mock sqlit.config not sqlit.app
The tests were mocking load_settings/save_settings in sqlit.app, but these functions are actually in sqlit.config.
1 parent 903b2af commit f7bdbb5

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,7 @@ jobs:
6868
--ignore=tests/test_turso.py \
6969
--ignore=tests/test_firebird.py \
7070
--ignore=tests/test_ssh.py \
71-
--ignore=tests/test_clickhouse.py \
72-
--ignore=tests/performance/
71+
--ignore=tests/test_clickhouse.py
7372
7473
test-sqlite:
7574
runs-on: ubuntu-latest

tests/performance/test_large_dataset_rendering.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ def mock_app_context():
102102
mock_settings = MockSettingsStore({"theme": "tokyo-night"})
103103

104104
return patch.multiple(
105-
"sqlit.app",
105+
"sqlit.config",
106106
load_connections=mock_connections.load_all,
107107
load_settings=mock_settings.load_all,
108108
save_settings=mock_settings.save_all,

0 commit comments

Comments
 (0)