resizing 1.2.1 → 1.3.1

data/test/resizing/configuration_test.rb

@@ -4,10 +4,11 @@ require 'test_helper'
 
 module Resizing
   class ConfigurationTest < Minitest::Test
+    ACTIVE_SUPPORT_OBJECT_EXTENSIONS = %i[present? blank? try].freeze
+
     def setup
       @template = {
         image_host: 'http://192.168.56.101:5000',
-        video_host: 'http://192.168.56.101:5000',
         project_id: '098a2a0d-c387-4135-a071-1254d6d7e70a',
         secret_token: '4g1cshg2lq8j93ufhvqrpjswxmtjz12yhfvq6w79jpwi7cr7nnknoqgwzkwerbs6',
         open_timeout: 10,
@@ -26,11 +27,13 @@ module Resizing
       assert_equal(config.image_host, Resizing::Configuration::DEFAULT_IMAGE_HOST)
     end
 
-    def test_that_it_has_default_video_host
-      template = @template.dup
-      template.delete(:video_host)
-      config = Resizing::Configuration.new template
-      assert_equal(config.video_host, Resizing::Configuration::DEFAULT_VIDEO_HOST)
+    # 動画 API の廃止にともない video_host は非推奨。値は保持するが参照時に警告を出す
+    def test_that_video_host_reader_warns_and_still_returns_the_default
+      config = Resizing::Configuration.new @template.dup
+
+      _out, err = capture_io { assert_equal('https://video.resizing.net', config.video_host) }
+
+      assert_includes err, Resizing::Configuration::DEPRECATED_VIDEO_HOST_MESSAGE
     end
 
     def test_that_it_need_raise_exception_if_host_presented
@@ -41,16 +44,77 @@ module Resizing
       end
     end
 
-    def test_that_it_has_same_image_host_value
+    def test_that_it_does_not_raise_exception_if_host_is_nil
       template = @template.dup
+      template[:host] = nil
       config = Resizing::Configuration.new template
       assert_equal(config.image_host, template[:image_host])
     end
 
-    def test_that_it_has_same_video_host_value
+    def test_that_it_does_not_raise_exception_if_host_is_empty_string
+      ['', '   '].each do |host|
+        template = @template.dup
+        template[:host] = host
+        config = Resizing::Configuration.new template
+        assert_equal(config.image_host, template[:image_host])
+      end
+    end
+
+    # Rails 外(ActiveSupport 未ロード)から利用されるため、
+    # Configuration は present? / blank? などの ActiveSupport 拡張に依存してはいけない。
+    # see: https://github.com/jksy/resizing-gem/issues/87
+    def test_that_it_does_not_depend_on_active_support_object_extensions
+      without_active_support_object_extensions do
+        template = @template.dup
+        config = Resizing::Configuration.new template
+        assert_equal(config.image_host, template[:image_host])
+
+        template_with_host = @template.dup
+        template_with_host[:host] = 'https://www.resizing.net'
+        assert_raises ConfigurationError do
+          Resizing::Configuration.new template_with_host
+        end
+      end
+    end
+
+    def test_that_it_has_same_image_host_value
       template = @template.dup
       config = Resizing::Configuration.new template
-      assert_equal(config.video_host, template[:video_host])
+      assert_equal(config.image_host, template[:image_host])
+    end
+
+    def test_that_passing_video_host_warns_but_keeps_the_value
+      template = @template.dup
+      template[:video_host] = 'http://192.168.56.101:5000'
+
+      _out, err = capture_io { @config = Resizing::Configuration.new template }
+
+      assert_includes err, Resizing::Configuration::DEPRECATED_VIDEO_HOST_MESSAGE
+      capture_io { assert_equal(template[:video_host], @config.video_host) }
+    end
+
+    # 非推奨だが == の比較対象からは外さない。比較そのものは警告を出さない
+    def test_equality_still_compares_the_deprecated_video_host
+      base = other = nil
+      capture_io do
+        base = Resizing::Configuration.new @template.merge(video_host: 'https://a.example.com')
+        other = Resizing::Configuration.new @template.merge(video_host: 'https://b.example.com')
+      end
+
+      _out, err = capture_io { refute_equal base, other }
+
+      assert_empty err
+    end
+
+    def test_that_the_default_video_host_constant_is_deprecated
+      previous = Warning[:deprecated]
+      Warning[:deprecated] = true
+
+      _out, err = capture_io { Resizing::Configuration::DEFAULT_VIDEO_HOST }
+
+      assert_match(/DEFAULT_VIDEO_HOST is deprecated/, err)
+    ensure
+      Warning[:deprecated] = previous
     end
 
     def test_that_it_has_no_project_id
@@ -69,7 +133,7 @@ module Resizing
 
     def test_that_it_has_no_secret_token
       template = @template.dup
-      template.delete(:project_id)
+      template.delete(:secret_token)
       assert_raises ConfigurationError do
         Resizing::Configuration.new template
       end
@@ -263,5 +327,166 @@ module Resizing
 
       assert_equal true, config.enable_mock
     end
+
+    def test_that_it_has_default_response_timeout
+      template = @template.dup
+      template.delete(:response_timeout)
+      config = Resizing::Configuration.new template
+      assert_equal(Resizing::Configuration::DEFAULT_RESPONSE_TIMEOUT, config.response_timeout)
+    end
+
+    def test_that_it_has_same_response_timeout
+      config = Resizing::Configuration.new @template
+      assert_equal(@template[:response_timeout], config.response_timeout)
+    end
+
+    def test_that_it_raises_when_initialized_without_hash
+      assert_raises(ConfigurationError) { Resizing::Configuration.new }
+      assert_raises(ConfigurationError) { Resizing::Configuration.new(nil) }
+      assert_raises(ConfigurationError) { Resizing::Configuration.new('not a hash') }
+    end
+
+    def test_that_it_can_be_initialized_with_only_required_keys
+      config = Resizing::Configuration.new(project_id: 'project', secret_token: 'token')
+
+      assert_equal 'project', config.project_id
+      assert_equal 'token', config.secret_token
+      assert_equal Resizing::Configuration::DEFAULT_IMAGE_HOST, config.image_host
+      assert_equal Resizing::Configuration::DEFAULT_OPEN_TIMEOUT, config.open_timeout
+      assert_equal Resizing::Configuration::DEFAULT_RESPONSE_TIMEOUT, config.response_timeout
+      assert_equal false, config.enable_mock
+    end
+
+    def test_that_string_attributes_are_frozen_copies_of_input
+      template = @template.dup
+      template[:image_host] = +'http://mutable.host'
+      template[:project_id] = +'mutable-project'
+      template[:secret_token] = +'mutable-token'
+      config = Resizing::Configuration.new template
+
+      assert config.image_host.frozen?
+      assert config.project_id.frozen?
+      assert config.secret_token.frozen?
+
+      # 入力の Hash の文字列を後から変更しても設定値には影響しない
+      template[:image_host] << '/changed'
+      template[:project_id] << '-changed'
+      template[:secret_token] << '-changed'
+      assert_equal 'http://mutable.host', config.image_host
+      assert_equal 'mutable-project', config.project_id
+      assert_equal 'mutable-token', config.secret_token
+    end
+
+    def test_that_auth_header_is_version_timestamp_and_sha256_of_timestamp_and_secret_token
+      # サーバー側 (ResizingTokenAuthenticator) は "v1,<unix time>,<sha256>" 形式を期待している
+      now = Time.parse('2024-01-02 03:04:05 UTC')
+      Timecop.freeze(now) do
+        config = Resizing::Configuration.new @template
+        header = config.generate_auth_header
+
+        assert_match(/\Av\d,\d+,[0-9a-f]{64}\z/, header)
+
+        version, timestamp, token = header.split(',')
+        assert_equal 'v1', version
+        assert_equal now.to_i.to_s, timestamp
+        assert_equal Digest::SHA2.hexdigest("#{now.to_i}|#{@template[:secret_token]}"), token
+      end
+    end
+
+    def test_that_auth_header_changes_with_time
+      config = Resizing::Configuration.new @template
+      first = Timecop.freeze(Time.parse('2024-01-01 00:00:00 UTC')) { config.generate_auth_header }
+      second = Timecop.freeze(Time.parse('2024-01-01 00:00:01 UTC')) { config.generate_auth_header }
+
+      refute_equal first, second
+    end
+
+    def test_that_it_return_image_url_with_transformations
+      config = Resizing::Configuration.new @template
+      assert_equal(
+        'http://192.168.56.101:5000/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/some-image-id/w_100,h_200',
+        config.generate_image_url('some-image-id', nil, [{ w: 100, h: 200 }])
+      )
+    end
+
+    def test_that_it_return_image_url_with_version_id_and_transformations
+      config = Resizing::Configuration.new @template
+      assert_equal(
+        'http://192.168.56.101:5000/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/some-image-id/vversion-id/w_40,h_40,c_fill/f_webp',
+        config.generate_image_url('some-image-id', 'version-id', [{ c: 'fill', w: 40, h: 40 }, { f: 'webp' }])
+      )
+    end
+
+    def test_that_it_return_image_url_accepts_single_hash_transformation
+      config = Resizing::Configuration.new @template
+      assert_equal(
+        'http://192.168.56.101:5000/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/some-image-id/w_100',
+        config.generate_image_url('some-image-id', nil, { w: 100 })
+      )
+    end
+
+    def test_that_it_return_image_url_without_trailing_slash_for_empty_transformation
+      config = Resizing::Configuration.new @template
+      expected = 'http://192.168.56.101:5000/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/some-image-id'
+
+      assert_equal expected, config.generate_image_url('some-image-id', nil, [])
+      assert_equal expected, config.generate_image_url('some-image-id', nil, [{}])
+      assert_equal expected, config.generate_image_url('some-image-id', nil, { unknown: 1 })
+    end
+
+    def test_transformation_path_normalizes_key_order_to_transform_options_order
+      # Hash#slice は引数の順で返すため、入力順に関係なく TRANSFORM_OPTIONS の順 (w, h, f, c, q) になる
+      config = Resizing::Configuration.new @template
+
+      assert_equal 'w_100,h_200', config.transformation_path(h: 200, w: 100)
+      assert_equal 'w_100,h_200', config.transformation_path(w: 100, h: 200)
+      assert_equal 'w_40,h_40,f_webp,c_fill,q_80', config.transformation_path(q: 80, c: 'fill', f: 'webp', h: 40, w: 40)
+    end
+
+    def test_transformation_path_with_quality
+      config = Resizing::Configuration.new @template
+
+      assert_equal 'q_80', config.transformation_path(q: 80)
+      assert_equal 'w_100,q_auto', config.transformation_path(w: 100, q: 'auto')
+    end
+
+    def test_equality_returns_false_when_any_compared_attribute_differs
+      base = Resizing::Configuration.new @template
+      {
+        secret_token: 'different-token',
+        open_timeout: 99,
+        response_timeout: 99
+      }.each do |key, value|
+        other = Resizing::Configuration.new @template.merge(key => value)
+        refute_equal base, other, "#{key} should be compared by =="
+      end
+    end
+
+    private
+
+    def without_active_support_object_extensions
+      removed = each_active_support_extension.map do |klass, method_name|
+        klass.send(:alias_method, backup_name(method_name), method_name)
+        klass.send(:undef_method, method_name)
+        [klass, method_name]
+      end
+
+      yield
+    ensure
+      removed&.each do |klass, method_name|
+        klass.send(:alias_method, method_name, backup_name(method_name))
+        klass.send(:remove_method, backup_name(method_name))
+      end
+    end
+
+    def each_active_support_extension
+      [NilClass, String].product(ACTIVE_SUPPORT_OBJECT_EXTENSIONS).select do |klass, method_name|
+        klass.method_defined?(method_name)
+      end
+    end
+
+    def backup_name(method_name)
+      :"__resizing_test_original_#{method_name}"
+    end
   end
 end

data/test/resizing/http_clientable_test.rb

@@ -80,5 +80,32 @@ module Resizing
         assert_includes e.message, 'TimeoutError'
       end
     end
+
+    def test_http_client_uses_multipart_and_url_encoded_request_middleware
+      handlers = @client.http_client.builder.handlers
+
+      assert_includes handlers, Faraday::Multipart::Middleware
+      assert_includes handlers, Faraday::Request::UrlEncoded
+    end
+
+    def test_handle_faraday_error_does_not_rescue_other_faraday_errors
+      # TimeoutError 以外の Faraday エラーはそのまま伝播する (現状の仕様)
+      assert_raises Faraday::ConnectionFailed do
+        @client.handle_faraday_error { raise Faraday::ConnectionFailed, 'connection refused' }
+      end
+    end
+
+    def test_handle_faraday_error_does_not_rescue_non_faraday_errors
+      assert_raises RuntimeError do
+        @client.handle_faraday_error { raise 'unexpected' }
+      end
+    end
+
+    def test_handle_timeout_error_message_includes_original_message
+      error = Faraday::TimeoutError.new('execution expired')
+
+      raised = assert_raises(Resizing::APIError) { @client.handle_timeout_error(error) }
+      assert_includes raised.message, 'execution expired'
+    end
   end
 end

data/test/resizing/mock_client_test.rb

@@ -34,9 +34,50 @@ module Resizing
         assert_includes result['public_id'], name
         assert result.key?('latest_version_id')
         assert result.key?('latest_etag')
+
+        # id と public_id の image_id が一致していること
+        public_id = Resizing::PublicId.new(result['public_id'])
+        assert_equal name, public_id.image_id
+        # カセットに含まれる元の image_id が残っていないこと
+        refute_includes result['public_id'], 'AWEaewfAreaweFAFASfwe'
+        # version がカセットの固定値から更新されていること
+        refute_equal 'fztekhN_WoeXo8ZkCZ4i5jcQvmPpZewR', result['version']
+        refute_includes result['public_id'], 'vfztekhN_WoeXo8ZkCZ4i5jcQvmPpZewR'
+        # version / latest_version_id / public_id の version が揃っていること
+        assert_equal result['version'], public_id.version
+        assert_equal result['latest_version_id'], public_id.version
       end
     end
 
+    def test_put_generates_new_version_for_each_call
+      name = 'test-image-123'
+      first = @client.put(name, nil, {})
+      second = @client.put(name, nil, {})
+
+      refute_equal first['version'], second['version']
+      refute_equal first['public_id'], second['public_id']
+    end
+
+    def test_metadata_returns_public_id_for_given_name
+      name = 'metadata-test-image'
+      result = @client.metadata(name)
+      public_id = Resizing::PublicId.new(result['public_id'])
+
+      assert_equal result['id'], public_id.image_id
+      assert_equal name, public_id.image_id
+    end
+
+    def test_delete_keeps_version_from_cassette
+      name = 'delete-test-image'
+      result = @client.delete(name)
+      public_id = Resizing::PublicId.new(result['public_id'])
+
+      assert_equal name, public_id.image_id
+      assert_equal result['project_id'], public_id.project_id
+      assert_equal result['latest_version_id'], public_id.version
+      refute_includes result['public_id'], '28c49144-c00d-4cb5-8619-98ce95977b9c'
+    end
+
     def test_delete_returns_parsed_json_with_modified_name
       VCR.use_cassette('client/delete', record: :none) do
         name = 'delete-test-image'
@@ -55,8 +96,17 @@ module Resizing
 
         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')
+
+        # id と public_id の image_id が一致していること
+        public_id = Resizing::PublicId.new(result['public_id'])
+        assert_equal name, public_id.image_id
+        assert_equal result['id'], public_id.image_id
+        # カセットに含まれる元の image_id が残っていないこと
+        refute_includes result['public_id'], '87263920-2081-498e-a107-9625f4fde01b'
+        # project_id と version はカセットの値を引き継ぐこと
+        assert_equal result['project_id'], public_id.project_id
+        assert_equal result['version'], public_id.version
       end
     end
 
@@ -71,5 +121,65 @@ module Resizing
         assert result['latest_etag'].is_a?(String)
       end
     end
+
+    def test_mock_client_does_not_perform_http_requests
+      # VCR はカセット外の HTTP 接続を禁止しているため、カセットなしで成功すれば通信していないことが分かる
+      post_result = @client.post('test/data/images/sample1.jpg', content_type: 'image/jpeg')
+      assert_equal '87263920-2081-498e-a107-9625f4fde01b', post_result['id']
+
+      refute_nil @client.put('img', nil, {})
+      refute_nil @client.delete('img')
+      refute_nil @client.metadata('img')
+    end
+
+    def test_post_returns_public_id_parsable_by_public_id
+      result = @client.post(nil)
+      public_id = Resizing::PublicId.new(result['public_id'])
+
+      assert_equal result['id'], public_id.image_id
+      assert_equal result['project_id'], public_id.project_id
+      assert_equal result['latest_version_id'], public_id.version
+    end
+
+    def test_put_returns_public_id_for_given_name
+      name = 'test-image-123'
+      result = @client.put(name, nil, {})
+      public_id = Resizing::PublicId.new(result['public_id'])
+
+      assert_equal name, public_id.image_id
+      refute_nil public_id.version
+    end
+
+    def test_delete_returns_public_id_for_given_name
+      name = 'delete-test-image'
+      result = @client.delete(name)
+      public_id = Resizing::PublicId.new(result['public_id'])
+
+      assert_equal name, public_id.image_id
+    end
+
+    def test_each_call_returns_independent_result
+      first = @client.post(nil)
+      second = @client.post(nil)
+
+      refute_same first, second
+      first['id'] = 'changed'
+      first['public_id'] << '/changed'
+      assert_equal '87263920-2081-498e-a107-9625f4fde01b', second['id']
+      refute_includes second['public_id'], 'changed'
+    end
+
+    def test_mock_client_is_selected_by_resizing_module_when_enable_mock
+      Resizing.configure = {
+        image_host: 'https://img.example.com',
+        project_id: 'project123',
+        secret_token: 'token123',
+        enable_mock: true
+      }
+
+      result = Resizing.post('test/data/images/sample1.jpg', content_type: 'image/jpeg')
+
+      assert_equal '87263920-2081-498e-a107-9625f4fde01b', result['id']
+    end
   end
 end

data/test/resizing/public_id_test.rb

@@ -37,5 +37,101 @@ module Resizing
       public_id = Resizing::PublicId.new @public_id_as_string
       assert_equal @public_id_as_string, public_id.to_s
     end
+
+    def test_empty_is_false_for_valid_public_id
+      public_id = Resizing::PublicId.new @public_id_as_string
+      refute public_id.empty?
+    end
+
+    def test_filename_returns_image_id
+      public_id = Resizing::PublicId.new @public_id_as_string
+      assert_equal @image_id, public_id.filename
+    end
+
+    def test_version_is_nil_when_public_id_has_no_version
+      public_id = Resizing::PublicId.new "/projects/#{@project_id}/upload/images/#{@image_id}"
+
+      assert_nil public_id.version
+      assert_equal @image_id, public_id.image_id
+      assert_equal @project_id, public_id.project_id
+      assert_equal "/projects/#{@project_id}/upload/images/#{@image_id}", public_id.identifier
+    end
+
+    def test_version_may_start_with_underscore
+      # 実際の API が返すバージョン ID の形式 (test/vcr/carrier_wave_test/save.yml より)
+      public_id = Resizing::PublicId.new "/projects/#{@project_id}/upload/images/#{@image_id}/v_8c5lEhDB5RT3PZp1Fn5PYGm9YVx_x0e"
+
+      assert_equal '_8c5lEhDB5RT3PZp1Fn5PYGm9YVx_x0e', public_id.version
+      assert_equal @image_id, public_id.image_id
+    end
+
+    def test_nil_public_id_is_empty_and_has_no_parts
+      public_id = Resizing::PublicId.new nil
+
+      assert public_id.empty?
+      assert_nil public_id.image_id
+      assert_nil public_id.project_id
+      assert_nil public_id.version
+      assert_equal '', public_id.to_s
+    end
+
+    def test_empty_string_public_id_is_empty_and_has_no_parts
+      public_id = Resizing::PublicId.new ''
+
+      assert public_id.empty?
+      assert_nil public_id.image_id
+      assert_nil public_id.project_id
+      assert_nil public_id.version
+      assert_equal '', public_id.to_s
+    end
+
+    def test_empty_string_public_id_does_not_raise
+      # Issue #94: nil は許容されるのに空文字は RuntimeError になっていた
+      Resizing::PublicId.new ''
+    end
+
+    def test_empty_string_public_id_behaves_same_as_nil
+      from_empty_string = Resizing::PublicId.new ''
+      from_nil = Resizing::PublicId.new nil
+
+      parts = ->(public_id) { [public_id.empty?, public_id.image_id, public_id.project_id, public_id.version, public_id.to_s] }
+
+      assert_equal parts.call(from_nil), parts.call(from_empty_string)
+    end
+
+    def test_whitespace_only_public_id_is_treated_as_empty
+      public_id = Resizing::PublicId.new "  \t\n "
+
+      assert public_id.empty?
+      assert_nil public_id.image_id
+      assert_nil public_id.project_id
+      assert_nil public_id.version
+    end
+
+    def test_invalid_public_id_raises_error
+      assert_raises(RuntimeError) { Resizing::PublicId.new 'not/a/public/id' }
+      assert_raises(RuntimeError) { Resizing::PublicId.new "/projects/#{@project_id}/upload/videos/#{@image_id}" }
+    end
+
+    def test_project_id_must_consist_of_hex_digits_and_hyphens
+      # public_id の project_id 部分は UUID 形式 ([0-9a-f-]+) のみ受け付ける
+      assert_raises(RuntimeError) { Resizing::PublicId.new "/projects/project123/upload/images/#{@image_id}" }
+    end
+
+    def test_image_id_accepts_non_uuid_value
+      public_id = Resizing::PublicId.new "/projects/#{@project_id}/upload/images/AWEaewfAreaweFAFASfwe/vabc"
+
+      assert_equal 'AWEaewfAreaweFAFASfwe', public_id.image_id
+      assert_equal 'abc', public_id.version
+    end
+
+    def test_parses_public_id_embedded_in_full_url
+      # match はアンカーなしのため image_host 付きの URL からも各要素を取り出せる
+      public_id = Resizing::PublicId.new "https://img.resizing.net#{@public_id_as_string}"
+
+      assert_equal @project_id, public_id.project_id
+      assert_equal @image_id, public_id.image_id
+      assert_equal @version, public_id.version
+    end
   end
 end

data/test/resizing_module_test.rb

@@ -203,4 +203,114 @@ class ResizingModuleTest < Minitest::Test
     assert_instance_of Hash, result
     assert_equal 'image_id', result['id']
   end
+
+  def test_post_delegates_to_client
+    Resizing.configure = {
+      image_host: 'https://img.example.com',
+      project_id: 'project123',
+      secret_token: 'token123',
+      enable_mock: true
+    }
+
+    result = Resizing.post('test/data/images/sample1.jpg', content_type: 'image/jpeg')
+
+    assert_instance_of Hash, result
+    assert_equal '87263920-2081-498e-a107-9625f4fde01b', result['id']
+  end
+
+  def test_client_raises_configuration_error_when_not_configured
+    assert_raises(Resizing::ConfigurationError) { Resizing.client }
+  end
+
+  def test_api_methods_raise_configuration_error_when_not_configured
+    assert_raises(Resizing::ConfigurationError) { Resizing.post('dummy', content_type: 'image/jpeg') }
+    assert_raises(Resizing::ConfigurationError) { Resizing.put('id', 'dummy', content_type: 'image/jpeg') }
+    assert_raises(Resizing::ConfigurationError) { Resizing.delete('id') }
+    assert_raises(Resizing::ConfigurationError) { Resizing.metadata('id', {}) }
+    assert_raises(Resizing::ConfigurationError) { Resizing.url_from_image_id('id') }
+    assert_raises(Resizing::ConfigurationError) { Resizing.generate_identifier }
+  end
+
+  def test_configure_setter_raises_for_invalid_hash
+    assert_raises(Resizing::ConfigurationError) { Resizing.configure = { project_id: 'only-project' } }
+    assert_raises(Resizing::ConfigurationError) { Resizing.configure = { secret_token: 'only-token' } }
+    assert_raises(Resizing::ConfigurationError) do
+      Resizing.configure = { project_id: 'p', secret_token: 's', host: 'deprecated-host' }
+    end
+  end
+
+  def test_configure_setter_replaces_previous_configuration
+    Resizing.configure = { project_id: 'first', secret_token: 'token' }
+    Resizing.configure = { project_id: 'second', secret_token: 'token' }
+
+    assert_equal 'second', Resizing.configure.project_id
+  end
+
+  def test_client_reflects_latest_configuration
+    Resizing.configure = { project_id: 'p', secret_token: 's', enable_mock: false }
+    assert_instance_of Resizing::Client, Resizing.client
+
+    Resizing.configure = { project_id: 'p', secret_token: 's', enable_mock: true }
+    assert_instance_of Resizing::MockClient, Resizing.client
+  end
+
+  def test_separate_public_id_extracts_project_id_image_id_and_version
+    matched = Resizing.separate_public_id(
+      '/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/28c49144-c00d-4cb5-8619-98ce95977b9c/v1Id850q34fgsaer23w'
+    )
+
+    assert_equal '098a2a0d-c387-4135-a071-1254d6d7e70a', matched[:project_id]
+    assert_equal '28c49144-c00d-4cb5-8619-98ce95977b9c', matched[:image_id]
+    assert_equal '1Id850q34fgsaer23w', matched[:version]
+  end
+
+  def test_separate_public_id_without_version
+    matched = Resizing.separate_public_id(
+      '/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/28c49144-c00d-4cb5-8619-98ce95977b9c'
+    )
+
+    assert_equal '28c49144-c00d-4cb5-8619-98ce95977b9c', matched[:image_id]
+    assert_nil matched[:version]
+  end
+
+  def test_separate_public_id_returns_nil_for_unrelated_string
+    assert_nil Resizing.separate_public_id('/foo/bar')
+    assert_nil Resizing.separate_public_id('')
+  end
+
+  def test_generate_identifier_returns_new_value_each_time
+    Resizing.configure = { project_id: '098a2a0d-c387-4135-a071-1254d6d7e70a', secret_token: 'token' }
+
+    first = Resizing.generate_identifier
+    second = Resizing.generate_identifier
+
+    refute_equal first, second
+    # 生成した identifier は PublicId として解釈できる
+    assert_equal '098a2a0d-c387-4135-a071-1254d6d7e70a', Resizing::PublicId.new(first).project_id
+  end
+
+  def test_url_from_image_id_uses_default_image_host_when_not_specified
+    Resizing.configure = { project_id: 'project123', secret_token: 'token123' }
+
+    assert_equal 'https://img.resizing.net/projects/project123/upload/images/image456', Resizing.url_from_image_id('image456')
+  end
+
+  # Issue #95: 実体のないファイルを指す autoload が残っていると
+  # その定数を参照した瞬間に LoadError になるため、
+  # 全 autoload 先が実際にロードできることを確認する
+  def test_all_autoload_targets_are_loadable
+    source = File.read(File.expand_path('../lib/resizing.rb', __dir__))
+    const_names = source.scan(/autoload\s+:(\w+),/).flatten
+
+    refute_empty const_names
+
+    const_names.each do |const_name|
+      assert Resizing.const_get(const_name), "Resizing::#{const_name} should be loadable"
+    end
+  end
+
+  def test_video_constant_is_not_registered
+    refute_includes Resizing.constants, :Video
+    assert_raises(NameError) { Resizing.const_get(:Video) }
+  end
 end

data/test/resizing_test.rb

@@ -6,4 +6,12 @@ class ResizingTest < Minitest::Test
   def test_that_it_has_a_version_number
     refute_nil ::Resizing::VERSION
   end
+
+  def test_version_follows_semantic_versioning
+    assert_match(/\A\d+\.\d+\.\d+(\.pre)?\z/, ::Resizing::VERSION)
+  end
+
+  def test_version_is_frozen_string
+    assert ::Resizing::VERSION.frozen?
+  end
 end