Remove [test] package from opentelemetry-exporter-zipkin

Fixes open-telemetry#3720
ocelotl · Mar 9, 2024 · 0c8cf8d · 0c8cf8d
1 parent 41cf788
commit 0c8cf8d
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 7 deletions.
diff --git a/exporter/opentelemetry-exporter-zipkin/pyproject.toml b/exporter/opentelemetry-exporter-zipkin/pyproject.toml
@@ -29,9 +29,6 @@ dependencies = [
   "opentelemetry-exporter-zipkin-proto-http == 1.24.0.dev",
 ]
 
-[project.optional-dependencies]
-test = []
-
 [project.entry-points.opentelemetry_traces_exporter]
 zipkin = "opentelemetry.exporter.zipkin.proto.http:ZipkinExporter"
 

diff --git a/exporter/opentelemetry-exporter-zipkin/test-requirements.txt b/exporter/opentelemetry-exporter-zipkin/test-requirements.txt
@@ -0,0 +1,22 @@
+asgiref==3.7.2
+attrs==23.2.0
+Deprecated==1.2.14
+flaky==3.7.0
+importlib-metadata==6.11.0
+iniconfig==2.0.0
+packaging==23.2
+pluggy==1.4.0
+py==1.11.0
+py-cpuinfo==9.0.0
+pytest==7.1.3
+pytest-benchmark==4.0.0
+tomli==2.0.1
+typing_extensions==4.10.0
+wrapt==1.16.0
+zipp==3.17.0
+-e opentelemetry-api
+-e exporter/opentelemetry-exporter-zipkin-json
+-e exporter/opentelemetry-exporter-zipkin-proto-http
+-e opentelemetry-sdk
+-e opentelemetry-semantic-conventions
+-e exporter/opentelemetry-exporter-zipkin
diff --git a/tox.ini b/tox.ini
@@ -142,9 +142,7 @@ commands_pre =
 
   exporter-prometheus: pip install {toxinidir}/exporter/opentelemetry-exporter-prometheus
 
-  exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-json
-  exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http
-  exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin
+  exporter-zipkin-combined: pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin/test-requirements.txt
 
   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
@@ -228,7 +226,7 @@ commands_pre =
   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]
-  python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-zipkin[test]
+  pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin/test-requirements.txt
   python -m pip install -e {toxinidir}/propagator/opentelemetry-propagator-b3[test]
   python -m pip install -e {toxinidir}/propagator/opentelemetry-propagator-jaeger[test]
   # Pin protobuf version due to lint failing on v3.20.0