Skip to content

Commit 9945b4a

Browse files
committed
fix(tests): Clean up unused imports and variables flagged by Ruff
- darshan: Remove unused AsyncMock import - jarvis: Remove unused os, sys, MagicMock, AsyncMock, HTTPException imports - pandas: Remove unused json, datetime, timedelta, patch, MagicMock, numpy imports - parallel-sort: Remove unused filter_logs import - parquet: Remove unused json, os imports and fix unused variable in error handling test - All Ruff linting checks now pass
1 parent 661ec80 commit 9945b4a

File tree

14 files changed

+364
-15
lines changed

14 files changed

+364
-15
lines changed

agent-toolkit-mcp-servers/darshan/tests/test_darshan_parser_posix_mpiio.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
"""Tests for POSIX and MPI-IO operation parsing functions in Darshan parser."""
22

33
import pytest
4-
from unittest.mock import patch, AsyncMock
4+
from unittest.mock import patch
55
import os
66
import sys
77

agent-toolkit-mcp-servers/hdf5/uv.lock

Lines changed: 120 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

agent-toolkit-mcp-servers/jarvis/tests/test_server_direct.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,7 @@
44
"""
55

66
import pytest
7-
import os
8-
import sys
9-
from unittest.mock import Mock, patch, MagicMock, AsyncMock
10-
from fastapi import HTTPException
7+
from unittest.mock import Mock, patch
118

129

1310
class TestPipelineToolsDirect:

agent-toolkit-mcp-servers/pandas/tests/test_data_io.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
import tempfile
99
import os
1010
import sys
11-
import json
1211

1312
# Add the parent directory to Python path so we can import src
1413
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))

agent-toolkit-mcp-servers/pandas/tests/test_memory_optimization.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
import tempfile
99
import os
1010
import sys
11-
from unittest.mock import patch, MagicMock
1211

1312
# Add the parent directory to Python path
1413
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))

agent-toolkit-mcp-servers/pandas/tests/test_time_series.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
import tempfile
99
import os
1010
import sys
11-
from datetime import datetime, timedelta
1211

1312
# Add the parent directory to Python path
1413
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))

agent-toolkit-mcp-servers/pandas/tests/test_transformations.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
import pytest
66
import pandas as pd
7-
import numpy as np
87
import tempfile
98
import os
109
import sys

agent-toolkit-mcp-servers/parallel-sort/tests/test_edge_cases.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
import tempfile
77
import os
88
from implementation.export_handler import export_to_json, export_to_csv, export_to_text, export_summary_report
9-
from implementation.filter_handler import filter_logs, filter_by_time_range
9+
from implementation.filter_handler import filter_by_time_range
1010
from implementation.sort_handler import sort_log_by_timestamp
1111
from implementation.parallel_processor import parallel_sort_large_file
1212
from implementation.statistics_handler import analyze_log_statistics

agent-toolkit-mcp-servers/parallel-sort/uv.lock

Lines changed: 120 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

agent-toolkit-mcp-servers/parquet/tests/test_additional_edge_cases.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@ async def test_column_preview_max_items_negative(self, edge_test_file):
8989
async def test_column_preview_generic_exception(self, edge_test_file):
9090
"""Test column preview with corrupted file to trigger generic exception."""
9191
from parquet_mcp.capabilities.parquet_handler import get_column_preview
92-
import os
9392

9493
# Create a corrupted file
9594
corrupted_file = edge_test_file.replace('.parquet', '_corrupt.parquet')
@@ -148,7 +147,6 @@ async def test_aggregate_unknown_operation_result_none(self, edge_test_file):
148147
async def test_aggregate_generic_exception(self, edge_test_file):
149148
"""Test aggregate with corrupted file to trigger generic exception."""
150149
from parquet_mcp.capabilities.parquet_handler import aggregate_column
151-
import os
152150

153151
# Create a corrupted file
154152
corrupted_file = edge_test_file.replace('.parquet', '_agg_corrupt.parquet')

0 commit comments

Comments
 (0)