Remove uses of with_indifferent_access

The tests are fine without them.
buildkite · Feb 15, 2024 · d3246b9 · d3246b9
1 parent 79fa893
commit d3246b9
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 4 deletions.
diff --git a/lib/buildkite/test_collector.rb b/lib/buildkite/test_collector.rb
@@ -12,7 +12,6 @@ module TestCollector
 require "timeout"
 require "tmpdir"
 
-require "active_support/core_ext/hash/indifferent_access"
 require "active_support/notifications"
 
 require_relative "test_collector/version"

diff --git a/lib/buildkite/test_collector/minitest_plugin/trace.rb b/lib/buildkite/test_collector/minitest_plugin/trace.rb
@@ -38,7 +38,7 @@ def as_hash
         failure_reason: failure_reason,
         failure_expanded: failure_expanded,
         history: history,
-      ).with_indifferent_access.select { |_, value| !value.nil? }
+      ).select { |_, value| !value.nil? }
     end
 
     private

diff --git a/lib/buildkite/test_collector/rspec_plugin/trace.rb b/lib/buildkite/test_collector/rspec_plugin/trace.rb
@@ -32,7 +32,7 @@ def as_hash
         failure_reason: failure_reason,
         failure_expanded: failure_expanded,
         history: history,
-      ).with_indifferent_access.select { |_, value| !value.nil? }
+      ).select { |_, value| !value.nil? }
     end
 
     private

diff --git a/lib/buildkite/test_collector/tracer.rb b/lib/buildkite/test_collector/tracer.rb
@@ -43,7 +43,7 @@ def as_hash
           duration: end_at - start_at,
           detail: detail,
           children: children.map(&:as_hash),
-        }.with_indifferent_access
+        }
       end
     end