use super::*;
use crate::config::workflow::{CompactionConfig, IntermediateCompaction};
use crate::template::RealGit;
use tempfile::TempDir;
fn cfg(trigger: CompactionTrigger, n: Option<u32>) -> CompactionConfig {
CompactionConfig {
intermediate: IntermediateCompaction { trigger, n },
}
}
fn st(commits: u32, seconds: u64, flush: bool) -> CheckpointState {
CheckpointState {
commits_since_checkpoint: commits,
seconds_since_checkpoint: seconds,
flush_requested: flush,
is_compactor: false,
}
}
#[test]
fn no_config_never_compacts() {
assert!(!due(None, &st(1000, 1000, true)));
}
#[test]
fn every_n_commits_fires_at_or_past_the_threshold() {
let c = cfg(CompactionTrigger::EveryNCommits, Some(3));
assert!(!due(Some(&c), &st(2, 0, false)));
assert!(due(Some(&c), &st(3, 0, false)));
assert!(due(Some(&c), &st(4, 0, false)));
}
#[test]
fn every_t_seconds_fires_at_or_past_the_threshold() {
let c = cfg(CompactionTrigger::EveryTSeconds, Some(10));
assert!(!due(Some(&c), &st(0, 9, false)));
assert!(due(Some(&c), &st(0, 10, false)));
}
#[test]
fn on_flush_fires_only_when_the_agent_elects_it() {
let c = cfg(CompactionTrigger::OnFlush, None);
assert!(!due(Some(&c), &st(9999, 9999, false)));
assert!(due(Some(&c), &st(0, 0, true)));
}
#[test]
fn a_malformed_threshold_fails_closed() {
assert!(!due(
Some(&cfg(CompactionTrigger::EveryNCommits, None)),
&st(100, 0, false)
));
assert!(!due(
Some(&cfg(CompactionTrigger::EveryTSeconds, Some(0))),
&st(0, 100, false)
));
}
#[test]
fn a_compactor_is_never_compaction_eligible() {
let compactor = CheckpointState {
is_compactor: true,
..st(9999, 9999, true)
};
for c in [
cfg(CompactionTrigger::EveryNCommits, Some(1)),
cfg(CompactionTrigger::EveryTSeconds, Some(1)),
cfg(CompactionTrigger::OnFlush, None),
] {
assert!(!due(Some(&c), &compactor), "{:?}", c.intermediate.trigger);
assert!(due(Some(&c), &st(9999, 9999, true)));
}
}
fn init(wt: &Path) {
let g = RealGit::new();
g.run(wt, &["init", "-b", "agents/p1"]).unwrap();
g.run(wt, &["config", "user.email", "t@t"]).unwrap();
g.run(wt, &["config", "core.hooksPath", "/dev/null"])
.unwrap();
g.run(wt, &["config", "user.name", "t"]).unwrap();
}
fn commit(wt: &Path, subject: &str, rel: &str, content: &str) {
let g = RealGit::new();
let f = wt.join(rel);
std::fs::create_dir_all(f.parent().unwrap()).unwrap();
std::fs::write(&f, content).unwrap();
g.run(wt, &["add", "-A"]).unwrap();
g.run(wt, &["commit", "-m", subject]).unwrap();
}
fn now_of(wt: &Path) -> u64 {
RealGit::new()
.run_capture(wt, &["log", "-n", "1", "--format=%ct", "HEAD"])
.unwrap()
.trim()
.parse()
.unwrap()
}
#[test]
fn state_counts_the_whole_branch_when_no_checkpoint_landed() {
let dir = TempDir::new().unwrap();
let wt = dir.path();
init(wt);
commit(wt, "root", "a.txt", "1");
let root_ct = now_of(wt);
commit(wt, "step", "b.txt", "2");
let s = state(wt, "p1", root_ct + 7, false, &RealGit::new()).unwrap();
assert_eq!(s.commits_since_checkpoint, 2, "root + one step");
assert_eq!(s.seconds_since_checkpoint, 7, "measured from the root");
assert!(!s.flush_requested);
}
#[test]
fn state_measures_from_the_last_compaction_merge() {
let dir = TempDir::new().unwrap();
let wt = dir.path();
init(wt);
commit(wt, "root", "a.txt", "1");
commit(wt, "compaction merge [p1-cmp]", "summary/001.md", "x");
let cmp_ct = now_of(wt);
commit(wt, "step after", "b.txt", "2");
let s = state(wt, "p1", cmp_ct + 42, true, &RealGit::new()).unwrap();
assert_eq!(s.commits_since_checkpoint, 1, "only the post-merge step");
assert_eq!(s.seconds_since_checkpoint, 42);
assert!(s.flush_requested);
}
#[test]
fn state_saturates_when_now_precedes_the_checkpoint() {
let dir = TempDir::new().unwrap();
let wt = dir.path();
init(wt);
commit(wt, "root", "a.txt", "1");
let s = state(wt, "p1", 0, false, &RealGit::new()).unwrap();
assert_eq!(s.seconds_since_checkpoint, 0, "no negative elapsed time");
}
#[test]
fn state_surfaces_a_git_failure() {
let dir = TempDir::new().unwrap();
let err = state(dir.path(), "p1", 0, false, &RealGit::new()).unwrap_err();
assert!(matches!(err, Error::Git { .. }), "{err:?}");
}
#[test]
fn state_measures_from_the_branchs_own_dispatch_commit_not_inherited_history() {
let dir = TempDir::new().unwrap();
let wt = dir.path();
init(wt);
for i in 0..25 {
commit(
wt,
&format!("inherited step {i}"),
&format!("m/{i}.md"),
"x",
);
}
commit(wt, "dispatch: compactor [p1-c1]", "goal.md", "g");
commit(wt, "step 001", "m/own.md", "x");
let s = state(wt, "p1-c1", now_of(wt), false, &RealGit::new()).unwrap();
assert_eq!(
s.commits_since_checkpoint, 1,
"only this branch's own commit, not the 26 it inherited"
);
let c = cfg(CompactionTrigger::EveryNCommits, Some(20));
assert!(
!due(Some(&c), &s),
"a seconds-old branch is below threshold"
);
}
#[test]
fn state_reads_the_branch_role_from_its_own_dispatch_commit() {
let dir = TempDir::new().unwrap();
let wt = dir.path();
init(wt);
commit(wt, "root", "a.txt", "1");
commit(wt, "dispatch: compactor [p1-c1]", "goal.md", "g");
let s = state(wt, "p1-c1", now_of(wt), false, &RealGit::new()).unwrap();
assert!(s.is_compactor, "role derived from the dispatch subject");
commit(wt, "dispatch: worker [p1-c1-w9]", "x.md", "x");
let child = state(wt, "p1-c1", now_of(wt), false, &RealGit::new()).unwrap();
assert!(child.is_compactor);
let worker = state(wt, "p1-c1-w9", now_of(wt), false, &RealGit::new()).unwrap();
assert!(!worker.is_compactor);
}
#[test]
fn state_falls_back_to_the_root_when_a_branch_has_no_dispatch_commit() {
let dir = TempDir::new().unwrap();
let wt = dir.path();
init(wt);
commit(wt, "root", "a.txt", "1");
commit(wt, "step", "b.txt", "2");
let s = state(wt, "nobody", now_of(wt), false, &RealGit::new()).unwrap();
assert_eq!(s.commits_since_checkpoint, 2);
assert!(!s.is_compactor);
}
struct FailOn(&'static str);
impl GitRunner for FailOn {
fn run(&self, _dest: &Path, _args: &[&str]) -> std::io::Result<()> {
unreachable!("checkpoint derivation only captures")
}
fn run_capture(&self, _dest: &Path, args: &[&str]) -> std::io::Result<String> {
if args.iter().any(|a| a.contains(self.0)) {
return Err(std::io::Error::other("stub git failure"));
}
Ok(match args {
a if a.contains(&"--grep") => String::new(),
a if a.contains(&"--count") => "1".to_string(),
a if a.contains(&"--max-parents=0") => "r00t".to_string(),
_ => "100".to_string(),
})
}
}
fn op_of(err: Error) -> &'static str {
match err {
Error::Git { op, .. } => op,
other => panic!("expected Error::Git, got {other:?}"),
}
}
#[test]
fn state_tags_a_commit_count_failure_with_its_op() {
let dir = TempDir::new().unwrap();
let err = state(dir.path(), "p1", 0, false, &FailOn("--count")).unwrap_err();
assert_eq!(op_of(err), "checkpoint rev-list count");
}
#[test]
fn state_tags_a_commit_time_failure_with_its_op() {
let dir = TempDir::new().unwrap();
let err = state(dir.path(), "p1", 0, false, &FailOn("%ct")).unwrap_err();
assert_eq!(op_of(err), "checkpoint commit time");
}
#[test]
fn state_tags_a_root_lookup_failure_with_its_op() {
let dir = TempDir::new().unwrap();
let err = state(dir.path(), "p1", 0, false, &FailOn("--max-parents=0")).unwrap_err();
assert_eq!(op_of(err), "checkpoint root rev-list");
}