[WIP] accept input characters for info queries, not just signals

Author: sunshowers

Cargo.lock

@@ -45,6 +45,7 @@ config = { version = "0.14.1", default-features = false, features = [
 chrono = "0.4.38"
 clap = { version = "4.5.22", features = ["derive"] }
 console-subscriber = "0.4.1"
+crossterm = { version = "0.28.1", features = ["event-stream"] }
 dialoguer = "0.11.0"
 debug-ignore = "1.0.5"
 duct = "0.13.7"
@@ -80,7 +81,7 @@ newtype-uuid = { version = "1.1.3", features = ["v4"] }
 nextest-filtering = { version = "0.12.0", path = "nextest-filtering" }
 nextest-metadata = { version = "0.12.1", path = "nextest-metadata" }
 nextest-workspace-hack = "0.1.0"
-nix = { version = "0.29.0", default-features = false, features = ["signal"] }
+nix = { version = "0.29.0", default-features = false, features = ["signal", "term"] }
 once_cell = "1.20.2"
 owo-colors = "4.1.0"
 pathdiff = { version = "0.2.3", features = ["camino"] }

Cargo.toml

@@ -22,6 +22,7 @@ use nextest_runner::{
     },
     double_spawn::DoubleSpawnInfo,
     errors::WriteTestListError,
+    input::InputHandlerKind,
     list::{
         BinaryList, OutputFormat, RustTestArtifact, SerializableFormat, TestExecuteContext,
         TestList,
@@ -986,6 +987,13 @@ struct TestReporterOpts {
     #[arg(long, env = "NEXTEST_HIDE_PROGRESS_BAR", value_parser = BoolishValueParser::new())]
     hide_progress_bar: bool,
 
+    /// Disable handling of input keys from the terminal.
+    ///
+    /// By default, when running a terminal, nextest accepts the `t` key to dump
+    /// test information. This flag disables that behavior.
+    #[arg(long, env = "NEXTEST_NO_INPUT_HANDLER", value_parser = BoolishValueParser::new())]
+    no_input_handler: bool,
+
     /// Format to use for test results (experimental).
     #[arg(
         long,
@@ -1780,12 +1788,16 @@ impl App {
             .color
             .should_colorize(supports_color::Stream::Stderr);
 
-        let mut reporter = reporter_opts
-            .to_builder(no_capture, should_colorize)
-            .set_verbose(self.base.output.verbose)
-            .build(&test_list, &profile, output, structured_reporter);
+        let signal_handler = SignalHandlerKind::Standard;
+        let input_handler = if reporter_opts.no_input_handler {
+            InputHandlerKind::Noop
+        } else {
+            // This means that the input handler determines whether it should be
+            // enabled.
+            InputHandlerKind::Standard
+        };
 
-        let handler = SignalHandlerKind::Standard;
+        // Make the runner.
         let runner_builder = match runner_opts.to_builder(cap_strat) {
             Some(runner_builder) => runner_builder,
             None => {
@@ -1798,11 +1810,18 @@ impl App {
             &test_list,
             &profile,
             cli_args,
-            handler,
+            signal_handler,
+            input_handler,
             double_spawn.clone(),
             target_runner.clone(),
         )?;
 
+        // Make the reporter.
+        let mut reporter = reporter_opts
+            .to_builder(no_capture, should_colorize)
+            .set_verbose(self.base.output.verbose)
+            .build(&test_list, &profile, output, structured_reporter);
+
         configure_handle_inheritance(no_capture)?;
         let run_stats = runner.try_execute(|event| {
             // Write and flush the event.

cargo-nextest/src/dispatch.rs

@@ -25,6 +25,7 @@ cargo_metadata.workspace = true
 config.workspace = true
 cfg-if.workspace = true
 chrono.workspace = true
+crossterm.workspace = true
 debug-ignore.workspace = true
 duct.workspace = true
 future-queue.workspace = true
@@ -69,6 +70,7 @@ thiserror.workspace = true
 # For parsing of .cargo/config.toml files
 tokio = { workspace = true, features = [
     "fs",
+    "io-std",
     "io-util",
     "macros",
     "process",