andre/crewAI
1
0
Fork 0
mirror of https://github.com/crewAIInc/crewAI.git synced 2026-01-08 15:48:29 +00:00
Code Issues Actions 4 Packages Projects Releases Wiki Activity

fix: resolve lint issues in dependency compatibility tests

Browse Source 
- Remove unused imports: pathlib.Path, opentelemetry modules
- Remove unused variable assignment for context
- All ruff checks now pass locally

Co-Authored-By: João <joao@crewai.com>
This commit is contained in:
Devin AI
2025-08-27 19:31:32 +00:00
parent 36de68ecd4
commit b42d2e8cf0
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/test_dependency_compatibility.py
View File

@@ -8,7 +8,6 @@ import subprocess
import sys import sys
import tempfile import tempfile
import pytest import pytest
from pathlib import Path
class TestDependencyCompatibility: class TestDependencyCompatibility:
@@ -17,11 +16,7 @@ class TestDependencyCompatibility:
def test_opentelemetry_protobuf_compatibility(self): def test_opentelemetry_protobuf_compatibility(self):
"""Test that opentelemetry versions work with protobuf<5.0.""" """Test that opentelemetry versions work with protobuf<5.0."""
try: try:
import opentelemetry
from opentelemetry import trace, baggage
from opentelemetry.context import attach, detach
from opentelemetry.sdk.trace import TracerProvider from opentelemetry.sdk.trace import TracerProvider
from opentelemetry.exporter.otlp.proto.http.trace_exporter import OTLPSpanExporter
tracer_provider = TracerProvider() tracer_provider = TracerProvider()
tracer = tracer_provider.get_tracer("test") tracer = tracer_provider.get_tracer("test")
@@ -54,7 +49,7 @@ class TestDependencyCompatibility:
telemetry = Telemetry() telemetry = Telemetry()
assert telemetry is not None assert telemetry is not None
context = get_crew_context() get_crew_context()
except ImportError as e: except ImportError as e:
pytest.fail(f"Failed to import CrewAI telemetry modules: {e}") pytest.fail(f"Failed to import CrewAI telemetry modules: {e}")