Migrate 10 more integrations to mock_http

[mwdd146980]

Motivation

Step 5 of the Test Decoupling Plan. Eliminate direct requests.Session.* patches in test files so tests are decoupled from the HTTP backend. Part of the httpx migration.

Approach

Per-integration migration to the mock_http fixture (introduced in #22676), one commit per integration. PR #22676 precedent followed for assertion rewrites. Drop boilerplate kwargs and assert only the contract being tested.

powerdns_recursor's test_metadata_unit was split into three focused tests with a _make_check() helper. The original sequenced three sub-cases sharing one mock_http.get with asymmetric reset_mock calls. That structure makes it hard to add new sub-cases without breaking existing ones. Migrating the file was a fitting cleanup window since every assertion line was already being touched.

For full details, see the PR 5.1 plan page.

Verification

• CI passes against mwdd146980/httpx-migration-base (this PR stacks on Feature branch for httpx migration #22676)
• Reviewer reads commit-by-commit (one integration per commit)
• Local: ddev test -fs <intg> + ddev --no-interactive test --recreate <intg> per integration

🤖 Generated with Claude Code

[datadog-prod-us1-5]

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 94.09% (+6.81%)

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 4a95df5 | Docs | Datadog PR Page | Give us feedback!}

[mwdd146980]

• Migrate 10 more integrations to mock_http #23536 👈 (View in Graphite)
• Remove requests.Session test patches from conftest.py files #23314
• Feature branch for httpx migration #22676 : 7 other dependent PRs (#22704 , #22710 , #22722 and 4 others)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (mwdd146980/httpx-migration-base@faddc6c). Learn more about missing BASE report.

Additional details and impacted files

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mwdd146980]

@codex review

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. Chef's kiss.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[dd-octo-sts]

Validation Report

All 20 validations passed.

Show details

Validation	Description	Status
agent-reqs	Verify check versions match the Agent requirements file	✅
ci	Validate CI configuration and Codecov settings	✅
codeowners	Validate every integration has a CODEOWNERS entry	✅
config	Validate default configuration files against spec.yaml	✅
dep	Verify dependency pins are consistent and Agent-compatible	✅
http	Validate integrations use the HTTP wrapper correctly	✅
imports	Validate check imports do not use deprecated modules	✅
integration-style	Validate check code style conventions	✅
jmx-metrics	Validate JMX metrics definition files and config	✅
labeler	Validate PR labeler config matches integration directories	✅
legacy-signature	Validate no integration uses the legacy Agent check signature	✅
license-headers	Validate Python files have proper license headers	✅
licenses	Validate third-party license attribution list	✅
metadata	Validate metadata.csv metric definitions	✅
models	Validate configuration data models match spec.yaml	✅
openmetrics	Validate OpenMetrics integrations disable the metric limit	✅
package	Validate Python package metadata and naming	✅
readmes	Validate README files have required sections	✅
saved-views	Validate saved view JSON file structure and fields	✅
version	Validate version consistency between package and changelog	✅

View full run