From f9129323085791c79a5f3652eb474428ff8b5d33 Mon Sep 17 00:00:00 2001 From: tlkahn Date: Mon, 20 Jul 2026 11:12:14 +0700 Subject: [PATCH] fix(test): isolate HF hub cache in test_registry_discover (#101) unsetenv(MLXD_HF_HUB_DIR) let reg_hf_hub_dir() fall through to the real $HOME/.cache/huggingface/hub, inflating discovery counts. Point the four fixture-only tests at /nonexistent instead. --- tests/test_registry_discover.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/test_registry_discover.c b/tests/test_registry_discover.c index f7da111..922d5a0 100644 --- a/tests/test_registry_discover.c +++ b/tests/test_registry_discover.c @@ -9,7 +9,7 @@ static void test_discover_finds_models(void) { setenv("MLXD_CACHE_DIR", FIXTURE_CACHE, 1); - unsetenv("MLXD_HF_HUB_DIR"); + setenv("MLXD_HF_HUB_DIR", "/nonexistent", 1); int count = -1; registry_model_info_t *models = registry_discover(&count); assert(count == 2); @@ -35,7 +35,7 @@ static void test_discover_finds_models(void) { static void test_discover_skips_no_config(void) { setenv("MLXD_CACHE_DIR", FIXTURE_CACHE, 1); - unsetenv("MLXD_HF_HUB_DIR"); + setenv("MLXD_HF_HUB_DIR", "/nonexistent", 1); int count = -1; registry_model_info_t *models = registry_discover(&count); for (int i = 0; i < count; i++) { @@ -47,7 +47,7 @@ static void test_discover_skips_no_config(void) { static void test_discover_skips_part_only(void) { setenv("MLXD_CACHE_DIR", FIXTURE_CACHE, 1); - unsetenv("MLXD_HF_HUB_DIR"); + setenv("MLXD_HF_HUB_DIR", "/nonexistent", 1); int count = -1; registry_model_info_t *models = registry_discover(&count); for (int i = 0; i < count; i++) { @@ -59,7 +59,7 @@ static void test_discover_skips_part_only(void) { static void test_discover_empty_cache(void) { setenv("MLXD_CACHE_DIR", "/nonexistent/path", 1); - unsetenv("MLXD_HF_HUB_DIR"); + setenv("MLXD_HF_HUB_DIR", "/nonexistent", 1); int count = -1; registry_model_info_t *models = registry_discover(&count); assert(count == 0);