resizing 1.2.0 → 1.2.2

data/test/resizing/configuration_test.rb

@@ -23,7 +23,6 @@ module Resizing
       template = @template.dup
       template.delete(:image_host)
       config = Resizing::Configuration.new template
-      assert_equal(config.host, Resizing::Configuration::DEFAULT_IMAGE_HOST)
       assert_equal(config.image_host, Resizing::Configuration::DEFAULT_IMAGE_HOST)
     end
 
@@ -38,7 +37,7 @@ module Resizing
       template = @template.dup
       template[:host] = 'need raise execption if host is presented'
       assert_raises ConfigurationError do
-        config = Resizing::Configuration.new template
+        _config = Resizing::Configuration.new template
       end
     end
 
@@ -147,5 +146,122 @@ module Resizing
       config = Resizing::Configuration.new @template
       assert_match %r{/projects/#{config.project_id}/upload/images/[\da-z-]}, config.generate_identifier
     end
+
+    def test_generate_image_id_returns_uuid
+      config = Resizing::Configuration.new @template
+      image_id = config.generate_image_id
+
+      assert_instance_of String, image_id
+      # UUID format: 8-4-4-4-12
+      assert_match(/\A[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}\z/, image_id)
+    end
+
+    def test_generate_image_id_returns_different_uuids
+      config = Resizing::Configuration.new @template
+      id1 = config.generate_image_id
+      id2 = config.generate_image_id
+
+      refute_equal id1, id2
+    end
+
+    def test_equality_returns_true_for_same_configurations
+      config1 = Resizing::Configuration.new @template
+      config2 = Resizing::Configuration.new @template
+
+      assert_equal config1, config2
+    end
+
+    def test_equality_returns_false_for_different_image_host
+      config1 = Resizing::Configuration.new @template
+      template2 = @template.dup
+      template2[:image_host] = 'http://different.host'
+      config2 = Resizing::Configuration.new template2
+
+      refute_equal config1, config2
+    end
+
+    def test_equality_returns_false_for_different_project_id
+      config1 = Resizing::Configuration.new @template
+      template2 = @template.dup
+      template2[:project_id] = 'different-project-id'
+      config2 = Resizing::Configuration.new template2
+
+      refute_equal config1, config2
+    end
+
+    def test_equality_returns_false_for_different_class
+      config = Resizing::Configuration.new @template
+
+      refute_equal config, 'not a configuration'
+      refute_equal config, @template
+    end
+
+    def test_transformation_path_with_multiple_transforms
+      config = Resizing::Configuration.new @template
+      transforms = [
+        { w: 100, h: 200 },
+        { f: 'webp', q: 80 }
+      ]
+
+      path = config.transformation_path(transforms)
+
+      assert_equal 'w_100,h_200/f_webp,q_80', path
+    end
+
+    def test_transformation_path_with_single_hash
+      config = Resizing::Configuration.new @template
+      transform = { w: 300, h: 300, c: 'fill' }
+
+      path = config.transformation_path(transform)
+
+      assert_equal 'w_300,h_300,c_fill', path
+    end
+
+    def test_transformation_path_ignores_unknown_options
+      config = Resizing::Configuration.new @template
+      transform = { w: 100, unknown_option: 'ignored', h: 200 }
+
+      path = config.transformation_path(transform)
+
+      assert_equal 'w_100,h_200', path
+      refute_includes path, 'unknown_option'
+    end
+
+    def test_transformation_path_with_empty_array
+      config = Resizing::Configuration.new @template
+      path = config.transformation_path([])
+
+      assert_equal '', path
+    end
+
+    def test_generate_identifier_includes_project_id
+      config = Resizing::Configuration.new @template
+      identifier = config.generate_identifier
+
+      assert_includes identifier, config.project_id
+    end
+
+    def test_generate_identifier_has_correct_format
+      config = Resizing::Configuration.new @template
+      identifier = config.generate_identifier
+
+      assert_match %r{\A/projects/[\da-z-]+/upload/images/[\da-z-]+\z}, identifier
+    end
+
+    def test_enable_mock_defaults_to_false
+      template = @template.dup
+      template.delete(:enable_mock)
+      config = Resizing::Configuration.new template
+
+      assert_equal false, config.enable_mock
+    end
+
+    def test_enable_mock_can_be_set_to_true
+      template = @template.dup
+      template[:enable_mock] = true
+      config = Resizing::Configuration.new template
+
+      assert_equal true, config.enable_mock
+    end
   end
 end

data/test/resizing/constants_test.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+module Resizing
+  class ConstantsTest < Minitest::Test
+    def test_http_status_ok_is_defined
+      assert_equal 200, Resizing::Constants::HTTP_STATUS_OK
+    end
+
+    def test_http_status_created_is_defined
+      assert_equal 201, Resizing::Constants::HTTP_STATUS_CREATED
+    end
+
+    def test_http_status_not_found_is_defined
+      assert_equal 404, Resizing::Constants::HTTP_STATUS_NOT_FOUND
+    end
+
+    def test_constants_are_integers
+      assert_instance_of Integer, Resizing::Constants::HTTP_STATUS_OK
+      assert_instance_of Integer, Resizing::Constants::HTTP_STATUS_CREATED
+      assert_instance_of Integer, Resizing::Constants::HTTP_STATUS_NOT_FOUND
+    end
+  end
+end

data/test/resizing/error_test.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+module Resizing
+  class ErrorTest < Minitest::Test
+    def test_error_is_standard_error_subclass
+      assert Resizing::Error < StandardError
+    end
+
+    def test_configuration_error_is_error_subclass
+      assert Resizing::ConfigurationError < Resizing::Error
+    end
+
+    def test_api_error_is_error_subclass
+      assert Resizing::APIError < Resizing::Error
+    end
+
+    def test_api_error_has_decoded_body_accessor
+      error = Resizing::APIError.new('test error')
+
+      assert_respond_to error, :decoded_body
+      assert_respond_to error, :decoded_body=
+    end
+
+    def test_api_error_decoded_body_defaults_to_empty_hash
+      error = Resizing::APIError.new('test error')
+
+      assert_equal({}, error.decoded_body)
+    end
+
+    def test_api_error_decoded_body_can_be_set_with_hash
+      error = Resizing::APIError.new('test error')
+      body = { 'error' => 'test', 'code' => 400 }
+
+      error.decoded_body = body
+
+      assert_equal body, error.decoded_body
+    end
+
+    def test_api_error_decoded_body_raises_argument_error_for_non_hash
+      error = Resizing::APIError.new('test error')
+
+      assert_raises ArgumentError do
+        error.decoded_body = 'not a hash'
+      end
+    end
+
+    def test_api_error_decoded_body_raises_argument_error_for_array
+      error = Resizing::APIError.new('test error')
+
+      assert_raises ArgumentError do
+        error.decoded_body = %w[not a hash]
+      end
+    end
+
+    def test_configuration_error_can_be_raised_with_message
+      error = assert_raises Resizing::ConfigurationError do
+        raise Resizing::ConfigurationError, 'test configuration error'
+      end
+
+      assert_equal 'test configuration error', error.message
+    end
+
+    def test_api_error_can_be_raised_with_message
+      error = assert_raises Resizing::APIError do
+        raise Resizing::APIError, 'test api error'
+      end
+
+      assert_equal 'test api error', error.message
+    end
+  end
+end

data/test/resizing/http_clientable_test.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+module Resizing
+  class HttpClientableTest < Minitest::Test
+    class TestClient
+      include HttpClientable
+
+      def config
+        @config ||= Configuration.new(
+          image_host: 'https://image.example.com',
+          project_id: 'test_project',
+          secret_token: 'test_token'
+        )
+      end
+    end
+
+    def setup
+      @client = TestClient.new
+    end
+
+    def teardown
+      # NOP
+    end
+
+    def test_http_client_initialization
+      http_client = @client.http_client
+
+      assert_instance_of Faraday::Connection, http_client
+    end
+
+    def test_http_client_has_open_timeout
+      http_client = @client.http_client
+
+      assert_equal @client.config.open_timeout, http_client.options[:open_timeout]
+    end
+
+    def test_http_client_has_response_timeout
+      http_client = @client.http_client
+
+      assert_equal @client.config.response_timeout, http_client.options[:timeout]
+    end
+
+    def test_http_client_is_cached
+      http_client1 = @client.http_client
+      http_client2 = @client.http_client
+
+      assert_equal http_client1.object_id, http_client2.object_id
+    end
+
+    def test_handle_faraday_error_yields_block
+      result = @client.handle_faraday_error { 'test_result' }
+
+      assert_equal 'test_result', result
+    end
+
+    def test_handle_faraday_error_catches_timeout_error
+      assert_raises Resizing::APIError do
+        @client.handle_faraday_error do
+          raise Faraday::TimeoutError, 'timeout'
+        end
+      end
+    end
+
+    def test_handle_timeout_error_raises_api_error
+      error = Faraday::TimeoutError.new('test timeout')
+
+      assert_raises Resizing::APIError do
+        @client.handle_timeout_error(error)
+      end
+    end
+
+    def test_handle_timeout_error_message_includes_error_info
+      error = Faraday::TimeoutError.new('test timeout')
+
+      begin
+        @client.handle_timeout_error(error)
+      rescue Resizing::APIError => e
+        assert_includes e.message, 'TimeoutError'
+      end
+    end
+  end
+end

data/test/resizing/mock_client_test.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+module Resizing
+  class MockClientTest < Minitest::Test
+    def setup
+      @client = Resizing::MockClient.new
+    end
+
+    def teardown
+      # NOP
+    end
+
+    def test_post_returns_parsed_json
+      VCR.use_cassette('client/post', record: :none) do
+        result = @client.post(nil)
+
+        assert_instance_of Hash, result
+        assert result.key?('id')
+        assert result.key?('public_id')
+        assert result.key?('latest_version_id')
+        assert result.key?('latest_etag')
+      end
+    end
+
+    def test_put_returns_parsed_json_with_modified_name
+      VCR.use_cassette('client/put', record: :none) do
+        name = 'test-image-123'
+        result = @client.put(name, nil, {})
+
+        assert_instance_of Hash, result
+        assert_equal name, result['id']
+        assert_includes result['public_id'], name
+        assert result.key?('latest_version_id')
+        assert result.key?('latest_etag')
+      end
+    end
+
+    def test_delete_returns_parsed_json_with_modified_name
+      VCR.use_cassette('client/delete', record: :none) do
+        name = 'delete-test-image'
+        result = @client.delete(name)
+
+        assert_instance_of Hash, result
+        assert_equal name, result['id']
+        assert_includes result['public_id'], name
+      end
+    end
+
+    def test_metadata_returns_parsed_json_with_modified_name
+      VCR.use_cassette('client/metadata', record: :none) do
+        name = 'metadata-test-image'
+        result = @client.metadata(name)
+
+        assert_instance_of Hash, result
+        assert_equal name, result['id']
+        # The cassette contains a fixed public_id, so we just check it exists
+        assert result.key?('public_id')
+      end
+    end
+
+    def test_post_response_contains_expected_fields
+      VCR.use_cassette('client/post', record: :none) do
+        result = @client.post(nil)
+
+        # Verify response structure
+        assert result['id'].is_a?(String)
+        assert result['public_id'].is_a?(String)
+        assert result['latest_version_id'].is_a?(String)
+        assert result['latest_etag'].is_a?(String)
+      end
+    end
+  end
+end

data/test/resizing/public_id_test.rb

@@ -30,7 +30,7 @@ module Resizing
 
     def test_expect_equal_identifier
       public_id = Resizing::PublicId.new @public_id_as_string
-      assert_equal @public_id_as_string.gsub(/\/v.*$/, ''), public_id.identifier
+      assert_equal @public_id_as_string.gsub(%r{/v.*$}, ''), public_id.identifier
     end
 
     def test_expect_equal_public_id

data/test/resizing_module_test.rb

@@ -0,0 +1,206 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class ResizingModuleTest < Minitest::Test
+  def setup
+    # Reset configure before each test to ensure clean state
+    return unless Resizing.instance_variable_defined?(:@configure)
+
+    Resizing.remove_instance_variable(:@configure)
+  end
+
+  def teardown
+    # Reset configure after each test
+    return unless Resizing.instance_variable_defined?(:@configure)
+
+    Resizing.remove_instance_variable(:@configure)
+  end
+
+  def test_configure_raises_error_when_not_initialized
+    assert_raises Resizing::ConfigurationError do
+      Resizing.configure
+    end
+  end
+
+  def test_configure_returns_duplicate_of_configuration
+    config = Resizing::Configuration.new(
+      image_host: 'https://test.example.com',
+      project_id: 'test_id',
+      secret_token: 'test_token'
+    )
+    Resizing.configure = config
+
+    result = Resizing.configure
+
+    assert_instance_of Resizing::Configuration, result
+    assert_equal config.image_host, result.image_host
+    assert_equal config.project_id, result.project_id
+    refute_equal config.object_id, result.object_id # Should be a duplicate
+  end
+
+  def test_configure_setter_accepts_configuration_object
+    config = Resizing::Configuration.new(
+      image_host: 'https://test.example.com',
+      project_id: 'test_id',
+      secret_token: 'test_token'
+    )
+
+    Resizing.configure = config
+
+    assert_equal config, Resizing.instance_variable_get(:@configure)
+  end
+
+  def test_configure_setter_converts_hash_to_configuration
+    config_hash = {
+      image_host: 'https://hash.example.com',
+      project_id: 'hash_id',
+      secret_token: 'hash_token'
+    }
+
+    Resizing.configure = config_hash
+
+    result = Resizing.instance_variable_get(:@configure)
+    assert_instance_of Resizing::Configuration, result
+    assert_equal 'https://hash.example.com', result.image_host
+    assert_equal 'hash_id', result.project_id
+  end
+
+  def test_get_raises_not_implemented_error
+    assert_raises NotImplementedError do
+      Resizing.get('test')
+    end
+  end
+
+  def test_url_from_image_id_returns_url_without_version_and_transforms
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123'
+    }
+
+    url = Resizing.url_from_image_id('image456')
+
+    assert_equal 'https://img.example.com/projects/project123/upload/images/image456', url
+  end
+
+  def test_url_from_image_id_returns_url_with_version
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123'
+    }
+
+    url = Resizing.url_from_image_id('image456', '789')
+
+    assert_equal 'https://img.example.com/projects/project123/upload/images/image456/v789', url
+  end
+
+  def test_url_from_image_id_returns_url_with_transformations
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123'
+    }
+
+    url = Resizing.url_from_image_id('image456', nil, [{ w: 100, h: 200 }])
+
+    assert_includes url, 'https://img.example.com/projects/project123/upload/images/image456/'
+    assert_includes url, 'w_100,h_200'
+  end
+
+  def test_url_from_image_id_returns_url_with_version_and_transformations
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123'
+    }
+
+    url = Resizing.url_from_image_id('image456', '789', [{ w: 100 }])
+
+    assert_includes url, 'image456/v789/'
+    assert_includes url, 'w_100'
+  end
+
+  def test_generate_identifier_returns_identifier_string
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123'
+    }
+
+    identifier = Resizing.generate_identifier
+
+    assert_instance_of String, identifier
+    assert_includes identifier, 'project123'
+    assert_match %r{/projects/project123/upload/images/}, identifier
+  end
+
+  def test_client_returns_mock_client_when_enable_mock_is_true
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123',
+      enable_mock: true
+    }
+
+    client = Resizing.client
+
+    assert_instance_of Resizing::MockClient, client
+  end
+
+  def test_client_returns_real_client_when_enable_mock_is_false
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123',
+      enable_mock: false
+    }
+
+    client = Resizing.client
+
+    assert_instance_of Resizing::Client, client
+  end
+
+  def test_put_delegates_to_client
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123',
+      enable_mock: true
+    }
+
+    result = Resizing.put('image_id', 'dummy', content_type: 'image/jpeg')
+
+    assert_instance_of Hash, result
+    assert_equal 'image_id', result['id']
+  end
+
+  def test_delete_delegates_to_client
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123',
+      enable_mock: true
+    }
+
+    result = Resizing.delete('image_id')
+
+    assert_instance_of Hash, result
+    assert_equal 'image_id', result['id']
+  end
+
+  def test_metadata_delegates_to_client
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123',
+      enable_mock: true
+    }
+
+    result = Resizing.metadata('image_id', {})
+
+    assert_instance_of Hash, result
+    assert_equal 'image_id', result['id']
+  end
+end