ruby_spriter 0.6.5 → 0.6.7

data/spec/ruby_spriter/utils/file_helper_spec.rb

@@ -62,10 +62,89 @@ RSpec.describe RubySpriter::Utils::FileHelper do
     it 'validates file exists and is readable' do
       file = File.join(@test_dir, 'test.txt')
       File.write(file, 'test')
-      
+
       expect {
         described_class.validate_readable!(file)
       }.not_to raise_error
     end
   end
+
+  describe '.unique_filename' do
+    it 'returns original filename when file does not exist' do
+      result = described_class.unique_filename('/path/to/output.png')
+      expect(result).to eq('/path/to/output.png')
+    end
+
+    it 'adds timestamp when file exists' do
+      file = File.join(@test_dir, 'existing.png')
+      File.write(file, 'test')
+
+      result = described_class.unique_filename(file)
+      expect(result).to match(/existing_\d{8}_\d{6}_\d{3}\.png$/)
+      expect(result).not_to eq(file)
+    end
+
+    it 'handles files with multiple dots in name' do
+      file = File.join(@test_dir, 'my.sprite.sheet.png')
+      File.write(file, 'test')
+
+      result = described_class.unique_filename(file)
+      expect(result).to match(/my\.sprite\.sheet_\d{8}_\d{6}_\d{3}\.png$/)
+    end
+
+    it 'preserves directory path' do
+      file = File.join(@test_dir, 'subdir', 'output.png')
+      FileUtils.mkdir_p(File.dirname(file))
+      File.write(file, 'test')
+
+      result = described_class.unique_filename(file)
+      expect(result).to start_with(File.join(@test_dir, 'subdir'))
+    end
+
+    it 'generates different filenames for consecutive calls' do
+      file = File.join(@test_dir, 'test.png')
+      File.write(file, 'test')
+
+      result1 = described_class.unique_filename(file)
+      sleep(0.01) # Small delay to ensure different timestamps
+      result2 = described_class.unique_filename(file)
+
+      expect(result1).not_to eq(result2)
+    end
+  end
+
+  describe '.ensure_unique_output' do
+    it 'returns original path when overwrite is true' do
+      file = File.join(@test_dir, 'output.png')
+      File.write(file, 'test')
+
+      result = described_class.ensure_unique_output(file, overwrite: true)
+      expect(result).to eq(file)
+    end
+
+    it 'returns original path when file does not exist and overwrite is false' do
+      file = File.join(@test_dir, 'new_output.png')
+
+      result = described_class.ensure_unique_output(file, overwrite: false)
+      expect(result).to eq(file)
+    end
+
+    it 'returns unique filename when file exists and overwrite is false' do
+      file = File.join(@test_dir, 'existing_output.png')
+      File.write(file, 'test')
+
+      result = described_class.ensure_unique_output(file, overwrite: false)
+      expect(result).to match(/existing_output_\d{8}_\d{6}_\d{3}\.png$/)
+      expect(result).not_to eq(file)
+    end
+
+    it 'defaults to overwrite false when not specified' do
+      file = File.join(@test_dir, 'default_test.png')
+      File.write(file, 'test')
+
+      result = described_class.ensure_unique_output(file)
+      expect(result).not_to eq(file)
+      expect(result).to match(/default_test_\d{8}_\d{6}_\d{3}\.png$/)
+    end
+  end
 end

data/spec/ruby_spriter/utils/spritesheet_splitter_spec.rb

@@ -0,0 +1,104 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe RubySpriter::Utils::SpritesheetSplitter do
+  describe '#split_into_frames' do
+    let(:spritesheet_file) { 'spritesheet.png' }
+    let(:output_dir) { '/tmp/frames' }
+    let(:columns) { 4 }
+    let(:rows) { 4 }
+    let(:frames) { 16 }
+    let(:tile_width) { 100 }
+    let(:tile_height) { 100 }
+
+    let(:splitter) { described_class.new }
+
+    before do
+      allow(FileUtils).to receive(:mkdir_p)
+      allow(File).to receive(:exist?).and_return(true)
+      allow(Open3).to receive(:capture3).and_return(['', '', instance_double(Process::Status, success?: true)])
+
+      # Mock ImageMagick identify to return dimensions
+      allow(Open3).to receive(:capture3).with(/identify/).and_return(
+        ["400x400\n", '', instance_double(Process::Status, success?: true)]
+      )
+    end
+
+    it 'creates output directory for frames' do
+      expect(FileUtils).to receive(:mkdir_p).with(output_dir)
+
+      splitter.split_into_frames(spritesheet_file, output_dir, columns, rows, frames)
+    end
+
+    it 'extracts each frame with FR prefix and zero-padded numbers' do
+      # Expect ImageMagick convert commands for each frame
+      expect(Open3).to receive(:capture3).with(/identify/).once.and_return(
+        ["400x400\n", '', instance_double(Process::Status, success?: true)]
+      )
+
+      (1..frames).each do |i|
+        expect(Open3).to receive(:capture3).with(/convert.*FR#{format('%03d', i)}_/).and_return(
+          ['', '', instance_double(Process::Status, success?: true)]
+        )
+      end
+
+      splitter.split_into_frames(spritesheet_file, output_dir, columns, rows, frames)
+    end
+
+    it 'calculates tile dimensions from spritesheet size and grid' do
+      # For 400x400 image with 4x4 grid, tiles should be 100x100
+      expect(Open3).to receive(:capture3).with(/identify/).and_return(
+        ["400x400\n", '', instance_double(Process::Status, success?: true)]
+      )
+
+      # Check that crop parameters use 100x100
+      expect(Open3).to receive(:capture3).with(/100x100\+0\+0/).and_return(
+        ['', '', instance_double(Process::Status, success?: true)]
+      )
+
+      allow(Open3).to receive(:capture3).and_return(['', '', instance_double(Process::Status, success?: true)])
+
+      splitter.split_into_frames(spritesheet_file, output_dir, columns, rows, 1)
+    end
+
+    it 'includes spritesheet basename in frame output names' do
+      basename = File.basename(spritesheet_file, '.*')
+
+      # Mock identify first
+      allow(Open3).to receive(:capture3).with(/identify/).and_return(
+        ["400x400\n", '', instance_double(Process::Status, success?: true)]
+      )
+
+      # Expect convert with frame filename
+      expect(Open3).to receive(:capture3).with(/FR001_#{basename}\.png/).and_return(
+        ['', '', instance_double(Process::Status, success?: true)]
+      )
+
+      splitter.split_into_frames(spritesheet_file, output_dir, columns, rows, 1)
+    end
+
+    it 'raises ProcessingError when ImageMagick fails' do
+      allow(Open3).to receive(:capture3).with(/identify/).and_return(
+        ["400x400\n", '', instance_double(Process::Status, success?: true)]
+      )
+
+      allow(Open3).to receive(:capture3).with(/convert/).and_return(
+        ['', 'Error message', instance_double(Process::Status, success?: false)]
+      )
+
+      expect {
+        splitter.split_into_frames(spritesheet_file, output_dir, columns, rows, 1)
+      }.to raise_error(RubySpriter::ProcessingError, /Could not extract frame/)
+    end
+
+    it 'displays progress information' do
+      expect(RubySpriter::Utils::OutputFormatter).to receive(:header).with(/Extracting Frames/)
+      expect(RubySpriter::Utils::OutputFormatter).to receive(:indent).with(/Splitting spritesheet into #{frames} frames/)
+      expect(RubySpriter::Utils::OutputFormatter).to receive(:indent).with(/Output directory:/)
+      expect(RubySpriter::Utils::OutputFormatter).to receive(:indent).with(/Frames extracted successfully/)
+
+      splitter.split_into_frames(spritesheet_file, output_dir, columns, rows, frames)
+    end
+  end
+end

data/spec/ruby_spriter/video_processor_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe RubySpriter::VideoProcessor do
+  describe '#create_spritesheet' do
+    let(:video_file) { 'test_video.mp4' }
+    let(:output_file) { 'spritesheet.png' }
+    let(:processor) { described_class.new(frame_count: 16, columns: 4) }
+
+    before do
+      allow(RubySpriter::Utils::FileHelper).to receive(:validate_readable!)
+      allow(File).to receive(:size).and_return(1000)
+      allow(File).to receive(:exist?).and_return(true)
+      allow(File).to receive(:delete)
+      allow(RubySpriter::MetadataManager).to receive(:embed)
+      allow(Open3).to receive(:capture3).and_return(['2.0', '', instance_double(Process::Status, success?: true)])
+    end
+
+    it 'creates spritesheet from video file' do
+      result = processor.create_spritesheet(video_file, output_file)
+
+      expect(result[:output_file]).to eq(output_file)
+      expect(result[:columns]).to eq(4)
+      expect(result[:rows]).to eq(4)
+      expect(result[:frames]).to eq(16)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby_spriter
 version: !ruby/object:Gem::Version
-  version: 0.6.5
+  version: 0.6.7
 platform: ruby
 authors:
 - scooter-indie
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-10-23 00:00:00.000000000 Z
+date: 2025-10-24 00:00:00.000000000 Z
 dependencies: []
 description: |
   Ruby Spriter is a cross-platform tool for creating spritesheets from video files
@@ -28,7 +28,9 @@ files:
 - README.md
 - bin/ruby_spriter
 - lib/ruby_spriter.rb
+- lib/ruby_spriter/batch_processor.rb
 - lib/ruby_spriter/cli.rb
+- lib/ruby_spriter/compression_manager.rb
 - lib/ruby_spriter/consolidator.rb
 - lib/ruby_spriter/dependency_checker.rb
 - lib/ruby_spriter/gimp_processor.rb
@@ -38,6 +40,7 @@ files:
 - lib/ruby_spriter/utils/file_helper.rb
 - lib/ruby_spriter/utils/output_formatter.rb
 - lib/ruby_spriter/utils/path_helper.rb
+- lib/ruby_spriter/utils/spritesheet_splitter.rb
 - lib/ruby_spriter/version.rb
 - lib/ruby_spriter/video_processor.rb
 - ruby_spriter.gemspec
@@ -47,7 +50,9 @@ files:
 - spec/fixtures/spritesheet_6x2.png
 - spec/fixtures/spritesheet_with_metadata.png
 - spec/fixtures/test_video.mp4
+- spec/ruby_spriter/batch_processor_spec.rb
 - spec/ruby_spriter/cli_spec.rb
+- spec/ruby_spriter/compression_manager_spec.rb
 - spec/ruby_spriter/consolidator_spec.rb
 - spec/ruby_spriter/dependency_checker_spec.rb
 - spec/ruby_spriter/gimp_processor_spec.rb
@@ -57,6 +62,7 @@ files:
 - spec/ruby_spriter/utils/file_helper_spec.rb
 - spec/ruby_spriter/utils/output_formatter_spec.rb
 - spec/ruby_spriter/utils/path_helper_spec.rb
+- spec/ruby_spriter/utils/spritesheet_splitter_spec.rb
 - spec/ruby_spriter/video_processor_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/scooter-indie/ruby-spriter