csvops 0.5.0.alpha → 0.6.0.alpha

data/lib/csvtool/interface/cli/workflows/run_csv_parity_workflow.rb

@@ -0,0 +1,66 @@
+# frozen_string_literal: true
+
+require "csvtool/application/use_cases/run_csv_parity"
+require "csvtool/interface/cli/errors/presenter"
+require "csvtool/interface/cli/prompts/file_path_prompt"
+require "csvtool/interface/cli/prompts/separator_prompt"
+require "csvtool/interface/cli/prompts/headers_present_prompt"
+require "csvtool/interface/cli/workflows/builders/csv_parity_session_builder"
+require "csvtool/interface/cli/workflows/presenters/csv_parity_presenter"
+require "csvtool/interface/cli/workflows/support/result_error_handler"
+require "csvtool/interface/cli/workflows/steps/workflow_step_pipeline"
+require "csvtool/interface/cli/workflows/steps/parity/collect_inputs_step"
+require "csvtool/interface/cli/workflows/steps/parity/build_session_step"
+require "csvtool/interface/cli/workflows/steps/parity/execute_step"
+
+module Csvtool
+  module Interface
+    module CLI
+      module Workflows
+        class RunCsvParityWorkflow
+          def initialize(stdin:, stdout:, use_case: Application::UseCases::RunCsvParity.new)
+            @stdin = stdin
+            @stdout = stdout
+            @use_case = use_case
+            @errors = Interface::CLI::Errors::Presenter.new(stdout: stdout)
+            @session_builder = Builders::CsvParitySessionBuilder.new
+            @presenter = Presenters::CsvParityPresenter.new(stdout: stdout)
+            @result_error_handler = Support::ResultErrorHandler.new(errors: @errors)
+          end
+
+          def call
+            context = {
+              use_case: @use_case,
+              session_builder: @session_builder,
+              presenter: @presenter,
+              handle_error: method(:handle_error)
+            }
+            pipeline = Steps::WorkflowStepPipeline.new(steps: [
+              Steps::Parity::CollectInputsStep.new(
+                file_path_prompt: Interface::CLI::Prompts::FilePathPrompt.new(stdin: @stdin, stdout: @stdout),
+                separator_prompt: Interface::CLI::Prompts::SeparatorPrompt.new(stdin: @stdin, stdout: @stdout, errors: @errors),
+                headers_present_prompt: Interface::CLI::Prompts::HeadersPresentPrompt.new(stdin: @stdin, stdout: @stdout)
+              ),
+              Steps::Parity::BuildSessionStep.new,
+              Steps::Parity::ExecuteStep.new
+            ])
+            pipeline.call(context)
+            nil
+          end
+
+          private
+
+          def handle_error(result)
+            @result_error_handler.call(result, {
+              file_not_found: ->(r, errors) { errors.file_not_found(r.data[:path]) },
+              could_not_parse_csv: ->(_r, errors) { errors.could_not_parse_csv },
+              cannot_read_file: ->(r, errors) { errors.cannot_read_file(r.data[:path]) },
+              no_headers: ->(_r, errors) { errors.no_headers },
+              header_mismatch: ->(_r, errors) { errors.header_mismatch }
+            })
+          end
+        end
+      end
+    end
+  end
+end

data/lib/csvtool/interface/cli/workflows/steps/parity/build_session_step.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module Csvtool
+  module Interface
+    module CLI
+      module Workflows
+        module Steps
+          module Parity
+            class BuildSessionStep
+              def call(context)
+                context[:session] = context.fetch(:session_builder).call(
+                  left_path: context.fetch(:left_path),
+                  right_path: context.fetch(:right_path),
+                  col_sep: context.fetch(:col_sep),
+                  headers_present: context.fetch(:headers_present)
+                )
+                nil
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/csvtool/interface/cli/workflows/steps/parity/collect_inputs_step.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module Csvtool
+  module Interface
+    module CLI
+      module Workflows
+        module Steps
+          module Parity
+            class CollectInputsStep
+              def initialize(file_path_prompt:, separator_prompt:, headers_present_prompt:)
+                @file_path_prompt = file_path_prompt
+                @separator_prompt = separator_prompt
+                @headers_present_prompt = headers_present_prompt
+              end
+
+              def call(context)
+                context[:left_path] = @file_path_prompt.call(label: "Left CSV file path: ")
+                context[:right_path] = @file_path_prompt.call(label: "Right CSV file path: ")
+                col_sep = @separator_prompt.call
+                return :halt if col_sep.nil?
+
+                context[:col_sep] = col_sep
+                context[:headers_present] = @headers_present_prompt.call
+                nil
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/csvtool/interface/cli/workflows/steps/parity/execute_step.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Csvtool
+  module Interface
+    module CLI
+      module Workflows
+        module Steps
+          module Parity
+            class ExecuteStep
+              def call(context)
+                result = context.fetch(:use_case).call(session: context.fetch(:session))
+                unless result.ok?
+                  context.fetch(:handle_error).call(result)
+                  return :halt
+                end
+
+                context.fetch(:presenter).print_summary(result.data)
+                nil
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/csvtool/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Csvtool
-  VERSION = "0.5.0.alpha"
+  VERSION = "0.6.0.alpha"
 end

data/test/csvtool/application/use_cases/run_csv_parity_test.rb

@@ -0,0 +1,160 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/application/use_cases/run_csv_parity"
+require "csvtool/domain/csv_parity_session/source_pair"
+require "csvtool/domain/csv_parity_session/parity_options"
+require "csvtool/domain/csv_parity_session/parity_session"
+
+class RunCsvParityTest < Minitest::Test
+  class EaccesComparator
+    def call(left_path:, right_path:, col_sep:, headers_present:, sample_limit: 5)
+      error = Errno::EACCES.new("/tmp/protected.csv")
+      def error.path
+        "/tmp/protected.csv"
+      end
+      raise error
+    end
+  end
+
+  def fixture_path(name)
+    File.expand_path("../../../fixtures/#{name}", __dir__)
+  end
+
+  def build_session(left_path:, right_path:, separator: ",", headers_present: true)
+    source_pair = Csvtool::Domain::CsvParitySession::SourcePair.new(
+      left_path: left_path,
+      right_path: right_path
+    )
+    options = Csvtool::Domain::CsvParitySession::ParityOptions.new(
+      separator: separator,
+      headers_present: headers_present
+    )
+    Csvtool::Domain::CsvParitySession::ParitySession.start(
+      source_pair: source_pair,
+      options: options
+    )
+  end
+
+  def test_returns_match_for_equivalent_files
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("sample_people.csv"),
+        right_path: fixture_path("parity_people_reordered.csv")
+      )
+    )
+
+    assert_equal true, result.ok?
+    assert_equal true, result.data[:match]
+    assert_equal 0, result.data[:left_only_count]
+    assert_equal 0, result.data[:right_only_count]
+  end
+
+  def test_returns_mismatch_counts_for_non_equivalent_files
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("sample_people.csv"),
+        right_path: fixture_path("parity_people_mismatch.csv")
+      )
+    )
+
+    assert_equal true, result.ok?
+    assert_equal false, result.data[:match]
+    assert_equal 1, result.data[:left_only_count]
+    assert_equal 1, result.data[:right_only_count]
+  end
+
+  def test_duplicate_count_differences_are_detected
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("parity_duplicates_left.csv"),
+        right_path: fixture_path("parity_duplicates_right.csv")
+      )
+    )
+
+    assert_equal true, result.ok?
+    assert_equal false, result.data[:match]
+    assert_equal 1, result.data[:left_only_count]
+    assert_equal 0, result.data[:right_only_count]
+    assert_equal "1,Alice", result.data[:left_only_examples][0][:row]
+    assert_equal 1, result.data[:left_only_examples][0][:count_delta]
+  end
+
+  def test_headered_mode_fails_when_headers_do_not_match
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("sample_people.csv"),
+        right_path: fixture_path("parity_people_header_mismatch.csv")
+      )
+    )
+
+    assert_equal false, result.ok?
+    assert_equal :header_mismatch, result.error
+  end
+
+  def test_headerless_mode_compares_all_rows_as_data
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("sample_people_no_headers.csv"),
+        right_path: fixture_path("sample_people_no_headers.csv"),
+        headers_present: false
+      )
+    )
+
+    assert_equal true, result.ok?
+    assert_equal true, result.data[:match]
+  end
+
+  def test_returns_file_not_found_for_left_side
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: "/tmp/nope-left.csv",
+        right_path: fixture_path("sample_people.csv")
+      )
+    )
+
+    assert_equal false, result.ok?
+    assert_equal :file_not_found, result.error
+    assert_equal "/tmp/nope-left.csv", result.data[:path]
+  end
+
+  def test_returns_file_not_found_for_right_side
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("sample_people.csv"),
+        right_path: "/tmp/nope-right.csv"
+      )
+    )
+
+    assert_equal false, result.ok?
+    assert_equal :file_not_found, result.error
+    assert_equal "/tmp/nope-right.csv", result.data[:path]
+  end
+
+  def test_returns_parse_error_for_malformed_csv
+    result = Csvtool::Application::UseCases::RunCsvParity.new.call(
+      session: build_session(
+        left_path: fixture_path("sample_people.csv"),
+        right_path: fixture_path("sample_people_bad_tail.csv")
+      )
+    )
+
+    assert_equal false, result.ok?
+    assert_equal :could_not_parse_csv, result.error
+  end
+
+  def test_returns_cannot_read_file_when_eacces_is_raised
+    result = Csvtool::Application::UseCases::RunCsvParity.new(
+      comparator: EaccesComparator.new
+    ).call(
+      session: build_session(
+        left_path: fixture_path("sample_people.csv"),
+        right_path: fixture_path("sample_people.csv")
+      )
+    )
+
+    assert_equal false, result.ok?
+    assert_equal :cannot_read_file, result.error
+    assert_equal "/tmp/protected.csv", result.data[:path]
+  end
+end

data/test/csvtool/cli_test.rb

@@ -11,7 +11,7 @@ class TestCli < Minitest::Test
 
   def test_menu_can_exit_cleanly
     output = StringIO.new
-    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("5\n"), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("6\n"), stdout: output, stderr: StringIO.new)
     assert_equal 0, status
     assert_includes output.string, "CSV Tool Menu"
   end
@@ -26,7 +26,7 @@ class TestCli < Minitest::Test
       "",
       "y",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     output = StringIO.new
@@ -58,7 +58,7 @@ class TestCli < Minitest::Test
       "2",
       "3",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -79,7 +79,7 @@ class TestCli < Minitest::Test
       "0",
       "3",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -98,7 +98,7 @@ class TestCli < Minitest::Test
       "2",
       "3",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -119,7 +119,7 @@ class TestCli < Minitest::Test
       "2",
       "3",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -144,7 +144,7 @@ class TestCli < Minitest::Test
         "3",
         "2",
         output_path,
-        "5"
+        "6"
       ].join("\n") + "\n"
 
       status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -164,7 +164,7 @@ class TestCli < Minitest::Test
       "1",
       "2",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -184,7 +184,7 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -209,7 +209,7 @@ class TestCli < Minitest::Test
         "",
         "2",
         output_path,
-        "5"
+        "6"
       ].join("\n") + "\n"
 
       status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -231,7 +231,7 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -250,7 +250,7 @@ class TestCli < Minitest::Test
       "n",
       "",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -270,7 +270,7 @@ class TestCli < Minitest::Test
       "",
       "",
       "abc",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -295,7 +295,7 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -329,7 +329,7 @@ class TestCli < Minitest::Test
       "",
       "2",
       output_path,
-      "5"
+      "6"
       ].join("\n") + "\n"
 
       status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -356,7 +356,7 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -382,7 +382,7 @@ class TestCli < Minitest::Test
       "",
       "",
       "",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -394,6 +394,160 @@ class TestCli < Minitest::Test
     assert_includes output.string, "Summary: source_rows=5 removed_rows=3 kept_rows=2"
   end
 
+  def test_parity_workflow_reports_match_and_returns_to_menu
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people.csv"),
+      fixture_path("sample_people.csv"),
+      "",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    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_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+
+  def test_parity_workflow_supports_tsv_separator
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people.tsv"),
+      fixture_path("parity_people_reordered.tsv"),
+      "2",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    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"
+  end
+
+  def test_parity_workflow_headerless_mode_compares_all_rows
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people_no_headers.csv"),
+      fixture_path("sample_people_no_headers.csv"),
+      "",
+      "n",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    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"
+  end
+
+  def test_parity_workflow_reports_header_mismatch_in_headered_mode
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people.csv"),
+      fixture_path("parity_people_header_mismatch.csv"),
+      "",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_includes output.string, "CSV headers do not match."
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+
+  def test_parity_workflow_prints_mismatch_examples_and_counts
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people.csv"),
+      fixture_path("parity_people_mismatch.csv"),
+      "",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    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 examples:"
+    assert_includes output.string, "Cara,Berlin (count +1)"
+    assert_includes output.string, "Right-only examples:"
+    assert_includes output.string, "Dina,Rome (count +1)"
+  end
+
+  def test_parity_workflow_missing_left_file_returns_to_menu
+    output = StringIO.new
+    input = [
+      "5",
+      "/tmp/not-there-left.csv",
+      fixture_path("sample_people.csv"),
+      "",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_includes output.string, "File not found: /tmp/not-there-left.csv"
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+    refute_includes output.string, "Traceback"
+  end
+
+  def test_parity_workflow_missing_right_file_returns_to_menu
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people.csv"),
+      "/tmp/not-there-right.csv",
+      "",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_includes output.string, "File not found: /tmp/not-there-right.csv"
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+    refute_includes output.string, "Traceback"
+  end
+
+  def test_parity_workflow_malformed_csv_returns_to_menu
+    output = StringIO.new
+    input = [
+      "5",
+      fixture_path("sample_people.csv"),
+      fixture_path("sample_people_bad_tail.csv"),
+      "",
+      "",
+      "6"
+    ].join("\n") + "\n"
+
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_includes output.string, "Could not parse CSV file."
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+    refute_includes output.string, "Traceback"
+  end
+
   def test_end_to_end_file_output_writes_expected_csv
     output = StringIO.new
     output_path = nil
@@ -410,7 +564,7 @@ class TestCli < Minitest::Test
         "y",
         "2",
         output_path,
-        "5"
+        "6"
       ].join("\n") + "\n"
 
       status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -430,7 +584,7 @@ class TestCli < Minitest::Test
       "1",
       "",
       "n",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     output = StringIO.new
@@ -445,7 +599,7 @@ class TestCli < Minitest::Test
     output = StringIO.new
     status = Csvtool::CLI.start(
       ["menu"],
-      stdin: StringIO.new("1\n/tmp/does-not-exist.csv\n4\n"),
+      stdin: StringIO.new("1\n/tmp/does-not-exist.csv\n4\n6\n"),
       stdout: output,
       stderr: StringIO.new
     )
@@ -466,7 +620,7 @@ class TestCli < Minitest::Test
       "y",
       "2",
       "/tmp/not-a-dir/out.csv",
-      "5"
+      "6"
     ].join("\n") + "\n"
 
     output = StringIO.new