Fix lint and type-checker issues in OAuth2 implementation

- Remove unused imports (List, Optional, MagicMock)
- Fix type annotation: change 'any' to 'Any' in oauth2_token_manager.py
- All OAuth2 tests still pass after fixes

Co-Authored-By: João <joao@crewai.com>
This commit is contained in:
Devin AI
2025-07-07 18:01:35 +00:00
parent ac1080afd8
commit 03ee4c59eb
3 changed files with 4 additions and 4 deletions

View File

@@ -1,5 +1,5 @@
from pathlib import Path from pathlib import Path
from typing import Dict, List, Optional from typing import Dict, Optional
import json import json
from pydantic import BaseModel, Field from pydantic import BaseModel, Field

View File

@@ -1,6 +1,6 @@
import time import time
import requests import requests
from typing import Dict, Optional from typing import Dict, Any
from .oauth2_config import OAuth2Config from .oauth2_config import OAuth2Config
@@ -19,7 +19,7 @@ class OAuth2TokenManager:
return self._acquire_new_token(config) return self._acquire_new_token(config)
def _is_token_valid(self, token_data: Dict[str, any]) -> bool: def _is_token_valid(self, token_data: Dict[str, Any]) -> bool:
"""Check if token is still valid (not expired)""" """Check if token is still valid (not expired)"""
if "expires_at" not in token_data: if "expires_at" not in token_data:
return False return False

View File

@@ -4,7 +4,7 @@ import requests
import tempfile import tempfile
import time import time
from pathlib import Path from pathlib import Path
from unittest.mock import Mock, patch, MagicMock from unittest.mock import Mock, patch
from crewai.llm import LLM from crewai.llm import LLM
from crewai.llms.oauth2_config import OAuth2Config, OAuth2ConfigLoader from crewai.llms.oauth2_config import OAuth2Config, OAuth2ConfigLoader
from crewai.llms.oauth2_token_manager import OAuth2TokenManager from crewai.llms.oauth2_token_manager import OAuth2TokenManager