Skip to content

Commit ffd6883

Browse files
committed
fix(medcare_bridge): drop the test temp dir instead of mem::forget
CodeRabbit (PR #582): the test helper leaked a temp dir via std::mem::forget(tmp) on every call. hydrate_once_sync parses the TTL into the in-memory OntologyRegistry, so the dir isn't needed afterward — let it drop. medcare_bridge tests 8/8 green. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EYvNjD8M8LMNYbRy3gq2FP
1 parent 4417a6f commit ffd6883

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

crates/lance-graph-ontology/src/bridges/medcare_bridge.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,10 @@ ogit.Healthcare:Patient
8383
registry
8484
.hydrate_once_sync(tmp.path(), &["Healthcare"])
8585
.unwrap();
86-
std::mem::forget(tmp);
86+
// `hydrate_once_sync` parses the TTL into the in-memory registry, so
87+
// the temp dir is no longer needed — let it drop (no leak). The
88+
// earlier `std::mem::forget(tmp)` kept it alive unnecessarily
89+
// (CodeRabbit, PR #582).
8790
registry
8891
}
8992

0 commit comments

Comments
 (0)