csvops 0.7.0.alpha → 0.9.0.alpha

data/test/csvtool/cli_test.rb

@@ -2,6 +2,7 @@
 
 require_relative "../test_helper"
 require "csvtool/cli"
+require "json"
 require "tmpdir"
 require "fileutils"
 
@@ -12,11 +13,75 @@ class TestCli < Minitest::Test
 
   def test_menu_can_exit_cleanly
     output = StringIO.new
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("7\n"), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("8\n"), stdout: output, stderr: output)
     assert_equal 0, status
     assert_includes output.string, "CSV Tool Menu"
   end
 
+  def test_stats_workflow_shell_can_run_and_return_to_menu
+    output = StringIO.new
+    input = [
+      "7",
+      fixture_path("sample_people.csv"),
+      "",
+      "",
+      "",
+      "8"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
+
+    assert_equal 0, status
+    assert_includes output.string, "CSV Stats Summary"
+    assert_includes output.string, "Metric"
+    assert_includes output.string, "Rows"
+    assert_includes output.string, "Columns"
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+
+  def test_stats_workflow_missing_file_returns_to_menu
+    output = StringIO.new
+    input = [
+      "7",
+      "/tmp/does-not-exist.csv",
+      "",
+      "",
+      "",
+      "8"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
+
+    assert_equal 0, status
+    assert_includes output.string, "File not found: /tmp/does-not-exist.csv"
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+
+  def test_stats_workflow_can_write_output_to_file
+    output = StringIO.new
+
+    Dir.mktmpdir do |dir|
+      output_path = File.join(dir, "stats.csv")
+      input = [
+        "7",
+        fixture_path("sample_people.csv"),
+        "",
+        "",
+        "2",
+        output_path,
+        "8"
+      ].join("\n") + "\n"
+
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
+
+      assert_equal 0, status
+      assert_includes output.string, "Wrote output to #{output_path}"
+      csv_text = File.read(output_path)
+      assert_includes csv_text, "metric,value"
+      assert_includes csv_text, "row_count,3"
+    end
+  end
+
   def test_split_workflow_splits_csv_in_menu_flow
     output = StringIO.new
     Dir.mktmpdir do |dir|
@@ -32,13 +97,13 @@ class TestCli < Minitest::Test
         "",
         "",
         "",
-        "7"
+        "8"
       ].join("\n") + "\n"
 
-      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
       assert_equal 0, status
-      assert_includes output.string, "Chunks written: 3"
+      assert_includes output.string, "Chunks written"
       assert File.file?(File.join(dir, "people_part_001.csv"))
       assert File.file?(File.join(dir, "people_part_002.csv"))
       assert File.file?(File.join(dir, "people_part_003.csv"))
@@ -53,10 +118,10 @@ class TestCli < Minitest::Test
       "",
       "",
       "0",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Chunk size must be a positive integer."
@@ -73,11 +138,11 @@ class TestCli < Minitest::Test
       "",
       "y",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
     output = StringIO.new
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_match(/\nAlice\nBob\nCara\n/, output.string)
@@ -96,6 +161,244 @@ class TestCli < Minitest::Test
     assert_equal "Alice\nBob\nCara\n", output.string
   end
 
+  def test_stats_command_writes_summary_to_stdout_only
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv")],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    assert_includes stdout.string, "CSV Stats Summary"
+    assert_includes stdout.string, "Metric"
+    assert_includes stdout.string, "Rows"
+    assert_includes stdout.string, "Columns"
+    assert_includes stdout.string, "Headers"
+    assert_equal "", stderr.string
+  end
+
+  def test_stats_command_writes_errors_to_stderr_only
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", "/tmp/not-there.csv"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 1, status
+    assert_equal "", stdout.string
+    assert_includes stderr.string, "File not found: /tmp/not-there.csv"
+  end
+
+  def test_stats_command_supports_json_format
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--format", "json"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    data = JSON.parse(stdout.string, symbolize_names: true)
+    assert_equal 3, data[:row_count]
+    assert_equal 2, data[:column_count]
+    assert_equal ["name", "city"], data[:headers]
+    assert_equal "", stderr.string
+  end
+
+  def test_stats_command_supports_csv_format
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--format=csv"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    lines = stdout.string.lines.map(&:strip)
+    assert_equal "metric,value", lines[0]
+    assert_includes lines, "row_count,3"
+    assert_includes lines, "column_count,2"
+    assert_includes lines, "headers,name|city"
+    assert_equal "", stderr.string
+  end
+
+  def test_stats_command_rejects_unknown_format
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--format", "yaml"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 1, status
+    assert_equal "", stdout.string
+    assert_includes stderr.string, "Invalid format: yaml"
+  end
+
+  def test_stats_command_color_auto_without_tty_has_no_ansi
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--color", "auto"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    refute_match(/\e\[[0-9;]*m/, stdout.string)
+    assert_equal "", stderr.string
+  end
+
+  def test_stats_command_color_always_adds_ansi
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--color=always"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    assert_match(/\e\[[0-9;]*m/, stdout.string)
+    assert_equal "", stderr.string
+  end
+
+  def test_stats_command_color_never_disables_ansi
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--color", "never"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    refute_match(/\e\[[0-9;]*m/, stdout.string)
+    assert_equal "", stderr.string
+  end
+
+  def test_stats_command_no_color_env_disables_auto
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--color", "auto"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr,
+      env: { "NO_COLOR" => "1" }
+    )
+
+    assert_equal 0, status
+    refute_match(/\e\[[0-9;]*m/, stdout.string)
+  end
+
+  def test_stats_command_color_always_overrides_no_color_env
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--color", "always"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr,
+      env: { "NO_COLOR" => "1" }
+    )
+
+    assert_equal 0, status
+    assert_match(/\e\[[0-9;]*m/, stdout.string)
+  end
+
+  def test_stats_command_rejects_unknown_color_mode
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(
+      ["stats", fixture_path("sample_people.csv"), "--color", "sometimes"],
+      stdin: StringIO.new,
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 1, status
+    assert_equal "", stdout.string
+    assert_includes stderr.string, "Invalid color mode: sometimes"
+  end
+
+  def test_menu_workflow_routes_ui_to_stderr_and_data_to_stdout
+    stdout = StringIO.new
+    stderr = StringIO.new
+    input = [
+      "7",
+      fixture_path("sample_people.csv"),
+      "",
+      "",
+      "",
+      "8"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(
+      ["menu"],
+      stdin: StringIO.new(input),
+      stdout: stdout,
+      stderr: stderr
+    )
+
+    assert_equal 0, status
+    assert_includes stderr.string, "CSV Tool Menu"
+    assert_includes stderr.string, "CSV file path:"
+    assert_includes stdout.string, "CSV Stats Summary"
+    refute_includes stdout.string, "CSV Tool Menu"
+    refute_includes stdout.string, "CSV file path:"
+  end
+
+  def test_stats_command_table_respects_narrow_terminal_width
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    Dir.mktmpdir do |dir|
+      path = File.join(dir, "long_headers.csv")
+      File.write(path, "very_long_column_name,another_really_long_column_name\nvalue_a,value_b\n")
+
+      status = Csvtool::CLI.start(
+        ["stats", path],
+        stdin: StringIO.new,
+        stdout: stdout,
+        stderr: stderr,
+        env: { "COLUMNS" => "32" }
+      )
+
+      assert_equal 0, status
+      lines = stdout.string.lines.map(&:chomp)
+      assert lines.all? { |line| line.length <= 32 }
+      assert_includes stdout.string, "..."
+      assert_equal "", stderr.string
+    end
+  end
+
   def test_row_range_workflow_prints_selected_rows
     output = StringIO.new
     input = [
@@ -105,10 +408,10 @@ class TestCli < Minitest::Test
       "2",
       "3",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "name,city"
@@ -126,10 +429,10 @@ class TestCli < Minitest::Test
       "0",
       "3",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Start row must be a positive integer."
@@ -145,10 +448,10 @@ class TestCli < Minitest::Test
       "2",
       "3",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "name,city"
@@ -166,10 +469,10 @@ class TestCli < Minitest::Test
       "2",
       "3",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "name,city"
@@ -191,10 +494,10 @@ class TestCli < Minitest::Test
         "3",
         "2",
         output_path,
-        "7"
+        "8"
       ].join("\n") + "\n"
 
-      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
       assert_equal 0, status
       assert_equal "name,city\nBob,Paris\nCara,Berlin\n", File.read(output_path)
     end
@@ -211,10 +514,10 @@ class TestCli < Minitest::Test
       "1",
       "2",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Alice,London"
@@ -231,10 +534,10 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "name,city"
@@ -256,10 +559,10 @@ class TestCli < Minitest::Test
         "",
         "2",
         output_path,
-        "7"
+        "8"
       ].join("\n") + "\n"
 
-      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
       assert_equal 0, status
       assert_includes output.string, "Wrote output to #{output_path}"
@@ -278,10 +581,10 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "name\tcity"
@@ -297,10 +600,10 @@ class TestCli < Minitest::Test
       "n",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     refute_includes output.string, "name,city"
@@ -317,10 +620,10 @@ class TestCli < Minitest::Test
       "",
       "",
       "abc",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Seed must be an integer."
@@ -342,10 +645,10 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Reference CSV file path:"
@@ -354,7 +657,7 @@ class TestCli < Minitest::Test
     assert_includes output.string, "customer_id,name"
     assert_includes output.string, "1,Alice"
     assert_includes output.string, "3,Cara"
-    assert_includes output.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
+    assert_includes output.string, "Summary"
   end
 
   def test_dedupe_workflow_can_write_to_file
@@ -376,15 +679,15 @@ class TestCli < Minitest::Test
       "",
       "2",
       output_path,
-      "7"
+      "8"
       ].join("\n") + "\n"
 
-      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
       assert_equal 0, status
       assert_includes output.string, "Wrote output to #{output_path}"
       assert_equal "customer_id,name\n1,Alice\n3,Cara\n", File.read(output_path)
-      assert_includes output.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
+      assert_includes output.string, "Summary"
     end
   end
 
@@ -403,10 +706,10 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "customer_id\tname"
@@ -429,16 +732,16 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     refute_includes output.string, "customer_id,name"
     assert_includes output.string, "1,Alice"
     assert_includes output.string, "3,Cara"
-    assert_includes output.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
+    assert_includes output.string, "Summary"
   end
 
   def test_parity_workflow_reports_match_and_returns_to_menu
@@ -449,16 +752,18 @@ class TestCli < Minitest::Test
       fixture_path("sample_people.csv"),
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Left CSV file path:"
     assert_includes output.string, "Right CSV file path:"
     assert_includes output.string, "MATCH"
-    assert_includes output.string, "Summary: left_rows=3 right_rows=3 left_only=0 right_only=0"
+    assert_includes output.string, "Metric"
+    assert_includes output.string, "Left rows"
+    assert_includes output.string, "Right rows"
     assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
   end
 
@@ -470,14 +775,15 @@ class TestCli < Minitest::Test
       fixture_path("parity_people_reordered.tsv"),
       "2",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "MATCH"
-    assert_includes output.string, "Summary: left_rows=3 right_rows=3 left_only=0 right_only=0"
+    assert_includes output.string, "Left rows"
+    assert_includes output.string, "Right rows"
   end
 
   def test_parity_workflow_headerless_mode_compares_all_rows
@@ -488,14 +794,15 @@ class TestCli < Minitest::Test
       fixture_path("sample_people_no_headers.csv"),
       "",
       "n",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "MATCH"
-    assert_includes output.string, "Summary: left_rows=3 right_rows=3 left_only=0 right_only=0"
+    assert_includes output.string, "Left rows"
+    assert_includes output.string, "Right rows"
   end
 
   def test_parity_workflow_reports_header_mismatch_in_headered_mode
@@ -506,10 +813,10 @@ class TestCli < Minitest::Test
       fixture_path("parity_people_header_mismatch.csv"),
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "CSV headers do not match."
@@ -524,14 +831,15 @@ class TestCli < Minitest::Test
       fixture_path("parity_people_mismatch.csv"),
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "MISMATCH"
-    assert_includes output.string, "Summary: left_rows=3 right_rows=3 left_only=1 right_only=1"
+    assert_includes output.string, "Left only"
+    assert_includes output.string, "Right only"
     assert_includes output.string, "Left-only examples:"
     assert_includes output.string, "Cara,Berlin (count +1)"
     assert_includes output.string, "Right-only examples:"
@@ -546,10 +854,10 @@ class TestCli < Minitest::Test
       fixture_path("sample_people.csv"),
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "File not found: /tmp/not-there-left.csv"
@@ -565,10 +873,10 @@ class TestCli < Minitest::Test
       "/tmp/not-there-right.csv",
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "File not found: /tmp/not-there-right.csv"
@@ -584,10 +892,10 @@ class TestCli < Minitest::Test
       fixture_path("sample_people_bad_tail.csv"),
       "",
       "",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Could not parse CSV file."
@@ -611,10 +919,10 @@ class TestCli < Minitest::Test
         "y",
         "2",
         output_path,
-        "7"
+        "8"
       ].join("\n") + "\n"
 
-      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
       assert_equal 0, status
       assert_equal "name\nAlice\nBob\nCara\n", File.read(output_path)
     end
@@ -631,11 +939,11 @@ class TestCli < Minitest::Test
       "1",
       "",
       "n",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
     output = StringIO.new
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Canceled."
@@ -648,7 +956,7 @@ class TestCli < Minitest::Test
       ["menu"],
       stdin: StringIO.new("1\n/tmp/does-not-exist.csv\n4\n7\n"),
       stdout: output,
-      stderr: StringIO.new
+      stderr: output
     )
 
     assert_equal 0, status
@@ -667,11 +975,11 @@ class TestCli < Minitest::Test
       "y",
       "2",
       "/tmp/not-a-dir/out.csv",
-      "7"
+      "8"
     ].join("\n") + "\n"
 
     output = StringIO.new
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: output)
 
     assert_equal 0, status
     assert_includes output.string, "Cannot write output file: /tmp/not-a-dir/out.csv"