Use map instead of tap to make test setup clearer58971-sentry-api-keyerror

1 files changed, 9 insertions, 8 deletions

diff --git a/spec/lib/sentry/client_spec.rb b/spec/lib/sentry/client_spec.rb
index ea6301913a7..3333f8307ae 100644
--- a/spec/lib/sentry/client_spec.rb
+++ b/spec/lib/sentry/client_spec.rb
@@ -144,8 +144,9 @@ describe Sentry::Client do
 
     context 'Older sentry versions where keys are not present' do
       let(:sentry_api_response) do
-        [] << issues_sample_response[0].tap do |issue|
+        issues_sample_response[0...1].map do |issue|
           issue[:project].delete(:id)
+          issue
         end
       end
 
@@ -159,8 +160,8 @@ describe Sentry::Client do
 
     context 'essential keys missing in API response' do
       let(:sentry_api_response) do
-        [] << issues_sample_response[0].tap do |issue|
-          issue.delete(:id)
+        issues_sample_response[0...1].map do |issue|
+          issue.except(:id)
         end
       end
 
@@ -186,8 +187,8 @@ describe Sentry::Client do
 
     context 'essential keys missing in API response' do
       let(:sentry_api_response) do
-        [] << projects_sample_response[0].tap do |project|
-          project.delete(:slug)
+        projects_sample_response[0...1].map do |project|
+          project.except(:slug)
         end
       end
 
@@ -198,10 +199,10 @@ describe Sentry::Client do
 
     context 'optional keys missing in sentry response' do
       let(:sentry_api_response) do
-        [] << projects_sample_response[0].tap do |project|
-          project.delete(:id)
-          project.delete(:status)
+        projects_sample_response[0...1].map do |project|
           project[:organization].delete(:id)
+          project.delete(:id)
+          project.except(:status)
         end
       end