CR feedback changes

lib/sentry/client.rb

@@ -34,7 +34,7 @@ module Sentry
     private
 
     def validate_size(issues)
-      return if valid_size?(issues)
+      return if Gitlab::Utils::DeepSize.new(issues).valid?
 
       raise Client::ResponseInvalidSizeError, "Sentry API response is too big. Limit is #{Gitlab::Utils::DeepSize.human_default_max_size}."
     end
@@ -101,7 +101,7 @@ module Sentry
         raise_error "Sentry response status code: #{response.code}"
       end
 
-      response
+      response.parsed_response
     end
 
     def raise_error(message)

spec/lib/sentry/client_spec.rb

@@ -194,9 +194,10 @@ describe Sentry::Client do
 
     context 'sentry api response too large' do
       it 'raises exception' do
-        allow(client).to receive(:valid_size?).and_return(false)
+        deep_size = double('Gitlab::Utils::DeepSize', valid?: false)
+        allow(Gitlab::Utils::DeepSize).to receive(:new).with(sentry_api_response).and_return(deep_size)
 
-        expect { subject }.to raise_error(Sentry::Client::ResponseInvalidSizeError, "Sentry API response is too big. Limit is #{Gitlab::Utils::DeepSize.human_default_max_size}.")
+        expect { subject }.to raise_error(Sentry::Client::ResponseInvalidSizeError, 'Sentry API response is too big. Limit is 1 MB.')
       end
     end