Remove requests.Session test patches from conftest.py files

[mwdd146980]

Motivation

Test fixtures that patch requests.Session directly will silently break when integrations switch from requests to httpx. The mock_http fixture intercepts HTTP at the AgentCheck.http level, making tests transport-agnostic.

Approach

• OM v2 / AgentCheck integrations: Replaced mocker.patch('requests.Session.get', ...) with mock_http fixture + MockHTTPResponse.
• OM v1 / Prometheus integrations: The v1 mixin's get_http_handler() constructs its own RequestsWrapper, bypassing self.http. Fixtures patch get_http_handler via mocker.patch to return mock_http. No production code changes. Shared fixtures mock_openmetrics_http / mock_prometheus_http extracted to the ddev pytest plugin.

Notable changes

• nutanix: Larger refactor than the other integrations. In conftest.py, mock_http_get was rewritten to dispatch URLs via MockHTTPResponse instead of building mocker.Mock(spec=Response) per branch.
• yarn test_ssl_verification: Old test asserted requests-library SSL behavior (raises SSLError on bad cert), which is already covered in datadog_checks_base/tests/. The mocked_bad_cert_request fixture was removed.
• haproxy fixture scope: Narrowed haproxy_mock, haproxy_mock_evil, and haproxy_mock_enterprise_version_info from scope="module" to function scope, since mock_http is function-scoped and pytest forbids broader-scoped fixtures from depending on narrower ones. These are mock-config fixtures only so runtime impact is negligible.

Verification

# All target integrations
for intg in temporal fluxcd dcgm impala nutanix yarn external_dns \
  datadog_cluster_agent scylla prometheus cilium gitlab_runner gitlab haproxy; do
  ddev test -fs $intg && ddev --no-interactive test $intg
done
# Regression (get_http_handler consumers)
for intg in kube_scheduler kube_controller_manager; do
  ddev --no-interactive test $intg
done

All target integrations pass. Cilium has 2 pre-existing legacy test failures (confirmed on clean branch).

[mwdd146980]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Migrate 10 more integrations to mock_http #23536
• Remove requests.Session test patches from conftest.py files #23314 👈 (View in Graphite)
• 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

❌ Patch coverage is 96.32353% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.90%. Comparing base (7e8968a) to head (8a139cc).
⚠️ Report is 1 commits behind head on mwdd146980/httpx-migration-base.

❗ There is a different number of reports uploaded between BASE (7e8968a) and HEAD (8a139cc). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (7e8968a)	HEAD (8a139cc)
	2	1

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.

[datadog-prod-us1-5]

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
• Patch Coverage: 96.32%
• Overall Coverage: 75.77% (-11.49%)

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8a139cc | Docs | Datadog PR Page | Give us 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