Increase timeout of hover_on_external_command (nushell#16818)

Recently `hover_on_external_command` has been timing out and failing CI
runs.

is 10 seconds long? yeah. is it quicker than re-running ci jobs? also
yeah.

Author: Bahex

crates/nu-lsp/src/hover.rs

@@ -228,6 +228,7 @@ mod hover_tests {
     use assert_json_diff::assert_json_eq;
     use nu_test_support::fs::fixtures;
     use rstest::rstest;
+    use std::time::Duration;
 
     #[rstest]
     #[case::variable("var.nu", (2, 0), "```\ntable\n``` \n---\nimmutable")]
@@ -260,7 +261,9 @@ mod hover_tests {
 
         open_unchecked(&client_connection, script.clone());
         let (line, character) = cursor;
-        let resp = send_hover_request(&client_connection, script, line, character);
+        let resp = send_hover_request(&client_connection, script, line, character)
+            .recv_timeout(Duration::from_secs(3))
+            .unwrap();
 
         assert_json_eq!(
             result_from_message(resp)["contents"]["value"],
@@ -277,7 +280,9 @@ mod hover_tests {
         let script = path_to_uri(&script);
 
         open_unchecked(&client_connection, script.clone());
-        let resp = send_hover_request(&client_connection, script, 6, 2);
+        let resp = send_hover_request(&client_connection, script, 6, 2)
+            .recv_timeout(Duration::from_secs(10))
+            .unwrap();
 
         let hover_text = result_from_message(resp)["contents"]["value"].to_string();
 
@@ -311,7 +316,9 @@ mod hover_tests {
 
         open_unchecked(&client_connection, script_uri.clone());
         let (line, character) = cursor;
-        let resp = send_hover_request(&client_connection, script_uri, line, character);
+        let resp = send_hover_request(&client_connection, script_uri, line, character)
+            .recv_timeout(Duration::from_secs(3))
+            .unwrap();
         let result = result_from_message(resp);
 
         let actual = result["contents"]["value"].to_string().replace("\\r", "");

crates/nu-lsp/src/lib.rs

@@ -671,7 +671,7 @@ mod tests {
         uri: Uri,
         line: u32,
         character: u32,
-    ) -> Message {
+    ) -> &crossbeam_channel::Receiver<Message> {
         client_connection
             .sender
             .send(Message::Request(lsp_server::Request {
@@ -688,9 +688,6 @@ mod tests {
             }))
             .unwrap();
 
-        client_connection
-            .receiver
-            .recv_timeout(Duration::from_secs(3))
-            .unwrap()
+        &client_connection.receiver
     }
 }

crates/nu-lsp/src/notification.rs

@@ -155,6 +155,7 @@ mod tests {
     use lsp_types::Range;
     use nu_test_support::fs::fixtures;
     use rstest::rstest;
+    use std::time::Duration;
 
     #[rstest]
     #[case::full(
@@ -186,7 +187,9 @@ hello"#,
 
         open_unchecked(&client_connection, script.clone());
         update(&client_connection, script.clone(), text, range);
-        let resp = send_hover_request(&client_connection, script, 3, 0);
+        let resp = send_hover_request(&client_connection, script, 3, 0)
+            .recv_timeout(Duration::from_secs(3))
+            .unwrap();
 
         assert_json_eq!(
             result_from_message(resp),

crates/nu-lsp/src/workspace.rs

@@ -552,6 +552,7 @@ mod tests {
     };
     use nu_test_support::fs::fixtures;
     use rstest::rstest;
+    use std::time::Duration;
 
     // Helper functions to reduce JSON duplication
     fn make_range(
@@ -663,7 +664,9 @@ mod tests {
 
         // use a hover request to interrupt
         if immediate_cancellation {
-            send_hover_request(client_connection, uri, line, character);
+            send_hover_request(client_connection, uri, line, character)
+                .recv_timeout(Duration::from_secs(3))
+                .unwrap();
         }
 
         (0..num)