fix(elasticsearch): Hook not being removed

[PROFeNoM]

Description

As mentioned in this GH comment from #2427, an elasticsearch hook wasn't removed

[ddtrace] [error] Could not add hook to Elastic\Elasticsearch\Endpoints\Indices::__construct with more than datadog.trace.hook_limit = 100 installed hooks in /opt/datadog-php/dd-trace-sources/bridge/_generated_integrations.php on line 5437; This message is only displayed once. Specify DD_TRACE_ONCE_LOGS=0 to show all messages.

dd_untrace was called on the Trait while it should have been called on the hooked class. With this PR, the installed hook is removed on the correct class.

Reviewer checklist

• Test coverage seems ok.
• Appropriate labels assigned.

[codecov-commenter]

Codecov Report

Merging #2615 (da4b652) into master (0d08bf6) will decrease coverage by 12.15%.
The diff coverage is 100.00%.

Additional details and impacted files

@@              Coverage Diff              @@
##             master    #2615       +/-   ##
=============================================
- Coverage     76.53%   64.39%   -12.15%     
  Complexity     2609     2609               
=============================================
  Files           217      217               
  Lines         23318    23318               
=============================================
- Hits          17846    15015     -2831     
- Misses         5472     8303     +2831     

Flag	Coverage Δ
tracer-extension	78.70% <ø> (ø)
tracer-php	46.83% <100.00%> (-27.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...ions/ElasticSearch/V8/ElasticSearchIntegration.php	99.13% <100.00%> (ø)

... and 19 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d08bf6...da4b652. Read the comment docs.

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-04-05 13:59:35

Comparing candidate commit da4b652 in PR branch alex/issue/gh2427-fourth with baseline commit 0d08bf6 in branch master.

Found 1 performance improvements and 1 performance regressions! Performance is the same for 180 metrics, 0 unstable metrics.

scenario:ComposerTelemetryBench/benchTelemetryParsing

• 🟥 execution_time [+1.153µs; +2.847µs] or [+2.112%; +5.214%]

scenario:SamplingRuleMatchingBench/benchRegexMatching4

• 🟩 execution_time [-354.047ns; -142.153ns] or [-5.576%; -2.239%]

[PROFeNoM]

Superseded by #2616