Remove [test] package from jinja2 instrumentation

Fixes #2204
open-telemetry · Feb 24, 2024 · 8904a20 · 8904a20
1 parent 8daa8ad
commit 8904a20
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 7 deletions.
diff --git a/instrumentation/opentelemetry-instrumentation-jinja2/pyproject.toml b/instrumentation/opentelemetry-instrumentation-jinja2/pyproject.toml
@@ -33,11 +33,6 @@ dependencies = [
 instruments = [
   "jinja2 >= 2.7, < 4.0",
 ]
-test = [
-  "opentelemetry-instrumentation-jinja2[instruments]",
-  "markupsafe==2.0.1",
-  "opentelemetry-test-utils == 0.45b0.dev",
-]
 
 [project.entry-points.opentelemetry_instrumentor]
 jinja2 = "opentelemetry.instrumentation.jinja2:Jinja2Instrumentor"

diff --git a/instrumentation/opentelemetry-instrumentation-jinja2/test-requirements.txt b/instrumentation/opentelemetry-instrumentation-jinja2/test-requirements.txt
@@ -0,0 +1,19 @@
+asgiref==3.7.2
+attrs==23.2.0
+Deprecated==1.2.14
+importlib-metadata==6.11.0
+iniconfig==2.0.0
+Jinja2==3.1.3
+MarkupSafe==2.0.1
+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.9.0
+wrapt==1.16.0
+zipp==3.17.0
+-e ../../../opentelemetry-instrumentation
+-e ../../../instrumentation/opentelemetry-instrumentation-jinja2
diff --git a/tox.ini b/tox.ini
@@ -454,7 +454,7 @@ commands_pre =
 
   tortoiseorm: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-tortoiseorm[test]
 
-  jinja2: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-jinja2[test]
+  jinja2: pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-jinja2/test-requirements.txt
 
   logging: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-logging[test]
 
@@ -557,7 +557,7 @@ commands_pre =
   python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-redis[test]
   python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-remoulade[test]
   python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-fastapi[test]
-  python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-jinja2[test]
+  python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-jinja2
   python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-kafka-python[test]
   python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-confluent-kafka[test]
   python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-logging[test]