resizing 0.4.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04cca28d70c6c4c7a361ae5714aadb8e0983a77827b7099ca3dcb18d60e41052
-  data.tar.gz: 653e75727cbeb23d3b9f799b9547df92010410b7f409d671bfaea529325060db
+  metadata.gz: 447b0dda55d606eec4ec8133edd826cd206719ebe44c839c6a2628092ff8eaca
+  data.tar.gz: 8bb995ba29217cd573fc1eeaec86aad1a0f999513aa89cb598eec8ea168a5ef0
 SHA512:
-  metadata.gz: 9f6395295e817926af6f50fdf66817c97beac4b9e4efbd9ffede1cfb597567ca43a56615a777ae333a96e841aac62f20b4804918f542dc8cf42a57c59c8e8433
-  data.tar.gz: b7368beeae3086929f303100059122f9310e18b9bbe6c2503d5d8cb11994272d5e95e480870a4a6a513162db5a87a247ccc785d1dec21dac731bd462ce7881c2
+  metadata.gz: 9d899143122b31fc588b6c721f1fd889edf203a43de5b09a2e0164dd3a075d92b3fb3936f7d0e0c154b6c994dddc748e190c2e7ab8327690aed1b31a96bc886c
+  data.tar.gz: a621e2b3c388c3d9b6cea438589f2fe4474465ca6fe78aa4e97977c10857da4ef48c89883690c3db4b782b8b58871fa1fa37e8ff69bc8ce67f05b84cb95d80ec

data/CHANGELOG.md

@@ -0,0 +1,149 @@
+# Changelog
+
+## [v0.5.3](https://github.com/jksy/resizing-gem/tree/v0.5.3) (2020-10-15)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.5.2...v0.5.3)
+
+**Merged pull requests:**
+
+- Feature/fix raise error [\#40](https://github.com/jksy/resizing-gem/pull/40) ([jksy](https://github.com/jksy))
+
+## [v0.5.2](https://github.com/jksy/resizing-gem/tree/v0.5.2) (2020-10-11)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.5.1...v0.5.2)
+
+## [v0.5.1](https://github.com/jksy/resizing-gem/tree/v0.5.1) (2020-10-11)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.5.0...v0.5.1)
+
+## [v0.5.0](https://github.com/jksy/resizing-gem/tree/v0.5.0) (2020-10-11)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.4.2...v0.5.0)
+
+## [v0.4.2](https://github.com/jksy/resizing-gem/tree/v0.4.2) (2020-10-01)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.4.1...v0.4.2)
+
+**Merged pull requests:**
+
+- fix raise error if tempfile is assinged [\#38](https://github.com/jksy/resizing-gem/pull/38) ([jksy](https://github.com/jksy))
+
+## [v0.4.1](https://github.com/jksy/resizing-gem/tree/v0.4.1) (2020-10-01)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.2.1...v0.4.1)
+
+**Closed issues:**
+
+- pass original filename into Resizing [\#36](https://github.com/jksy/resizing-gem/issues/36)
+- columnがnilの時はdefault\_urlを返す [\#33](https://github.com/jksy/resizing-gem/issues/33)
+
+**Merged pull requests:**
+
+- save original filename [\#37](https://github.com/jksy/resizing-gem/pull/37) ([jksy](https://github.com/jksy))
+- columnがnilの時はdefault\_urlを返す \#33 [\#34](https://github.com/jksy/resizing-gem/pull/34) ([jksy](https://github.com/jksy))
+
+## [v0.2.1](https://github.com/jksy/resizing-gem/tree/v0.2.1) (2020-09-29)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.4.0...v0.2.1)
+
+## [v0.4.0](https://github.com/jksy/resizing-gem/tree/v0.4.0) (2020-09-28)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.3.2...v0.4.0)
+
+**Merged pull requests:**
+
+- change version string when put request with mockclient [\#32](https://github.com/jksy/resizing-gem/pull/32) ([jksy](https://github.com/jksy))
+- add delete method [\#31](https://github.com/jksy/resizing-gem/pull/31) ([jksy](https://github.com/jksy))
+- Feature/add identifier to public [\#30](https://github.com/jksy/resizing-gem/pull/30) ([jksy](https://github.com/jksy))
+- add identifier [\#29](https://github.com/jksy/resizing-gem/pull/29) ([jksy](https://github.com/jksy))
+- Feature/refactor public [\#28](https://github.com/jksy/resizing-gem/pull/28) ([jksy](https://github.com/jksy))
+- divide to PublicId class [\#27](https://github.com/jksy/resizing-gem/pull/27) ([jksy](https://github.com/jksy))
+- fix some bug & errors [\#26](https://github.com/jksy/resizing-gem/pull/26) ([jksy](https://github.com/jksy))
+- fix mockclient [\#25](https://github.com/jksy/resizing-gem/pull/25) ([jksy](https://github.com/jksy))
+- fix raise no method presented, no cotent type is presented [\#24](https://github.com/jksy/resizing-gem/pull/24) ([jksy](https://github.com/jksy))
+
+## [v0.3.2](https://github.com/jksy/resizing-gem/tree/v0.3.2) (2020-09-11)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.3.1...v0.3.2)
+
+**Merged pull requests:**
+
+- fix raise error when call remove! with nil column [\#23](https://github.com/jksy/resizing-gem/pull/23) ([jksy](https://github.com/jksy))
+
+## [v0.3.1](https://github.com/jksy/resizing-gem/tree/v0.3.1) (2020-09-11)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.3.0...v0.3.1)
+
+## [v0.3.0](https://github.com/jksy/resizing-gem/tree/v0.3.0) (2020-08-21)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.2.0...v0.3.0)
+
+**Merged pull requests:**
+
+- Feature/fix mock client [\#22](https://github.com/jksy/resizing-gem/pull/22) ([jksy](https://github.com/jksy))
+
+## [v0.2.0](https://github.com/jksy/resizing-gem/tree/v0.2.0) (2020-08-12)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.1.4...v0.2.0)
+
+**Merged pull requests:**
+
+- version 0.2.0 [\#21](https://github.com/jksy/resizing-gem/pull/21) ([jksy](https://github.com/jksy))
+- support f\_auto, q\_auto parameter [\#20](https://github.com/jksy/resizing-gem/pull/20) ([jksy](https://github.com/jksy))
+
+## [v0.1.4](https://github.com/jksy/resizing-gem/tree/v0.1.4) (2020-08-08)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.1.3...v0.1.4)
+
+**Merged pull requests:**
+
+- Feature/update gems and change default hostname [\#19](https://github.com/jksy/resizing-gem/pull/19) ([jksy](https://github.com/jksy))
+
+## [v0.1.3](https://github.com/jksy/resizing-gem/tree/v0.1.3) (2020-06-12)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.1.2...v0.1.3)
+
+**Merged pull requests:**
+
+- fix error when record is not found [\#18](https://github.com/jksy/resizing-gem/pull/18) ([jksy](https://github.com/jksy))
+
+## [v0.1.2](https://github.com/jksy/resizing-gem/tree/v0.1.2) (2020-06-12)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.1.1...v0.1.2)
+
+**Merged pull requests:**
+
+- ignore response if delete method is received 404 status code [\#17](https://github.com/jksy/resizing-gem/pull/17) ([jksy](https://github.com/jksy))
+
+## [v0.1.1](https://github.com/jksy/resizing-gem/tree/v0.1.1) (2020-06-12)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/v0.1.0...v0.1.1)
+
+**Merged pull requests:**
+
+- Feature/fix raise error [\#16](https://github.com/jksy/resizing-gem/pull/16) ([jksy](https://github.com/jksy))
+- change mysql port on docker-compose [\#9](https://github.com/jksy/resizing-gem/pull/9) ([jksy](https://github.com/jksy))
+
+## [v0.1.0](https://github.com/jksy/resizing-gem/tree/v0.1.0) (2020-06-07)
+
+[Full Changelog](https://github.com/jksy/resizing-gem/compare/e817d9b883425ddae3e42439ccd1a9d2e270c363...v0.1.0)
+
+**Merged pull requests:**
+
+- add mock client for test environments [\#14](https://github.com/jksy/resizing-gem/pull/14) ([jksy](https://github.com/jksy))
+- Feature/add test test for remove xxx [\#13](https://github.com/jksy/resizing-gem/pull/13) ([jksy](https://github.com/jksy))
+- add rubocop & fix some tests [\#12](https://github.com/jksy/resizing-gem/pull/12) ([jksy](https://github.com/jksy))
+- fix the column cant be update by null value [\#11](https://github.com/jksy/resizing-gem/pull/11) ([jksy](https://github.com/jksy))
+- fix empty url when model is reload [\#10](https://github.com/jksy/resizing-gem/pull/10) ([jksy](https://github.com/jksy))
+- fix return invalid url when model is reloaded [\#8](https://github.com/jksy/resizing-gem/pull/8) ([jksy](https://github.com/jksy))
+- add circleci [\#7](https://github.com/jksy/resizing-gem/pull/7) ([jksy](https://github.com/jksy))
+- divide some classes and add tests [\#6](https://github.com/jksy/resizing-gem/pull/6) ([jksy](https://github.com/jksy))
+- fix connect host [\#5](https://github.com/jksy/resizing-gem/pull/5) ([jksy](https://github.com/jksy))
+- Feature/carrier wave [\#4](https://github.com/jksy/resizing-gem/pull/4) ([jksy](https://github.com/jksy))
+- faraday version is down. and some fix bugs [\#3](https://github.com/jksy/resizing-gem/pull/3) ([jksy](https://github.com/jksy))
+- Feature/add default host [\#2](https://github.com/jksy/resizing-gem/pull/2) ([jksy](https://github.com/jksy))
+- first implementation [\#1](https://github.com/jksy/resizing-gem/pull/1) ([jksy](https://github.com/jksy))
+
+
+
+\* *This Changelog was automatically generated by [github_changelog_generator](https://github.com/github-changelog-generator/github-changelog-generator)*

data/bin/generate-changelog

@@ -0,0 +1,3 @@
+#!/usr/bin/bash
+
+github_changelog_generator -u jksy -p resizing-gem

data/lib/resizing.rb

@@ -13,7 +13,16 @@ module Resizing
 
   class Error < StandardError; end
   class ConfigurationError < Error; end
-  class APIError < Error; end
+  class APIError < Error;
+    def decoded_body
+      @decoded_body ||= {}
+    end
+
+    def decoded_body=(value)
+      raise ArgumentError, 'The decoded_body is expected to be passed a Hash.' unless value.is_a? Hash
+      @decoded_body = value
+    end
+  end
 
   def self.configure
     raise ConfigurationError, 'Resizing.configure is not initialized' unless defined? @configure

data/lib/resizing/carrier_wave/storage/file.rb

@@ -8,6 +8,7 @@ module Resizing
 
         def initialize(uploader, identifier = nil)
           @uploader = uploader
+          @file = nil
           @content_type = nil
           @public_id = Resizing::PublicId.new identifier
         end
@@ -76,6 +77,11 @@ module Resizing
           !!file
         end
 
+        def current_path
+          @current_path = model.send :read_attribute, serialization_column
+        end
+        alias path current_path
+
         def store(new_file)
           if new_file.is_a?(self.class)
             # new_file.copy_to(path)
@@ -88,15 +94,23 @@ module Resizing
             # guess content-type from extension
             @content_type ||= MIME::Types.type_for(new_file.path).first.content_type
           end
-          @public_id = PublicId.new(model.send :read_attribute, serialization_column)
 
-          image_id = if @public_id.empty?
-                       Resizing.configure.generate_image_id
-                     else
-                       @public_id.image_id
-                     end
+          original_filename = new_file.try(:original_filename) || new_file.try(:filename) || new_file.try(:path)
+          if original_filename.present?
+            original_filename = ::File.basename(original_filename)
+          end
 
-          @response = Resizing.put(image_id, new_file.read, { content_type: @content_type, filename: new_file.original_filename })
+          content = if new_file.respond_to?(:to_io)
+                      new_file.to_io.tap(&:rewind)
+                    elsif new_file.respond_to?(:read) && new_file.respond_to?(:rewind)
+                      new_file.read.tap do
+                        new_file.rewind
+                      end
+                    else
+                      new_file
+                    end
+
+          @response = Resizing.post(content, { content_type: @content_type, filename: original_filename })
           @public_id = Resizing::PublicId.new(@response['public_id'])
 
           # force update column
@@ -176,12 +190,6 @@ module Resizing
           parameters.count == 2 && parameters[1].include?(:options)
         end
 
-        # def store! sanitized_file
-        #   puts sanitized_file.inspect
-        #   client = Resizing::Client.new
-        #   @store_response = client.post(sanitized_file.to_file, {content_type: sanitized_file.content_type})
-        # end
-
         def retrieve!(identifier)
           raise NotImplementedError, "retrieve! #{identifier}"
         end

data/lib/resizing/client.rb

@@ -63,6 +63,8 @@ module Resizing
 
       result = handle_create_response(response)
       result
+    rescue Faraday::TimeoutError => e
+      handle_timeout_error e
     end
 
     def put(image_id, file_or_binary, options)
@@ -83,6 +85,8 @@ module Resizing
 
       result = handle_create_response(response)
       result
+    rescue Faraday::TimeoutError => e
+      handle_timeout_error e
     end
 
     def delete(image_id)
@@ -94,6 +98,8 @@ module Resizing
 
       result = handle_delete_response(response)
       result
+    rescue Faraday::TimeoutError => e
+      handle_timeout_error e
     end
 
     def metadata(image_id, options = {})
@@ -105,6 +111,8 @@ module Resizing
 
       result = handle_metadata_response(response)
       result
+    rescue Faraday::TimeoutError => e
+      handle_timeout_error e
     end
 
     private
@@ -145,13 +153,13 @@ module Resizing
     end
 
     def to_io(data)
+      return data.to_io if data.respond_to? :to_io
+
       case data
-      when IO
-        data
       when String
         StringIO.new(data)
       else
-        raise ArgumentError, 'file_or_binary is required IO class or String'
+        raise ArgumentError, "file_or_binary is required IO class or String:#{data.class}"
       end
     end
 
@@ -166,7 +174,10 @@ module Resizing
       when HTTP_STATUS_OK, HTTP_STATUS_CREATED
         JSON.parse(response.body)
       else
-        raise APIError, "invalid http status code #{response.status}"
+        result = JSON.parse(response.body) rescue {}
+        err = APIError.new("invalid http status code #{response.status}")
+        err.decoded_body = result
+        raise err
       end
     end
 
@@ -177,7 +188,10 @@ module Resizing
       when HTTP_STATUS_OK, HTTP_STATUS_NOT_FOUND
         JSON.parse(response.body)
       else
-        raise APIError, "invalid http status code #{response.status}"
+        result = JSON.parse(response.body) rescue {}
+        err = APIError.new("invalid http status code #{response.status}")
+        err.decoded_body = result
+        raise err
       end
     end
 
@@ -188,8 +202,16 @@ module Resizing
       when HTTP_STATUS_OK, HTTP_STATUS_NOT_FOUND
         JSON.parse(response.body)
       else
-        raise APIError, "invalid http status code #{response.status}"
+        result = JSON.parse(response.body) rescue {}
+        err = APIError.new("invalid http status code #{response.status}")
+        err.decoded_body = result
+        raise err
       end
     end
+
+    def handle_timeout_error error
+      # error: Faraday::TimeoutError
+      raise APIError.new("TimeoutError: #{error.inspect}")
+    end
   end
 end

data/lib/resizing/mock_client.rb

@@ -42,7 +42,7 @@ module Resizing
     end
 
     def library_root
-      File.join(File.dirname(__FILE__), '..', '..')
+      @library_root ||= File.expand_path('../../../', __FILE__)
     end
   end
 end

data/lib/resizing/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Resizing
-  VERSION = '0.4.1'
+  VERSION = '0.7.0'
 end

data/test/resizing/carrier_wave_test.rb

@@ -10,8 +10,8 @@ module Resizing
 
       @configuration_template = {
         host: 'http://192.168.56.101:5000',
-        project_id: '098a2a0d-c387-4135-a071-1254d6d7e70a',
-        secret_token: '4g1cshg2lq8j93ufhvqrpjswxmtjz12yhfvq6w79jpwi7cr7nnknoqgwzkwerbs6',
+        project_id: 'e06e710d-f026-4dcf-b2c0-eab0de8bb83f',
+        secret_token: 'ewbym2r1pk49x1d2lxdbiiavnqp25j2kh00hsg3koy0ppm620x5mhlmgl3rq5ci8',
         open_timeout: 10,
         response_timeout: 20
       }
@@ -70,7 +70,7 @@ module Resizing
       assert_equal('jpg', model.resizing_picture.format)
     end
 
-    def test_url_is_return_default_url
+    def test_url_is_return_default_url_as_nil
       model = TestModel.new
       model.save!
       assert_nil model.resizing_picture_url
@@ -82,13 +82,19 @@ module Resizing
       assert_equal('http://example.com/test.jpg', model.resizing_picture_url)
     end
 
+    def test_is_successful
+      model = prepare_model_with_tempfile TestModel
+      model.save!
+      # assert_equal('http://192.168.56.101:5000/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/28c49144-c00d-4cb5-8619-98ce95977b9c/v1Id850__tqNsnoGWWUibtIBZ5NgjV45M/', model.resizing_picture_url)
+    end
+
     def expect_url
-      'http://192.168.56.101:5000/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/'+
-        'upload/images/28c49144-c00d-4cb5-8619-98ce95977b9c/v1Id850__tqNsnoGWWUibtIBZ5NgjV45M'
+      'http://192.168.56.101:5000/projects/e06e710d-f026-4dcf-b2c0-eab0de8bb83f/'+
+        'upload/images/14ea7aac-a194-4330-931f-6b562aec413d/v_8c5lEhDB5RT3PZp1Fn5PYGm9YVx_x0e'
     end
 
     def prepare_model model
-      VCR.use_cassette 'carrier_wave_test/save' do
+      VCR.use_cassette 'carrier_wave_test/save', record: :once do
         SecureRandom.stub :uuid, '28c49144-c00d-4cb5-8619-98ce95977b9c' do
           model = model.new
           file = File.open('test/data/images/sample1.jpg', 'r')
@@ -103,5 +109,16 @@ module Resizing
         end
       end
     end
+
+    def prepare_model_with_tempfile model
+      VCR.use_cassette 'carrier_wave_test/save', record: :once do
+        SecureRandom.stub :uuid, '28c49144-c00d-4cb5-8619-98ce95977b9c' do
+          model = model.new
+          file = File.open('test/data/images/sample1.jpg', 'r')
+          model.resizing_picture = file
+          return model
+        end
+      end
+    end
   end
 end

data/test/resizing/client_test.rb

@@ -8,8 +8,8 @@ module Resizing
       # NOP
       @configuration_template = {
         host: 'http://192.168.56.101:5000',
-        project_id: '098a2a0d-c387-4135-a071-1254d6d7e70a',
-        secret_token: '4g1cshg2lq8j93ufhvqrpjswxmtjz12yhfvq6w79jpwi7cr7nnknoqgwzkwerbs6',
+        project_id: 'e06e710d-f026-4dcf-b2c0-eab0de8bb83f',
+        secret_token: 'ewbym2r1pk49x1d2lxdbiiavnqp25j2kh00hsg3koy0ppm620x5mhlmgl3rq5ci8',
         open_timeout: 10,
         response_timeout: 20
       }
@@ -41,14 +41,14 @@ module Resizing
       VCR.use_cassette 'client/post', record: :once  do
         f = File.open('test/data/images/sample1.jpg', 'r')
         r = client.post(f, content_type: 'image/jpeg')
-        assert_equal(r['id'], 'bfdaf2b3-7ec5-41f4-9caa-d53247dd9666')
+        assert_equal(r['id'], '87263920-2081-498e-a107-9625f4fde01b')
         assert_equal(r['project_id'], Resizing.configure.project_id)
         assert_equal(r['content_type'], 'image/jpeg')
         assert(!r['latest_version_id'].nil?)
         assert(!r['latest_etag'].nil?)
         assert(!r['created_at'].nil?)
         assert(!r['updated_at'].nil?)
-        assert_equal(r['public_id'], '/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/bfdaf2b3-7ec5-41f4-9caa-d53247dd9666/vAyWaxx96gLaAzB9Bq.VbX1_pxfXJ0Jcq')
+        assert_equal(r['public_id'], '/projects/e06e710d-f026-4dcf-b2c0-eab0de8bb83f/upload/images/87263920-2081-498e-a107-9625f4fde01b/vHg9VFvdI6HRzLFbV495VdwVmHIspLRCo')
         assert_equal(r['filename'], 'sample1.jpg')
       end
     end
@@ -70,11 +70,41 @@ module Resizing
         assert(!r['updated_at'].nil?)
         assert_equal(
           r['public_id'],
-          "/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/#{name}/v6Ew3HmDAYfb3NMRdLxR45i_gXMbLlGyi"
+          "/projects/e06e710d-f026-4dcf-b2c0-eab0de8bb83f/upload/images/#{name}/vfztekhN_WoeXo8ZkCZ4i5jcQvmPpZewR"
         )
       end
     end
 
+    def test_raise_error
+      Resizing.configure = @configuration_template
+
+      client = Resizing::Client.new
+      VCR.use_cassette 'client/error', record: :once do
+        f = File.open('test/data/images/empty_file.jpg', 'r')
+        assert_raises Resizing::APIError do
+          client.post(f, content_type: 'image/jpeg')
+        end
+      end
+    end
+
+    def test_handleable_response_body_from_resizing
+      Resizing.configure = @configuration_template
+
+      client = Resizing::Client.new
+      VCR.use_cassette 'client/error', record: :once do
+        f = File.open('test/data/images/empty_file.jpg', 'r')
+
+        response = nil
+
+        begin
+          client.post(f, content_type: 'image/jpeg')
+        rescue Resizing::APIError => e
+          response = e.decoded_body
+        end
+        assert_equal response, {"error"=>"Magick::ImageMagickError", "message"=>"invalid image format found"}
+      end
+    end
+
     def test_get_the_metadata
       # TODO
 
@@ -82,7 +112,7 @@ module Resizing
 
       client = Resizing::Client.new
       VCR.use_cassette 'client/metadata', record: :once do
-        name = 'bfdaf2b3-7ec5-41f4-9caa-d53247dd9666'
+        name = '87263920-2081-498e-a107-9625f4fde01b'
         r = client.metadata(name)
         assert_equal(r['id'], name)
         assert_equal(r['project_id'], Resizing.configure.project_id)
@@ -95,7 +125,7 @@ module Resizing
         assert(!r['width'].nil?)
         assert_equal(
           r['public_id'],
-          "/projects/098a2a0d-c387-4135-a071-1254d6d7e70a/upload/images/#{name}/v6Ew3HmDAYfb3NMRdLxR45i_gXMbLlGyi"
+          "/projects/e06e710d-f026-4dcf-b2c0-eab0de8bb83f/upload/images/87263920-2081-498e-a107-9625f4fde01b/vHg9VFvdI6HRzLFbV495VdwVmHIspLRCo"
         )
       end
     end