fix: test missing timeout_ms field, unused title warning
This commit is contained in:
parent
b86717f7dc
commit
20e97b932b
2 changed files with 4 additions and 1 deletions
|
|
@ -362,7 +362,9 @@ async fn handle_message(
|
||||||
}
|
}
|
||||||
|
|
||||||
ServerMessage::PromptRequest { request_id, message, title } => {
|
ServerMessage::PromptRequest { request_id, message, title } => {
|
||||||
let title = title.unwrap_or_else(|| "Helios Remote".to_string());
|
let _title = title.unwrap_or_else(|| "Helios Remote".to_string());
|
||||||
|
#[cfg(windows)]
|
||||||
|
let title = _title.clone();
|
||||||
log_cmd!("prompt › {}", &message[..message.len().min(60)]);
|
log_cmd!("prompt › {}", &message[..message.len().min(60)]);
|
||||||
#[cfg(windows)]
|
#[cfg(windows)]
|
||||||
{
|
{
|
||||||
|
|
|
||||||
|
|
@ -154,6 +154,7 @@ mod tests {
|
||||||
let msg = ServerMessage::ExecRequest {
|
let msg = ServerMessage::ExecRequest {
|
||||||
request_id: Uuid::nil(),
|
request_id: Uuid::nil(),
|
||||||
command: "echo hello".into(),
|
command: "echo hello".into(),
|
||||||
|
timeout_ms: None,
|
||||||
};
|
};
|
||||||
let json = serde_json::to_string(&msg).unwrap();
|
let json = serde_json::to_string(&msg).unwrap();
|
||||||
assert!(json.contains("exec_request"));
|
assert!(json.contains("exec_request"));
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue