Add tests, make README.MD similar to other projects, remove safety_check

Author: tishun

.github/workflows/ci.yml

@@ -66,14 +66,12 @@ jobs:
       run: |
         uv python install 3.12
         uv sync --all-extras --dev
-        uv add --dev bandit safety
+        uv add --dev bandit
 
     - name: ⚙️ Run security scan with bandit
       run: |
         uv run bandit -r src/ -f json -o bandit-report.json || true
         uv run bandit -r src/
-        uv run safety check --output json > safety-report.json || true
-        uv run safety check
 
     - name: ⚙️ Upload security reports
       uses: actions/upload-artifact@v4
@@ -82,7 +80,6 @@ jobs:
         name: security-reports
         path: |
           bandit-report.json
-          safety-report.json
         retention-days: 30
 
 

.github/workflows/release.yml

@@ -55,12 +55,11 @@ jobs:
       run: |
         uv python install 3.12
         uv sync --all-extras --dev
-        uv add --dev bandit safety
+        uv add --dev bandit
 
     - name: ⚙️ Run security scan with bandit
       run: |
         uv run bandit -r src/
-        uv run safety check
 
   test:
     runs-on: ubuntu-latest

README.md

@@ -1,8 +1,17 @@
 # Redis MCP Server
-[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
+[![Integration](https://github.com/redis/mcp-redis/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/redis/lettuce/actions/workflows/integration.yml)
 [![Python Version](https://img.shields.io/badge/python-3.13%2B-blue)](https://www.python.org/downloads/)
+[![MIT licensed](https://img.shields.io/badge/license-MIT-blue.svg)](./LICENSE.txt)
 [![smithery badge](https://smithery.ai/badge/@redis/mcp-redis)](https://smithery.ai/server/@redis/mcp-redis)
 [![Verified on MseeP](https://mseep.ai/badge.svg)](https://mseep.ai/app/70102150-efe0-4705-9f7d-87980109a279)
+[![codecov](https://codecov.io/gh/redis/mcp-redis/branch/master/graph/badge.svg?token=yenl5fzxxr)](https://codecov.io/gh/redis/mcp-redis)
+
+
+[![Discord](https://img.shields.io/discord/697882427875393627.svg?style=social&logo=discord)](https://discord.gg/redis)
+[![Twitch](https://img.shields.io/twitch/status/redisinc?style=social)](https://www.twitch.tv/redisinc)
+[![YouTube](https://img.shields.io/youtube/channel/views/UCD78lHSwYqMlyetR0_P4Vig?style=social)](https://www.youtube.com/redisinc)
+[![Twitter](https://img.shields.io/twitter/follow/redisinc?style=social)](https://twitter.com/redisinc)
+[![Stack Exchange questions](https://img.shields.io/stackexchange/stackoverflow/t/mcp-redis?style=social&logo=stackoverflow&label=Stackoverflow)](https://stackoverflow.com/questions/tagged/mcp-redis)
 
 ## Overview
 The Redis MCP Server is a **natural language interface** designed for agentic applications to efficiently manage and search data in Redis. It integrates seamlessly with **MCP (Model Content Protocol) clients**, enabling AI-driven workflows to interact with structured and unstructured data in Redis. Using this MCP Server, you can ask questions like:

pyproject.toml

@@ -52,21 +52,7 @@ skips = ["B101", "B601"]  # Skip assert_used and shell_injection_process_args if
 [tool.bandit.assert_used]
 skips = ["*_test.py", "*/test_*.py"]
 
-# Test configuration
-[tool.pytest.ini_options]
-testpaths = ["tests"]
-python_files = ["test_*.py"]
-python_classes = ["Test*"]
-python_functions = ["test_*"]
-addopts = [
-    "--strict-markers",
-    "--strict-config",
-    "--verbose",
-]
-markers = [
-    "slow: marks tests as slow",
-    "integration: marks tests as integration tests",
-]
+
 
 [dependency-groups]
 dev = [
@@ -78,7 +64,69 @@ dev = [
     "pytest>=8.4.1",
     "pytest-asyncio>=1.1.0",
     "pytest-cov>=6.2.1",
+    "pytest-mock>=3.12.0",
     "ruff>=0.12.5",
     "safety>=3.6.0",
     "twine>=4.0",
 ]
+
+test = [
+    "pytest>=8.4.1",
+    "pytest-asyncio>=1.1.0",
+    "pytest-cov>=6.2.1",
+    "pytest-mock>=3.12.0",
+    "coverage>=7.10.1",
+]
+
+# Testing configuration
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+python_files = ["test_*.py"]
+python_classes = ["Test*"]
+python_functions = ["test_*"]
+addopts = [
+    "--strict-markers",
+    "--strict-config",
+    "--verbose",
+    "--cov=src",
+    "--cov-report=html",
+    "--cov-report=term",
+    "--cov-report=xml",
+    "--cov-fail-under=80",
+]
+markers = [
+    "unit: marks tests as unit tests",
+    "integration: marks tests as integration tests",
+    "slow: marks tests as slow running",
+]
+asyncio_mode = "auto"
+filterwarnings = [
+    "ignore::DeprecationWarning",
+    "ignore::PendingDeprecationWarning",
+]
+
+[tool.coverage.run]
+source = ["src"]
+omit = [
+    "*/tests/*",
+    "*/test_*.py",
+    "*/__pycache__/*",
+    "*/venv/*",
+    "*/.venv/*",
+]
+
+[tool.coverage.report]
+exclude_lines = [
+    "pragma: no cover",
+    "def __repr__",
+    "if self.debug:",
+    "if settings.DEBUG",
+    "raise AssertionError",
+    "raise NotImplementedError",
+    "if 0:",
+    "if __name__ == .__main__.:",
+    "class .*\\bProtocol\\):",
+    "@(abc\\.)?abstractmethod",
+]
+
+

tests/__init__.py

@@ -0,0 +1 @@
+# Tests package for Redis MCP Server

tests/conftest.py

@@ -0,0 +1,134 @@
+"""
+Pytest configuration and fixtures for Redis MCP Server tests.
+"""
+
+import pytest
+from unittest.mock import Mock, patch
+import redis
+from redis.exceptions import RedisError, ConnectionError, TimeoutError
+
+
+@pytest.fixture
+def mock_redis():
+    """Create a mock Redis connection."""
+    mock = Mock(spec=redis.Redis)
+    return mock
+
+
+@pytest.fixture
+def mock_redis_cluster():
+    """Create a mock Redis Cluster connection."""
+    mock = Mock(spec=redis.cluster.RedisCluster)
+    return mock
+
+
+@pytest.fixture
+def mock_redis_connection_manager():
+    """Mock the RedisConnectionManager to return a mock Redis connection."""
+    with patch('src.common.connection.RedisConnectionManager.get_connection') as mock_get_conn:
+        mock_redis = Mock(spec=redis.Redis)
+        mock_get_conn.return_value = mock_redis
+        yield mock_redis
+
+
+@pytest.fixture
+def redis_config():
+    """Sample Redis configuration for testing."""
+    return {
+        "host": "localhost",
+        "port": 6379,
+        "db": 0,
+        "username": None,
+        "password": "",
+        "ssl": False,
+        "ssl_ca_path": None,
+        "ssl_keyfile": None,
+        "ssl_certfile": None,
+        "ssl_cert_reqs": "required",
+        "ssl_ca_certs": None,
+        "cluster_mode": False,
+    }
+
+
+@pytest.fixture
+def redis_uri_samples():
+    """Sample Redis URIs for testing."""
+    return {
+        "basic": "redis://localhost:6379/0",
+        "with_auth": "redis://user:pass@localhost:6379/0",
+        "ssl": "rediss://user:pass@localhost:6379/0",
+        "with_query": "redis://localhost:6379/0?ssl_cert_reqs=required",
+        "cluster": "redis://localhost:6379/0?cluster_mode=true",
+    }
+
+
+@pytest.fixture
+def sample_vector():
+    """Sample vector for testing vector operations."""
+    return [0.1, 0.2, 0.3, 0.4, 0.5]
+
+
+@pytest.fixture
+def sample_json_data():
+    """Sample JSON data for testing."""
+    return {
+        "name": "John Doe",
+        "age": 30,
+        "city": "New York",
+        "hobbies": ["reading", "swimming"]
+    }
+
+
+@pytest.fixture
+def redis_error_scenarios():
+    """Common Redis error scenarios for testing."""
+    return {
+        "connection_error": ConnectionError("Connection refused"),
+        "timeout_error": TimeoutError("Operation timed out"),
+        "generic_error": RedisError("Generic Redis error"),
+        "auth_error": RedisError("NOAUTH Authentication required"),
+        "wrong_type": RedisError("WRONGTYPE Operation against a key holding the wrong kind of value"),
+    }
+
+
+@pytest.fixture(autouse=True)
+def reset_connection_manager():
+    """Reset the RedisConnectionManager singleton before each test."""
+    from src.common.connection import RedisConnectionManager
+    RedisConnectionManager._instance = None
+    yield
+    RedisConnectionManager._instance = None
+
+
+@pytest.fixture
+def mock_numpy_array():
+    """Mock numpy array for vector testing."""
+    with patch('numpy.array') as mock_array:
+        mock_array.return_value.tobytes.return_value = b'mock_binary_data'
+        yield mock_array
+
+
+@pytest.fixture
+def mock_numpy_frombuffer():
+    """Mock numpy frombuffer for vector testing."""
+    with patch('numpy.frombuffer') as mock_frombuffer:
+        mock_frombuffer.return_value.tolist.return_value = [0.1, 0.2, 0.3]
+        yield mock_frombuffer
+
+
+# Async test helpers
+@pytest.fixture
+def event_loop():
+    """Create an event loop for async tests."""
+    import asyncio
+    loop = asyncio.new_event_loop()
+    yield loop
+    loop.close()
+
+
+# Mark configurations
+def pytest_configure(config):
+    """Configure pytest markers."""
+    config.addinivalue_line("markers", "unit: mark test as a unit test")
+    config.addinivalue_line("markers", "integration: mark test as an integration test")
+    config.addinivalue_line("markers", "slow: mark test as slow running")