vinzenz/hyperhive
vinzenz/hyperhive
Watch
0
0
Fork
You've already forked hyperhive
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

rename(162): update manager test to use root

Browse source
This commit is contained in:
damocles 2026-06-01 16:59:27 +02:00 • committed by mara
commit 9654315a20
1 changed files with 5 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

17
hive-c0re/src/reminder_scheduler.rs
View file

@ -249,21 +249,14 @@ mod tests {
#[test] #[test]
fn manager_uses_container_name_prefix() { fn manager_uses_container_name_prefix() {
// Manager's container view of its state is at // Manager's container view of its state is at `/agents/root/state/`.
// `/agents/<MANAGER_NAME>/state/` (= `/agents/hm1nd/state/`), assert_eq!(container_state_prefix("root"), "/agents/root/state/");
// same as every other agent. let p = resolve_host_path("root", "/agents/root/state/reminders/x.md").unwrap();
assert_eq!(container_state_prefix("manager"), "/agents/hm1nd/state/");
let p = resolve_host_path("manager", "/agents/hm1nd/state/reminders/x.md").unwrap();
// NB: the host path still resolves under `agents/manager/`
// (Coordinator::agent_notes_dir takes the broker LOGICAL name).
// That's a pre-existing manager-logical-vs-container-name
// discrepancy tracked separately; out of scope here.
assert_eq!( assert_eq!(
p, p,
PathBuf::from("/var/lib/hyperhive/agents/manager/state/reminders/x.md") PathBuf::from("/var/lib/hyperhive/agents/root/state/reminders/x.md")
); );
// And the legacy `/state/` prefix must NOT be accepted anymore. assert!(resolve_host_path("root", "/state/x.md").is_err());
assert!(resolve_host_path("manager", "/state/x.md").is_err());
} }
#[test] #[test]