Skip to content

Commit 41cf788

Browse files
ocelotllzchen
andauthored
Remove [test] package from opentelemetry-sdk (#3735)
Fixes #3717 Co-authored-by: Leighton Chen <[email protected]>
1 parent 8ad10f7 commit 41cf788

File tree

3 files changed

+23
-4
lines changed

3 files changed

+23
-4
lines changed

Diff for: opentelemetry-sdk/pyproject.toml

-3
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,6 @@ dependencies = [
3030
"typing-extensions >= 3.7.4",
3131
]
3232

33-
[project.optional-dependencies]
34-
test = []
35-
3633
[project.entry-points.opentelemetry_environment_variables]
3734
sdk = "opentelemetry.sdk.environment_variables"
3835

Diff for: opentelemetry-sdk/test-requirements.txt

+19
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
asgiref==3.7.2
2+
attrs==23.2.0
3+
Deprecated==1.2.14
4+
flaky==3.7.0
5+
importlib-metadata==6.11.0
6+
iniconfig==2.0.0
7+
packaging==23.2
8+
pluggy==1.4.0
9+
py==1.11.0
10+
py-cpuinfo==9.0.0
11+
pytest==7.1.3
12+
pytest-benchmark==4.0.0
13+
tomli==2.0.1
14+
typing_extensions==4.10.0
15+
wrapt==1.16.0
16+
zipp==3.17.0
17+
-e opentelemetry-api
18+
-e opentelemetry-semantic-conventions
19+
-e opentelemetry-sdk

Diff for: tox.ini

+4-1
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,9 @@ setenv =
9999
commands_pre =
100100
; Install without -e to test the actual installation
101101
py3{8,9,10,11}: python -m pip install -U pip setuptools wheel
102+
103+
sdk: pip install -r {toxinidir}/opentelemetry-sdk/test-requirements.txt
104+
102105
; Install common packages for all the tests. These are not needed in all the
103106
; cases but it saves a lot of boilerplate in this file.
104107
opentelemetry: pip install {toxinidir}/opentelemetry-api {toxinidir}/opentelemetry-semantic-conventions {toxinidir}/opentelemetry-sdk {toxinidir}/tests/opentelemetry-test-utils
@@ -212,7 +215,7 @@ deps =
212215
commands_pre =
213216
python -m pip install -e {toxinidir}/opentelemetry-api[test]
214217
python -m pip install -e {toxinidir}/opentelemetry-semantic-conventions[test]
215-
python -m pip install -e {toxinidir}/opentelemetry-sdk[test]
218+
pip install -r {toxinidir}/opentelemetry-sdk/test-requirements.txt
216219
python -m pip install -e {toxinidir}/opentelemetry-proto[test]
217220
python -m pip install -e {toxinidir}/tests/opentelemetry-test-utils[test]
218221
python -m pip install -e {toxinidir}/shim/opentelemetry-opentracing-shim[test]

0 commit comments

Comments
 (0)