Skip to content

Conversation

@sohankshirsagar
Copy link
Contributor

@sohankshirsagar sohankshirsagar commented Jan 16, 2026

Note

Consolidates e2e test request logic to shared utilities.

  • Replace local helper implementations in aiohttp/e2e-tests/src/test_requests.py and urllib3/e2e-tests/src/test_requests.py with drift.instrumentation.e2e_common.test_utils.make_request
  • Standardize completion output by calling print_request_summary() instead of printing a static message
  • Remove per-file dependencies on requests, time, and inline BASE_URL handling

Written by Cursor Bugbot for commit ad39032. This will update automatically on new commits. Configure here.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@sohankshirsagar sohankshirsagar merged commit b4b4565 into main Jan 16, 2026
17 checks passed
@sohankshirsagar sohankshirsagar deleted the sohan/fix-outbound-http-e2e-tests branch January 16, 2026 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants