andre/crewAI
1
0
Fork 0
mirror of https://github.com/crewAIInc/crewAI.git synced 2026-01-23 15:18:14 +00:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

fix: resolve lint issues in human input event streaming implementation

Browse Source 
- Remove unused imports from server modules and test files
- Fix undefined variable references in test assertions
- All ruff checks now pass locally

Co-Authored-By: João <joao@crewai.com>
This commit is contained in:
Devin AI
2025-08-02 16:10:35 +00:00
parent ea560d0af1
commit f133d13a00
5 changed files with 5 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/crewai/server/event_stream_manager.py
View File

@@ -2,7 +2,6 @@ import asyncio
import json import json
import uuid import uuid
from typing import Dict, List, Optional, Set from typing import Dict, List, Optional, Set
from datetime import datetime, timezone
from crewai.utilities.events.crewai_event_bus import crewai_event_bus from crewai.utilities.events.crewai_event_bus import crewai_event_bus
from crewai.utilities.events.task_events import HumanInputRequiredEvent, HumanInputCompletedEvent from crewai.utilities.events.task_events import HumanInputRequiredEvent, HumanInputCompletedEvent

4
src/crewai/server/human_input_server.py
View File

@@ -1,8 +1,6 @@
import asyncio import asyncio
import json import json
import queue from typing import Optional
import uuid
from typing import Optional, Dict, Any
from datetime import datetime, timezone from datetime import datetime, timezone
try: try:

5
tests/server/test_event_stream_manager.py
View File

@@ -1,10 +1,7 @@
import pytest
import asyncio
import json
from unittest.mock import MagicMock, patch from unittest.mock import MagicMock, patch
from crewai.server.event_stream_manager import EventStreamManager from crewai.server.event_stream_manager import EventStreamManager
from crewai.utilities.events.task_events import HumanInputRequiredEvent, HumanInputCompletedEvent from crewai.utilities.events.task_events import HumanInputRequiredEvent
class TestEventStreamManager: class TestEventStreamManager:

3
tests/server/test_human_input_server.py
View File

@@ -1,7 +1,4 @@
import pytest import pytest
import asyncio
import json
from unittest.mock import patch, MagicMock
try: try:
from fastapi.testclient import TestClient from fastapi.testclient import TestClient

6
tests/test_human_input_event_integration.py
View File

@@ -167,7 +167,7 @@ class TestHumanInputEventIntegration:
with patch.object(crewai_event_bus, 'emit', side_effect=capture_event), \ with patch.object(crewai_event_bus, 'emit', side_effect=capture_event), \
patch('builtins.input', return_value='test'): patch('builtins.input', return_value='test'):
result = self.executor._ask_human_input("Test result") self.executor._ask_human_input("Test result")
assert len(events_captured) == 2 assert len(events_captured) == 2
required_event = events_captured[0][1] required_event = events_captured[0][1]
@@ -185,7 +185,7 @@ class TestHumanInputEventIntegration:
with patch.object(crewai_event_bus, 'emit', side_effect=capture_event), \ with patch.object(crewai_event_bus, 'emit', side_effect=capture_event), \
patch('builtins.input', return_value='test'): patch('builtins.input', return_value='test'):
result = self.executor._ask_human_input("Test result") self.executor._ask_human_input("Test result")
assert len(events_captured) == 2 assert len(events_captured) == 2
required_event = events_captured[0][1] required_event = events_captured[0][1]
@@ -202,7 +202,7 @@ class TestHumanInputEventIntegration:
with patch.object(crewai_event_bus, 'emit', side_effect=capture_event), \ with patch.object(crewai_event_bus, 'emit', side_effect=capture_event), \
patch('builtins.input', return_value='test'): patch('builtins.input', return_value='test'):
result = self.executor._ask_human_input("Test result") self.executor._ask_human_input("Test result")
assert len(events_captured) == 2 assert len(events_captured) == 2
required_event = events_captured[0][1] required_event = events_captured[0][1]