in tests, force terminal width to be 80

crates/dropshot-api-manager/src/output.rs

@@ -393,7 +393,7 @@ pub fn display_resolution(
             "\n{}\n",
             textwrap::fill(
                 &n.to_string(),
-                textwrap::Options::with_termwidth()
+                textwrap::Options::new(term_width())
                     .initial_indent(&initial_indent)
                     .subsequent_indent(&more_indent)
             )
@@ -539,7 +539,7 @@ where
                     "{}",
                     textwrap::fill(
                         &issue.to_string(),
-                        textwrap::Options::with_termwidth()
+                        textwrap::Options::new(term_width())
                             .initial_indent(&nested_first_indent)
                             .subsequent_indent(&nested_more_indent)
                     )
@@ -705,7 +705,7 @@ fn write_problem_header(
         "{}",
         textwrap::fill(
             &InlineErrorChain::new(error).to_string(),
-            textwrap::Options::with_termwidth()
+            textwrap::Options::new(term_width())
                 .initial_indent(&first_indent)
                 .subsequent_indent(&more_indent)
         )
@@ -731,7 +731,7 @@ fn write_fix_summary(
             "{}",
             textwrap::fill(
                 &format!("will {}", s),
-                textwrap::Options::with_termwidth()
+                textwrap::Options::new(term_width())
                     .initial_indent(&first_indent)
                     .subsequent_indent(&more_indent)
             )
@@ -762,6 +762,28 @@ impl fmt::Display for InlineErrorChain<'_> {
     }
 }
 
+/// Returns the wrap width to use for terminal output.
+///
+/// Honors the `OPENAPI_MGR_TERM_WIDTH` environment variable as an override.
+/// Otherwise falls back to [`textwrap::termwidth`], which queries the terminal
+/// connected to stdout, or returns 80 when stdout isn't a tty.
+///
+/// The override exists for snapshot determinism. Under `cargo nextest run` by
+/// default, stdout is captured, so `termwidth` returns 80 and snapshots are
+/// deterministic. Under `cargo nextest run --no-capture` (or `cargo test`),
+/// however, stdout may be the developer's tty, and width is wherever the window
+/// happens to be sized. Setting `OPENAPI_MGR_TERM_WIDTH=80` explicitly, as we
+/// do in our tests, ensures that snapshots are deterministic in this scenario
+/// as well.
+pub(crate) fn term_width() -> usize {
+    match std::env::var("OPENAPI_MGR_TERM_WIDTH") {
+        Ok(s) => s.parse().unwrap_or_else(|err| {
+            panic!("OPENAPI_MGR_TERM_WIDTH={s:?} is not a valid width: {err}")
+        }),
+        Err(_) => textwrap::termwidth(),
+    }
+}
+
 /// Output headers.
 pub(crate) mod headers {
     // Same width as Cargo's output.

crates/integration-tests/src/environment.rs

@@ -113,10 +113,21 @@ pub struct TestEnvironment {
     vcs_mode: VcsMode,
 }
 
+/// Set the wrap width used by `output.rs::term_width` to ensure that snapshots
+/// are deterministic.
+fn set_test_term_width() {
+    // SAFETY:
+    // https://nexte.st/docs/configuration/env-vars/#altering-the-environment-within-tests
+    unsafe {
+        std::env::set_var("OPENAPI_MGR_TERM_WIDTH", "80");
+    }
+}
+
 impl TestEnvironment {
     /// Create a new test environment with temporary directories, backed by a
     /// Git repository.
     pub fn new_git() -> Result<Self> {
+        set_test_term_width();
         let temp_dir =
             Utf8TempDir::with_prefix("dropshot-api-manager-integration-")
                 .context("failed to create temporary directory")?;
@@ -177,6 +188,7 @@ impl TestEnvironment {
     /// The non-colocated nature of the Jujutsu repository means that operations
     /// are forced to go through jj.
     pub fn new_jj() -> Result<Self> {
+        set_test_term_width();
         let temp_dir = Utf8TempDir::with_prefix("dropshot-api-manager-jj-")
             .context("failed to create temporary directory")?;