csvops 0.8.0.alpha → 0.9.0.alpha

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

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/output/colorizer"
+
+class ColorizerTest < Minitest::Test
+  class FakePolicy
+    def initialize(enabled)
+      @enabled = enabled
+    end
+
+    def enabled?
+      @enabled
+    end
+  end
+
+  def test_wraps_text_when_enabled
+    colorizer = Csvtool::Interface::CLI::Output::Colorizer.new(policy: FakePolicy.new(true))
+
+    assert_equal "\e[31mMISMATCH\e[0m", colorizer.call("MISMATCH", code: "31")
+  end
+
+  def test_returns_text_when_disabled
+    colorizer = Csvtool::Interface::CLI::Output::Colorizer.new(policy: FakePolicy.new(false))
+
+    assert_equal "MATCH", colorizer.call("MATCH", code: "32")
+  end
+end

data/test/csvtool/interface/cli/output/formatters/csv_row_formatter_test.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require_relative "../../../../../test_helper"
+require "csvtool/interface/cli/output/formatters/csv_row_formatter"
+
+class CsvRowFormatterTest < Minitest::Test
+  def test_formats_row_with_separator
+    formatter = Csvtool::Interface::CLI::Output::Formatters::CsvRowFormatter.new
+
+    row = formatter.call(fields: ["Alice", "London"], col_sep: ",")
+
+    assert_equal "Alice,London", row
+  end
+
+  def test_quotes_values_when_needed
+    formatter = Csvtool::Interface::CLI::Output::Formatters::CsvRowFormatter.new
+
+    row = formatter.call(fields: ["Alice, Jr", "London"], col_sep: ",")
+
+    assert_equal '"Alice, Jr",London', row
+  end
+end

data/test/csvtool/interface/cli/output/formatters/stats_formatter_test.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require_relative "../../../../../test_helper"
+require "csvtool/interface/cli/output/formatters/stats_formatter"
+require "csvtool/interface/cli/output/table_renderer"
+require "json"
+
+class StatsFormatterTest < Minitest::Test
+  def sample_data
+    {
+      row_count: 3,
+      column_count: 2,
+      headers: ["name", "city"],
+      column_stats: [
+        { name: "name", non_blank_count: 3, blank_count: 0 },
+        { name: "city", non_blank_count: 2, blank_count: 1 }
+      ]
+    }
+  end
+
+  def formatter
+    Csvtool::Interface::CLI::Output::Formatters::StatsFormatter.new(
+      table_renderer: Csvtool::Interface::CLI::Output::TableRenderer.new
+    )
+  end
+
+  def test_formats_text_summary
+    text = formatter.call(data: sample_data, format: "text", max_width: 80)
+
+    assert_includes text, "CSV Stats Summary"
+    assert_includes text, "Metric"
+    assert_includes text, "Column completeness:"
+  end
+
+  def test_formats_json_summary
+    json = formatter.call(data: sample_data, format: "json", max_width: 80)
+
+    parsed = JSON.parse(json, symbolize_names: true)
+    assert_equal 3, parsed[:row_count]
+    assert_equal 2, parsed[:column_count]
+  end
+
+  def test_formats_csv_summary
+    csv = formatter.call(data: sample_data, format: "csv", max_width: 80)
+
+    lines = csv.lines.map(&:chomp)
+    assert_equal "metric,value", lines.first
+    assert_includes lines, "row_count,3"
+    assert_includes lines, "column.city.blank,1"
+  end
+end

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

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/output/streams"
+
+class StreamsTest < Minitest::Test
+  def test_builds_data_and_ui_streams
+    data = StringIO.new
+    ui = StringIO.new
+
+    streams = Csvtool::Interface::CLI::Output::Streams.build(data: data, ui: ui)
+
+    assert_same data, streams.data
+    assert_same ui, streams.ui
+  end
+
+  def test_defaults_ui_to_data_stream
+    data = StringIO.new
+
+    streams = Csvtool::Interface::CLI::Output::Streams.build(data: data)
+
+    assert_same data, streams.data
+    assert_same data, streams.ui
+  end
+end

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/output/table_renderer"
+
+class TableRendererTest < Minitest::Test
+  def test_renders_aligned_table
+    renderer = Csvtool::Interface::CLI::Output::TableRenderer.new
+
+    text = renderer.render(
+      headers: ["Metric", "Value"],
+      rows: [["Rows", "3"], ["Columns", "2"]],
+      max_width: 80
+    )
+
+    lines = text.lines.map(&:chomp)
+    assert_equal "Metric  | Value", lines[0]
+    assert_equal "--------+------", lines[1]
+    assert_equal "Rows    | 3    ", lines[2]
+    assert_equal "Columns | 2    ", lines[3]
+  end
+
+  def test_truncates_cells_when_width_is_narrow
+    renderer = Csvtool::Interface::CLI::Output::TableRenderer.new
+
+    text = renderer.render(
+      headers: ["Column", "Non-blank", "Blank"],
+      rows: [["very_long_column_name", "123456", "0"]],
+      max_width: 26
+    )
+
+    lines = text.lines.map(&:chomp)
+    assert lines.all? { |line| line.length <= 26 }
+    assert_includes lines[2], "..."
+  end
+end

data/test/csvtool/interface/cli/workflows/presenters/cross_csv_dedupe_presenter_test.rb

@@ -14,7 +14,10 @@ class CrossCsvDedupePresenterTest < Minitest::Test
 
     assert_includes out.string, "\nid,name\n"
     assert_includes out.string, "1,Alice"
-    assert_includes out.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
+    assert_includes out.string, "Summary"
+    assert_includes out.string, "Source rows"
+    assert_includes out.string, "Removed rows"
+    assert_includes out.string, "Kept rows"
   end
 
   def test_prints_zero_and_all_removed_messages

data/test/csvtool/interface/cli/workflows/presenters/csv_parity_presenter_test.rb

@@ -17,7 +17,11 @@ class CsvParityPresenterTest < Minitest::Test
     )
 
     assert_includes out.string, "MATCH"
-    assert_includes out.string, "Summary: left_rows=3 right_rows=3 left_only=0 right_only=0"
+    assert_includes out.string, "Metric"
+    assert_includes out.string, "Left rows"
+    assert_includes out.string, "Right rows"
+    assert_includes out.string, "Left only"
+    assert_includes out.string, "Right only"
   end
 
   def test_prints_mismatch_examples

data/test/csvtool/interface/cli/workflows/presenters/csv_split_presenter_test.rb

@@ -17,10 +17,28 @@ class CsvSplitPresenterTest < Minitest::Test
     )
 
     assert_includes out.string, "Split complete."
-    assert_includes out.string, "Chunk size: 10"
-    assert_includes out.string, "Data rows: 25"
-    assert_includes out.string, "Chunks written: 3"
-    assert_includes out.string, "Manifest: /tmp/manifest.csv"
+    assert_includes out.string, "Metric"
+    assert_includes out.string, "Chunk size"
+    assert_includes out.string, "Data rows"
+    assert_includes out.string, "Chunks written"
+    assert_includes out.string, "Manifest"
     assert_includes out.string, "/tmp/people_part_001.csv"
   end
+
+  def test_truncates_summary_table_for_narrow_width
+    out = StringIO.new
+    presenter = Csvtool::Interface::CLI::Workflows::Presenters::CsvSplitPresenter.new(stdout: out, max_width: 26)
+
+    presenter.print_summary(
+      chunk_size: 10,
+      data_rows: 25,
+      chunk_count: 3,
+      manifest_path: "/tmp/very/long/path/manifest.csv",
+      chunk_paths: []
+    )
+
+    lines = out.string.lines.map(&:chomp)
+    assert lines.all? { |line| line.empty? || line.length <= 26 }
+    assert_includes out.string, "..."
+  end
 end

data/test/csvtool/interface/cli/workflows/presenters/csv_stats_presenter_test.rb

@@ -19,11 +19,13 @@ class CsvStatsPresenterTest < Minitest::Test
     )
 
     assert_includes out.string, "CSV Stats Summary"
-    assert_includes out.string, "Rows: 3"
-    assert_includes out.string, "Columns: 2"
-    assert_includes out.string, "Headers: name, city"
-    assert_includes out.string, "name: non_blank=3 blank=0"
-    assert_includes out.string, "city: non_blank=2 blank=1"
+    assert_includes out.string, "Metric"
+    assert_includes out.string, "Rows"
+    assert_includes out.string, "Columns"
+    assert_includes out.string, "Headers"
+    assert_includes out.string, "Column completeness:"
+    assert_includes out.string, "name"
+    assert_includes out.string, "city"
   end
 
   def test_prints_file_written_message

data/test/csvtool/interface/cli/workflows/run_cross_csv_dedupe_workflow_test.rb

@@ -38,7 +38,10 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
     assert_includes output.string, "3,Cara"
     refute_includes output.string, "2,Bob"
     refute_includes output.string, "4,Dan"
-    assert_includes output.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
+    assert_includes output.string, "Summary"
+    assert_includes output.string, "Source rows"
+    assert_includes output.string, "Removed rows"
+    assert_includes output.string, "Kept rows"
   end
 
   def test_can_write_deduped_rows_to_file
@@ -67,7 +70,7 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
 
       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
 
@@ -119,7 +122,7 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
     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_reports_column_not_found_when_missing
@@ -164,7 +167,7 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
       .new(stdin: StringIO.new(input), stdout: output)
       .call
 
-    assert_includes output.string, "Summary: source_rows=5 removed_rows=0 kept_rows=5"
+    assert_includes output.string, "Summary"
     assert_includes output.string, "No rows removed; no matching keys found."
   end
 
@@ -188,7 +191,7 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
       .new(stdin: StringIO.new(input), stdout: output)
       .call
 
-    assert_includes output.string, "Summary: source_rows=5 removed_rows=5 kept_rows=0"
+    assert_includes output.string, "Summary"
     assert_includes output.string, "All source rows were removed by dedupe."
   end
 
@@ -215,7 +218,7 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
     refute_includes output.string, " A1 ,Alice"
     refute_includes output.string, "c3,Cara"
     assert_includes output.string, "B2,Bob"
-    assert_includes output.string, "Summary: source_rows=3 removed_rows=2 kept_rows=1"
+    assert_includes output.string, "Summary"
   end
 
   def test_normalization_disabled_preserves_exact_match_behavior
@@ -241,6 +244,6 @@ class RunCrossCsvDedupeWorkflowTest < Minitest::Test
     assert_includes output.string, " A1 ,Alice"
     assert_includes output.string, "B2,Bob"
     assert_includes output.string, "c3,Cara"
-    assert_includes output.string, "Summary: source_rows=3 removed_rows=0 kept_rows=3"
+    assert_includes output.string, "Summary"
   end
 end

data/test/csvtool/interface/cli/workflows/run_csv_parity_workflow_test.rb

@@ -62,7 +62,9 @@ class RunCsvParityWorkflowTest < Minitest::Test
     assert_includes stdout.string, "Choose separator:"
     assert_includes stdout.string, "Headers present? [Y/n]: "
     assert_includes stdout.string, "MATCH"
-    assert_includes stdout.string, "Summary: left_rows=3 right_rows=3 left_only=0 right_only=0"
+    assert_includes stdout.string, "Metric"
+    assert_includes stdout.string, "Left rows"
+    assert_includes stdout.string, "Right rows"
   end
 
   def test_prints_mismatch_examples_when_not_equal

data/test/csvtool/interface/cli/workflows/run_csv_split_workflow_test.rb

@@ -24,8 +24,9 @@ class RunCsvSplitWorkflowTest < Minitest::Test
       ).call
 
       assert_includes out.string, "Split complete."
-      assert_includes out.string, "Chunk size: 10"
-      assert_includes out.string, "Chunks written: 3"
+      assert_includes out.string, "Metric"
+      assert_includes out.string, "Chunk size"
+      assert_includes out.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"))
@@ -194,7 +195,8 @@ class RunCsvSplitWorkflowTest < Minitest::Test
       assert_includes lines[1], ",10"
       assert_includes lines[2], ",10"
       assert_includes lines[3], ",5"
-      assert_includes out.string, "Manifest: #{manifest_path}"
+      assert_includes out.string, "Manifest"
+      assert_includes out.string, "manifest.csv"
     end
   end
 end

data/test/csvtool/interface/cli/workflows/run_csv_stats_workflow_test.rb

@@ -19,12 +19,13 @@ class RunCsvStatsWorkflowTest < Minitest::Test
     ).call
 
     assert_includes out.string, "CSV Stats Summary"
-    assert_includes out.string, "Rows: 3"
-    assert_includes out.string, "Columns: 2"
-    assert_includes out.string, "Headers: name, city"
+    assert_includes out.string, "Metric"
+    assert_includes out.string, "Rows"
+    assert_includes out.string, "Columns"
+    assert_includes out.string, "Headers"
     assert_includes out.string, "Column completeness:"
-    assert_includes out.string, "name: non_blank=3 blank=0"
-    assert_includes out.string, "city: non_blank=3 blank=0"
+    assert_includes out.string, "name"
+    assert_includes out.string, "city"
   end
 
   def test_workflow_supports_tsv_separator
@@ -36,9 +37,9 @@ class RunCsvStatsWorkflowTest < Minitest::Test
       stdout: out
     ).call
 
-    assert_includes out.string, "Rows: 3"
-    assert_includes out.string, "Columns: 2"
-    assert_includes out.string, "Headers: name, city"
+    assert_includes out.string, "Rows"
+    assert_includes out.string, "Columns"
+    assert_includes out.string, "Headers"
   end
 
   def test_workflow_supports_headerless_mode
@@ -50,11 +51,11 @@ class RunCsvStatsWorkflowTest < Minitest::Test
       stdout: out
     ).call
 
-    assert_includes out.string, "Rows: 3"
-    assert_includes out.string, "Columns: 2"
-    refute_includes out.string, "Headers:"
-    assert_includes out.string, "column_1: non_blank=3 blank=0"
-    assert_includes out.string, "column_2: non_blank=3 blank=0"
+    assert_includes out.string, "Rows"
+    assert_includes out.string, "Columns"
+    refute_includes out.string, "Headers |"
+    assert_includes out.string, "column_1"
+    assert_includes out.string, "column_2"
   end
 
   def test_workflow_supports_custom_separator
@@ -66,9 +67,9 @@ class RunCsvStatsWorkflowTest < Minitest::Test
       stdout: out
     ).call
 
-    assert_includes out.string, "Rows: 3"
-    assert_includes out.string, "Columns: 2"
-    assert_includes out.string, "Headers: name, city"
+    assert_includes out.string, "Rows"
+    assert_includes out.string, "Columns"
+    assert_includes out.string, "Headers"
   end
 
   def test_workflow_prints_column_completeness_for_blank_values
@@ -80,8 +81,12 @@ class RunCsvStatsWorkflowTest < Minitest::Test
       stdout: out
     ).call
 
-    assert_includes out.string, "name: non_blank=3 blank=2"
-    assert_includes out.string, "city: non_blank=4 blank=1"
+    assert_includes out.string, "name"
+    assert_includes out.string, "3"
+    assert_includes out.string, "2"
+    assert_includes out.string, "city"
+    assert_includes out.string, "4"
+    assert_includes out.string, "1"
   end
 
   def test_workflow_reports_missing_file

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: csvops
 version: !ruby/object:Gem::Version
-  version: 0.8.0.alpha
+  version: 0.9.0.alpha
 platform: ruby
 authors:
 - Robert Hall
@@ -68,6 +68,7 @@ files:
 - bin/tool
 - csvops.gemspec
 - docs/architecture.md
+- docs/cli-output-conventions.md
 - docs/release-v0.1.0-alpha.md
 - docs/release-v0.2.0-alpha.md
 - docs/release-v0.3.0-alpha.md
@@ -76,6 +77,7 @@ files:
 - docs/release-v0.6.0-alpha.md
 - docs/release-v0.7.0-alpha.md
 - docs/release-v0.8.0-alpha.md
+- docs/release-v0.9.0-alpha.md
 - exe/csvtool
 - lib/csvtool/application/use_cases/run_cross_csv_dedupe.rb
 - lib/csvtool/application/use_cases/run_csv_parity.rb
@@ -132,6 +134,12 @@ files:
 - lib/csvtool/infrastructure/output/csv_stats_file_writer.rb
 - lib/csvtool/interface/cli/errors/presenter.rb
 - lib/csvtool/interface/cli/menu_loop.rb
+- lib/csvtool/interface/cli/output/color_policy.rb
+- lib/csvtool/interface/cli/output/colorizer.rb
+- lib/csvtool/interface/cli/output/formatters/csv_row_formatter.rb
+- lib/csvtool/interface/cli/output/formatters/stats_formatter.rb
+- lib/csvtool/interface/cli/output/streams.rb
+- lib/csvtool/interface/cli/output/table_renderer.rb
 - lib/csvtool/interface/cli/prompts/chunk_size_prompt.rb
 - lib/csvtool/interface/cli/prompts/column_selector_prompt.rb
 - lib/csvtool/interface/cli/prompts/confirm_prompt.rb
@@ -249,6 +257,12 @@ files:
 - test/csvtool/infrastructure/output/csv_stats_file_writer_test.rb
 - test/csvtool/interface/cli/errors/presenter_test.rb
 - test/csvtool/interface/cli/menu_loop_test.rb
+- test/csvtool/interface/cli/output/color_policy_test.rb
+- test/csvtool/interface/cli/output/colorizer_test.rb
+- test/csvtool/interface/cli/output/formatters/csv_row_formatter_test.rb
+- test/csvtool/interface/cli/output/formatters/stats_formatter_test.rb
+- test/csvtool/interface/cli/output/streams_test.rb
+- test/csvtool/interface/cli/output/table_renderer_test.rb
 - test/csvtool/interface/cli/prompts/chunk_size_prompt_test.rb
 - test/csvtool/interface/cli/prompts/column_selector_prompt_test.rb
 - test/csvtool/interface/cli/prompts/confirm_prompt_test.rb