From 32e5b5b097c50d0b1d8a3dbcf93163aabf1ee8fb Mon Sep 17 00:00:00 2001 From: Ben Sima Date: Wed, 26 Nov 2025 09:40:59 -0500 Subject: Fix test database isolation: TASK_TEST_MODE takes precedence Tests were polluting production database because TASK_DB_PATH took precedence over TASK_TEST_MODE. Now test mode always uses .tasks/tasks-test.db regardless of TASK_DB_PATH setting. --- Omni/Task/Core.hs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'Omni/Task/Core.hs') diff --git a/Omni/Task/Core.hs b/Omni/Task/Core.hs index 4f7a3d3..2110170 100644 --- a/Omni/Task/Core.hs +++ b/Omni/Task/Core.hs @@ -237,9 +237,9 @@ getTasksDbPath :: IO FilePath getTasksDbPath = do customPath <- lookupEnv "TASK_DB_PATH" testMode <- lookupEnv "TASK_TEST_MODE" - let path = case (customPath, testMode) of - (Just p, _) -> p -- Custom path wins (even in test mode, to allow specific test DBs) - (_, Just "1") -> ".tasks/tasks-test.db" + let path = case (testMode, customPath) of + (Just "1", _) -> ".tasks/tasks-test.db" -- Test mode always uses test db + (_, Just p) -> p -- Custom path for production _ -> ".tasks/tasks.db" pure path -- cgit v1.2.3