Fix specs after rebase

getsentry · Oct 31, 2024 · 98355b1 · 98355b1
1 parent d7522f9
commit 98355b1
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 13 deletions.
diff --git a/sentry-rails/spec/sentry/rails/tracing/active_support_subscriber_spec.rb b/sentry-rails/spec/sentry/rails/tracing/active_support_subscriber_spec.rb
@@ -23,7 +23,7 @@
       transaction.finish
 
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
 
       expect(cache_transaction[:spans].count).to eq(1)
@@ -45,7 +45,7 @@
 
       expect(Rails.cache.read("my_cache_key")).to eq(1)
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(1)
       expect(cache_transaction[:spans][0][:op]).to eq("cache.put")
@@ -64,7 +64,7 @@
       transaction.finish
 
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(1)
       expect(cache_transaction[:spans][0][:op]).to eq("cache.put")
@@ -79,7 +79,7 @@
       transaction.finish
 
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(1)
       expect(cache_transaction[:spans][0][:op]).to eq("cache.get")
@@ -95,7 +95,7 @@
       transaction.finish
 
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(1)
       expect(cache_transaction[:spans][0][:op]).to eq("cache.get")
@@ -112,7 +112,7 @@
       transaction.finish
 
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(2)
       expect(cache_transaction[:spans][1][:op]).to eq("cache.flush")
@@ -130,7 +130,7 @@
 
       transaction.finish
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(2)
       expect(cache_transaction[:spans][0][:op]).to eq("cache.get")
@@ -152,7 +152,7 @@
 
       transaction.finish
       expect(transport.events.count).to eq(1)
-      cache_transaction = transport.events.first.to_hash
+      cache_transaction = transport.events.first.to_h
       expect(cache_transaction[:type]).to eq("transaction")
       expect(cache_transaction[:spans].count).to eq(1)
       expect(cache_transaction[:spans][0][:op]).to eq("cache.remove")

diff --git a/sentry-rails/spec/sentry/rails/tracing_spec.rb b/sentry-rails/spec/sentry/rails/tracing_spec.rb
@@ -117,7 +117,7 @@
 
       it "does not record sensitive params" do
         get "/posts?foo=bar&password=42&secret=baz"
-        transaction = transport.events.last.to_hash
+        transaction = transport.events.last.to_h
 
         params = transaction[:spans][0][:data][:params]
         expect(params["foo"]).to eq("bar")
@@ -139,7 +139,7 @@
 
       it "records all params" do
         get "/posts?foo=bar&password=42&secret=baz"
-        transaction = transport.events.last.to_hash
+        transaction = transport.events.last.to_h
 
         params = transaction[:spans][0][:data][:params]
         expect(params["foo"]).to eq("bar")

diff --git a/sentry-ruby/lib/sentry/rspec.rb b/sentry-ruby/lib/sentry/rspec.rb
@@ -70,7 +70,7 @@ def find_matched_event(event_message, sentry_events)
   end
 
   def dump_events(sentry_events)
-    sentry_events.map(&Kernel.method(:Hash)).map do |hash|
+    sentry_events.map(&:to_h).map do |hash|
       hash.select { |k, _| [:message, :contexts, :tags, :exception].include?(k) }
     end.map do |hash|
       JSON.pretty_generate(hash)

diff --git a/sentry-ruby/spec/sentry/event_spec.rb b/sentry-ruby/spec/sentry/event_spec.rb
@@ -146,7 +146,7 @@
       it "doesn't overwrite already set ip address" do
         Sentry.set_user({ ip_address: "3.3.3.3" })
         Sentry.get_current_scope.apply_to_event(event)
-        expect(event.to_hash[:user][:ip_address]).to eq("3.3.3.3")
+        expect(event.to_h[:user][:ip_address]).to eq("3.3.3.3")
       end
 
       context "with config.trusted_proxies = [\"2.2.2.2\"]" do

diff --git a/sentry-ruby/spec/sentry/profiler_spec.rb b/sentry-ruby/spec/sentry/profiler_spec.rb
@@ -181,7 +181,7 @@
       end
 
       it 'returns empty' do
-        expect(subject.to_hash).to eq({})
+        expect(subject.to_h).to eq({})
       end
 
       it 'records lost event' do