hello_sign 0.6.0 → 1.0.0

data/spec/integration/signature_request_spec.rb

@@ -6,69 +6,69 @@ describe HelloSign do
       let(:text_file_io)  { File.new('spec/fixtures/test.txt') }
       let(:image_io)      { File.new('spec/fixtures/test.jpg') }
 
-      before do
-        stub_post_with_auth('/signature_request/send')
+      before { stub_post_with_auth('/signature_request/send') }
+
+      example do
         HelloSign.signature_request.deliver do |request|
           request.title   = 'Lease'
           request.subject = 'Sign this'
           request.message = 'You must sign this.'
           request.ccs     = ['lawyer@lawfirm.com', 'spouse@family.com']
           request.signers = [
-            {:name => 'Jack', :email_address => 'jack@hill.com'},
-            {:name => 'Jill', :email_address => 'jill@hill.com'}
+            {name: 'Jack', email_address: 'jack@hill.com'},
+            {name: 'Jill', email_address: 'jill@hill.com'}
           ]
           request.files   = [
-            {:name => 'test.txt', :io => text_file_io, :mime => 'text/plain'},
-            {:name => 'test.jpg', :io => image_io,     :mime => 'image/jpeg'}
+            {filename: 'test.txt', io: text_file_io, mime: 'text/plain'},
+            {filename: 'spec/fixtures/test.txt', mime: 'text/xml'},
+            {filename: 'spec/fixtures/test.jpg'},
+            {io: image_io, mime: 'image/jpeg'}
           ]
         end
-      end
 
-      it "sends a signature request to the HelloSign API" do
         expect(a_post_with_auth('/signature_request/send')
-          .with(:headers => {'Content-Type' => /multipart\/form-data/}, :body => /This is a test upload file\./)
+          .with(headers: {'Content-Type' => /multipart\/form-data/}, body: /This is a test upload file\./)
         ).to have_been_made
       end
     end
 
     context "when using a reusable form" do
-      before do
-        stub_post_with_auth('/signature_request/send_with_reusable_form')
-        HelloSign.signature_request.deliver(:form => 'form_id') do |request|
+      before { stub_post_with_auth('/signature_request/send_with_reusable_form') }
+
+      example do
+        HelloSign.signature_request.deliver(form: 'form_id') do |request|
           request.title         = 'Lease'
           request.subject       = 'Sign this'
           request.message       = 'You must sign this.'
           request.ccs           = [
-            {:email_address => 'lawyer@lawfirm.com', :role => 'lawyer'},
-            {:email_address => 'accountant@llc.com', :role => 'accountant'}
+            {email_address: 'lawyer@lawfirm.com', role: 'lawyer'},
+            {email_address: 'accountant@llc.com', role: 'accountant'}
           ]
           request.signers       = [
-            {:name => 'Jack', :email_address => 'jack@hill.com', :role => 'consultant'},
-            {:name => 'Jill', :email_address => 'jill@hill.com', :role => 'client'}
+            {name: 'Jack', email_address: 'jack@hill.com', role: 'consultant'},
+            {name: 'Jill', email_address: 'jill@hill.com', role: 'client'}
           ]
           request.custom_fields = [
-            {:name => 'cost', :value => '$20,000'},
-            {:name => 'time', :value => 'two weeks'}
+            {name: 'cost', value: '$20,000'},
+            {name: 'time', value: 'two weeks'}
           ]
         end
-      end
 
-      it "sends a signature request using a reusable form to the HelloSign API" do
         expect(a_post_with_auth('/signature_request/send_with_reusable_form')
-          .with(:body => {
-            :reusable_form_id => 'form_id',
-            :title            => 'Lease',
-            :subject          => 'Sign this',
-            :message          => 'You must sign this.',
-            :ccs              => {
-              'lawyer'     => {:email_address => 'lawyer@lawfirm.com'},
-              'accountant' => {:email_address => 'accountant@llc.com'}
+          .with(body: {
+            reusable_form_id: 'form_id',
+            title:            'Lease',
+            subject:          'Sign this',
+            message:          'You must sign this.',
+            ccs:              {
+              'lawyer'     => {email_address: 'lawyer@lawfirm.com'},
+              'accountant' => {email_address: 'accountant@llc.com'}
             },
-            :signers          => {
-              'consultant' => {:name => 'Jack', :email_address => 'jack@hill.com'}, 
-              'client'     => {:name => 'Jill', :email_address => 'jill@hill.com'}
+            signers:          {
+              'consultant' => {name: 'Jack', email_address: 'jack@hill.com'}, 
+              'client'     => {name: 'Jill', email_address: 'jill@hill.com'}
             },
-            :custom_fields    => {
+            custom_fields:    {
               'cost' => '$20,000',
               'time' => 'two weeks'
             }
@@ -79,58 +79,53 @@ describe HelloSign do
   end
 
   context "when fetching a signature request" do
-    before do
-      stub_get_with_auth('/signature_request/request_id')
-      HelloSign.signature_request.status('request_id')
-    end
+    before { stub_get_with_auth('/signature_request/request_id') }
+
+    example do
+      HelloSign.signature_request('request_id').status
 
-    it "fetches the signature request information from the HelloSign API" do
       expect(a_get_with_auth('/signature_request/request_id')).to have_been_made
     end
   end
 
   context "when getting a list of signature requests" do
-    before do
-      stub_get_with_auth('/signature_request/list?page=1')
+    before { stub_get_with_auth('/signature_request/list?page=1') }
+
+    example do
       HelloSign.signature_request.list
-    end
 
-    it "fetches a list of signature requests from the HelloSign API" do
       expect(a_get_with_auth('/signature_request/list?page=1')).to have_been_made
     end
   end
 
   context "when sending a signature request reminder" do
-    before do
-      stub_post_with_auth('/signature_request/remind/request_id')
-      HelloSign.signature_request.remind('request_id', :email => 'john@johnson.com')
-    end
+    before { stub_post_with_auth('/signature_request/remind/request_id') }
+
+    example do
+      HelloSign.signature_request('request_id').remind(email: 'john@johnson.com')
 
-    it "sends a reminder request to the HelloSign API" do
       expect(a_post_with_auth('/signature_request/remind/request_id')
-        .with(:body => {:email_address => 'john@johnson.com'})
+        .with(body: {email_address: 'john@johnson.com'})
       ).to have_been_made
     end
   end
 
   context "when canceling a signature request" do
-    before do
-      stub_post_with_auth('/signature_request/cancel/request_id')
-      HelloSign.signature_request.cancel('request_id')
-    end
+    before { stub_post_with_auth('/signature_request/cancel/request_id') }
+
+    example do
+      HelloSign.signature_request('request_id').cancel
 
-    it "sends a signature request cancellation to the HelloSign API" do
       expect(a_post_with_auth('/signature_request/cancel/request_id')).to have_been_made
     end
   end
 
   context "when fetching a final copy of a signature request" do
-    before do
-      stub_get_with_auth('/signature_request/final_copy/request_id')
-      HelloSign.signature_request.final_copy('request_id')
-    end
+    before { stub_get_with_auth('/signature_request/final_copy/request_id') }
+
+    example do
+      HelloSign.signature_request('request_id').final_copy
 
-    it "fetches a final copy of the signature request from the HelloSign API" do
       expect(a_get_with_auth('/signature_request/final_copy/request_id')).to have_been_made
     end
   end

data/spec/integration/team_spec.rb

@@ -5,10 +5,10 @@ describe HelloSign do
     before { stub_post_with_auth('/team/create') }
 
     example do
-      HelloSign.team.create(:name => 'The Browncoats')
+      HelloSign.team.create(name: 'The Browncoats')
 
       expect(a_post_with_auth('/team/create')
-        .with(:body => {:name => 'The Browncoats'}))
+        .with(body: {name: 'The Browncoats'}))
         .to have_been_made
     end
   end
@@ -28,10 +28,10 @@ describe HelloSign do
     before { stub_post_with_auth('/team') }
 
     example do
-      HelloSign.team.update(:name => 'The Bluecoats')
+      HelloSign.team.update(name: 'The Bluecoats')
 
       expect(a_post_with_auth('/team')
-        .with(:name => 'The Bluecoats'))
+        .with(name: 'The Bluecoats'))
         .to have_been_made
     end
   end
@@ -51,10 +51,10 @@ describe HelloSign do
     before { stub_post_with_auth('/team/add_member') }
 
     example do
-      HelloSign.team.add_member(:email => 'bob@smith.com')
+      HelloSign.team.add_member(email: 'bob@smith.com')
 
       expect(a_post_with_auth('/team/add_member')
-        .with(:email => 'bob@smith.com'))
+        .with(email: 'bob@smith.com'))
         .to have_been_made
     end
   end
@@ -63,10 +63,10 @@ describe HelloSign do
     before { stub_post_with_auth('/team/remove_member') }
 
     example do
-      HelloSign.team.remove_member(:email => 'bob@smith.com')
+      HelloSign.team.remove_member(email: 'bob@smith.com')
 
       expect(a_post_with_auth('/team/remove_member')
-        .with(:email => 'bob@smith.com'))
+        .with(email: 'bob@smith.com'))
         .to have_been_made
     end
   end

data/spec/integration/unclaimed_draft_spec.rb

@@ -10,13 +10,13 @@ describe HelloSign do
     example do
       HelloSign.unclaimed_draft.create do |draft|
         draft.files = [
-          {:name => 'test.txt', :io => text_io,  :mime => 'text/plain'},
-          {:name => 'test.jpg', :io => image_io, :mime => 'image/jpeg'}
+          {name: 'test.txt', io: text_io,  mime: 'text/plain'},
+          {name: 'test.jpg', io: image_io, mime: 'image/jpeg'}
         ]
       end
 
       expect(a_post_with_auth('/unclaimed_draft/create')
-        .with(:headers => {'Content-Type' => /multipart\/form-data/}, :body => /This is a test upload file\./)
+        .with(headers: {'Content-Type' => /multipart\/form-data/}, body: /This is a test upload file\./)
       ).to have_been_made
     end
   end

data/spec/shared_examples/proxy.rb

@@ -14,25 +14,35 @@ shared_examples_for 'a proxy' do
   describe "#signature_request" do
     before { client.signature_request_proxy_source = proxy_source }
 
-    it "returns an signature request proxy" do
-      proxy_source.should_receive(:new).with(client).and_return(proxy)
+    it "returns a signature request proxy" do
+      proxy_source.should_receive(:new).with(client, nil).and_return(proxy)
       expect(client.signature_request).to eq proxy
     end
+
+    it "passes on an optional signature request ID" do
+      proxy_source.should_receive(:new).with(client, 'signature_request_id')
+      client.signature_request('signature_request_id')
+    end
   end
 
   describe "#reusable_form" do
     before { client.reusable_form_proxy_source = proxy_source }
 
-    it "returns an reusable form proxy" do
-      proxy_source.should_receive(:new).with(client).and_return(proxy)
+    it "returns a reusable form proxy" do
+      proxy_source.should_receive(:new).with(client, nil).and_return(proxy)
       expect(client.reusable_form).to eq proxy
     end
+
+    it "passes on an optional reusable form ID" do
+      proxy_source.should_receive(:new).with(client, 'reusable_form_id')
+      client.reusable_form('reusable_form_id')
+    end
   end
 
   describe "#team" do
     before { client.team_proxy_source = proxy_source }
 
-    it "returns an team proxy" do
+    it "returns a team proxy" do
       proxy_source.should_receive(:new).with(client).and_return(proxy)
       expect(client.team).to eq proxy
     end

data/spec/unit/client_spec.rb

@@ -45,17 +45,17 @@ describe HelloSign::Client do
   end
 
   context "when a hash is passed to the constructor" do
-    subject(:hs_client) { HelloSign::Client.new(:email_address => email_address, :password => password) }
+    subject(:hs_client) { HelloSign::Client.new(email_address: email_address, password: password) }
 
     its(:email_address) { should eq email_address }
     its(:password)      { should eq password }
 
     it "raises an exception if an email address is not provided" do
-      expect { HelloSign::Client.new(:password => 'space') }.to raise_error ArgumentError
+      expect { HelloSign::Client.new(password: 'space') }.to raise_error ArgumentError
     end
 
     it "raises an exception if a password is not provided" do
-      expect { HelloSign::Client.new(:email_address => 'david@bowman.com') }.to raise_error ArgumentError
+      expect { HelloSign::Client.new(email_address: 'david@bowman.com') }.to raise_error ArgumentError
     end
   end
 end

data/spec/unit/middleware/raise_error_spec.rb

@@ -21,7 +21,7 @@ describe HelloSign::Middleware::RaiseError do
   private
 
   def call_with_error(error)
-    env = {:body => {:error => {:error_name => error}}}
+    env = {body: {error: {error_name: error}}}
     middleware.call(env)
   end
 

data/spec/unit/parameters/reusable_form_signature_request_spec.rb

@@ -11,34 +11,34 @@ describe HelloSign::Parameters::ReusableFormSignatureRequest do
       request_parameters.subject = 'Sign this'
       request_parameters.message = 'You must sign this.'
       request_parameters.ccs     = [
-        {:email_address => 'lawyer@lawfirm.com', :role => 'lawyer'},
-        {:email_address => 'accountant@llc.com', :role => 'accountant'}
+        {email_address: 'lawyer@lawfirm.com', role: 'lawyer'},
+        {email_address: 'accountant@llc.com', role: 'accountant'}
       ]
       request_parameters.signers = [
-        {:name => 'Jack', :email_address => 'jack@hill.com', :role => 'consultant'},
-        {:name => 'Jill', :email_address => 'jill@hill.com', :role => 'client'}
+        {name: 'Jack', email_address: 'jack@hill.com', role: 'consultant'},
+        {name: 'Jill', email_address: 'jill@hill.com', role: 'client'}
       ]
       request_parameters.custom_fields = [
-        {:name => 'cost', :value => '$20,000'},
-        {:name => 'time', :value => 'two weeks'}
+        {name: 'cost', value: '$20,000'},
+        {name: 'time', value: 'two weeks'}
       ]
     end
 
     it "returns formatted parameters" do
       expected = {
-        :reusable_form_id => 'form_id',
-        :title            => 'Lease',
-        :subject          => 'Sign this',
-        :message          => 'You must sign this.',
-        :ccs              => {
-          'lawyer'     => {:email_address => 'lawyer@lawfirm.com'},
-          'accountant' => {:email_address => 'accountant@llc.com'}
+        reusable_form_id: 'form_id',
+        title:            'Lease',
+        subject:          'Sign this',
+        message:          'You must sign this.',
+        ccs:              {
+          'lawyer'     => {email_address: 'lawyer@lawfirm.com'},
+          'accountant' => {email_address: 'accountant@llc.com'}
         },
-        :signers          => {
-          'consultant' => {:name => 'Jack', :email_address => 'jack@hill.com'}, 
-          'client'     => {:name => 'Jill', :email_address => 'jill@hill.com'}
+        signers:          {
+          'consultant' => {name: 'Jack', email_address: 'jack@hill.com'}, 
+          'client'     => {name: 'Jill', email_address: 'jill@hill.com'}
         },
-        :custom_fields    => {
+        custom_fields:    {
           'cost' => '$20,000',
           'time' => 'two weeks'
         }

data/spec/unit/parameters/signature_request_spec.rb

@@ -4,20 +4,24 @@ require 'hello_sign/parameters/signature_request'
 describe HelloSign::Parameters::SignatureRequest do
   describe "#formatted" do
     let(:request_parameters) { HelloSign::Parameters::SignatureRequest.new }
+    let(:upload_io_source)   { double('upload IO source') }
     let(:text_file)          { double('text file') }
     let(:image_file)         { double('image file') }
+
+    before { request_parameters.upload_io_source = upload_io_source }
+
     context "when all required arguments are set" do
       let(:expected) do
         {
-          :title => 'Lease',
-          :subject => 'Sign this',
-          :message => 'You must sign this.',
-          :cc_email_addresses => ['lawyer@lawfirm.com', 'spouse@family.com'], # BUGBUG: should have explicit indexes
-          :signers => {
-            0 => {:name => 'Jack', :email_address => 'jack@hill.com', :order => 0}, 
-            1 => {:name => 'Jill', :email_address => 'jill@hill.com', :order => 1}
+          title: 'Lease',
+          subject: 'Sign this',
+          message: 'You must sign this.',
+          cc_email_addresses: ['lawyer@lawfirm.com', 'spouse@family.com'], # BUGBUG: should have explicit indexes
+          signers: {
+            0 => {name: 'Jack', email_address: 'jack@hill.com', order: 0}, 
+            1 => {name: 'Jill', email_address: 'jill@hill.com', order: 1}
           },
-          :file => {1 => text_file, 2 => image_file}
+          file: {1 => text_file, 2 => image_file}
         }
       end
 
@@ -27,18 +31,19 @@ describe HelloSign::Parameters::SignatureRequest do
         request_parameters.message = 'You must sign this.'
         request_parameters.ccs     = ['lawyer@lawfirm.com', 'spouse@family.com']
         request_parameters.signers = [
-          {:name => 'Jack', :email_address => 'jack@hill.com'},
-          {:name => 'Jill', :email_address => 'jill@hill.com'}
+          {name: 'Jack', email_address: 'jack@hill.com'},
+          {name: 'Jill', email_address: 'jill@hill.com'}
         ]
         request_parameters.files   = [
-          {:name => 'test.txt', :io => 'text file IO object', :mime => 'text/plain'},
-          {:name => 'test.jpg', :io => 'image file IO object', :mime => 'image/jpeg'}
+          @file_data_1 = {filename: 'test.txt', io: 'text file IO object', mime: 'text/plain'},
+          @file_data_2 = {filename: 'test.jpg', io: 'image file IO object', mime: 'image/jpeg'}
         ]
       end
 
       it "returns formatted parameters" do
-        Faraday::UploadIO.should_receive(:new).with('text file IO object', 'text/plain', 'test.txt').and_return(text_file)
-        Faraday::UploadIO.should_receive(:new).with('image file IO object', 'image/jpeg', 'test.jpg').and_return(image_file)
+        upload_io_source.should_receive(:new).with(@file_data_1).and_return(text_file)
+        upload_io_source.should_receive(:new).with(@file_data_2).and_return(image_file)
+        [text_file, image_file].each { |file| file.should_receive(:upload).and_return(file) }
 
         expect(request_parameters.formatted).to eq expected
       end
@@ -47,12 +52,12 @@ describe HelloSign::Parameters::SignatureRequest do
     context "when required parameters are omitted" do
       let(:expected) do
         {
-          :title              => nil,
-          :subject            => nil,
-          :message            => nil,
-          :cc_email_addresses => nil,
-          :signers            => {},
-          :file               => {}
+          title:              nil,
+          subject:            nil,
+          message:            nil,
+          cc_email_addresses: nil,
+          signers:            {},
+          file:               {}
         }
       end
 

data/spec/unit/parameters/unclaimed_draft_spec.rb

@@ -4,27 +4,28 @@ require 'hello_sign/parameters/unclaimed_draft'
 require 'stringio'
 
 describe HelloSign::Parameters::UnclaimedDraft do
-  let(:draft_parameters) { HelloSign::Parameters::UnclaimedDraft.new }
+  let(:upload_io_source)     { double('upload IO source') }
+  subject(:draft_parameters) { HelloSign::Parameters::UnclaimedDraft.new }
 
-  it "interfaces with Faraday::UploadIO properly" do
-    draft_parameters.files = [{:name => 'test.txt', :io => StringIO.new('foobar'),  :mime => 'text/plain'}]
-    expect(draft_parameters.formatted).to be_a Hash
-  end
+  before { draft_parameters.upload_io_source = upload_io_source }
 
   describe "#formatted" do
-    let(:text_file)        { double('text file') }
-    let(:image_file)       { double('image file') }
+    let(:text_file) { double('text file') }
+    let(:image)     { double('image') }
 
     before do
       draft_parameters.files = [
-        {:name => 'test.txt', :io => 'text file IO object',  :mime => 'text/plain'},
-        {:name => 'test.jpg', :io => 'image file IO object', :mime => 'image/jpeg'}
+        @file_data_1 = {filename: 'test.txt', io: text_file,  mime: 'text/plain'},
+        @file_data_2 = {filename: 'test.jpg', io: image, mime: 'image/jpeg'}
       ]
     end
 
     it "returns formatted parameters" do
-      Faraday::UploadIO.stub(:new).and_return(text_file, image_file)
-      expect(draft_parameters.formatted).to eq({:file => {0 => text_file, 1 => image_file}})
+      upload_io_source.should_receive(:new).with(@file_data_1).and_return(text_file)
+      upload_io_source.should_receive(:new).with(@file_data_2).and_return(image)
+      [text_file, image].each { |file| file.should_receive(:upload).and_return(file) }
+
+      expect(draft_parameters.formatted).to eq({file: {0 => text_file, 1 => image}})
     end
   end
 end

data/spec/unit/proxy/account_spec.rb

@@ -18,20 +18,20 @@ describe HelloSign::Proxy::Account do
     it "sends a request to create an account" do
       client.should_receive(:post).with(
         '/account/create',
-        :body => {:email_address => 'david@bowman.com', :password => 'space'},
-        :auth_not_required => true
+        body: {email_address: 'david@bowman.com', password: 'space'},
+        auth_not_required: true
       )
       account_proxy.create(
-        :email_address => 'david@bowman.com',
-        :password      => 'space'
+        email_address: 'david@bowman.com',
+        password:      'space'
       )
     end
 
     it "returns the API response" do
       expect(
         account_proxy.create(
-          :email_address => 'david@bowman.com',
-          :password      => 'space'
+          email_address: 'david@bowman.com',
+          password:      'space'
         )
       ). to eq api_response
     end

data/spec/unit/proxy/reusable_form_spec.rb

@@ -6,19 +6,20 @@ describe HelloSign::Proxy::ReusableForm do
   let(:api_response)  { double('API response') }
   let(:form_id)       { 'form_id' }
   let(:email_address) { 'bob@example.com' }
-  subject(:rf_proxy)  { HelloSign::Proxy::ReusableForm.new(client) }
+  subject(:rf_proxy)  { HelloSign::Proxy::ReusableForm.new(client, form_id) }
 
   before do
     client.stub(:get).and_return(api_response)
     client.stub(:post).and_return(api_response)
   end
 
-  its(:client) { should eq client }
+  its(:client)  { should eq client }
+  its(:form_id) { should eq form_id }
 
   describe "#list" do
     it "sends a request to fetch the list of reusable forms" do
-      client.should_receive(:get).with('/reusable_form/list', :params => {:page => 10})
-      rf_proxy.list(:page => 10)
+      client.should_receive(:get).with('/reusable_form/list', params: {page: 10})
+      rf_proxy.list(page: 10)
     end
 
     it "returns the API response" do
@@ -29,33 +30,33 @@ describe HelloSign::Proxy::ReusableForm do
   describe "#show" do
     it "sends a request to fetch the details of a reusable form" do
       client.should_receive(:get).with("/reusable_form/#{form_id}")
-      rf_proxy.show(form_id)
+      rf_proxy.show
     end
 
     it "returns the API response" do
-      expect(rf_proxy.show(form_id)).to eq api_response
+      expect(rf_proxy.show).to eq api_response
     end
   end
 
   describe "#grant_access" do
     it "sends a request to grant access" do
-      client.should_receive(:post).with("/reusable_form/add_user/#{form_id}", :body => {:email_address => email_address})
-      rf_proxy.grant_access(form_id, :email_address => email_address)
+      client.should_receive(:post).with("/reusable_form/add_user/#{form_id}", body: {email_address: email_address})
+      rf_proxy.grant_access(email_address: email_address)
     end
 
     it "returns the API response" do
-      expect(rf_proxy.grant_access(form_id, :email => email_address)).to eq api_response
+      expect(rf_proxy.grant_access(:email => email_address)).to eq api_response
     end
   end
 
   describe "#revoke_access" do
     it "sends a request to revoke access" do
-      client.should_receive(:post).with("/reusable_form/remove_user/#{form_id}", :body => {:email_address => email_address})
-      rf_proxy.revoke_access(form_id, :email_address => email_address)
+      client.should_receive(:post).with("/reusable_form/remove_user/#{form_id}", body: {email_address: email_address})
+      rf_proxy.revoke_access(email_address: email_address)
     end
 
     it "returns the API response" do
-      expect(rf_proxy.revoke_access(form_id, :email_address => email_address)).to eq api_response
+      expect(rf_proxy.revoke_access(email_address: email_address)).to eq api_response
     end
   end
 end

data/spec/unit/proxy/settings_spec.rb

@@ -18,12 +18,12 @@ describe HelloSign::Proxy::Settings do
     before { client.stub(:post).and_return(api_response) }
 
     it "sends a request to update the account's settings" do
-      client.should_receive(:post).with('/account', :body => {:callback_url => callback_url})
-      settings.update(:callback_url => callback_url)
+      client.should_receive(:post).with('/account', body: {callback_url: callback_url})
+      settings.update(callback_url: callback_url)
     end
 
     it "returns the API response" do
-      expect(settings.update(:callback_url => callback_url)).to be api_response
+      expect(settings.update(callback_url: callback_url)).to be api_response
     end
   end
 end