csvops 0.8.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,7 +13,7 @@ class TestCli < Minitest::Test
 
   def test_menu_can_exit_cleanly
     output = StringIO.new
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("8\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
@@ -28,12 +29,13 @@ class TestCli < Minitest::Test
       "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 Stats Summary"
-    assert_includes output.string, "Rows: 3"
-    assert_includes output.string, "Columns: 2"
+    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
 
@@ -48,7 +50,7 @@ class TestCli < Minitest::Test
       "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/does-not-exist.csv"
@@ -70,7 +72,7 @@ class TestCli < Minitest::Test
         "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}"
@@ -98,10 +100,10 @@ class TestCli < Minitest::Test
         "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"))
@@ -119,7 +121,7 @@ class TestCli < Minitest::Test
       "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."
@@ -140,7 +142,7 @@ class TestCli < Minitest::Test
     ].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)
@@ -159,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 = [
@@ -171,7 +411,7 @@ class TestCli < Minitest::Test
       "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"
@@ -192,7 +432,7 @@ class TestCli < Minitest::Test
       "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."
@@ -211,7 +451,7 @@ class TestCli < Minitest::Test
       "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"
@@ -232,7 +472,7 @@ class TestCli < Minitest::Test
       "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"
@@ -257,7 +497,7 @@ class TestCli < Minitest::Test
         "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
@@ -277,7 +517,7 @@ class TestCli < Minitest::Test
       "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"
@@ -297,7 +537,7 @@ class TestCli < Minitest::Test
       "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"
@@ -322,7 +562,7 @@ class TestCli < Minitest::Test
         "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}"
@@ -344,7 +584,7 @@ class TestCli < Minitest::Test
       "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"
@@ -363,7 +603,7 @@ class TestCli < Minitest::Test
       "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"
@@ -383,7 +623,7 @@ class TestCli < Minitest::Test
       "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."
@@ -408,7 +648,7 @@ class TestCli < Minitest::Test
       "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:"
@@ -417,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
@@ -442,12 +682,12 @@ class TestCli < Minitest::Test
       "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
 
@@ -469,7 +709,7 @@ class TestCli < Minitest::Test
       "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"
@@ -495,13 +735,13 @@ class TestCli < Minitest::Test
       "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
@@ -515,13 +755,15 @@ class TestCli < Minitest::Test
       "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
 
@@ -536,11 +778,12 @@ class TestCli < Minitest::Test
       "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
@@ -554,11 +797,12 @@ class TestCli < Minitest::Test
       "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
@@ -572,7 +816,7 @@ class TestCli < Minitest::Test
       "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."
@@ -590,11 +834,12 @@ class TestCli < Minitest::Test
       "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:"
@@ -612,7 +857,7 @@ class TestCli < Minitest::Test
       "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"
@@ -631,7 +876,7 @@ class TestCli < Minitest::Test
       "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"
@@ -650,7 +895,7 @@ class TestCli < Minitest::Test
       "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."
@@ -677,7 +922,7 @@ class TestCli < Minitest::Test
         "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
@@ -698,7 +943,7 @@ class TestCli < Minitest::Test
     ].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."
@@ -711,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
@@ -734,7 +979,7 @@ class TestCli < Minitest::Test
     ].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"

data/test/csvtool/cli_unit_test.rb

@@ -16,7 +16,7 @@ class CliUnitTest < Minitest::Test
   end
 
   def test_menu_command_can_exit_zero
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("6\n"), stdout: StringIO.new, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("8\n"), stdout: StringIO.new, stderr: StringIO.new)
     assert_equal 0, status
   end
 
@@ -28,7 +28,7 @@ class CliUnitTest < Minitest::Test
   def test_menu_routes_to_row_range_shell
     stdout = StringIO.new
     fixture = File.expand_path("../fixtures/sample_people.csv", __dir__)
-    input = ["2", fixture, "", "2", "3", "", "6"].join("\n") + "\n"
+    input = ["2", fixture, "", "2", "3", "", "8"].join("\n") + "\n"
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: stdout, stderr: StringIO.new)
     assert_equal 0, status
     assert_includes stdout.string, "name,city"
@@ -39,7 +39,7 @@ class CliUnitTest < Minitest::Test
   def test_menu_routes_to_randomize_rows_shell
     stdout = StringIO.new
     fixture = File.expand_path("../fixtures/sample_people.csv", __dir__)
-    input = ["3", fixture, "", "", "", "", "6"].join("\n") + "\n"
+    input = ["3", fixture, "", "", "", "", "8"].join("\n") + "\n"
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: stdout, stderr: StringIO.new)
     assert_equal 0, status
     assert_includes stdout.string, "name,city"
@@ -52,12 +52,12 @@ class CliUnitTest < Minitest::Test
     stdout = StringIO.new
     source_fixture = File.expand_path("../fixtures/dedupe_source.csv", __dir__)
     reference_fixture = File.expand_path("../fixtures/dedupe_reference.csv", __dir__)
-    input = ["4", source_fixture, "", "", reference_fixture, "", "", "customer_id", "external_id", "", "", "", "6"].join("\n") + "\n"
+    input = ["4", source_fixture, "", "", reference_fixture, "", "", "customer_id", "external_id", "", "", "", "8"].join("\n") + "\n"
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: stdout, stderr: StringIO.new)
     assert_equal 0, status
     assert_includes stdout.string, "customer_id,name"
     assert_includes stdout.string, "1,Alice"
     assert_includes stdout.string, "3,Cara"
-    assert_includes stdout.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
+    assert_includes stdout.string, "Summary"
   end
 end

data/test/csvtool/interface/cli/output/color_policy_test.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/output/color_policy"
+
+class ColorPolicyTest < Minitest::Test
+  class TtyIO
+    def tty? = true
+  end
+
+  class NonTtyIO
+    def tty? = false
+  end
+
+  def test_auto_uses_tty
+    enabled = Csvtool::Interface::CLI::Output::ColorPolicy.new(mode: "auto", io: TtyIO.new, env: {}).enabled?
+    disabled = Csvtool::Interface::CLI::Output::ColorPolicy.new(mode: "auto", io: NonTtyIO.new, env: {}).enabled?
+
+    assert_equal true, enabled
+    assert_equal false, disabled
+  end
+
+  def test_never_disables_color
+    policy = Csvtool::Interface::CLI::Output::ColorPolicy.new(mode: "never", io: TtyIO.new, env: {})
+
+    assert_equal false, policy.enabled?
+  end
+
+  def test_always_enables_color_even_with_no_color
+    policy = Csvtool::Interface::CLI::Output::ColorPolicy.new(mode: "always", io: NonTtyIO.new, env: { "NO_COLOR" => "1" })
+
+    assert_equal true, policy.enabled?
+  end
+
+  def test_no_color_disables_auto
+    policy = Csvtool::Interface::CLI::Output::ColorPolicy.new(mode: "auto", io: TtyIO.new, env: { "NO_COLOR" => "1" })
+
+    assert_equal false, policy.enabled?
+  end
+end