fix: correct test assertion for history search escape behavior

swananan · swananan · commit 068631f45478 · 2025-10-06T02:06:22.000+08:00
diff --git a/ghostscope-ui/tests/additional_ui_coverage_test.rs b/ghostscope-ui/tests/additional_ui_coverage_test.rs
@@ -101,7 +101,8 @@ mod history_search_tests {
         let actions = InputHandler::handle_key_event(&mut state, esc_key);
 
         assert!(!state.is_in_history_search());
-        assert!(actions
+        // Should NOT add empty response - would overwrite previous command's response
+        assert!(!actions
             .iter()
             .any(|a| matches!(a, Action::AddResponse { .. })));
     }

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,8 @@ mod history_search_tests {`
`101`	`101`	`let actions = InputHandler::handle_key_event(&mut state, esc_key);`
`102`	`102`
`103`	`103`	`assert!(!state.is_in_history_search());`
`104`		`- assert!(actions`
	`104`	`+ // Should NOT add empty response - would overwrite previous command's response`
	`105`	`+ assert!(!actions`
`105`	`106`	`.iter()`
`106`	`107`	`.any(\|a\| matches!(a, Action::AddResponse { .. })));`
`107`	`108`	`}`