66f4471e53
**Critical Fixes:**
1. **Fixed Error Hierarchy** (wikijs/exceptions.py)
- ConnectionError and TimeoutError now properly inherit from APIError
- Ensures consistent exception handling across the SDK
- Added proper __init__ methods with status_code=None
2. **Fixed test_connection Method** (wikijs/client.py)
- Changed from basic HTTP GET to proper GraphQL query validation
- Now uses query { site { title } } to validate API connectivity
- Provides better error messages for authentication failures
- Validates both connectivity AND API access
3. **Implemented JWT Token Refresh** (wikijs/auth/jwt.py)
- Added base_url parameter to JWTAuth class
- Implemented complete refresh() method with HTTP request to /api/auth/refresh
- Handles token, refresh token, and expiration updates
- Proper error handling for network failures and auth errors
**Bonus Fixes:**
- Dynamic user agent version (uses __version__ from version.py instead of hardcoded)
- Updated all JWT tests to include required base_url parameter
- Updated test mocks to match new GraphQL-based test_connection
**Test Results:**
- All 231 tests passing ✅
- Test coverage: 91.64% (target: 85%) ✅
- No test failures or errors
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
71 lines
1.8 KiB
Python
71 lines
1.8 KiB
Python
"""Test configuration and fixtures for wikijs-python-sdk."""
|
|
|
|
import pytest
|
|
import responses
|
|
|
|
from wikijs.auth import APIKeyAuth, JWTAuth, NoAuth
|
|
|
|
|
|
@pytest.fixture
|
|
def mock_api_key():
|
|
"""Fixture providing a test API key."""
|
|
return "test-api-key-12345"
|
|
|
|
|
|
@pytest.fixture
|
|
def mock_jwt_token():
|
|
"""Fixture providing a test JWT token."""
|
|
return "eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiYWRtaW4iOnRydWV9.TJVA95OrM7E2cBab30RMHrHDcEfxjoYZgeFONFh7HgQ"
|
|
|
|
|
|
@pytest.fixture
|
|
def api_key_auth(mock_api_key):
|
|
"""Fixture providing APIKeyAuth instance."""
|
|
return APIKeyAuth(mock_api_key)
|
|
|
|
|
|
@pytest.fixture
|
|
def jwt_auth(mock_jwt_token, mock_wiki_base_url):
|
|
"""Fixture providing JWTAuth instance."""
|
|
return JWTAuth(mock_jwt_token, mock_wiki_base_url)
|
|
|
|
|
|
@pytest.fixture
|
|
def no_auth():
|
|
"""Fixture providing NoAuth instance."""
|
|
return NoAuth()
|
|
|
|
|
|
@pytest.fixture
|
|
def mock_wiki_base_url():
|
|
"""Fixture providing test Wiki.js base URL."""
|
|
return "https://wiki.example.com"
|
|
|
|
|
|
@pytest.fixture
|
|
def mock_responses():
|
|
"""Fixture providing responses mock for HTTP requests."""
|
|
with responses.RequestsMock() as rsps:
|
|
yield rsps
|
|
|
|
|
|
@pytest.fixture
|
|
def sample_page_data():
|
|
"""Fixture providing sample page data."""
|
|
return {
|
|
"id": 1,
|
|
"title": "Test Page",
|
|
"path": "test-page",
|
|
"content": "This is a test page content.",
|
|
"created_at": "2023-01-01T00:00:00Z",
|
|
"updated_at": "2023-01-01T12:00:00Z",
|
|
"author": {"id": 1, "name": "Test User", "email": "test@example.com"},
|
|
"tags": ["test", "example"],
|
|
}
|
|
|
|
|
|
@pytest.fixture
|
|
def sample_error_response():
|
|
"""Fixture providing sample error response."""
|
|
return {"error": {"message": "Not found", "code": "PAGE_NOT_FOUND"}}
|