csvops 0.1.0.alpha

data/test/csvtool/cli_test.rb

@@ -0,0 +1,134 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+require "csvtool/cli"
+require "tmpdir"
+
+class TestCli < Minitest::Test
+  def fixture_path(name)
+    File.expand_path("../fixtures/#{name}", __dir__)
+  end
+
+  def test_menu_can_exit_cleanly
+    output = StringIO.new
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("2\n"), stdout: output, stderr: StringIO.new)
+    assert_equal 0, status
+    assert_includes output.string, "CSV Tool Menu"
+  end
+
+  def test_end_to_end_console_happy_path_prints_expected_values
+    input = [
+      "1",
+      fixture_path("sample_people.csv"),
+      "1",
+      "",
+      "1",
+      "",
+      "y",
+      "",
+      "2"
+    ].join("\n") + "\n"
+
+    output = StringIO.new
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_match(/\nAlice\nBob\nCara\n/, output.string)
+  end
+
+  def test_column_command_prints_expected_values
+    output = StringIO.new
+    status = Csvtool::CLI.start(
+      ["column", fixture_path("sample_people.csv"), "name"],
+      stdin: StringIO.new,
+      stdout: output,
+      stderr: StringIO.new
+    )
+
+    assert_equal 0, status
+    assert_equal "Alice\nBob\nCara\n", output.string
+  end
+
+  def test_end_to_end_file_output_writes_expected_csv
+    output = StringIO.new
+    output_path = nil
+
+    Dir.mktmpdir do |dir|
+      output_path = File.join(dir, "names.csv")
+      input = [
+        "1",
+        fixture_path("sample_people.csv"),
+        "1",
+        "",
+        "1",
+        "",
+        "y",
+        "2",
+        output_path,
+        "2"
+      ].join("\n") + "\n"
+
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      assert_equal 0, status
+      assert_equal "name\nAlice\nBob\nCara\n", File.read(output_path)
+    end
+
+    assert_includes output.string, "Wrote output to #{output_path}"
+  end
+
+  def test_preview_cancel_returns_to_menu
+    input = [
+      "1",
+      fixture_path("sample_people_many.csv"),
+      "1",
+      "",
+      "1",
+      "",
+      "n",
+      "2"
+    ].join("\n") + "\n"
+
+    output = StringIO.new
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_includes output.string, "Canceled."
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+
+  def test_missing_file_shows_error_and_returns_to_menu
+    output = StringIO.new
+    status = Csvtool::CLI.start(
+      ["menu"],
+      stdin: StringIO.new("1\n/tmp/does-not-exist.csv\n2\n"),
+      stdout: output,
+      stderr: StringIO.new
+    )
+
+    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_invalid_output_path_shows_error_and_returns_to_menu
+    input = [
+      "1",
+      fixture_path("sample_people.csv"),
+      "1",
+      "",
+      "1",
+      "",
+      "y",
+      "2",
+      "/tmp/not-a-dir/out.csv",
+      "2"
+    ].join("\n") + "\n"
+
+    output = StringIO.new
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+
+    assert_equal 0, status
+    assert_includes output.string, "Cannot write output file: /tmp/not-a-dir/out.csv"
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+end

data/test/csvtool/cli_unit_test.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require_relative "../test_helper"
+require "csvtool/cli"
+
+class CliUnitTest < Minitest::Test
+  def test_unknown_command_prints_usage_and_returns_one
+    stdout = StringIO.new
+    stderr = StringIO.new
+
+    status = Csvtool::CLI.start(["unknown"], stdin: StringIO.new, stdout: stdout, stderr: stderr)
+
+    assert_equal 1, status
+    assert_includes stderr.string, "csvtool menu"
+    assert_includes stderr.string, "csvtool column <file> <column>"
+  end
+
+  def test_menu_command_can_exit_zero
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("2\n"), stdout: StringIO.new, stderr: StringIO.new)
+    assert_equal 0, status
+  end
+
+  def test_column_command_requires_file_and_column_args
+    status = Csvtool::CLI.start(["column"], stdin: StringIO.new, stdout: StringIO.new, stderr: StringIO.new)
+    assert_equal 1, status
+  end
+end

data/test/csvtool/domain/extraction_session/column_selection_test.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/column_selection"
+
+class ColumnSelectionTest < Minitest::Test
+  def test_stores_name
+    selection = Csvtool::Domain::ExtractionSession::ColumnSelection.new(name: "city")
+    assert_equal "city", selection.name
+  end
+end

data/test/csvtool/domain/extraction_session/csv_source_test.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/csv_source"
+require "csvtool/domain/extraction_session/separator"
+
+class CsvSourceTest < Minitest::Test
+  def test_stores_path_and_separator
+    separator = Csvtool::Domain::ExtractionSession::Separator.new(",")
+    source = Csvtool::Domain::ExtractionSession::CsvSource.new(path: "/tmp/a.csv", separator: separator)
+    assert_equal "/tmp/a.csv", source.path
+    assert_equal separator, source.separator
+  end
+end

data/test/csvtool/domain/extraction_session/extraction_options_test.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/extraction_options"
+
+class ExtractionOptionsTest < Minitest::Test
+  def test_exposes_options
+    options = Csvtool::Domain::ExtractionSession::ExtractionOptions.new(skip_blanks: true, preview_limit: 10)
+    assert_equal true, options.skip_blanks?
+    assert_equal 10, options.preview_limit
+  end
+
+  def test_non_positive_preview_limit_raises
+    assert_raises(ArgumentError) do
+      Csvtool::Domain::ExtractionSession::ExtractionOptions.new(skip_blanks: true, preview_limit: 0)
+    end
+  end
+end

data/test/csvtool/domain/extraction_session/extraction_session_test.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/extraction_session"
+require "csvtool/domain/extraction_session/csv_source"
+require "csvtool/domain/extraction_session/separator"
+require "csvtool/domain/extraction_session/column_selection"
+require "csvtool/domain/extraction_session/extraction_options"
+require "csvtool/domain/extraction_session/preview"
+require "csvtool/domain/extraction_session/extraction_value"
+require "csvtool/domain/extraction_session/output_destination"
+
+class ExtractionSessionTest < Minitest::Test
+  def test_state_transitions
+    session = Csvtool::Domain::ExtractionSession::ExtractionSession.start(
+      source: Csvtool::Domain::ExtractionSession::CsvSource.new(
+        path: "/tmp/in.csv",
+        separator: Csvtool::Domain::ExtractionSession::Separator.new(",")
+      ),
+      column_selection: Csvtool::Domain::ExtractionSession::ColumnSelection.new(name: "name"),
+      options: Csvtool::Domain::ExtractionSession::ExtractionOptions.new(skip_blanks: true, preview_limit: 10)
+    )
+
+    preview = Csvtool::Domain::ExtractionSession::Preview.new(
+      values: [Csvtool::Domain::ExtractionSession::ExtractionValue.new("Alice")]
+    )
+    session = session.with_preview(preview).confirm!.with_output_destination(
+      Csvtool::Domain::ExtractionSession::OutputDestination.console
+    )
+
+    assert_equal true, session.confirmed?
+    assert_equal preview, session.preview
+    assert_equal true, session.output_destination.console?
+  end
+end

data/test/csvtool/domain/extraction_session/extraction_value_test.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/extraction_value"
+
+class ExtractionValueTest < Minitest::Test
+  def test_stringifies_value
+    value = Csvtool::Domain::ExtractionSession::ExtractionValue.new(123)
+    assert_equal "123", value.value
+  end
+end

data/test/csvtool/domain/extraction_session/output_destination_test.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/output_destination"
+
+class OutputDestinationTest < Minitest::Test
+  def test_console_factory
+    destination = Csvtool::Domain::ExtractionSession::OutputDestination.console
+    assert_equal true, destination.console?
+    assert_equal false, destination.file?
+  end
+
+  def test_file_factory
+    destination = Csvtool::Domain::ExtractionSession::OutputDestination.file(path: "/tmp/out.csv")
+    assert_equal true, destination.file?
+    assert_equal "/tmp/out.csv", destination.path
+  end
+end

data/test/csvtool/domain/extraction_session/preview_test.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/preview"
+require "csvtool/domain/extraction_session/extraction_value"
+
+class PreviewTest < Minitest::Test
+  def test_exposes_size_and_string_values
+    values = [
+      Csvtool::Domain::ExtractionSession::ExtractionValue.new("Alice"),
+      Csvtool::Domain::ExtractionSession::ExtractionValue.new("Bob")
+    ]
+    preview = Csvtool::Domain::ExtractionSession::Preview.new(values: values)
+
+    assert_equal 2, preview.size
+    assert_equal %w[Alice Bob], preview.to_strings
+  end
+end

data/test/csvtool/domain/extraction_session/separator_test.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/extraction_session/separator"
+
+class SeparatorTest < Minitest::Test
+  def test_stores_value
+    separator = Csvtool::Domain::ExtractionSession::Separator.new(",")
+    assert_equal ",", separator.value
+  end
+
+  def test_empty_value_raises
+    assert_raises(ArgumentError) { Csvtool::Domain::ExtractionSession::Separator.new("") }
+  end
+end

data/test/csvtool/infrastructure/csv/header_reader_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/infrastructure/csv/header_reader"
+
+class InfrastructureHeaderReaderTest < Minitest::Test
+  def fixture_path(name)
+    File.expand_path("../../../fixtures/#{name}", __dir__)
+  end
+
+  def test_returns_headers
+    reader = Csvtool::Infrastructure::CSV::HeaderReader.new
+    headers = reader.call(file_path: fixture_path("sample_people.csv"), col_sep: ",")
+    assert_equal %w[name city], headers
+  end
+end

data/test/csvtool/infrastructure/csv/value_streamer_test.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/infrastructure/csv/value_streamer"
+
+class InfrastructureValueStreamerTest < Minitest::Test
+  def fixture_path(name)
+    File.expand_path("../../../fixtures/#{name}", __dir__)
+  end
+
+  def test_filters_blanks_when_enabled
+    streamer = Csvtool::Infrastructure::CSV::ValueStreamer.new
+    values = []
+    streamer.each(
+      file_path: fixture_path("sample_people_blanks.csv"),
+      column_name: "name",
+      col_sep: ",",
+      skip_blanks: true
+    ) { |v| values << v }
+    assert_equal %w[Alice Bob Cara], values
+  end
+end

data/test/csvtool/infrastructure/output/console_writer_test.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/infrastructure/output/console_writer"
+
+class InfrastructureConsoleWriterTest < Minitest::Test
+  class FakeStreamer
+    def each(file_path:, column_name:, col_sep:, skip_blanks:)
+      %w[Alice Bob].each { |value| yield value }
+    end
+  end
+
+  def test_writes_streamed_values_to_stdout
+    out = StringIO.new
+    writer = Csvtool::Infrastructure::Output::ConsoleWriter.new(stdout: out, value_streamer: FakeStreamer.new)
+    writer.call(file_path: "x.csv", column_name: "name", col_sep: ",", skip_blanks: true)
+    assert_equal "Alice\nBob\n", out.string
+  end
+end

data/test/csvtool/infrastructure/output/csv_file_writer_test.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/infrastructure/output/csv_file_writer"
+require "csvtool/interface/cli/errors/presenter"
+require "tmpdir"
+
+class InfrastructureCsvFileWriterTest < Minitest::Test
+  class FakeStreamer
+    def each(file_path:, column_name:, col_sep:, skip_blanks:)
+      %w[Alice Bob].each { |value| yield value }
+    end
+  end
+
+  def test_writes_header_and_values
+    stdout = StringIO.new
+    writer = Csvtool::Infrastructure::Output::CsvFileWriter.new(
+      stdout: stdout,
+      errors: Csvtool::Interface::CLI::Errors::Presenter.new(stdout: stdout),
+      value_streamer: FakeStreamer.new
+    )
+
+    Dir.mktmpdir do |dir|
+      output_path = File.join(dir, "names.csv")
+      writer.call(
+        file_path: "ignored.csv",
+        column_name: "name",
+        col_sep: ",",
+        skip_blanks: true,
+        output_path: output_path
+      )
+      assert_equal "name\nAlice\nBob\n", File.read(output_path)
+    end
+  end
+end

data/test/csvtool/interface/cli/errors/presenter_test.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/errors/presenter"
+
+class ErrorsPresenterTest < Minitest::Test
+  def test_all_messages_are_presented
+    out = StringIO.new
+    presenter = Csvtool::Interface::CLI::Errors::Presenter.new(stdout: out)
+
+    presenter.file_not_found("/tmp/x.csv")
+    presenter.no_headers
+    presenter.column_not_found
+    presenter.could_not_parse_csv
+    presenter.cannot_read_file("/tmp/y.csv")
+    presenter.cannot_write_output_file("/tmp/z.csv", Errno::ENOENT)
+    presenter.empty_output_path
+    presenter.invalid_output_destination
+    presenter.empty_custom_separator
+    presenter.invalid_separator_choice
+    presenter.canceled
+
+    text = out.string
+    assert_includes text, "File not found: /tmp/x.csv"
+    assert_includes text, "No headers found."
+    assert_includes text, "Column not found."
+    assert_includes text, "Could not parse CSV file."
+    assert_includes text, "Cannot read file: /tmp/y.csv"
+    assert_includes text, "Cannot write output file: /tmp/z.csv (Errno::ENOENT)"
+    assert_includes text, "Output file path cannot be empty."
+    assert_includes text, "Invalid output destination."
+    assert_includes text, "Separator cannot be empty."
+    assert_includes text, "Invalid separator choice."
+    assert_includes text, "Canceled."
+  end
+end

data/test/csvtool/interface/cli/menu_loop_test.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/interface/cli/menu_loop"
+
+class MenuLoopTest < Minitest::Test
+  class FakeAction
+    attr_reader :runs
+
+    def initialize
+      @runs = 0
+    end
+
+    def call
+      @runs += 1
+    end
+  end
+
+  def test_routes_extract_then_exit
+    action = FakeAction.new
+    stdout = StringIO.new
+    menu = Csvtool::Interface::CLI::MenuLoop.new(
+      stdin: StringIO.new("1\n2\n"),
+      stdout: stdout,
+      menu_options: ["Extract column", "Exit"],
+      extract_action: action
+    )
+
+    status = menu.run
+
+    assert_equal 0, status
+    assert_equal 1, action.runs
+    assert_includes stdout.string, "CSV Tool Menu"
+  end
+
+  def test_invalid_choice_shows_prompt
+    action = FakeAction.new
+    stdout = StringIO.new
+    menu = Csvtool::Interface::CLI::MenuLoop.new(
+      stdin: StringIO.new("x\n2\n"),
+      stdout: stdout,
+      menu_options: ["Extract column", "Exit"],
+      extract_action: action
+    )
+
+    menu.run
+
+    assert_includes stdout.string, "Please choose 1 or 2."
+    assert_equal 0, action.runs
+  end
+end

data/test/csvtool/interface/cli/prompts/column_selector_prompt_test.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/prompts/column_selector_prompt"
+
+class ColumnSelectorPromptTest < Minitest::Test
+  class FakeErrors
+    attr_reader :calls
+
+    def initialize
+      @calls = []
+    end
+
+    def column_not_found = @calls << :column_not_found
+  end
+
+  def test_selects_filtered_header
+    errors = FakeErrors.new
+    prompt = Csvtool::Interface::CLI::Prompts::ColumnSelectorPrompt.new(stdin: StringIO.new("ci\n1\n"), stdout: StringIO.new, errors: errors)
+    assert_equal "city", prompt.call(%w[name city])
+    assert_empty errors.calls
+  end
+end

data/test/csvtool/interface/cli/prompts/confirm_prompt_test.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/prompts/confirm_prompt"
+
+class ConfirmPromptTest < Minitest::Test
+  class FakeErrors
+    attr_reader :calls
+
+    def initialize
+      @calls = []
+    end
+
+    def canceled = @calls << :canceled
+  end
+
+  def test_calls_canceled_on_no
+    errors = FakeErrors.new
+    prompt = Csvtool::Interface::CLI::Prompts::ConfirmPrompt.new(stdin: StringIO.new("n\n"), stdout: StringIO.new, errors: errors)
+    assert_equal false, prompt.call(%w[a b])
+    assert_includes errors.calls, :canceled
+  end
+end

data/test/csvtool/interface/cli/prompts/file_path_prompt_test.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/prompts/file_path_prompt"
+
+class FilePathPromptTest < Minitest::Test
+  def test_returns_trimmed_path
+    prompt = Csvtool::Interface::CLI::Prompts::FilePathPrompt.new(stdin: StringIO.new("  /tmp/a.csv  \n"), stdout: StringIO.new)
+    assert_equal "/tmp/a.csv", prompt.call
+  end
+end

data/test/csvtool/interface/cli/prompts/output_destination_prompt_test.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/prompts/output_destination_prompt"
+require "csvtool/interface/cli/errors/presenter"
+
+class OutputDestinationPromptTest < Minitest::Test
+  def test_defaults_to_console
+    stdout = StringIO.new
+    prompt = Csvtool::Interface::CLI::Prompts::OutputDestinationPrompt.new(
+      stdin: StringIO.new("\n"),
+      stdout: stdout,
+      errors: Csvtool::Interface::CLI::Errors::Presenter.new(stdout: stdout)
+    )
+    assert_equal({ mode: :console }, prompt.call)
+  end
+
+  def test_file_mode_requires_path
+    stdout = StringIO.new
+    prompt = Csvtool::Interface::CLI::Prompts::OutputDestinationPrompt.new(
+      stdin: StringIO.new("2\n\n"),
+      stdout: stdout,
+      errors: Csvtool::Interface::CLI::Errors::Presenter.new(stdout: stdout)
+    )
+    assert_nil prompt.call
+    assert_includes stdout.string, "Output file path cannot be empty."
+  end
+end

data/test/csvtool/interface/cli/prompts/separator_prompt_test.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/prompts/separator_prompt"
+
+class SeparatorPromptTest < Minitest::Test
+  class FakeErrors
+    attr_reader :calls
+
+    def initialize
+      @calls = []
+    end
+
+    def empty_custom_separator = @calls << :empty_custom_separator
+    def invalid_separator_choice = @calls << :invalid_separator_choice
+  end
+
+  def test_defaults_to_comma
+    errors = FakeErrors.new
+    prompt = Csvtool::Interface::CLI::Prompts::SeparatorPrompt.new(stdin: StringIO.new("\n"), stdout: StringIO.new, errors: errors)
+    assert_equal ",", prompt.call
+    assert_empty errors.calls
+  end
+
+  def test_custom_empty_reports_error
+    errors = FakeErrors.new
+    prompt = Csvtool::Interface::CLI::Prompts::SeparatorPrompt.new(stdin: StringIO.new("5\n\n"), stdout: StringIO.new, errors: errors)
+    assert_nil prompt.call
+    assert_includes errors.calls, :empty_custom_separator
+  end
+end

data/test/csvtool/interface/cli/prompts/skip_blanks_prompt_test.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require_relative "../../../../test_helper"
+require "csvtool/interface/cli/prompts/skip_blanks_prompt"
+
+class SkipBlanksPromptTest < Minitest::Test
+  def test_default_true_and_no_false
+    prompt_yes = Csvtool::Interface::CLI::Prompts::SkipBlanksPrompt.new(stdin: StringIO.new("\n"), stdout: StringIO.new)
+    prompt_no = Csvtool::Interface::CLI::Prompts::SkipBlanksPrompt.new(stdin: StringIO.new("n\n"), stdout: StringIO.new)
+    assert_equal true, prompt_yes.call
+    assert_equal false, prompt_no.call
+  end
+end

data/test/csvtool/services/preview_builder_test.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require_relative "../../test_helper"
+require "csvtool/services/preview_builder"
+
+class PreviewBuilderTest < Minitest::Test
+  class FakeStreamer
+    def initialize(values)
+      @values = values
+    end
+
+    def each(file_path:, column_name:, col_sep:, skip_blanks:)
+      @values.each { |value| yield value }
+    end
+  end
+
+  def test_builds_preview_up_to_limit
+    builder = Csvtool::Services::PreviewBuilder.new(value_streamer: FakeStreamer.new(%w[a b c d]))
+    preview = builder.call(file_path: "x.csv", column_name: "name", col_sep: ",", skip_blanks: true, limit: 2)
+    assert_equal %w[a b], preview
+  end
+end

data/test/fixtures/sample_people.csv

@@ -0,0 +1,4 @@
+name,city
+Alice,London
+Bob,Paris
+Cara,Berlin

data/test/fixtures/sample_people.tsv

@@ -0,0 +1,4 @@
+name	city
+Alice	London
+Bob	Paris
+Cara	Berlin