chore(ci_visibility): add unique tests request to test vis API client

[romainkomorndatadog]

This adds the unique tests request (self.fetch_unique_tests()) to the API client to support Early Flake Detection work.

A refactor wraps the client's _do_request() method is also wrapped in _do_request_with_telemetry() to DRY the three _do_request() callers. The telemetry calls are also refactored into a api_request telemetry module.

This also fixes the test verifying the skippable API request payload which had been shamefully copy/pasted and was actually not testing what it was supposed to.

Checklist

• 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
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• 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 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

[github-actions]

CODEOWNERS have been resolved as:

ddtrace/internal/ci_visibility/telemetry/api_request.py                 @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/telemetry/early_flake_detection.py       @DataDog/ci-app-libraries
tests/ci_visibility/api_client/test_ci_visibility_api_client_unique_tests_responses.py  @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/_api_client.py                           @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/constants.py                             @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/recorder.py                              @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/telemetry/git.py                         @DataDog/ci-app-libraries
ddtrace/internal/ci_visibility/telemetry/itr.py                         @DataDog/ci-app-libraries
tests/ci_visibility/api_client/_util.py                                 @DataDog/ci-app-libraries
tests/ci_visibility/api_client/test_ci_visibility_api_client.py         @DataDog/ci-app-libraries
tests/ci_visibility/api_client/test_ci_visibility_api_client_setting_responses.py  @DataDog/ci-app-libraries
tests/ci_visibility/test_ci_visibility.py                               @DataDog/ci-app-libraries

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-09-27 15:17:21

Comparing candidate commit e5fc750 in PR branch romain.komorn/SDTEST-906/add_new_tests_to_test_vis_api_client with baseline commit 0e1f4eb in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 371 metrics, 53 unstable metrics.