Remove [test] package from opentelemetry-exporter-zipkin-proto-http

Fixes open-telemetry#3726
ocelotl · Mar 6, 2024 · 4dd5d6e · 4dd5d6e
1 parent 216411f
commit 4dd5d6e
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 6 deletions.
diff --git a/exporter/opentelemetry-exporter-zipkin-proto-http/pyproject.toml b/exporter/opentelemetry-exporter-zipkin-proto-http/pyproject.toml
@@ -32,9 +32,6 @@ dependencies = [
   "requests ~= 2.7",
 ]
 
-[project.optional-dependencies]
-test = []
-
 [project.entry-points.opentelemetry_traces_exporter]
 zipkin_proto = "opentelemetry.exporter.zipkin.proto.http:ZipkinExporter"
 

diff --git a/exporter/opentelemetry-exporter-zipkin-proto-http/test-requirements.txt b/exporter/opentelemetry-exporter-zipkin-proto-http/test-requirements.txt
@@ -0,0 +1,24 @@
+asgiref==3.7.2
+attrs==23.2.0
+certifi==2024.2.2
+charset-normalizer==3.3.2
+Deprecated==1.2.14
+flaky==3.7.0
+idna==3.6
+importlib-metadata==6.11.0
+iniconfig==2.0.0
+packaging==23.2
+pluggy==1.4.0
+protobuf==3.20.3
+py==1.11.0
+py-cpuinfo==9.0.0
+pytest==7.1.3
+pytest-benchmark==4.0.0
+requests==2.31.0
+tomli==2.0.1
+typing_extensions==4.10.0
+urllib3==2.2.1
+wrapt==1.16.0
+zipp==3.17.0
+-e opentelemetry-exporter-zipkin-json
+-e opentelemetry-exporter-zipkin-proto-http
diff --git a/tox.ini b/tox.ini
@@ -143,8 +143,7 @@ commands_pre =
   exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http
   exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin
 
-  exporter-zipkin-proto-http: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-json
-  exporter-zipkin-proto-http: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http
+  exporter-zipkin-proto-http: pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http/test-requirements.txt
 
   exporter-zipkin-json: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-json
 
@@ -224,7 +223,7 @@ commands_pre =
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-otlp[test]
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-prometheus[test]
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-zipkin-json[test]
-  python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http[test]
+  pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http/test-requirements.txt
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-zipkin[test]
   python -m pip install -e {toxinidir}/propagator/opentelemetry-propagator-b3[test]
   python -m pip install -e {toxinidir}/propagator/opentelemetry-propagator-jaeger[test]