mirror of
https://github.com/instructkr/claw-code.git
synced 2026-04-10 18:14:50 +08:00
fix(tests): update test assertions for null model in resume /status; drop unused import
Two integration tests expected 'model':'restored-session' in the /status
JSON output but dc4fa55 changed resume mode to emit null for model.
Updated both assertions to assert model is null (correct behavior).
Also remove unused 'estimate_session_tokens' import in compact.rs tests
(surfaced as warning in CI, kept failing CI green noise).
All workspace tests pass.
This commit is contained in:
@@ -555,7 +555,7 @@ fn extract_summary_timeline(summary: &str) -> Vec<String> {
|
|||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use super::{
|
use super::{
|
||||||
collect_key_files, compact_session, estimate_session_tokens, format_compact_summary,
|
collect_key_files, compact_session, format_compact_summary,
|
||||||
get_compact_continuation_message, infer_pending_work, should_compact, CompactionConfig,
|
get_compact_continuation_message, infer_pending_work, should_compact, CompactionConfig,
|
||||||
};
|
};
|
||||||
use crate::session::{ContentBlock, ConversationMessage, MessageRole, Session};
|
use crate::session::{ContentBlock, ConversationMessage, MessageRole, Session};
|
||||||
|
|||||||
@@ -253,7 +253,8 @@ fn doctor_and_resume_status_emit_json_when_requested() {
|
|||||||
],
|
],
|
||||||
);
|
);
|
||||||
assert_eq!(resumed["kind"], "status");
|
assert_eq!(resumed["kind"], "status");
|
||||||
assert_eq!(resumed["model"], "restored-session");
|
// model is null in resume mode (not known without --model flag)
|
||||||
|
assert!(resumed["model"].is_null());
|
||||||
assert_eq!(resumed["usage"]["messages"], 1);
|
assert_eq!(resumed["usage"]["messages"], 1);
|
||||||
assert!(resumed["workspace"]["cwd"].as_str().is_some());
|
assert!(resumed["workspace"]["cwd"].as_str().is_some());
|
||||||
assert!(resumed["sandbox"]["filesystem_mode"].as_str().is_some());
|
assert!(resumed["sandbox"]["filesystem_mode"].as_str().is_some());
|
||||||
|
|||||||
@@ -261,7 +261,8 @@ fn resumed_status_command_emits_structured_json_when_requested() {
|
|||||||
let parsed: Value =
|
let parsed: Value =
|
||||||
serde_json::from_str(stdout.trim()).expect("resume status output should be json");
|
serde_json::from_str(stdout.trim()).expect("resume status output should be json");
|
||||||
assert_eq!(parsed["kind"], "status");
|
assert_eq!(parsed["kind"], "status");
|
||||||
assert_eq!(parsed["model"], "restored-session");
|
// model is null in resume mode (not known without --model flag)
|
||||||
|
assert!(parsed["model"].is_null());
|
||||||
assert_eq!(parsed["permission_mode"], "danger-full-access");
|
assert_eq!(parsed["permission_mode"], "danger-full-access");
|
||||||
assert_eq!(parsed["usage"]["messages"], 1);
|
assert_eq!(parsed["usage"]["messages"], 1);
|
||||||
assert!(parsed["usage"]["turns"].is_number());
|
assert!(parsed["usage"]["turns"].is_number());
|
||||||
|
|||||||
Reference in New Issue
Block a user