ruby_spriter 0.6.7 → 0.7.0.1

data/spec/ruby_spriter/utils/file_helper_spec.rb

@@ -1,150 +1,150 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-RSpec.describe RubySpriter::Utils::FileHelper do
-  describe '.spritesheet_filename' do
-    it 'generates correct filename from video' do
-      result = described_class.spritesheet_filename('/path/to/video.mp4')
-      expect(result).to eq('/path/to/video_spritesheet.png')
-    end
-
-    it 'handles different extensions' do
-      result = described_class.spritesheet_filename('C:\\videos\\clip.avi')
-      expect(result).to match(/clip_spritesheet\.png$/)
-    end
-  end
-
-  describe '.output_filename' do
-    it 'generates filename with suffix' do
-      result = described_class.output_filename('/path/to/image.png', 'scaled')
-      expect(result).to eq('/path/to/image-scaled.png')
-    end
-
-    it 'preserves directory path' do
-      result = described_class.output_filename('/some/deep/path/file.png', 'nobg')
-      expect(result).to start_with('/some/deep/path/')
-    end
-  end
-
-  describe '.format_size' do
-    it 'formats bytes correctly' do
-      expect(described_class.format_size(500)).to eq('500 bytes')
-    end
-
-    it 'formats kilobytes correctly' do
-      expect(described_class.format_size(2048)).to eq('2.0 KB')
-    end
-
-    it 'formats megabytes correctly' do
-      expect(described_class.format_size(5_242_880)).to eq('5.0 MB')
-    end
-  end
-
-  describe '.validate_exists!' do
-    it 'raises error for non-existent file' do
-      expect {
-        described_class.validate_exists!('/nonexistent/file.txt')
-      }.to raise_error(RubySpriter::ValidationError, /File not found/)
-    end
-
-    it 'does not raise for existing file' do
-      file = File.join(@test_dir, 'test.txt')
-      File.write(file, 'test')
-      
-      expect {
-        described_class.validate_exists!(file)
-      }.not_to raise_error
-    end
-  end
-
-  describe '.validate_readable!' 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
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe RubySpriter::Utils::FileHelper do
+  describe '.spritesheet_filename' do
+    it 'generates correct filename from video' do
+      result = described_class.spritesheet_filename('/path/to/video.mp4')
+      expect(result).to eq('/path/to/video_spritesheet.png')
+    end
+
+    it 'handles different extensions' do
+      result = described_class.spritesheet_filename('C:\\videos\\clip.avi')
+      expect(result).to match(/clip_spritesheet\.png$/)
+    end
+  end
+
+  describe '.output_filename' do
+    it 'generates filename with suffix' do
+      result = described_class.output_filename('/path/to/image.png', 'scaled')
+      expect(result).to eq('/path/to/image-scaled.png')
+    end
+
+    it 'preserves directory path' do
+      result = described_class.output_filename('/some/deep/path/file.png', 'nobg')
+      expect(result).to start_with('/some/deep/path/')
+    end
+  end
+
+  describe '.format_size' do
+    it 'formats bytes correctly' do
+      expect(described_class.format_size(500)).to eq('500 bytes')
+    end
+
+    it 'formats kilobytes correctly' do
+      expect(described_class.format_size(2048)).to eq('2.0 KB')
+    end
+
+    it 'formats megabytes correctly' do
+      expect(described_class.format_size(5_242_880)).to eq('5.0 MB')
+    end
+  end
+
+  describe '.validate_exists!' do
+    it 'raises error for non-existent file' do
+      expect {
+        described_class.validate_exists!('/nonexistent/file.txt')
+      }.to raise_error(RubySpriter::ValidationError, /File not found/)
+    end
+
+    it 'does not raise for existing file' do
+      file = File.join(@test_dir, 'test.txt')
+      File.write(file, 'test')
+      
+      expect {
+        described_class.validate_exists!(file)
+      }.not_to raise_error
+    end
+  end
+
+  describe '.validate_readable!' 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/path_helper_spec.rb

@@ -1,78 +1,78 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-RSpec.describe RubySpriter::Utils::PathHelper do
-  describe '.quote_path' do
-    context 'on Windows' do
-      before do
-        allow(RubySpriter::Platform).to receive(:windows?).and_return(true)
-      end
-
-      it 'wraps path in double quotes' do
-        expect(described_class.quote_path('C:\\test\\file.txt')).to eq('"C:\\test\\file.txt"')
-      end
-    end
-
-    context 'on Unix-like systems' do
-      before do
-        allow(RubySpriter::Platform).to receive(:windows?).and_return(false)
-      end
-
-      it 'wraps path in single quotes' do
-        expect(described_class.quote_path('/test/file.txt')).to eq("'/test/file.txt'")
-      end
-
-      it 'escapes single quotes in path' do
-        result = described_class.quote_path("/test/file's.txt")
-        # Should wrap in single quotes and escape internal single quotes
-        expect(result).to start_with("'")
-        expect(result).to end_with("'")
-        expect(result).to include("\\'")  # Should contain escaped single quote
-      end
-    end
-  end
-
-  describe '.normalize_for_python' do
-    it 'returns absolute path' do
-      result = described_class.normalize_for_python('.')
-      # Should return an absolute path (Unix: starts with /, Windows: starts with drive letter)
-      is_unix_absolute = result.start_with?('/')
-      is_windows_absolute = result.match?(/^[A-Z]:/i)
-      expect(is_unix_absolute || is_windows_absolute).to be true
-    end
-
-    context 'on Windows' do
-      before do
-        allow(RubySpriter::Platform).to receive(:windows?).and_return(true)
-      end
-
-      it 'converts backslashes to forward slashes' do
-        allow(File).to receive(:absolute_path).and_return('C:\\test\\file.txt')
-        expect(described_class.normalize_for_python('file.txt')).to eq('C:/test/file.txt')
-      end
-    end
-  end
-
-  describe '.to_native' do
-    context 'on Windows' do
-      before do
-        allow(RubySpriter::Platform).to receive(:windows?).and_return(true)
-      end
-
-      it 'converts forward slashes to backslashes' do
-        expect(described_class.to_native('C:/test/file.txt')).to eq('C:\\test\\file.txt')
-      end
-    end
-
-    context 'on Unix-like systems' do
-      before do
-        allow(RubySpriter::Platform).to receive(:windows?).and_return(false)
-      end
-
-      it 'converts backslashes to forward slashes' do
-        expect(described_class.to_native('C:\\test\\file.txt')).to eq('C:/test/file.txt')
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe RubySpriter::Utils::PathHelper do
+  describe '.quote_path' do
+    context 'on Windows' do
+      before do
+        allow(RubySpriter::Platform).to receive(:windows?).and_return(true)
+      end
+
+      it 'wraps path in double quotes' do
+        expect(described_class.quote_path('C:\\test\\file.txt')).to eq('"C:\\test\\file.txt"')
+      end
+    end
+
+    context 'on Unix-like systems' do
+      before do
+        allow(RubySpriter::Platform).to receive(:windows?).and_return(false)
+      end
+
+      it 'wraps path in single quotes' do
+        expect(described_class.quote_path('/test/file.txt')).to eq("'/test/file.txt'")
+      end
+
+      it 'escapes single quotes in path' do
+        result = described_class.quote_path("/test/file's.txt")
+        # Should wrap in single quotes and escape internal single quotes
+        expect(result).to start_with("'")
+        expect(result).to end_with("'")
+        expect(result).to include("\\'")  # Should contain escaped single quote
+      end
+    end
+  end
+
+  describe '.normalize_for_python' do
+    it 'returns absolute path' do
+      result = described_class.normalize_for_python('.')
+      # Should return an absolute path (Unix: starts with /, Windows: starts with drive letter)
+      is_unix_absolute = result.start_with?('/')
+      is_windows_absolute = result.match?(/^[A-Z]:/i)
+      expect(is_unix_absolute || is_windows_absolute).to be true
+    end
+
+    context 'on Windows' do
+      before do
+        allow(RubySpriter::Platform).to receive(:windows?).and_return(true)
+      end
+
+      it 'converts backslashes to forward slashes' do
+        allow(File).to receive(:absolute_path).and_return('C:\\test\\file.txt')
+        expect(described_class.normalize_for_python('file.txt')).to eq('C:/test/file.txt')
+      end
+    end
+  end
+
+  describe '.to_native' do
+    context 'on Windows' do
+      before do
+        allow(RubySpriter::Platform).to receive(:windows?).and_return(true)
+      end
+
+      it 'converts forward slashes to backslashes' do
+        expect(described_class.to_native('C:/test/file.txt')).to eq('C:\\test\\file.txt')
+      end
+    end
+
+    context 'on Unix-like systems' do
+      before do
+        allow(RubySpriter::Platform).to receive(:windows?).and_return(false)
+      end
+
+      it 'converts backslashes to forward slashes' do
+        expect(described_class.to_native('C:\\test\\file.txt')).to eq('C:/test/file.txt')
+      end
+    end
+  end
+end

data/spec/ruby_spriter/utils/spritesheet_splitter_spec.rb

@@ -1,104 +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
+# 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