fix(starlette): updated snapshot tests with latest starlette version (#4238) (#4291)

## Description
New version 0.21.0 of Starlette Replaced requests with httpx in TestClient  https://www.starlette.io/release-notes/
CI fails due to one of the snapshot tests of starlette verify requests wasn't patched

## Reviewer Checklist
- [ ] Title is accurate.
- [ ] Description motivates each change.
- [ ] No unnecessary changes were introduced in this PR.
- [ ] PR cannot be broken up into smaller PRs.
- [ ] Avoid breaking [API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces) changes unless absolutely necessary.
- [ ] Tests provided or description of manual testing performed is included in the code or PR.
- [ ] Release note has been added for fixes and features, or else `changelog/no-changelog` label added.
- [ ] All relevant GitHub issues are correctly linked.
- [ ] Backports are identified and tagged with Mergifyio.
- [ ] Add to milestone.

(cherry picked from commit 8936d35)

Co-authored-by: Alberto Vara <[email protected]>

Author: mergify[bot]

tests/contrib/starlette/test_starlette.py

@@ -4,6 +4,7 @@
 import httpx
 import pytest
 import sqlalchemy
+import starlette
 from starlette.testclient import TestClient
 
 import ddtrace
@@ -21,6 +22,10 @@
 from tests.utils import snapshot
 
 
+starlette_version_str = getattr(starlette, "__version__", "0.0.0")
+starlette_version = tuple([int(i) for i in starlette_version_str.split(".")])
+
+
 @pytest.fixture
 def engine():
     sql_patch()
@@ -435,6 +440,9 @@ def test_table_query_snapshot(snapshot_client):
     assert r_get.text == "[{'id': 1, 'text': 'test', 'completed': 1}]"
 
 
+@pytest.mark.skipif(
+    starlette_version >= (0, 21, 0), reason="Starlette>=0.21.0 replaced requests with httpx in TestClient"
+)
 @snapshot()
 def test_incorrect_patching(run_python_code_in_subprocess):
     """
@@ -450,15 +458,16 @@ def test_incorrect_patching(run_python_code_in_subprocess):
 import sqlalchemy
 
 from ddtrace import patch_all
+
+
 from tests.contrib.starlette.app import get_app
 
 engine = sqlalchemy.create_engine("sqlite:///test.db")
 app = get_app(engine)
 
 # Calling patch_all late
 # DEV: The test client uses `requests` so we want to ignore them for this scenario
-patch_all(requests=False)
-
+patch_all(requests=False, http=False)
 with TestClient(app) as test_client:
     r = test_client.get("/200")