restforce 6.0.0 → 8.0.1

data/lib/restforce/error_code.rb

@@ -19,6 +19,8 @@ module Restforce
 
     class ApexError < ResponseError; end
 
+    class ApexRestServicesDisabled < ResponseError; end
+
     class ApiCurrentlyDisabled < ResponseError; end
 
     class ApiDisabledForOrg < ResponseError; end
@@ -43,6 +45,8 @@ module Restforce
 
     class CannotDeleteLastDatedConversionRate < ResponseError; end
 
+    class CannotDeleteOwner < ResponseError; end
+
     class CannotDeleteManagedObject < ResponseError; end
 
     class CannotDisableLastAdmin < ResponseError; end
@@ -55,6 +59,8 @@ module Restforce
 
     class CannotModifyManagedObject < ResponseError; end
 
+    class CannotPostToArchivedGroup < ResponseError; end
+
     class CannotRenameApexReferencedField < ResponseError; end
 
     class CannotRenameApexReferencedObject < ResponseError; end
@@ -171,6 +177,8 @@ module Restforce
 
     class InsertUpdateDeleteNotAllowedDuringMaintenance < ResponseError; end
 
+    class InsufficientAccess < ResponseError; end
+
     class InsufficientAccessOnCrossReferenceEntity < ResponseError; end
 
     class InsufficientAccessOrReadonly < ResponseError; end
@@ -217,8 +225,12 @@ module Restforce
 
     class InvalidInetAddress < ResponseError; end
 
+    class InvalidInput < ResponseError; end
+
     class InvalidLineitemCloneState < ResponseError; end
 
+    class InvalidMarkup < ResponseError; end
+
     class InvalidMasterOrTranslatedSolution < ResponseError; end
 
     class InvalidMessageIdReference < ResponseError; end
@@ -245,6 +257,8 @@ module Restforce
 
     class InvalidSaveAsActivityFlag < ResponseError; end
 
+    class InvalidSearch < ResponseError; end
+
     class InvalidSessionId < ResponseError; end
 
     class InvalidSignupCountry < ResponseError; end
@@ -417,6 +431,7 @@ module Restforce
       "ALL_OR_NONE_OPERATION_ROLLED_BACK" => AllOrNoneOperationRolledBack,
       "ALREADY_IN_PROCESS" => AlreadyInProcess,
       "APEX_ERROR" => ApexError,
+      "APEX_REST_SERVICES_DISABLED" => ApexRestServicesDisabled,
       "API_CURRENTLY_DISABLED" => ApiCurrentlyDisabled,
       "API_DISABLED_FOR_ORG" => ApiDisabledForOrg,
       "ASSIGNEE_TYPE_REQUIRED" => AssigneeTypeRequired,
@@ -434,11 +449,13 @@ module Restforce
       "CANNOT_DELETE_LAST_DATED_CONVERSION_RATE" =>
    CannotDeleteLastDatedConversionRate,
       "CANNOT_DELETE_MANAGED_OBJECT" => CannotDeleteManagedObject,
+      "CANNOT_DELETE_OWNER" => CannotDeleteOwner,
       "CANNOT_DISABLE_LAST_ADMIN" => CannotDisableLastAdmin,
       "CANNOT_ENABLE_IP_RESTRICT_REQUESTS" => CannotEnableIpRestrictRequests,
       "CANNOT_EXECUTE_FLOW_TRIGGER" => CannotExecuteFlowTrigger,
       "CANNOT_INSERT_UPDATE_ACTIVATE_ENTITY" => CannotInsertUpdateActivateEntity,
       "CANNOT_MODIFY_MANAGED_OBJECT" => CannotModifyManagedObject,
+      "CANNOT_POST_TO_ARCHIVED_GROUP" => CannotPostToArchivedGroup,
       "CANNOT_RENAME_APEX_REFERENCED_FIELD" => CannotRenameApexReferencedField,
       "CANNOT_RENAME_APEX_REFERENCED_OBJECT" => CannotRenameApexReferencedObject,
       "CANNOT_REPARENT_RECORD" => CannotReparentRecord,
@@ -500,6 +517,8 @@ module Restforce
       "INACTIVE_OWNER_OR_USER" => InactiveOwnerOrUser,
       "INSERT_UPDATE_DELETE_NOT_ALLOWED_DURING_MAINTENANCE" =>
    InsertUpdateDeleteNotAllowedDuringMaintenance,
+      "INSUFFICIENT_ACCESS" =>
+   InsufficientAccess,
       "INSUFFICIENT_ACCESS_ON_CROSS_REFERENCE_ENTITY" =>
    InsufficientAccessOnCrossReferenceEntity,
       "INSUFFICIENT_ACCESS_OR_READONLY" => InsufficientAccessOrReadonly,
@@ -524,7 +543,9 @@ module Restforce
       "INVALID_FILTER_ACTION" => InvalidFilterAction,
       "INVALID_ID_FIELD" => InvalidIdField,
       "INVALID_INET_ADDRESS" => InvalidInetAddress,
+      "INVALID_INPUT" => InvalidInput,
       "INVALID_LINEITEM_CLONE_STATE" => InvalidLineitemCloneState,
+      "INVALID_MARKUP" => InvalidMarkup,
       "INVALID_MASTER_OR_TRANSLATED_SOLUTION" => InvalidMasterOrTranslatedSolution,
       "INVALID_MESSAGE_ID_REFERENCE" => InvalidMessageIdReference,
       "INVALID_OPERATION" => InvalidOperation,
@@ -539,6 +560,7 @@ module Restforce
       "INVALID_READ_ONLY_USER_DML" => InvalidReadOnlyUserDml,
       "INVALID_REPLICATION_DATE" => InvalidReplicationDate,
       "INVALID_SAVE_AS_ACTIVITY_FLAG" => InvalidSaveAsActivityFlag,
+      "INVALID_SEARCH" => InvalidSearch,
       "INVALID_SESSION_ID" => InvalidSessionId,
       "INVALID_SIGNUP_COUNTRY" => InvalidSignupCountry,
       "INVALID_STATUS" => InvalidStatus,
@@ -644,7 +666,7 @@ module Restforce
            "defined. If you're sure that this is a valid Salesforce error, then " \
            "please create an issue on GitHub at <#{GITHUB_ISSUE_URL}>."
 
-      super(constant_name)
+      super
     end
   end
 end

data/lib/restforce/file_part.rb

@@ -29,4 +29,6 @@ end
 
 # This patch is only needed with multipart-post < 2.0.0
 # 2.0.0 was released in 2013.
-require 'restforce/patches/parts' unless Parts::Part.method(:new).arity.abs == 4
+if defined?(Parts) && Parts::Part.method(:new).arity.abs != 4
+  require 'restforce/patches/parts'
+end

data/lib/restforce/mash.rb

@@ -46,10 +46,10 @@ module Restforce
       end
     end
 
-    def initialize(source_hash = nil, client = nil, default = nil, &blk)
+    def initialize(source_hash = nil, client = nil, default = nil, &)
       @client = client
       deep_update(source_hash) if source_hash
-      default ? super(default) : super(&blk)
+      default ? super(default) : super(&)
     end
 
     def dup

data/lib/restforce/middleware/authentication/client_credential.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module Restforce
+  class Middleware
+    class Authentication
+      class ClientCredential < Restforce::Middleware::Authentication
+        def params
+          { grant_type: 'client_credentials',
+            client_id: @options[:client_id],
+            client_secret: @options[:client_secret] }
+        end
+      end
+    end
+  end
+end

data/lib/restforce/middleware/authentication.rb

@@ -9,6 +9,7 @@ module Restforce
     autoload :Password,  'restforce/middleware/authentication/password'
     autoload :Token,     'restforce/middleware/authentication/token'
     autoload :JWTBearer, 'restforce/middleware/authentication/jwt_bearer'
+    autoload :ClientCredential, 'restforce/middleware/authentication/client_credential'
 
     # Rescue from 401's, authenticate then raise the error again so the client
     # can reissue the request.
@@ -50,7 +51,7 @@ module Restforce
         builder.use Faraday::Request::UrlEncoded
         builder.use Restforce::Middleware::Mashify, nil, @options
         builder.use Faraday::FollowRedirects::Middleware
-        builder.response :json
+        builder.response :restforce_json
 
         if Restforce.log?
           builder.use Restforce::Middleware::Logger,

data/lib/restforce/middleware/gzip.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'stringio'
 require 'zlib'
 
 module Restforce
@@ -28,6 +29,10 @@ module Restforce
     # Internal: Decompresses a gzipped string.
     def decompress(body)
       Zlib::GzipReader.new(StringIO.new(body)).read
+    rescue Zlib::GzipFile::Error
+      # We thought the response was gzipped, but it wasn't. Return the original
+      # body back to the caller. See https://github.com/restforce/restforce/issues/761.
+      body
     end
   end
 end

data/lib/restforce/middleware/json_request.rb

@@ -31,7 +31,7 @@ module Restforce
     CONTENT_TYPE = 'Content-Type'
 
     MIME_TYPE = 'application/json'
-    MIME_TYPE_REGEX = %r{^application/(vnd\..+\+)?json$}.freeze
+    MIME_TYPE_REGEX = %r{^application/(vnd\..+\+)?json$}
 
     #
     # Taken from `lib/faraday/middleware.rb` in the `faraday`
@@ -87,4 +87,4 @@ module Restforce
   end
 end
 
-Faraday::Request.register_middleware(json: Restforce::Middleware::JsonRequest)
+Faraday::Request.register_middleware(restforce_json: Restforce::Middleware::JsonRequest)

data/lib/restforce/middleware/json_response.rb

@@ -82,4 +82,4 @@ module Restforce
   end
 end
 
-Faraday::Response.register_middleware(json: Restforce::Middleware::JsonResponse)
+Faraday::Response.register_middleware(restforce_json: Restforce::Middleware::JsonResponse)

data/lib/restforce/middleware/raise_error.rb

@@ -60,7 +60,7 @@ module Restforce
       }
     end
 
-    ERROR_CODE_MATCHER = /\A[A-Z_]+\z/.freeze
+    ERROR_CODE_MATCHER = /\A[A-Z_]+\z/
 
     def exception_class_for_error_code(error_code)
       return Restforce::ResponseError unless ERROR_CODE_MATCHER.match?(error_code)

data/lib/restforce/patches/parts.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'stringio'
+
 module Faraday
   module Parts
     module Part

data/lib/restforce/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Restforce
-  VERSION = '6.0.0'
+  VERSION = '8.0.1'
 end

data/lib/restforce.rb

@@ -49,7 +49,6 @@ module Restforce
   UnauthorizedError   = Class.new(Faraday::ClientError)
   APIVersionError     = Class.new(Error)
   BatchAPIError       = Class.new(Error)
-  CompositeAPIError   = Class.new(Error)
 
   # Inherit from Faraday::ResourceNotFound for backwards-compatibility
   # Consumers of this library that rescue and handle Faraday::ResourceNotFound
@@ -60,6 +59,7 @@ module Restforce
   # Consumers of this library that rescue and handle Faraday::ClientError
   # can continue to do so.
   ResponseError       = Class.new(Faraday::ClientError)
+  CompositeAPIError   = Class.new(ResponseError)
   MatchesMultipleError= Class.new(ResponseError)
   EntityTooLargeError = Class.new(ResponseError)
 

data/restforce.gemspec

@@ -22,12 +22,12 @@ Gem::Specification.new do |gem|
 'rubygems_mfa_required' => 'true'
   }
 
-  gem.required_ruby_version = '>= 2.7'
+  gem.required_ruby_version = '>= 3.1'
 
-  gem.add_dependency 'faraday', '< 2.5.0', '>= 1.1.0'
-  gem.add_dependency 'faraday-follow_redirects', '<= 0.3.0', '< 1.0.0'
+  gem.add_dependency 'faraday', '< 3.0.0', '>= 1.1.0'
+  gem.add_dependency 'faraday-follow_redirects', '< 0.6.0'
   gem.add_dependency 'faraday-multipart', '>= 1.0.0', '< 2.0.0'
-  gem.add_dependency 'faraday-net_http', '< 3.0.0'
+  gem.add_dependency 'faraday-net_http', '< 4.0.0'
   gem.add_dependency 'hashie', '>= 1.2.0', '< 6.0'
   gem.add_dependency 'jwt', ['>= 1.5.6']
 end

data/spec/integration/abstract_client_spec.rb

@@ -144,7 +144,10 @@ shared_examples_for Restforce::AbstractClient do
   describe '.update' do
     context 'with missing Id' do
       subject { lambda { client.update('Account', Name: 'Foobar') } }
-      it { should raise_error ArgumentError, 'ID field missing from provided attributes' }
+      it {
+        expect { subject.call }.
+          to raise_error ArgumentError, 'ID field missing from provided attributes'
+      }
     end
 
     context 'with invalid Id' do
@@ -248,7 +251,7 @@ shared_examples_for Restforce::AbstractClient do
                status: 404
 
       subject { lambda { destroy! } }
-      it { should raise_error Faraday::ResourceNotFound }
+      it { expect { subject.call }.to raise_error Faraday::ResourceNotFound }
     end
 
     context 'with success' do
@@ -431,7 +434,7 @@ shared_examples_for Restforce::AbstractClient do
       end
 
       subject { lambda { client.query('SELECT some, fields FROM object') } }
-      it { should raise_error Restforce::UnauthorizedError }
+      it { expect { subject.call }.to raise_error Restforce::UnauthorizedError }
     end
   end
 

data/spec/unit/concerns/authentication_spec.rb

@@ -62,6 +62,17 @@ describe Restforce::Concerns::Authentication do
 
       it { should eq Restforce::Middleware::Authentication::JWTBearer }
     end
+
+    context 'when client_id and client_secret options are provided' do
+      before do
+        client.stub username_password?: false
+        client.stub oauth_refresh?: false
+        client.stub jwt?: false
+        client.stub client_credential?: true
+      end
+
+      it { should eq Restforce::Middleware::Authentication::ClientCredential }
+    end
   end
 
   describe '.username_password?' do
@@ -135,4 +146,26 @@ describe Restforce::Concerns::Authentication do
       it { should_not be true }
     end
   end
+
+  describe '.client_credential?' do
+    subject       { client.client_credential? }
+    let(:options) { {} }
+
+    before do
+      client.stub options: options
+    end
+
+    context 'when oauth client credential options are provided' do
+      let(:options) do
+        { client_id: 'client',
+          client_secret: 'secret' }
+      end
+
+      it { should be_truthy }
+    end
+
+    context 'when oauth client credential options are not provided' do
+      it { should_not be true }
+    end
+  end
 end

data/spec/unit/concerns/base_spec.rb

@@ -28,7 +28,7 @@ describe Restforce::Concerns::Base do
 
   describe '.options' do
     subject { lambda { client.options } }
-    it { should_not raise_error }
+    it { expect { subject.call }.not_to raise_error }
   end
 
   describe '.instance_url' do

data/spec/unit/concerns/composite_api_spec.rb

@@ -82,6 +82,23 @@ describe Restforce::Concerns::CompositeAPI do
       end
     end
 
+    it '#find' do
+      client.
+        should_receive(:api_post).
+        with(endpoint, { compositeRequest: [
+          {
+            method: 'GET',
+            url: "/services/data/v38.0/sobjects/Object/00312345",
+            referenceId: 'find_ref'
+          }
+        ], allOrNone: all_or_none, collateSubrequests: false }.to_json).
+        and_return(response)
+
+      client.send(method) do |subrequests|
+        subrequests.find('Object', 'find_ref', '00312345')
+      end
+    end
+
     it 'multiple subrequests' do
       client.
         should_receive(:api_post).
@@ -123,6 +140,32 @@ describe Restforce::Concerns::CompositeAPI do
         end
       end.to raise_error(ArgumentError)
     end
+
+    it 'has response in CompositeAPIError' do
+      response = double('Faraday::Response',
+                        body: { 'compositeResponse' =>
+                                  [{ 'httpStatusCode' => 400,
+                                     'body' => [{ 'errorCode' =>
+                                                    'DUPLICATE_VALUE' }] }] })
+      client.
+        should_receive(:api_post).
+        with(endpoint, { compositeRequest: [
+          {
+            method: 'POST',
+            url: '/services/data/v38.0/sobjects/Object',
+            body: { name: 'test' },
+            referenceId: 'create_ref'
+          }
+        ], allOrNone: true, collateSubrequests: false }.to_json).
+        and_return(response)
+      arg = method == :composite ? { all_or_none: true } : {}
+      expect do
+        client.send(method, **arg) do |subrequests|
+          subrequests.create('Object', 'create_ref', name: 'test')
+        end
+      end.to raise_error(an_instance_of(Restforce::CompositeAPIError).
+        and(having_attributes(response: response)))
+    end
   end
 
   describe '#composite' do

data/spec/unit/concerns/connection_spec.rb

@@ -85,6 +85,8 @@ describe Restforce::Concerns::Connection do
       client.stub options: { adapter: :typhoeus }
     end
 
-    its(:adapter) { should eq(:typhoeus) }
+    it "uses typheous adapter" do
+      expect(client.send(:adapter)).to be(:typhoeus)
+    end
   end
 end

data/spec/unit/config_spec.rb

@@ -62,7 +62,7 @@ describe Restforce do
     %i[username password security_token client_id client_secret compress
        timeout oauth_token refresh_token instance_url api_version host mashify
        authentication_retries proxy_uri authentication_callback ssl
-       request_headers log_level logger].each do |attr|
+       request_headers log_level logger clear_authorization_header].each do |attr|
       it "allows #{attr} to be set" do
         Restforce.configure do |config|
           config.send("#{attr}=", 'foobar')
@@ -70,6 +70,21 @@ describe Restforce do
         expect(Restforce.configuration.send(attr)).to eq 'foobar'
       end
     end
+
+    it 'takes precedence over environment variables' do
+      { 'SALESFORCE_USERNAME'  => 'env_foo',
+        'SALESFORCE_CLIENT_ID' => 'env_client id' }.
+        each { |var, value| ENV.stub(:fetch).with(var, anything).and_return(value) }
+
+      { username: 'config_foo', client_id: nil }.each do |attr, value|
+        Restforce.configure do |config|
+          config.send("#{attr}=", value)
+        end
+      end
+
+      expect(Restforce.configuration.send(:username)).to eq 'config_foo'
+      expect(Restforce.configuration.send(:client_id)).to eq 'env_client id'
+    end
   end
 
   describe '#log?' do
@@ -138,5 +153,17 @@ describe Restforce do
         checker.check!
       end
     end
+
+    it 'shows the precedence over config' do
+      ENV['SALESFORCE_USERNAME'] = 'env_foo'
+      Restforce.configure do |config|
+        config.username = 'config_foo'
+      end
+      client = Restforce.new(username: 'foo')
+
+      expect(client.options[:username]).to eq 'foo'
+
+      Restforce.instance_variable_set :@configuration, nil
+    end
   end
 end

data/spec/unit/middleware/authentication/client_credential_spec.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Restforce::Middleware::Authentication::ClientCredential do
+  let(:options) do
+    { host: 'login.salesforce.com',
+      client_id: 'client_id',
+      client_secret: 'client_secret',
+      adapter: :net_http }
+  end
+
+  it_behaves_like 'authentication middleware' do
+    let(:success_request) do
+      stub_login_request(
+        body: "grant_type=client_credentials&" \
+              "client_id=client_id&client_secret=client_secret"
+      ).to_return(
+        status: 200,
+        body: fixture(:auth_success_response),
+        headers: { "Content-Type" => "application/json" }
+      )
+    end
+
+    let(:fail_request) do
+      stub_login_request(
+        body: "grant_type=client_credentials&" \
+              "client_id=client_id&client_secret=client_secret"
+      ).to_return(
+        status: 400,
+        body: fixture(:refresh_error_response),
+        headers: { "Content-Type" => "application/json" }
+      )
+    end
+  end
+end

data/spec/unit/middleware/authentication_spec.rb

@@ -15,7 +15,7 @@ describe Restforce::Middleware::Authentication do
 
   describe '.authenticate!' do
     subject { lambda { middleware.authenticate! } }
-    it      { should raise_error NotImplementedError }
+    it      { expect { subject.call }.to raise_error NotImplementedError }
   end
 
   describe '.call' do
@@ -26,7 +26,7 @@ describe Restforce::Middleware::Authentication do
         app.should_receive(:call).once
       end
 
-      it { should_not raise_error }
+      it { expect { subject.call }.not_to raise_error }
     end
 
     context 'when an exception is thrown' do
@@ -37,7 +37,7 @@ describe Restforce::Middleware::Authentication do
           and_raise(Restforce::UnauthorizedError.new('something bad'))
       end
 
-      it { should raise_error Restforce::UnauthorizedError }
+      it { expect { subject.call }.to raise_error Restforce::UnauthorizedError }
     end
   end
 

data/spec/unit/middleware/authorization_spec.rb

@@ -8,6 +8,10 @@ describe Restforce::Middleware::Authorization do
   describe '.call' do
     subject { lambda { middleware.call(env) } }
 
-    it { should change { env[:request_headers]['Authorization'] }.to eq 'OAuth token' }
+    it {
+      expect { subject.call }.to change {
+                                   env[:request_headers]['Authorization']
+                                 }.to eq 'OAuth token'
+    }
   end
 end

data/spec/unit/middleware/custom_headers_spec.rb

@@ -9,13 +9,17 @@ describe Restforce::Middleware::CustomHeaders do
     context 'when :request_headers are a Hash' do
       let(:options) { { request_headers: { 'x-test-header' => 'Test Value' } } }
 
-      it { should change { env[:request_headers]['x-test-header'] }.to eq 'Test Value' }
+      it {
+        expect { subject.call }.to change {
+                                     env[:request_headers]['x-test-header']
+                                   }.to eq 'Test Value'
+      }
     end
 
     context 'when :request_headers are not a Hash' do
       let(:options) { { request_headers: 'bad header' } }
 
-      it { should_not(change { env[:request_headers] }) }
+      it { expect { subject.call }.not_to(change { env[:request_headers] }) }
     end
   end
 end

data/spec/unit/middleware/gzip_spec.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'stringio'
+
 require 'spec_helper'
 
 describe Restforce::Middleware::Gzip do
@@ -17,29 +19,73 @@ describe Restforce::Middleware::Gzip do
   describe '.call' do
     subject { lambda { middleware.call(env) } }
 
-    before do
-      app.should_receive(:on_complete) { middleware.on_complete(env) }
-      app.should_receive(:call) do
-        env[:body] = gzip fixture('sobject/query_success_response')
-        env[:response_headers]['Content-Encoding'] = 'gzip'
-        app
+    context 'when the response is gzipped' do
+      before do
+        app.should_receive(:on_complete) { middleware.on_complete(env) }
+        app.should_receive(:call) do
+          env[:body] = gzip fixture('sobject/query_success_response')
+          env[:response_headers]['Content-Encoding'] = 'gzip'
+          app
+        end
+      end
+
+      it 'decompresses the body' do
+        expect { subject.call }.to change {
+                                     env[:body]
+                                   }.to(fixture('sobject/query_success_response'))
+      end
+
+      context 'when :compress is false' do
+        it 'does not set request headers to ask the response to be compressed' do
+          expect { subject.call }.
+            not_to(change { env[:request_headers]['Accept-Encoding'] })
+        end
       end
-    end
 
-    it 'decompresses the body' do
-      should change { env[:body] }.to(fixture('sobject/query_success_response'))
+      context 'when :compress is true' do
+        before do
+          options[:compress] = true
+        end
+
+        it 'sets request headers to ask the response to be compressed' do
+          expect { subject.call }.to change {
+                                       env[:request_headers]['Accept-Encoding']
+                                     }.to('gzip')
+        end
+      end
     end
 
-    context 'when :compress is false' do
-      it { should_not(change { env[:request_headers]['Accept-Encoding'] }) }
+    context 'when the response claims to be gzipped, but is not' do
+      before do
+        app.should_receive(:on_complete) { middleware.on_complete(env) }
+        app.should_receive(:call) do
+          env[:body] = fixture('sobject/query_success_response')
+          env[:response_headers]['Content-Encoding'] = 'gzip'
+          app
+        end
+      end
+
+      it 'does not decompress the body' do
+        expect { subject.call }.to change {
+                                     env[:body]
+                                   }.to(fixture('sobject/query_success_response'))
+      end
     end
 
-    context 'when :compress is true' do
+    context 'when the response does not even claim to be gzipped' do
       before do
-        options[:compress] = true
+        app.should_receive(:on_complete) { middleware.on_complete(env) }
+        app.should_receive(:call) do
+          env[:body] = fixture('sobject/query_success_response')
+          app
+        end
       end
 
-      it { should(change { env[:request_headers]['Accept-Encoding'] }.to('gzip')) }
+      it 'does not decompress the body' do
+        expect { subject.call }.to change {
+                                     env[:body]
+                                   }.to(fixture('sobject/query_success_response'))
+      end
     end
   end