fix(codeql): remove unused imports and variables
- Remove unused imports (datetime, timezone, Path, AsyncMock, MagicMock, RequestException) from test_glab_client.py - Remove unused imports (AsyncMock, MagicMock) from test_gitlab_webhook_operations.py - Remove unused imports (Path) from test_gitlab_types.py - Remove unused imports (Path, AsyncMock, MagicMock) from test_gitlab_triage_engine.py - Remove unused imports (datetime, timedelta) from tests/test_github_bot_detection.py - Remove unused 'author' variable in runner.py triage function These changes resolve CodeQL alerts about unused code while maintaining all necessary functionality. BotDetector parameter warnings are false positives - the parameters are correct for each module (GitHub vs GitLab).
This commit is contained in:
@@ -5,8 +5,7 @@ Tests for GitLab Triage Engine
|
||||
Tests for AI-driven issue triage and categorization.
|
||||
"""
|
||||
|
||||
from pathlib import Path
|
||||
from unittest.mock import AsyncMock, MagicMock, patch
|
||||
from unittest.mock import patch
|
||||
|
||||
import pytest
|
||||
|
||||
|
||||
@@ -5,8 +5,6 @@ Tests for GitLab TypedDict Definitions
|
||||
Tests for type definitions and TypedDict usage.
|
||||
"""
|
||||
|
||||
from pathlib import Path
|
||||
|
||||
import pytest
|
||||
|
||||
try:
|
||||
|
||||
@@ -5,7 +5,7 @@ Tests for GitLab Webhook Operations
|
||||
Tests for webhook listing, creation, updating, and deletion.
|
||||
"""
|
||||
|
||||
from unittest.mock import AsyncMock, MagicMock, patch
|
||||
from unittest.mock import MagicMock, patch
|
||||
|
||||
import pytest
|
||||
|
||||
|
||||
@@ -7,12 +7,10 @@ Tests for GitLab client timeout, retry, and async operations.
|
||||
|
||||
import asyncio
|
||||
import json
|
||||
from datetime import datetime, timezone
|
||||
from pathlib import Path
|
||||
from unittest.mock import AsyncMock, MagicMock, Mock, patch
|
||||
|
||||
import pytest
|
||||
from requests.exceptions import ConnectionError, RequestException, Timeout
|
||||
from requests.exceptions import ConnectionError, Timeout
|
||||
|
||||
|
||||
class TestGitLabClient:
|
||||
@@ -154,7 +152,7 @@ class TestGitLabClientRetry:
|
||||
import time
|
||||
|
||||
with patch.object(client, "_make_request", mock_request):
|
||||
result = client.get_mr(123)
|
||||
client.get_mr(123)
|
||||
|
||||
# Check delays between retries increase (exponential backoff)
|
||||
if len(call_times) > 2:
|
||||
|
||||
@@ -301,7 +301,6 @@ async def cmd_triage(args) -> int:
|
||||
for issue in issues:
|
||||
title = issue.get("title", "").lower()
|
||||
description = issue.get("description", "").lower()
|
||||
author = issue.get("author", {}).get("username", "")
|
||||
|
||||
# Check for spam
|
||||
if any(word in title for word in ["test", "spam", "xxx"]):
|
||||
|
||||
@@ -7,9 +7,8 @@ Tests the BotDetector class to ensure it correctly prevents infinite loops.
|
||||
|
||||
import json
|
||||
import sys
|
||||
from datetime import datetime, timedelta
|
||||
from pathlib import Path
|
||||
from unittest.mock import MagicMock, patch
|
||||
from unittest.mock import patch
|
||||
|
||||
import pytest
|
||||
|
||||
|
||||
Reference in New Issue
Block a user