Skip to content

Commit 999113b

Browse files
ci: Remove responses from requirements-testing.txt (#6621)
Remove the package from `requirements-testing.txt` and add it only as a dependency of the test suites that rely on it. Add `fastapi[test]` as a dependency for `graphene` tests, to avoid unhandled `ImportError` exceptions that arose after moving the `responses` dependency. This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.
1 parent 44be095 commit 999113b

4 files changed

Lines changed: 10 additions & 3 deletions

File tree

requirements-testing.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,4 @@ pytest-cov
33
dataclasses; python_version < "3.7"
44
executing
55
asttokens
6-
responses
76
setuptools

scripts/populate_tox/config.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,9 @@
167167
},
168168
"gql": {
169169
"package": "gql[all]",
170+
"deps": {
171+
"*": ["responses"],
172+
},
170173
"num_versions": 2,
171174
},
172175
"google_genai": {
@@ -179,7 +182,7 @@
179182
"graphene": {
180183
"package": "graphene",
181184
"deps": {
182-
"*": ["blinker", "fastapi", "flask", "httpx"],
185+
"*": ["blinker", "fastapi[test]", "flask", "httpx"],
183186
"py3.6": ["aiocontextvars"],
184187
},
185188
},

scripts/populate_tox/tox.jinja

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,13 +137,15 @@ deps =
137137
aws_lambda: fastapi
138138
aws_lambda: requests
139139
aws_lambda: uvicorn
140+
aws_lambda: pyyaml
140141

141142
# OpenTelemetry (OTel)
142143
opentelemetry: opentelemetry-distro
143144
opentelemetry: pytest-forked
144145

145146
# OpenTelemetry with OTLP
146147
otlp: opentelemetry-distro[otlp]
148+
otlp: responses
147149

148150
# OpenTelemetry Experimental (POTel)
149151
potel: opentelemetry-distro

tox.ini

Lines changed: 4 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)