Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DEBUG-2334 remove unused method stubs from DI test suite #4107

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

p-datadog
Copy link
Contributor

What does this PR do?
Removes allow calls from test suite for the relocated propagate_all_exceptions setting.

Motivation:
Test suite cleanup.

Change log entry
None

Additional Notes:

How to test the change?

@p-datadog p-datadog requested a review from a team as a code owner November 13, 2024 02:54
@github-actions github-actions bot added the dev/testing Involves testing processes (e.g. RSpec) label Nov 13, 2024
@pr-commenter
Copy link

pr-commenter bot commented Nov 13, 2024

Benchmarks

Benchmark execution time: 2024-11-13 03:34:38

Comparing candidate commit 39da4d8 in PR branch di-unused-stubs with baseline commit 48e5f09 in branch master.

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

scenario:profiler - sample timeline=false

  • 🟥 throughput [-0.591op/s; -0.582op/s] or [-8.637%; -8.507%]

@p-datadog p-datadog merged commit 34d1947 into master Nov 13, 2024
264 of 266 checks passed
@p-datadog p-datadog deleted the di-unused-stubs branch November 13, 2024 18:41
@github-actions github-actions bot added this to the 2.8.0 milestone Nov 13, 2024
p-datadog pushed a commit that referenced this pull request Nov 15, 2024
* origin/di-virtual: (30 commits)
  DEBUG-2334 Ruby 2 compatibility for dynamic instrumentation (#4120)
  Fix CI flakiness by adding dockerhub login
  Update filter with splitted  gemfile and task
  [🤖] Lock Dependency: https://github.com/DataDog/dd-trace-rb/actions/runs/11846139851
  Lock sqlite3
  Lock ActiveRecord
  Fix deprecation behavior
  Fix trilogy/patcher_spec
  Update 3.4_relational_db.gemfile
  Update Ruby 3.4 Gemfile
  Add 3.4 to circleci
  DEBUG-2334 Use :return/:b_return trace points to target 'end' line of methods and blocks with line probes (#4109)
  Temporarily disable asan ruby job
  Update developement guide
  Lint
  Fix release_gem_spec.rb
  Gemfile migration
  Update usage
  Appraisal group generation replacement
  DEBUG-2334 remove unused method stubs from DI test suite (#4107)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/testing Involves testing processes (e.g. RSpec)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants