dolibarr-mcp/tests/test_config.py

"""
Test configuration module for Dolibarr MCP Server.
"""

import os
import pytest
from unittest.mock import patch
from pathlib import Path

# Import only existing modules
import sys
sys.path.insert(0, 'src')
from dolibarr_mcp.config import Config


class TestConfig:
    """Test configuration loading and validation."""
    
    def test_config_from_env(self):
        """Test configuration loading from environment variables."""
        with patch.dict(os.environ, {
            'DOLIBARR_URL': 'https://test.dolibarr.com',
            'DOLIBARR_API_KEY': 'test_key_123',
            'LOG_LEVEL': 'DEBUG'
        }):
            config = Config(
                dolibarr_url=os.getenv('DOLIBARR_URL'),
                dolibarr_api_key=os.getenv('DOLIBARR_API_KEY'),
                log_level=os.getenv('LOG_LEVEL')
            )
            assert config.dolibarr_url == 'https://test.dolibarr.com/api/index.php'
            assert config.dolibarr_api_key == 'test_key_123'
            assert config.log_level == 'DEBUG'
    
    def test_config_defaults(self):
        """Test configuration defaults when env vars not set."""
        with patch.dict(os.environ, {}, clear=True):
            config = Config()
            assert config.log_level == 'INFO'  # Default log level
    
    def test_config_url_normalization(self):
        """Test URL normalization (adding API path)."""
        with patch.dict(os.environ, {
            'DOLIBARR_URL': 'https://test.dolibarr.com/',
            'DOLIBARR_API_KEY': 'test_key'
        }):
            config = Config(
                dolibarr_url=os.getenv('DOLIBARR_URL'),
                dolibarr_api_key=os.getenv('DOLIBARR_API_KEY')
            )
            # Should add /api/index.php
            assert config.dolibarr_url == 'https://test.dolibarr.com/api/index.php'
            assert not config.dolibarr_url.endswith('//')
    
    def test_config_from_dotenv(self, tmp_path):
        """Test configuration loading from .env file."""
        env_file = tmp_path / ".env"
        env_file.write_text(
            "DOLIBARR_URL=https://env.dolibarr.com\n"
            "DOLIBARR_API_KEY=env_key_456\n"
            "LOG_LEVEL=WARNING\n"
        )
        
        with patch.dict(os.environ, {'DOTENV_PATH': str(env_file)}):
            # Load from env file
            from dotenv import load_dotenv
            load_dotenv(str(env_file))
            
            config = Config(
                dolibarr_url=os.getenv('DOLIBARR_URL'),
                dolibarr_api_key=os.getenv('DOLIBARR_API_KEY'),
                log_level=os.getenv('LOG_LEVEL')
            )
            assert config.dolibarr_url == 'https://env.dolibarr.com/api/index.php'
            assert config.dolibarr_api_key == 'env_key_456'
            assert config.log_level == 'WARNING'
    
    def test_config_precedence(self):
        """Test that environment variables take precedence over defaults."""
        with patch.dict(os.environ, {
            'DOLIBARR_URL': 'https://env.dolibarr.com',
            'DOLIBARR_API_KEY': 'env_key'
        }):
            config = Config(
                dolibarr_url=os.getenv('DOLIBARR_URL'),
                dolibarr_api_key=os.getenv('DOLIBARR_API_KEY')
            )
            assert config.dolibarr_url == 'https://env.dolibarr.com/api/index.php'
            assert config.dolibarr_api_key == 'env_key'
    
    def test_log_level_validation(self):
        """Test log level validation."""
        valid_levels = ['DEBUG', 'INFO', 'WARNING', 'ERROR', 'CRITICAL']
        
        for level in valid_levels:
            with patch.dict(os.environ, {
                'DOLIBARR_URL': 'https://test.com',
                'DOLIBARR_API_KEY': 'key',
                'LOG_LEVEL': level
            }):
                config = Config(
                    dolibarr_url=os.getenv('DOLIBARR_URL'),
                    dolibarr_api_key=os.getenv('DOLIBARR_API_KEY'),
                    log_level=os.getenv('LOG_LEVEL')
                )
                assert config.log_level == level
    
    def test_invalid_log_level(self):
        """Test invalid log level falls back to default."""
        with patch.dict(os.environ, {
            'DOLIBARR_URL': 'https://test.com',
            'DOLIBARR_API_KEY': 'key',
            'LOG_LEVEL': 'INVALID'
        }):
            config = Config(
                dolibarr_url=os.getenv('DOLIBARR_URL'),
                dolibarr_api_key=os.getenv('DOLIBARR_API_KEY'),
                log_level='INVALID'
            )
            assert config.log_level == 'INFO'  # Should fall back to default
    
    def test_api_key_alias(self):
        """Test backward compatibility alias for api_key."""
        config = Config(
            dolibarr_url='https://test.com',
            dolibarr_api_key='test_key'
        )
        assert config.api_key == 'test_key'  # Should work via alias