test: set OTEL_ env vars correctly for tests (strands-agents#169)

awsarron · web-flow · commit da55dc859025 · 2025-06-03T11:04:29.000-04:00
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -24,6 +24,8 @@ def moto_env(monkeypatch):
     monkeypatch.setenv("AWS_SECRET_ACCESS_KEY", "test")
     monkeypatch.setenv("AWS_SECURITY_TOKEN", "test")
     monkeypatch.setenv("AWS_DEFAULT_REGION", "us-west-2")
+    monkeypatch.delenv("OTEL_EXPORTER_OTLP_ENDPOINT", raising=False)
+    monkeypatch.delenv("OTEL_EXPORTER_OTLP_HEADERS", raising=False)
 
 
 @pytest.fixture
diff --git a/tests/strands/tools/test_executor.py b/tests/strands/tools/test_executor.py
@@ -1,6 +1,7 @@
 import concurrent
 import functools
 import unittest.mock
+import uuid
 
 import pytest
 
@@ -9,6 +10,11 @@
 from strands.types.content import Message
 
 
+@pytest.fixture(autouse=True)
+def moto_autouse(moto_env):
+    _ = moto_env
+
+
 @pytest.fixture
 def tool_handler(request):
     def handler(tool_use):
@@ -52,10 +58,10 @@ def invalid_tool_use_ids(request):
     return request.param if hasattr(request, "param") else []
 
 
-@unittest.mock.patch.object(strands.telemetry.metrics, "uuid4", return_value="trace1")
 @pytest.fixture
 def cycle_trace():
-    return strands.telemetry.metrics.Trace(name="test trace", raw_name="raw_name")
+    with unittest.mock.patch.object(uuid, "uuid4", return_value="trace1"):
+        return strands.telemetry.metrics.Trace(name="test trace", raw_name="raw_name")
 
 
 @pytest.fixture
