Skip to content

Conversation

@p-datadog
Copy link
Member

@p-datadog p-datadog commented Nov 24, 2025

What does this PR do?

Extracts a test helper to reduce duplication in DI tests

Motivation:

Less copy/paste

Change log entry
None

Additional Notes:

How to test the change?

Change is in tests only

@p-datadog p-datadog changed the title DI: extract instance_double_agent_settings_with_stubs DI: extract instance_double_agent_settings_with_stubs to DRY tests Nov 24, 2025
@github-actions github-actions bot added the dev/testing Involves testing processes (e.g. RSpec) label Nov 24, 2025
@github-actions
Copy link

github-actions bot commented Nov 24, 2025

Thank you for updating Change log entry section 👏

Visited at: 2025-11-24 18:00:59 UTC

@datadog-datadog-prod-us1
Copy link
Contributor

datadog-datadog-prod-us1 bot commented Nov 24, 2025

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 95.16% (-0.02%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 78eea14 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@p-datadog p-datadog marked this pull request as ready for review November 24, 2025 18:00
@p-datadog p-datadog requested a review from a team as a code owner November 24, 2025 18:00
Copy link
Member

@Strech Strech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's fine, but this could be completely avoided by doing the double differently

Co-authored-by: Sergey Fedorov <oni.strech@gmail.com>
@p-datadog p-datadog merged commit d134643 into master Nov 27, 2025
266 checks passed
@p-datadog p-datadog deleted the di-dry-tests branch November 27, 2025 14:45
@github-actions github-actions bot added this to the 2.23.0 milestone Nov 27, 2025
p-datadog pushed a commit that referenced this pull request Nov 27, 2025
* master:
  Transports: DRY HTTP client code (#5095)
  DI: extract instance_double_agent_settings_with_stubs to DRY tests (#5087)
  [PROF-13115] Fix profiler ractor specs failing on Ruby 4
  [PROF-13115] Bootstrap installing dependencies on Ruby 4.0.0-preview2
  Clarify support for `rb_obj_info` and why it's OK to not have it
  Tweak pending to not apply to all Ruby preview versions
  Do not try to use `rb_obj_info` on Ruby 4.0
  Adjust stack collector spec to account for changed Ruby 4 behavior
  [PROF-13115] Disable heap profiling on Ruby 4 preview due to incompatibility
  Stub sampling in integration tests
  Rewrite security response tests
  Bump the gh-actions-packages group across 3 directories with 13 updates
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.

4 participants