huskies: merge 1216 story Shorten long-turn notification to "Working..."
This commit is contained in:
@@ -19,7 +19,7 @@ use super::format_user_prompt;
|
|||||||
|
|
||||||
/// Text posted to the room by [`spawn_digging_in_watcher`] when a turn runs
|
/// Text posted to the room by [`spawn_digging_in_watcher`] when a turn runs
|
||||||
/// long without emitting any user-facing text.
|
/// long without emitting any user-facing text.
|
||||||
const DIGGING_IN_MESSAGE: &str = "Still digging in — this turn is taking a bit longer than usual.";
|
const DIGGING_IN_MESSAGE: &str = "Working...";
|
||||||
|
|
||||||
/// Spawns a background watcher that posts a single "digging in" notice to
|
/// Spawns a background watcher that posts a single "digging in" notice to
|
||||||
/// `room_id` if `threshold` elapses before `sent_any_text` becomes `true`.
|
/// `room_id` if `threshold` elapses before `sent_any_text` becomes `true`.
|
||||||
@@ -392,7 +392,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
/// AC 1: a turn that runs longer than the threshold without sending any
|
/// AC 1: a turn that runs longer than the threshold without sending any
|
||||||
/// text gets exactly one "digging in" notice.
|
/// text gets exactly one "Working..." notice.
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn digging_in_fires_after_threshold_when_no_text_sent() {
|
async fn digging_in_fires_after_threshold_when_no_text_sent() {
|
||||||
let transport = Arc::new(CapturingTransport::new());
|
let transport = Arc::new(CapturingTransport::new());
|
||||||
@@ -405,14 +405,7 @@ mod tests {
|
|||||||
);
|
);
|
||||||
handle.await.unwrap();
|
handle.await.unwrap();
|
||||||
assert_eq!(transport.sent_count(), 1);
|
assert_eq!(transport.sent_count(), 1);
|
||||||
assert!(
|
assert_eq!(transport.last_message().unwrap(), "Working...");
|
||||||
transport
|
|
||||||
.last_message()
|
|
||||||
.unwrap()
|
|
||||||
.to_lowercase()
|
|
||||||
.contains("digging in"),
|
|
||||||
"notice should mention 'digging in'"
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/// AC 2: if text is sent before the threshold elapses, the watcher must
|
/// AC 2: if text is sent before the threshold elapses, the watcher must
|
||||||
|
|||||||
Reference in New Issue
Block a user