cloudinary 1.8.1 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 07e321996ca2cae3a8eec12d27de1aed453853e7
-  data.tar.gz: 579c2b752735619c42ae516c9cbec268136ddbbe
+  metadata.gz: 8dee4c14d8ddf334a1ff60a96cc7bcb85ef82b25
+  data.tar.gz: 644319b8bdf30ba48b3ab57facc45364aedb3a3d
 SHA512:
-  metadata.gz: 4f0bdc46e84bf87fb80a1ea3a3934088b2079d910f4dfcdbe192f1318c760d386b639d66d902d5ccdfedba3a90827483c96c0c7ff6cc2ad50fd594e389b2b8e8
-  data.tar.gz: 1b80560e163319804899b0aa6fb378fcdedc260ae087977938a3de172d88b0ffff762dfa0b8b9688a9cbfde2facfe956ce1b67e6eada52d874ababd16ceb5176
+  metadata.gz: 3af0f955aa2c7a8eeb9f66c2fc92fdc04be4cfeb4a0bc9f6fc203d07ba096e2eaa850d8ca750f402c798b47326c180623e42ddc5d59b258e284ba25a7edef4ff
+  data.tar.gz: f583837bfc36ac1e0db6dd5b5335600fbac2fa7c20cb9a28c0b0578c989e094fb3bacebc22509e9f30944bc0065fcf478e649f4ffbb3f34011b246a3efd1e844

data/.travis.yml

@@ -1,3 +1,8 @@
+language: ruby
+
+before_install:
+  - gem install bundler
+
 rvm:
 - 1.9.3
 - 2.2.3

data/CHANGELOG.md

@@ -1,4 +1,16 @@
 
+1.8.2 / 2017-11-22
+==================
+
+  * Fix URL signature
+  * Use the correct method for updating a column
+  * Add support for `named` parameter in list transformation API
+  * load environment when running sync_static task
+  * Fix the overwritten initializer for hash (#273)
+  * Force TravisCI to install bundler
+  * Fix CloudinaryFile::exists? method. Solves #193 #205
+  * Update Readme to point to HTTPS URLs of cloudinary.com
+
 1.8.1 / 2017-05-16
 ==================
 

data/README.md

@@ -13,7 +13,7 @@ Cloudinary provides URL and HTTP based APIs that can be easily integrated with a
 For Ruby on Rails, Cloudinary provides a GEM for simplifying the integration even further.
 
 ## Getting started guide
-![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png)  **Take a look at our [Getting started guide of Ruby on Rails](http://cloudinary.com/documentation/rails_integration#getting_started_guide)**.
+![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png)  **Take a look at our [Getting started guide of Ruby on Rails](https://cloudinary.com/documentation/rails_integration#getting_started_guide)**.
 
 ## Setup ######################################################################
 
@@ -68,7 +68,7 @@ Converting to a 150x100 PNG with rounded corners of 20 pixels:
 
 ![Sample 150x150 Rounded PNG](https://res.cloudinary.com/demo/image/upload/w_150,h_100,c_fill,r_20/sample.png "Sample 150x150 Rounded PNG")
 
-For plenty more transformation options, see our [image transformations documentation](http://cloudinary.com/documentation/image_transformations).
+For plenty more transformation options, see our [image transformations documentation](https://cloudinary.com/documentation/image_transformations).
 
 Generating a 120x90 thumbnail based on automatic face detection of the Facebook profile picture of Bill Clinton:
  
@@ -76,7 +76,7 @@ Generating a 120x90 thumbnail based on automatic face detection of the Facebook
     
 ![Facebook 90x120](https://res.cloudinary.com/demo/image/facebook/c_thumb,g_face,h_90,w_120/billclinton.jpg "Facebook 90x200")
 
-For more details, see our documentation for embedding [Facebook](http://cloudinary.com/documentation/facebook_profile_pictures) and [Twitter](http://cloudinary.com/documentation/twitter_profile_pictures) profile pictures. 
+For more details, see our documentation for embedding [Facebook](https://cloudinary.com/documentation/facebook_profile_pictures) and [Twitter](https://cloudinary.com/documentation/twitter_profile_pictures) profile pictures. 
 
 
 ## Usage
@@ -84,7 +84,7 @@ For more details, see our documentation for embedding [Facebook](http://cloudina
 ### Configuration
 
 Each request for building a URL of a remote cloud resource must have the `cloud_name` parameter set. 
-Each request to our secure APIs (e.g., image uploads, eager sprite generation) must have the `api_key` and `api_secret` parameters set. See [API, URLs and access identifiers](http://cloudinary.com/documentation/api_and_access_identifiers) for more details.
+Each request to our secure APIs (e.g., image uploads, eager sprite generation) must have the `api_key` and `api_secret` parameters set. See [API, URLs and access identifiers](https://cloudinary.com/documentation/api_and_access_identifiers) for more details.
 
 Setting the `cloud_name`, `api_key` and `api_secret` parameters can be done either directly in each call to a Cloudinary method or by globally setting using a YAML configuration file.
 
@@ -130,7 +130,7 @@ Same goes for Twitter:
 
     twitter_name_profile_image_tag("billclinton.jpg")
 
-![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png) **See [our documentation](http://cloudinary.com/documentation/rails_image_manipulation) for more information about displaying and transforming images in Rails**.                                         
+![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png) **See [our documentation](https://cloudinary.com/documentation/rails_image_manipulation) for more information about displaying and transforming images in Rails**.                                         
 
 
 
@@ -157,7 +157,7 @@ You can also specify your own public ID:
     http://res.cloudinary.com/demo/image/upload/sample_remote.jpg
 
 
-![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png) **See [our documentation](http://cloudinary.com/documentation/rails_image_upload) for plenty more options of uploading to the cloud from your Ruby code or directly from the browser**.
+![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png) **See [our documentation](https://cloudinary.com/documentation/rails_image_upload) for plenty more options of uploading to the cloud from your Ruby code or directly from the browser**.
 
 
 ### CarrierWave Integration
@@ -172,9 +172,9 @@ Cloudinary's Ruby GEM includes an optional plugin for [CarrierWave](https://gith
         ...  
     end
 
-![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png) **For more details on CarrierWave integration see [our documentation](http://cloudinary.com/documentation/rails_carrierwave)**.
+![More](http://res.cloudinary.com/cloudinary/image/upload/see_more_bullet.png) **For more details on CarrierWave integration see [our documentation](https://cloudinary.com/documentation/rails_carrierwave)**.
 
-We also published an interesting blog post about [Ruby on Rails image uploads with CarrierWave and Cloudinary](http://cloudinary.com/blog/ruby_on_rails_image_uploads_with_carrierwave_and_cloudinary).
+We also published an interesting blog post about [Ruby on Rails image uploads with CarrierWave and Cloudinary](https://cloudinary.com/blog/ruby_on_rails_image_uploads_with_carrierwave_and_cloudinary).
 
 #### Neo4j integration
 
@@ -205,7 +205,7 @@ To automatically set-up Cloudinary's configuration, include the following line i
 
 #### Uploading images from the browser
 
-The Cloudinary JavaScript library utilizes the Blueimp File Upload library to support image uploading from the browser. See the [documentation](http://cloudinary.com/documentation/jquery_image_upload) for more details.
+The Cloudinary JavaScript library utilizes the Blueimp File Upload library to support image uploading from the browser. See the [documentation](https://cloudinary.com/documentation/jquery_image_upload) for more details.
 
 |Important|
 |---------|
@@ -233,21 +233,22 @@ Please consult with the [README file](https://github.com/cloudinary/cloudinary_g
 
 Additional resources are available at:
 
-* [Website](http://cloudinary.com)
-* [Documentation](http://cloudinary.com/documentation)
-* [Knowledge Base](http://support.cloudinary.com/forums)
-* [Documentation for Ruby on Rails integration](http://cloudinary.com/documentation/rails_integration)
-* [Ruby on Rails image upload documentation](http://cloudinary.com/documentation/rails_image_upload)
-* [Ruby on Rails image manipulation documentation](http://cloudinary.com/documentation/rails_image_manipulation)
-* [Image transformations documentation](http://cloudinary.com/documentation/image_transformations)
+* [Website](https://cloudinary.com)
+* [Interactive demo](https://demo.cloudinary.com/default)
+* [Documentation](https://cloudinary.com/documentation)
+* [Knowledge Base](https://support.cloudinary.com/hc/en-us)
+* [Documentation for Ruby on Rails integration](https://cloudinary.com/documentation/rails_integration)
+* [Ruby on Rails image upload documentation](https://cloudinary.com/documentation/rails_image_upload)
+* [Ruby on Rails image manipulation documentation](https://cloudinary.com/documentation/rails_image_manipulation)
+* [Image transformations documentation](https://cloudinary.com/documentation/image_transformations)
 
 ## Support
 
 You can [open an issue through GitHub](https://github.com/cloudinary/cloudinary_gem/issues).
 
-Contact us [http://cloudinary.com/contact](http://cloudinary.com/contact)
+Contact us [https://cloudinary.com/contact](https://cloudinary.com/contact)
 
-Stay tuned for updates, tips and tutorials: [Blog](http://cloudinary.com/blog), [Twitter](https://twitter.com/cloudinary), [Facebook](http://www.facebook.com/Cloudinary).
+Stay tuned for updates, tips and tutorials: [Blog](https://cloudinary.com/blog), [Twitter](https://twitter.com/cloudinary), [Facebook](https://www.facebook.com/Cloudinary).
 
 
 ## License #######################################################################

data/lib/cloudinary/api.rb

@@ -10,14 +10,19 @@ class Cloudinary::Api
   class BadRequest < Error; end
   class GeneralError < Error; end
   class AuthorizationRequired < Error; end
+
   class Response < Hash
     attr_reader :rate_limit_reset_at, :rate_limit_remaining, :rate_limit_allowed
 
-    def initialize(response)
-      self.update(Cloudinary::Api.send(:parse_json_response, response))
-      @rate_limit_allowed   = response.headers[:x_featureratelimit_limit].to_i
-      @rate_limit_reset_at  = Time.parse(response.headers[:x_featureratelimit_reset])
-      @rate_limit_remaining = response.headers[:x_featureratelimit_remaining].to_i
+    def initialize(response=nil)
+      if response
+        # This sets the instantiated self as the response Hash
+        update Cloudinary::Api.parse_json_response response
+
+        @rate_limit_allowed   = response.headers[:x_featureratelimit_limit].to_i
+        @rate_limit_reset_at  = Time.parse(response.headers[:x_featureratelimit_reset])
+        @rate_limit_remaining = response.headers[:x_featureratelimit_remaining].to_i
+      end
     end
   end
 
@@ -160,7 +165,7 @@ class Cloudinary::Api
   end
 
   def self.transformations(options={})
-    call_api(:get, "transformations", only(options, :next_cursor, :max_results), options)
+    call_api(:get, "transformations", only(options, :named, :next_cursor, :max_results), options)
   end
 
   def self.transformation(transformation, options={})

data/lib/cloudinary/carrier_wave.rb

@@ -176,7 +176,8 @@ module Cloudinary::CarrierWave
     end
     
     def exists?
-      Cloudinary::Uploader.exists?(self.identifier, :type=>self.storage_type, :resource_type=>self.resource_type)
+      public_id = @resource_type == "raw" ? self.filename : self.public_id
+      Cloudinary::Uploader.exists?(public_id, :version=>self.version, :type=>self.storage_type, :resource_type=>self.resource_type)
     end
     
     def read(options={})

data/lib/cloudinary/carrier_wave/storage.rb

@@ -89,7 +89,7 @@ class Cloudinary::CarrierWave::Storage < ::CarrierWave::Storage::Abstract
         uploader.model.set(column, name)
       end
     elsif defined?(Neo4j::VERSION) && Neo4j::VERSION.split(".").first.to_i >= 5
-        uploader.model.set(column, name)
+        uploader.model.write_attribute(column, name)
     elsif defined?(Sequel::Model) && uploader.model.is_a?(Sequel::Model)
       # Sequel support
       uploader.model.this.update(column => name)

data/lib/cloudinary/utils.rb

@@ -397,11 +397,7 @@ class Cloudinary::Utils
     transformation = transformation.gsub(%r(([^:])//), '\1/')
     if sign_url && ( !auth_token || auth_token.empty?)
       to_sign = [transformation, sign_version && version, source_to_sign].reject(&:blank?).join("/")
-      i = 0
-      while to_sign != CGI.unescape(to_sign) && i <10
-        to_sign = CGI.unescape(to_sign)
-        i = i + 1
-      end
+      to_sign = fully_unescape(to_sign)
       signature = 's--' + Base64.urlsafe_encode64(Digest::SHA1.digest(to_sign + secret))[0,8] + '--'
     end
 
@@ -848,8 +844,24 @@ class Cloudinary::Utils
     Cloudinary::AuthToken.generate options
 
   end
+  
   private
 
+
+  # Repeatedly unescapes the source until no more unescaping is possible or 10 cycles elapsed
+  # @param [String] source - a (possibly) escaped string
+  # @return [String] the fully unescaped string
+  # @private
+  def self.fully_unescape(source)
+    i = 0
+    while source != CGI.unescape(source) && i <10
+      source = CGI.unescape(source.gsub('+', '%2B')) # don't let unescape replace '+' with space
+      i = i + 1
+    end
+    source
+  end
+  private_class_method :fully_unescape
+  
   def self.hash_query_params(hash)
     if hash.respond_to?("to_query")
       hash.to_query

data/lib/cloudinary/version.rb

@@ -1,4 +1,4 @@
 # Copyright Cloudinary
 module Cloudinary
-  VERSION = "1.8.1"
+  VERSION = "1.8.2"
 end

data/lib/tasks/cloudinary.rake

@@ -1,9 +1,9 @@
 unless Rake::Task.task_defined?('cloudinary:sync_static') # prevent double-loading/execution
   namespace :cloudinary do
     desc "Sync static resources with cloudinary"
-    task :sync_static do
+    task :sync_static => :environment do
       delete_missing = ENV['DELETE_MISSING'] == 'true' || ENV['DELETE_MISSING'] == '1'
       Cloudinary::Static.sync(:delete_missing => delete_missing)
     end
   end
-end
+end

data/spec/api_spec.rb

@@ -78,7 +78,7 @@ describe Cloudinary::Api do
     expect(resources.map{|resource| resource["tags"]}.flatten).to include(TEST_TAG, TIMESTAMP_TAG)
     expect(resources.map{|resource| resource["context"]}).to include({"custom" => {"key" => "value"}})
   end
-  
+
   it "should allow listing resources by context" do
     resources = @api.resources_by_context(test_key)["resources"]
     expect(resources.count).to eq(2)
@@ -94,7 +94,7 @@ describe Cloudinary::Api do
     expect(resources.map{|resource| resource["tags"]}.flatten).to include(TEST_TAG, TIMESTAMP_TAG)
     expect(resources.map{|resource| resource["context"]}).to include({"custom" => {"key" => "value"}})
   end
-  
+
   it "should allow listing resources by start date", :start_at => true do
     start_at = Time.now
     expect(RestClient::Request).to receive(:execute).with(deep_hash_value( {[:payload, :start_at] => start_at, [:payload, :direction] => "asc"}))
@@ -129,7 +129,7 @@ describe Cloudinary::Api do
     expect(resource["bytes"]).to eq(3381)
     expect(resource["derived"].length).to eq(1)
   end
-  
+
   it "should allow deleting derived resource" do
     derived_resource_id = "derived_id"
     expect(RestClient::Request).to receive(:execute).with(deep_hash_value( {[:payload, :derived_resource_ids] => derived_resource_id}))
@@ -199,7 +199,7 @@ describe Cloudinary::Api do
     tags = @api.tags(:prefix=>"api_test_no_such_tag")["tags"]
     expect(tags).to be_blank
   end
-  
+
   describe 'transformations' do
     it "should allow listing transformations" do
       transformation = @api.transformations(max_results: 500)["transformations"].find { |transformation| transformation["name"] == TEST_TRANSFOMATION }
@@ -265,6 +265,11 @@ describe Cloudinary::Api do
         @api.update_transformation(public_id, :unsafe_update => { "crop" => "scale", "width" => 103 })
       end
 
+      it "should allow listing of named transformations" do
+        expect(RestClient::Request).to receive(:execute).with(deep_hash_value( [:payload, :named ]=> true))
+        @api.transformations :named => true
+      end
+      
     end
     it "should allow deleting implicit transformation" do
       @api.transformation(TEST_TRANSFOMATION)
@@ -272,7 +277,7 @@ describe Cloudinary::Api do
       expect { @api.transformation(TEST_TRANSFOMATION) }.to raise_error(Cloudinary::Api::NotFound)
     end
   end
-  
+
   it "should allow creating upload_presets" do
     expected = {:url => /.*\/upload_presets$/,
                 [:payload, :name] => "new_preset",
@@ -305,7 +310,7 @@ describe Cloudinary::Api do
     expect(preset["settings"]["context"]).to eq({"a" => "b", "c" => "d"})
     expect(preset["settings"]["tags"]).to eq(["a","b","c", TEST_TAG, TIMESTAMP_TAG])
   end
-  
+
   it "should allow deleting upload_presets", :upload_preset => true do
     id = "#{prefix}_upload_preset"
     @api.create_upload_preset(:name => id, :folder => "folder", :tags => [TEST_TAG, TIMESTAMP_TAG])
@@ -313,7 +318,7 @@ describe Cloudinary::Api do
     @api.delete_upload_preset(id)
     expect{preset = @api.upload_preset(id)}.to raise_error(Cloudinary::Api::NotFound)
   end
-  
+
   it "should allow updating upload_presets", :upload_preset => true do
     name = @api.create_upload_preset(:folder => "folder", :tags => [TEST_TAG, TIMESTAMP_TAG])["name"]
     preset = @api.upload_preset(name)
@@ -323,7 +328,7 @@ describe Cloudinary::Api do
     expect(preset["unsigned"]).to eq(true)
     expect(preset["settings"]).to eq({"folder" => "folder", "colors" => true, "disallow_public_id" => true, "tags" => [TEST_TAG, TIMESTAMP_TAG]})
   end
-  
+
   # this test must be last because it deletes (potentially) all dependent transformations which some tests rely on. Excluded by default.
   skip "should allow deleting all resources", :delete_all=>true do
     Cloudinary::Uploader.upload(TEST_IMG, :public_id=>"api_test5", :eager=>[:width=>101,:crop=>:scale], :tags => [TEST_TAG, TIMESTAMP_TAG])
@@ -335,7 +340,7 @@ describe Cloudinary::Api do
     expect(resource).not_to be_blank
     expect(resource["derived"].length).to eq(0)
   end
-  
+
   it "should support setting manual moderation status" do
     result = Cloudinary::Uploader.upload(TEST_IMG, {:moderation => :manual, :tags => [TEST_TAG, TIMESTAMP_TAG]})
     expect(result["moderation"][0]["status"]).to eq("pending")
@@ -344,17 +349,17 @@ describe Cloudinary::Api do
     expect(api_result["moderation"][0]["status"]).to eq("approved")
     expect(api_result["moderation"][0]["kind"]).to eq("manual")
   end
-    
+
   it "should support requesting raw conversion" do
     result = Cloudinary::Uploader.upload("spec/docx.docx", :resource_type => :raw, :tags => [TEST_TAG, TIMESTAMP_TAG])
     expect{Cloudinary::Api.update(result["public_id"], {:resource_type => :raw, :raw_convert => :illegal})}.to raise_error(Cloudinary::Api::BadRequest, /^Illegal value|not a valid/)
   end
-  
+
   it "should support requesting categorization" do
     result = Cloudinary::Uploader.upload(TEST_IMG, :tags => [TEST_TAG, TIMESTAMP_TAG])
     expect{Cloudinary::Api.update(result["public_id"], {:categorization => :illegal})}.to raise_error(Cloudinary::Api::BadRequest, /^Illegal value/)
   end
-  
+
   it "should support requesting detection with server notification", :focus => true do
     expected = {
       [:payload, :detection] => "adv_face",
@@ -363,12 +368,12 @@ describe Cloudinary::Api do
     expect(RestClient::Request).to receive(:execute).with(deep_hash_value(expected))
     Cloudinary::Api.update("public_id", {:detection => "adv_face", :notification_url => "http://example.com"})
   end
-  
+
   it "should support requesting auto_tagging" do
     expect(RestClient::Request).to receive(:execute).with(deep_hash_value( [:payload, :auto_tagging] => 0.5))
     Cloudinary::Api.update("public_id", {:auto_tagging => 0.5})
   end
-  
+
   it "should support listing by moderation kind and value" do
     expect(RestClient::Request).to receive(:execute).with(deep_hash_value([:url] => /.*manual\/approved$/, [:payload, :max_results] => 1000))
     Cloudinary::Api.resources_by_moderation(:manual, :approved, :max_results => 1000)
@@ -462,12 +467,12 @@ describe Cloudinary::Api do
 
       expect(result["published"]).to be_an_instance_of(Array)
       expect(result["published"].length).to eq(1)
-      
+
       resource = result["published"][0]
-      
+
       expect(resource["public_id"]).to eq(publicId)
       expect(resource["type"]).to eq('upload')
-      
+
       bytes = resource["bytes"]
     end
     it "should publish resources by prefix and overwrite" do
@@ -475,13 +480,13 @@ describe Cloudinary::Api do
 
       expect(result["published"]).to be_an_instance_of(Array)
       expect(result["published"].length).to eq(1)
-      
+
       resource = result["published"][0]
-      
+
       expect(resource["public_id"]).to eq(publicId)
       expect(resource["bytes"]).not_to eq(bytes)
       expect(resource["type"]).to eq('upload')
-      
+
       bytes = resource["bytes"]
     end
     it "should publish resources by tag and overwrite" do
@@ -489,15 +494,52 @@ describe Cloudinary::Api do
 
       expect(result["published"]).to be_an_instance_of(Array)
       expect(result["published"].length).to eq(1)
-      
+
       resource = result["published"][0]
-      
+
       expect(resource["public_id"]).to eq(publicId)
       expect(resource["bytes"]).not_to eq(bytes)
       expect(resource["type"]).to eq('upload')
-      
+
       bytes = resource["bytes"]
     end
+  end
+end
+
+describe Cloudinary::Api::Response do
+  let(:api_response) { described_class.new }
+
+  shared_examples 'a Hash' do
+    it 'inherits from Hash' do
+      expect(api_response).to be_a Hash
+    end
+  end
+
+  context 'when there is no argument given on instantiation' do
+    it 'does not raise an error' do
+      expect { api_response }.to_not raise_error
+    end
+
+    it_behaves_like 'a Hash'
+  end
+
+  context 'when the response is nil' do
+    it 'does not raise an error' do
+      expect { described_class.new nil }.to_not raise_error
+    end
+
+    it_behaves_like 'a Hash'
+  end
+
+  context 'when the response is present' do
+    let(:body)          { { 'foo' => 'bar' } }
+    let(:http_response) { double code: 200, body: body.to_json, headers: { x_featureratelimit_reset: Time.new.to_s } }
+    let(:api_response)  { described_class.new http_response }
+
+    it 'sets the instantiated self as the parsed response which is a Hash' do
+      expect(api_response).to eq body
+    end
 
+    it_behaves_like 'a Hash'
   end
 end

data/spec/storage_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+require 'cloudinary'
+
+module CarrierWave
+  module Storage
+    class Abstract
+      def initialize(uploader)
+        @uploader = uploader
+      end
+
+      attr_accessor :uploader
+    end
+  end
+  class SanitizedFile; end
+end
+
+RSpec.describe Cloudinary::CarrierWave::Storage do
+  describe '#store_cloudinary_identifier' do
+    let(:column) { 'example_field' }
+    let(:model) { double :model, _mounter: mount, write_attribute: true }
+    let(:mount) { double :mount, serialization_column: column }
+    let(:storage) { Cloudinary::CarrierWave::Storage.new(uploader) }
+    let(:store_identifier) { storage.store_cloudinary_identifier('1', 'test.png') }
+    let(:uploader) { double :uploader, model: model, mounted_as: :example, use_extended_identifier?: false }
+
+    describe 'when the ORM is Neo4j 5 and above' do
+      before { stub_const('Neo4j::VERSION', '5.0') }
+
+      subject! { store_identifier }
+
+      it 'writes the name to the datastore without triggering validations' do
+        expect(model).to have_received(:write_attribute).with(column, 'v1/test.png')
+      end
+    end
+
+    describe 'when the ORM is Neo4j 4' do
+      before { stub_const('Neo4j::VERSION', '4.0') }
+
+      it 'raises an unsupported exception' do
+        expect { store_identifier }.to raise_error(CloudinaryException)
+      end
+    end
+  end
+end

data/spec/utils_spec.rb

@@ -120,6 +120,17 @@ describe Cloudinary::Utils do
             .and empty_options
   end
 
+  it "should sign a url" do
+    expected = Cloudinary::Utils.cloudinary_url "some_public_id.jpg", 
+                                     :cloud_name => "test", 
+                                     :api_key => "123456789012345", 
+                                     :api_secret => "AbcdEfghIjklmnopq1234567890", 
+                                     :type => "authenticated", 
+                                     :sign_url => true, 
+                                     :overlay => "text:Helvetica_50:test+text"
+    expect(expected).to eq("http://res.cloudinary.com/test/image/authenticated/s--j5Z1ILxd--/l_text:Helvetica_50:test+text/some_public_id.jpg")
+  end
+
   it "should not sign the url_suffix" do
     expected_signture = Cloudinary::Utils.cloudinary_url("test", :format => "jpg", :sign_url => true).match(/s--[0-9A-Za-z_-]{8}--/).to_s
     expect(["test", { :url_suffix => "hello", :private_cdn => true, :format => "jpg", :sign_url => true }])

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cloudinary
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.8.2
 platform: ruby
 authors:
 - Nadav Soferman
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-16 00:00:00.000000000 Z
+date: 2017-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws_cf_signer
@@ -184,6 +184,7 @@ files:
 - spec/sample_asset_file.tsv
 - spec/search_spec.rb
 - spec/spec_helper.rb
+- spec/storage_spec.rb
 - spec/streaminig_profiles_api_spec.rb
 - spec/support/helpers/temp_file_helpers.rb
 - spec/support/shared_contexts/rake.rb
@@ -224,7 +225,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: cloudinary
-rubygems_version: 2.6.10
+rubygems_version: 2.6.12
 signing_key: 
 specification_version: 4
 summary: Client library for easily using the Cloudinary service
@@ -244,6 +245,7 @@ test_files:
 - spec/sample_asset_file.tsv
 - spec/search_spec.rb
 - spec/spec_helper.rb
+- spec/storage_spec.rb
 - spec/streaminig_profiles_api_spec.rb
 - spec/support/helpers/temp_file_helpers.rb
 - spec/support/shared_contexts/rake.rb