csvops 0.1.0.alpha → 0.2.0.alpha

data/lib/csvtool/infrastructure/output/csv_row_console_writer.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require "csv"
+
+module Csvtool
+  module Infrastructure
+    module Output
+      class CsvRowConsoleWriter
+        def initialize(stdout:, row_streamer:)
+          @stdout = stdout
+          @row_streamer = row_streamer
+        end
+
+        def call(file_path:, col_sep:, headers:, start_row:, end_row:)
+          wrote_header = false
+          stats = @row_streamer.each_in_range(
+            file_path: file_path,
+            col_sep: col_sep,
+            start_row: start_row,
+            end_row: end_row
+          ) do |fields|
+            unless wrote_header
+              @stdout.puts ::CSV.generate_line(headers, row_sep: "").chomp
+              wrote_header = true
+            end
+            @stdout.puts ::CSV.generate_line(fields, row_sep: "").chomp
+          end
+
+          stats
+        end
+      end
+    end
+  end
+end

data/lib/csvtool/infrastructure/output/csv_row_file_writer.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require "csv"
+
+module Csvtool
+  module Infrastructure
+    module Output
+      class CsvRowFileWriter
+        def initialize(stdout:, errors:, row_streamer:)
+          @stdout = stdout
+          @errors = errors
+          @row_streamer = row_streamer
+        end
+
+        def call(file_path:, col_sep:, headers:, start_row:, end_row:, output_path:)
+          csv = nil
+          wrote_rows = false
+
+          stats = @row_streamer.each_in_range(
+            file_path: file_path,
+            col_sep: col_sep,
+            start_row: start_row,
+            end_row: end_row
+          ) do |fields|
+            unless wrote_rows
+              csv = ::CSV.open(output_path, "w")
+              csv << headers
+              wrote_rows = true
+            end
+            csv << fields
+          end
+
+          csv&.close
+          @stdout.puts "Wrote output to #{output_path}" if wrote_rows
+          stats
+        rescue Errno::EACCES, Errno::ENOENT => e
+          @errors.cannot_write_output_file(output_path, e.class)
+          nil
+        ensure
+          csv&.close unless csv&.closed?
+        end
+      end
+    end
+  end
+end

data/lib/csvtool/interface/cli/errors/presenter.rb

@@ -52,6 +52,22 @@ module Csvtool
           def canceled
             @stdout.puts "Canceled."
           end
+
+          def invalid_start_row
+            @stdout.puts "Start row must be a positive integer."
+          end
+
+          def invalid_end_row
+            @stdout.puts "End row must be a positive integer."
+          end
+
+          def invalid_row_range_order
+            @stdout.puts "End row must be greater than or equal to start row."
+          end
+
+          def row_range_out_of_bounds(total_rows)
+            @stdout.puts "Row range is out of bounds. File has #{total_rows} data rows."
+          end
         end
       end
     end

data/lib/csvtool/interface/cli/menu_loop.rb

@@ -4,25 +4,30 @@ module Csvtool
   module Interface
     module CLI
       class MenuLoop
-        def initialize(stdin:, stdout:, menu_options:, extract_action:)
+        def initialize(stdin:, stdout:, menu_options:, extract_column_action:, extract_rows_action:)
           @stdin = stdin
           @stdout = stdout
           @menu_options = menu_options
-          @extract_action = extract_action
+          @extract_column_action = extract_column_action
+          @extract_rows_action = extract_rows_action
         end
 
         def run
           loop do
             print_menu
             @stdout.print "> "
+            choice = @stdin.gets
+            return 0 if choice.nil?
 
-            case @stdin.gets&.strip
+            case choice.strip
             when "1"
-              @extract_action.call
+              @extract_column_action.call
             when "2"
+              @extract_rows_action.call
+            when "3"
               return 0
             else
-              @stdout.puts "Please choose 1 or 2."
+              @stdout.puts "Please choose 1, 2, or 3."
             end
           end
         end

data/lib/csvtool/version.rb

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

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

@@ -0,0 +1,140 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/application/use_cases/run_row_extraction"
+require "tmpdir"
+
+class RunRowExtractionTest < Minitest::Test
+  def test_use_case_prints_selected_row_range_with_header
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+    input = [fixture, "", "2", "3", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "name,city"
+    assert_includes out.string, "Bob,Paris"
+    assert_includes out.string, "Cara,Berlin"
+    refute_includes out.string, "Alice,London"
+  end
+
+  def test_rejects_non_numeric_start_row
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+    input = [fixture, "", "abc", "3", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "Start row must be a positive integer."
+    refute_includes out.string, "name,city"
+  end
+
+  def test_rejects_non_numeric_end_row
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+    input = [fixture, "", "1", "xyz", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "End row must be a positive integer."
+    refute_includes out.string, "name,city"
+  end
+
+  def test_rejects_end_before_start
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+    input = [fixture, "", "3", "2", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "End row must be greater than or equal to start row."
+    refute_includes out.string, "name,city"
+  end
+
+  def test_handles_out_of_bounds_start_row
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+    input = [fixture, "", "10", "12", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "Row range is out of bounds. File has 3 data rows."
+    refute_includes out.string, "name,city"
+  end
+
+  def test_use_case_supports_tsv_separator
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.tsv", __dir__)
+    input = [fixture, "2", "2", "3", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "name,city"
+    assert_includes out.string, "Bob,Paris"
+    assert_includes out.string, "Cara,Berlin"
+  end
+
+  def test_use_case_supports_custom_separator
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people_colon.txt", __dir__)
+    input = [fixture, "5", ":", "2", "3", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "name,city"
+    assert_includes out.string, "Bob,Paris"
+    assert_includes out.string, "Cara,Berlin"
+  end
+
+  def test_use_case_can_write_selected_rows_to_file
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+
+    Dir.mktmpdir do |dir|
+      output_path = File.join(dir, "rows.csv")
+      input = [fixture, "", "2", "3", "2", output_path].join("\n") + "\n"
+
+      use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+      use_case.call
+
+      assert_equal "name,city\nBob,Paris\nCara,Berlin\n", File.read(output_path)
+      assert_includes out.string, "Wrote output to #{output_path}"
+    end
+  end
+
+  def test_stops_parsing_after_end_row_for_console_output
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people_bad_tail.csv", __dir__)
+    input = [fixture, "", "1", "2", ""].join("\n") + "\n"
+
+    use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+    use_case.call
+
+    assert_includes out.string, "name,city"
+    assert_includes out.string, "Alice,London"
+    assert_includes out.string, "Bob,Paris"
+    refute_includes out.string, "Could not parse CSV file."
+  end
+
+  def test_out_of_bounds_file_mode_reports_error
+    out = StringIO.new
+    fixture = File.expand_path("../../../fixtures/sample_people.csv", __dir__)
+
+    Dir.mktmpdir do |dir|
+      output_path = File.join(dir, "rows.csv")
+      input = [fixture, "", "10", "12", "2", output_path].join("\n") + "\n"
+
+      use_case = Csvtool::Application::UseCases::RunRowExtraction.new(stdin: StringIO.new(input), stdout: out)
+      use_case.call
+    end
+
+    assert_includes out.string, "Row range is out of bounds. File has 3 data rows."
+  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("2\n"), stdout: output, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("3\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",
       "",
-      "2"
+      "3"
     ].join("\n") + "\n"
 
     output = StringIO.new
@@ -49,6 +49,132 @@ class TestCli < Minitest::Test
     assert_equal "Alice\nBob\nCara\n", output.string
   end
 
+  def test_row_range_workflow_prints_selected_rows
+    output = StringIO.new
+    input = [
+      "2",
+      fixture_path("sample_people.csv"),
+      "",
+      "2",
+      "3",
+      "",
+      "3"
+    ].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, "name,city"
+    assert_includes output.string, "Bob,Paris"
+    assert_includes output.string, "Cara,Berlin"
+    refute_includes output.string, "Alice,London"
+  end
+
+  def test_row_range_invalid_inputs_return_to_menu
+    output = StringIO.new
+    input = [
+      "2",
+      fixture_path("sample_people.csv"),
+      "",
+      "0",
+      "3",
+      "",
+      "3"
+    ].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, "Start row must be a positive integer."
+    assert_operator output.string.scan("CSV Tool Menu").length, :>=, 2
+  end
+
+  def test_row_range_workflow_supports_tsv_separator
+    output = StringIO.new
+    input = [
+      "2",
+      fixture_path("sample_people.tsv"),
+      "2",
+      "2",
+      "3",
+      "",
+      "3"
+    ].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, "name,city"
+    assert_includes output.string, "Bob,Paris"
+    assert_includes output.string, "Cara,Berlin"
+  end
+
+  def test_row_range_workflow_supports_custom_separator
+    output = StringIO.new
+    input = [
+      "2",
+      fixture_path("sample_people_colon.txt"),
+      "5",
+      ":",
+      "2",
+      "3",
+      "",
+      "3"
+    ].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, "name,city"
+    assert_includes output.string, "Bob,Paris"
+    assert_includes output.string, "Cara,Berlin"
+  end
+
+  def test_row_range_workflow_can_write_selected_rows_to_file
+    output = StringIO.new
+    output_path = nil
+
+    Dir.mktmpdir do |dir|
+      output_path = File.join(dir, "row_range.csv")
+      input = [
+        "2",
+        fixture_path("sample_people.csv"),
+        "",
+        "2",
+        "3",
+        "2",
+        output_path,
+        "3"
+      ].join("\n") + "\n"
+
+      status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
+      assert_equal 0, status
+      assert_equal "name,city\nBob,Paris\nCara,Berlin\n", File.read(output_path)
+    end
+
+    assert_includes output.string, "Wrote output to #{output_path}"
+  end
+
+  def test_row_range_workflow_stops_before_malformed_tail
+    output = StringIO.new
+    input = [
+      "2",
+      fixture_path("sample_people_bad_tail.csv"),
+      "",
+      "1",
+      "2",
+      "",
+      "3"
+    ].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, "Alice,London"
+    assert_includes output.string, "Bob,Paris"
+    refute_includes output.string, "Could not parse CSV file."
+  end
+
   def test_end_to_end_file_output_writes_expected_csv
     output = StringIO.new
     output_path = nil
@@ -65,7 +191,7 @@ class TestCli < Minitest::Test
         "y",
         "2",
         output_path,
-        "2"
+        "3"
       ].join("\n") + "\n"
 
       status = Csvtool::CLI.start(["menu"], stdin: StringIO.new(input), stdout: output, stderr: StringIO.new)
@@ -85,7 +211,7 @@ class TestCli < Minitest::Test
       "1",
       "",
       "n",
-      "2"
+      "3"
     ].join("\n") + "\n"
 
     output = StringIO.new
@@ -100,7 +226,7 @@ class TestCli < Minitest::Test
     output = StringIO.new
     status = Csvtool::CLI.start(
       ["menu"],
-      stdin: StringIO.new("1\n/tmp/does-not-exist.csv\n2\n"),
+      stdin: StringIO.new("1\n/tmp/does-not-exist.csv\n3\n"),
       stdout: output,
       stderr: StringIO.new
     )
@@ -121,7 +247,7 @@ class TestCli < Minitest::Test
       "y",
       "2",
       "/tmp/not-a-dir/out.csv",
-      "2"
+      "3"
     ].join("\n") + "\n"
 
     output = StringIO.new

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("2\n"), stdout: StringIO.new, stderr: StringIO.new)
+    status = Csvtool::CLI.start(["menu"], stdin: StringIO.new("3\n"), stdout: StringIO.new, stderr: StringIO.new)
     assert_equal 0, status
   end
 
@@ -24,4 +24,15 @@ class CliUnitTest < Minitest::Test
     status = Csvtool::CLI.start(["column"], stdin: StringIO.new, stdout: StringIO.new, stderr: StringIO.new)
     assert_equal 1, status
   end
+
+  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", "", "3"].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"
+    assert_includes stdout.string, "Bob,Paris"
+    assert_includes stdout.string, "Cara,Berlin"
+  end
 end

data/test/csvtool/domain/{extraction_session → column_session}/column_selection_test.rb

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

data/test/csvtool/domain/column_session/column_session_test.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/column_session/column_session"
+require "csvtool/domain/column_session/csv_source"
+require "csvtool/domain/column_session/separator"
+require "csvtool/domain/column_session/column_selection"
+require "csvtool/domain/column_session/extraction_options"
+require "csvtool/domain/column_session/preview"
+require "csvtool/domain/column_session/extraction_value"
+require "csvtool/domain/column_session/output_destination"
+
+class ColumnSessionTest < Minitest::Test
+  def test_state_transitions
+    session = Csvtool::Domain::ColumnSession::ColumnSession.start(
+      source: Csvtool::Domain::ColumnSession::CsvSource.new(
+        path: "/tmp/in.csv",
+        separator: Csvtool::Domain::ColumnSession::Separator.new(",")
+      ),
+      column_selection: Csvtool::Domain::ColumnSession::ColumnSelection.new(name: "name"),
+      options: Csvtool::Domain::ColumnSession::ExtractionOptions.new(skip_blanks: true, preview_limit: 10)
+    )
+
+    preview = Csvtool::Domain::ColumnSession::Preview.new(
+      values: [Csvtool::Domain::ColumnSession::ExtractionValue.new("Alice")]
+    )
+    session = session.with_preview(preview).confirm!.with_output_destination(
+      Csvtool::Domain::ColumnSession::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/column_session/csv_source_test.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/column_session/csv_source"
+require "csvtool/domain/column_session/separator"
+
+class CsvSourceTest < Minitest::Test
+  def test_stores_path_and_separator
+    separator = Csvtool::Domain::ColumnSession::Separator.new(",")
+    source = Csvtool::Domain::ColumnSession::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 → column_session}/extraction_options_test.rb

@@ -1,18 +1,18 @@
 # frozen_string_literal: true
 
 require_relative "../../../test_helper"
-require "csvtool/domain/extraction_session/extraction_options"
+require "csvtool/domain/column_session/extraction_options"
 
 class ExtractionOptionsTest < Minitest::Test
   def test_exposes_options
-    options = Csvtool::Domain::ExtractionSession::ExtractionOptions.new(skip_blanks: true, preview_limit: 10)
+    options = Csvtool::Domain::ColumnSession::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)
+      Csvtool::Domain::ColumnSession::ExtractionOptions.new(skip_blanks: true, preview_limit: 0)
     end
   end
 end

data/test/csvtool/domain/{extraction_session → column_session}/extraction_value_test.rb

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

data/test/csvtool/domain/{extraction_session → column_session}/output_destination_test.rb

@@ -1,17 +1,17 @@
 # frozen_string_literal: true
 
 require_relative "../../../test_helper"
-require "csvtool/domain/extraction_session/output_destination"
+require "csvtool/domain/column_session/output_destination"
 
 class OutputDestinationTest < Minitest::Test
   def test_console_factory
-    destination = Csvtool::Domain::ExtractionSession::OutputDestination.console
+    destination = Csvtool::Domain::ColumnSession::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")
+    destination = Csvtool::Domain::ColumnSession::OutputDestination.file(path: "/tmp/out.csv")
     assert_equal true, destination.file?
     assert_equal "/tmp/out.csv", destination.path
   end

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

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require_relative "../../../test_helper"
+require "csvtool/domain/column_session/preview"
+require "csvtool/domain/column_session/extraction_value"
+
+class PreviewTest < Minitest::Test
+  def test_exposes_size_and_string_values
+    values = [
+      Csvtool::Domain::ColumnSession::ExtractionValue.new("Alice"),
+      Csvtool::Domain::ColumnSession::ExtractionValue.new("Bob")
+    ]
+    preview = Csvtool::Domain::ColumnSession::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 → column_session}/separator_test.rb

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