chore(ci_visibility): remove code (#12798)

CI Visibility: Remove code no longer used

## Checklist
- [x] PR author has checked that all the criteria below are met
- The PR description includes an overview of the change
- The PR description articulates the motivation for the change
- The change includes tests OR the PR description describes a testing
strategy
- The PR description notes risks associated with the change, if any
- Newly-added code is easy to change
- The change follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] Reviewer has checked that all the criteria below are met 
- Title is accurate
- All changes are related to the pull request's stated goal
- Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes
- Testing strategy adequately addresses listed risks
- Newly-added code is easy to change
- Release note makes sense to a user of the library
- If necessary, author has acknowledged and discussed the performance
implications of this PR as reported in the benchmarks PR comment
- Backport labels are set in a manner that is consistent with the
[release branch maintenance
policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Author: gnufede

ddtrace/internal/ci_visibility/recorder.py

@@ -32,7 +32,6 @@
 from ddtrace.internal import atexit
 from ddtrace.internal import core
 from ddtrace.internal import telemetry
-from ddtrace.internal.agent import get_connection
 from ddtrace.internal.ci_visibility._api_client import AgentlessTestVisibilityAPIClient
 from ddtrace.internal.ci_visibility._api_client import EarlyFlakeDetectionSettings
 from ddtrace.internal.ci_visibility._api_client import EVPProxyTestVisibilityAPIClient
@@ -84,8 +83,6 @@
 from ddtrace.internal.test_visibility.api import InternalTest
 from ddtrace.internal.test_visibility.coverage_lines import CoverageLines
 from ddtrace.internal.utils.formats import asbool
-from ddtrace.internal.utils.http import verify_url
-from ddtrace.internal.writer.writer import Response
 from ddtrace.settings import IntegrationConfig
 from ddtrace.trace import Span
 from ddtrace.trace import Tracer
@@ -144,23 +141,6 @@ def _get_custom_configurations() -> Dict[str, str]:
     return custom_configurations
 
 
-def _do_request(
-    method: str, url: str, payload: str, headers: Dict[str, str], timeout: int = DEFAULT_TIMEOUT
-) -> Response:
-    try:
-        parsed_url = verify_url(url)
-        url_path = parsed_url.path
-        conn = get_connection(url, timeout=timeout)
-        log.debug("Sending request: %s %s %s %s", method, url_path, payload, headers)
-        conn.request("POST", url_path, payload, headers)
-        resp = conn.getresponse()
-        log.debug("Response status: %s", resp.status)
-        result = Response.from_http_response(resp)
-    finally:
-        conn.close()
-    return result
-
-
 class CIVisibilityTracer(Tracer):
     def __init__(self, *args, **kwargs) -> None:
         # Allows for multiple instances of the civis tracer to be created without logging a warning

tests/ci_visibility/util.py

@@ -137,8 +137,6 @@ def _mock_upload_git_metadata(obj, **kwargs):
         _upload_packfiles=classmethod(lambda *args, **kwargs: None),
         upload_git_metadata=_mock_upload_git_metadata,
         _do_request=NotImplementedError,
-    ), mock.patch(
-        "ddtrace.internal.ci_visibility.recorder._do_request", side_effect=NotImplementedError
     ):
         yield