scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-09-15 23:32:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

test(cli): normalize json run snapshot llm defaults

Browse source
This commit is contained in:
Bryan Helmkamp 2026-04-05 11:06:13 -04:00
parent d07850cb7f
commit e8ab762984
No known key found for this signature in database
1 changed files with 16 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
lib/crates/fabro-cli/tests/it/cmd/run.rs
View file

@ -229,12 +229,25 @@ fn json_run_implies_auto_approve_for_human_gates() {
String::from_utf8_lossy(&output.stderr)
);
let progress: Vec<Value> = String::from_utf8(output.stdout)
let mut progress: Vec<Value> = String::from_utf8(output.stdout)
.expect("stdout should be UTF-8")
.lines()
.filter(|line| !line.trim().is_empty())
.map(|line| serde_json::from_str(line).expect("run JSON output should be JSONL"))
.collect();
for event in &mut progress {
let Some(llm) = event.pointer_mut("/properties/settings/llm") else {
continue;
};
let Some(llm) = llm.as_object_mut() else {
continue;
};
llm.insert("model".to_string(), Value::String("[LLM_MODEL]".to_string()));
llm.insert(
"provider".to_string(),
Value::String("[LLM_PROVIDER]".to_string()),
);
}
fabro_json_snapshot!(context, &progress, @r#"
[
{
@ -348,8 +361,8 @@ fn json_run_implies_auto_approve_for_human_gates() {
"goal": "Route through the default approval path",
"llm": {
"fallbacks": null,
"model": "gpt-5.4",
"provider": "openai"
"model": "[LLM_MODEL]",
"provider": "[LLM_PROVIDER]"
},
"mode": "standalone",
"no_retro": true,