Enable automatic graphql tracing with a new :graphql patch

[sl0thentr0py]

closes #2091

[codecov]

Codecov Report

Attention: Patch coverage is 80.85106% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 98.66%. Comparing base (d0c9897) to head (7f8dcdd).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2308      +/-   ##
==========================================
- Coverage   98.73%   98.66%   -0.08%     
==========================================
  Files         203      205       +2     
  Lines       13282    13329      +47     
==========================================
+ Hits        13114    13151      +37     
- Misses        168      178      +10     

Components	Coverage Δ
sentry-ruby	99.03% <80.85%> (-0.09%)	⬇️
sentry-rails	97.37% <ø> (ø)
sentry-sidekiq	96.98% <ø> (ø)
sentry-resque	96.76% <ø> (-0.33%)	⬇️
sentry-delayed_job	98.91% <ø> (ø)
sentry-opentelemetry	100.00% <ø> (ø)

Files	Coverage Δ
sentry-ruby/lib/sentry-ruby.rb	96.60% <100.00%> (+0.01%)	⬆️
sentry-ruby/lib/sentry/graphql.rb	75.00% <75.00%> (ø)
sentry-ruby/spec/sentry/graphql_spec.rb	80.95% <80.95%> (ø)

... and 3 files with indirect coverage changes

[sl0thentr0py]

ignore the coverage report here because the uncovered lines are covered in older ruby versions which don't report their coverage

[adinauer]

LGTM, maybe we should consider requiring a higher version of graphql since there seems to have been a bug that was fixed in v2.2.8?

[sl0thentr0py]

the gem requirement is just for testing, we only have the error message for better DX

[adinauer]

I assume it won't explode if set_transaction_name isn't a known param yet for old versions, right? Seems to have been added in https://github.com/rmosolgo/graphql-ruby/pull/4853/files which was released in v2.2.11.

[sl0thentr0py]

yes its a keyword arg so its safe