mirror of
https://github.com/langbot-app/LangBot.git
synced 2026-06-02 12:05:54 +00:00
* fix(ci): update unit-test workflow paths to match current source layout Replace stale pkg/** filter with src/langbot/** and add uv.lock. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * docs(tests): update README to reflect current test layout - Fix stale paths: tests/pipeline → tests/unit_tests/pipeline - Update CI Python versions: 3.11, 3.12, 3.13 - Add test directory structure for box, config, platform, plugin, provider, storage - Document pytest markers and uv commands - Mention planned E2E tests Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add shared test factories package Create tests/factories/ with reusable test factories: - FakeApp: mock application with all dependencies - Message chains: text_chain, mention_chain, image_chain - Query factories: text_query, group_text_query, command_query, etc. No test changes - maintains backward compatibility. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add fake provider factory Add tests/factories/provider.py with: - FakeProvider: deterministic fake LLM provider - Error simulation: timeout, auth, rate-limit, malformed - Request capture for assertions - fake_model: mock model with attached provider Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add fake platform factory Add tests/factories/platform.py with: - FakePlatform: simulated platform adapter - Inbound message construction: friend/group/image - Mention-bot flag simulation - Outbound message capture for assertions - Streaming output support simulation - Send failure simulation Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add comprehensive message/query factories Extend tests/factories/message.py with: - file_query: file attachment query - unsupported_query: unknown message segment - voice_query: audio/voice query - at_all_query: group @All mention - query_with_session: query with session object - query_with_config: query with custom pipeline config Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add fake message flow smoke test Create tests/smoke/test_fake_message_flow.py: - TestFakeMessageFlow: factory verification tests - TestMessageFlowIntegration: minimal flow smoke test - Tests FakeApp, FakeProvider, FakePlatform, query factories - Verifies LANGBOT_FAKE_PONG marker response - Captures outbound messages for assertions Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add developer test-quick command Add scripts/test-quick.sh and Makefile with: - test-quick: runs ruff check + unit tests + smoke tests - No real provider keys or platform accounts required - Suitable for local branch self-test Update tests/README.md: - Document test-quick command - Document test factories package - Add smoke tests and factories directory structure Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(test): make test-quick reliable as developer gate Fixes for D-001验收问题: 1. test-quick.sh: use set -euo pipefail, uv run ruff, no tail pipe 2. Remove unused imports in factories (app.py, platform.py, provider.py) 3. Fix unused variable in smoke test 4. Add noqa: E402 to test_n8nsvapi.py lazy imports 5. Update smoke test docs: "minimal fake flow" not full pipeline Now test-quick is a reliable gate: lint failures exit 1, test failures propagate. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(unit): add preproc and taskmgr unit tests U-001: Pipeline Preprocessor tests - Normal text message processing - Empty message handling - Image segment with/without vision model - Model selection and fallback - Variable extraction U-004: Core Task Manager tests (pattern-based) - Task creation and tracking patterns - Task cancellation patterns - Scope-based cancellation - Task type filtering - Pruning completed tasks - Wait all tasks Taskmgr tests use pattern-based approach to avoid circular import in source code (taskmgr → app → http_controller → migration → taskmgr). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(unit): add config loader unit tests U-005: Config Loader tests - Valid YAML config loading - Valid JSON config loading - Invalid YAML/JSON error behavior - Missing config file creation from template - Template completion for missing keys - ConfigManager load/dump operations - Exists check for both YAML and JSON All tests use tmp_path fixture, no real project config. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(unit): add chat and command handler pattern tests U-002: Chat Handler tests (pattern-based) - Normal message event emission pattern - prevent_default handling - User message alteration pattern - Runner selection pattern - Streaming/non-streaming response patterns - Exception handling modes (show-error, show-hint, hide) - Message history update pattern - Telemetry payload pattern U-003: Command Handler tests (pattern-based) - Command parsing and text extraction - Event creation pattern - Privilege/admin check pattern - Command result handling (text, error, image) - prevent_default handling - String truncation helper Uses pattern-based testing to avoid circular import issues in source code. Direct imports of handler modules trigger circular import chain. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * style: fix unused imports after ruff auto-fix Remove unused imports in test files: - test_config_loader.py: remove unused os - test_taskmgr.py: remove unused Mock - test_preproc.py: remove unused unsupported_query, image_chain Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(unit): improve taskmgr tests to test real classes U-004 improved: Tests now import and test actual classes: - TaskContext: new(), trace(), to_dict(), placeholder() - TaskWrapper: task creation, context, exception/result capture, cancel, to_dict - AsyncTaskManager: create_task, create_user_task, cancel_task, cancel_by_scope - Task pruning behavior Uses pre-mocking technique: - Mock langbot.pkg.core.app before import (breaks circular chain) - Mock langbot.pkg.core.entities with proper Enum All 24 tests now test real class behavior, not patterns. taskmgr.py coverage should improve significantly. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * refactor(test): consolidate FakeApp and add sys.modules isolation utility - Extract tests/utils/import_isolation.py with isolated_sys_modules context manager - Extend tests/factories/app.py FakeApp with handler-specific attributes - Refactor test_chat_handler.py to use centralized FakeApp and cached imports - Refactor test_command_handler.py with mock_execute_factory fixture - Refactor test_smoke.py to move import-time sys.modules manipulation into fixture - Add SQLite migration integration tests (G-002) - Add HTTP API smoke integration tests (G-005) - Update CI workflow to call pytest for SQLite migrations (G-004) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add developer quality gate consolidation (G-007) - Add scripts/test-integration-fast.sh for fast integration tests - Add scripts/test-coverage.sh with 12% baseline threshold - Update Makefile with test-integration-fast, test-coverage, test-all-local - Update CI workflow with integration and coverage jobs - Add smoke marker to pytest.ini - Update tests/README.md with quality gate layers documentation - Add tests/integration/pipeline/ for pipeline stage-chain tests Quality gate layers: - Quick: ruff + unit + smoke (~2 min) - Fast Integration: SQLite/API/Pipeline (~3 min) - Coverage: 12% threshold gate (~8 min) - Full Local: all three combined Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): add PostgreSQL migration slow integration tests (G-003) - Add tests/integration/persistence/test_migrations_postgres.py - All tests marked with @pytest.mark.slow - Tests skip when TEST_POSTGRES_URL is not set (no local PostgreSQL) - Database isolation via clean_tables and clean_alembic_version fixtures - Update CI workflow to use pytest instead of inline Python script - Remove TODO(G-003) comment - Update tests/README.md with PostgreSQL test documentation Covered scenarios: - Baseline stamp sets revision - Upgrade from baseline to head - Upgrade idempotent - Get current on unstamped DB returns None Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(test): Phase 1.5 coverage expansion - COV-001 to COV-013 Coverage baseline raised from 13.65% to 26% (+12.35%) Gate raised from 12% to 18% Tasks completed: - COV-001: Command system unit tests (100% coverage) - COV-002: API service unit tests batch 1 (user/apikey/model/provider) - COV-003: Provider model manager unit tests - COV-004: Pipeline remaining stage tests (aggregator/cntfilter/longtext/msgtrun) - COV-005: Storage and utils coverage pass - COV-006: Gate ratchet 12%→15% - COV-007: Gate ratchet 15%→18% - COV-008: API service batch 2 (bot/pipeline/webhook/space/maintenance/mcp) - COV-009: Blocked - API controller circular import issue documented - COV-010: Plugin runtime unit tests (+0.08%) - COV-011: RAG and vector unit tests (+0.68%) - COV-012: Core boot and migration unit tests - COV-013: Provider requester logic unit tests (+0.62%) Key additions: - tests/utils/import_isolation.py: sys.modules isolation for circular imports - Provider requester mock tests: proved HTTP-dependent code can be tested locally - Vector filter utilities: 100% coverage on pure functions - API services: fake persistence pattern for unit testing Blocked issue COV-009 documented in langbot-test-plan/1.5/issues/ Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(phase1): add unit tests for telemetry, plugin, rag, persistence Add initial unit tests for Phase 1 of test coverage improvement: - telemetry: test initialization, payload sanitization, early returns (14.3% → 62.9%) - plugin: test _parse_plugin_id static method - rag: test _to_i18n_name static method - persistence: test serialize_model with datetime handling Overall core coverage: 41.9% → 42.2% Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(phase2): add unit tests for core, persistence, plugin, utils - Add test_handler_helpers.py for plugin handler helpers (7 tests) - Add test_mgr_methods.py for persistence manager (5 tests) - Add test_app_config_validation.py for core app config (12 tests) - Add test_knowledge_service.py for API knowledge service (22 tests) - Add test_kbmgr.py for RAG knowledge base manager (39 tests) - Add test_survey_manager.py for survey manager (22 tests) - Add test_connector_methods.py for plugin connector (24 tests) - Add test_funcschema.py for utils function schema (9 tests) - Add test_platform.py for utils platform detection (7 tests) - Add test_extract_deps.py for plugin deps extraction (7 tests) - Add test_database_decorator.py for persistence decorator (7 tests) - Add test_load_config.py for core config loading (19 tests) - Add COVERAGE_EXCLUSIONS.md documenting external adapter exclusions - Fix test_chat_session_limit.py path for portability Coverage: core 28% → 30%, persistence 24% → 24.4%, plugin 27% → 28% Total: 1082 tests passed, core module coverage 45.5% Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(integration): add API controller integration tests - Add test_pipelines.py (10 tests) covering pipelines CRUD operations - GET/POST/PUT/DELETE on /api/v1/pipelines - Extensions endpoint - Metadata endpoint - Coverage: pipelines controller 27% → 80% - Add test_providers.py (10 tests) covering provider/model management - Provider CRUD with model counts - LLM model CRUD - Coverage: providers controller 23% → 81%, models 29% → 45% Tests use Quart TestClient with mocked services for real HTTP behavior without external dependencies. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(integration): add knowledge, bots, and model endpoints tests - Add test_knowledge.py (10 tests) covering knowledge base management - CRUD operations on /api/v1/knowledge/bases - Files management endpoints - Retrieve endpoint with validation - Coverage: knowledge/base.py 26% → 91% - Add test_bots.py (9 tests) covering bot management - CRUD operations on /api/v1/platform/bots - Logs endpoint - Send message endpoint with validation - Coverage: platform/bots.py 24% → 87% - Extend test_providers.py (+4 tests) for embedding/rerank models - Embedding models CRUD - Rerank models CRUD - Coverage: provider/models.py 29% → 60% Total integration tests: 53 (smoke 12 + pipelines 10 + providers 14 + knowledge 10 + bots 9) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(integration): add embed and monitoring endpoint tests Add integration tests for embed widget and monitoring API endpoints: - test_embed.py: 15 tests for widget.js, logo, turnstile, messages, reset, feedback - test_monitoring.py: 15 tests for overview, messages, llm-calls, sessions, errors, export Coverage improvements: - embed.py: 17% → 56% - monitoring.py: 17% → 93% Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(e2e): add minimal startup E2E tests Add E2E tests for LangBot startup flow: - tests/e2e/utils/config_factory.py: minimal config generation - tests/e2e/utils/process_manager.py: LangBot subprocess management - tests/e2e/conftest.py: E2E fixtures (session-scoped process) - tests/e2e/test_startup.py: 12 tests for startup verification Tests verify: - boot.py + stages execution - database initialization (SQLite) - API availability - migrations applied Uses embedded databases (SQLite, Chroma) - no external dependencies. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test(quality): fix fake tests and add missing coverage P0 fixes: - telemetry: rewrite fake tests with real behavior verification (25 tests) - config: delete copied-source tests, use proper imports (2 deleted) - persistence: fix try-except pass to verify specific errors P1 fixes: - pipeline: add real FixedWindowAlgo tests instead of mocks (12 tests) - provider: add SessionManager and ToolManager tests (25 tests) - storage: add S3StorageProvider tests with moto mock (16 tests) - plugin: add handler action tests for setting inheritance (15 tests) - rag: add file storage and ZIP processing tests (21 tests) - vector: add VDB filter conversion tests (30 tests) P2 fixes: - pipeline/msgtrun: strengthen assertions for exact message count - api: add response structure validation in integration tests New test files: - provider/test_session_manager.py - provider/test_tool_manager.py - storage/test_s3storage.py - plugin/test_handler_actions.py - rag/test_file_storage.py - vector/test_vdb_filter_conversion.py Source code bugs documented: - provider: TokenManager.next_token() ZeroDivisionError - telemetry: send_tasks class variable shared state - command: empty command IndexError, unused parameters - utils: funcschema KeyError - entity: vector.py independent declarative_base Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * docs(test): update coverage stats and test structure - Update coverage from 22% to 30% - Add new test files to structure: - provider: session_manager, tool_manager - storage: s3storage - plugin: handler_actions - rag: file_storage - vector: vdb_filter_conversion - telemetry: rewritten tests - Update module coverage percentages Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * test: add 105 new unit tests for untested core functionality Add comprehensive tests for B-class issues (core functionality untested): Pipeline: - test_pool.py: QueryPool ID generation, caching, async context (12 tests) - test_ratelimit.py: Fixed timing-sensitive test tolerance - test_pipelinemgr.py: Use real Pydantic StageProcessResult instead of Mock Utils: - test_version.py: Version comparison functions (20 tests) - test_logcache.py: Log page management and retrieval (18 tests) - test_httpclient.py: HTTP session pool management (10 tests) - test_proxy.py: Proxy configuration from env and config (10 tests) - test_image.py: URL parsing and base64 extraction (12 tests) - test_pkgmgr.py: Pip command generation (8 tests) Discover: - test_engine.py: I18nString, Metadata, Component manifest (15 tests) Test count: 1193 → 1298 (+105 tests) Note: Some B-class issues cannot be tested due to circular import bugs filed as GitHub issues #2175 (pipeline) and #2176 (persistence). * test: tighten phase 1 coverage contracts * test: align ci integration isolation --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
634 lines
21 KiB
Python
634 lines
21 KiB
Python
"""
|
|
Unit tests for ProviderAPIRequester base class and runtime entities in provider/modelmgr.
|
|
|
|
Tests requester initialization, configuration handling, token management,
|
|
and runtime model/provider behavior without calling real LLM APIs.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import pytest
|
|
from unittest.mock import AsyncMock, Mock
|
|
from types import SimpleNamespace
|
|
|
|
from langbot.pkg.provider.modelmgr import requester
|
|
from langbot.pkg.provider.modelmgr import token
|
|
from langbot.pkg.entity.persistence import model as persistence_model
|
|
from langbot.pkg.provider.modelmgr.errors import RequesterError
|
|
|
|
|
|
# ============================================================================
|
|
# ProviderAPIRequester Base Class Tests
|
|
# ============================================================================
|
|
|
|
|
|
class TestableRequester(requester.ProviderAPIRequester):
|
|
"""Testable requester subclass for testing base class behavior."""
|
|
|
|
name = 'testable-requester'
|
|
|
|
default_config = {
|
|
'base_url': 'https://default.example.com',
|
|
'timeout': 60,
|
|
'max_retries': 3,
|
|
}
|
|
|
|
async def invoke_llm(
|
|
self,
|
|
query,
|
|
model: requester.RuntimeLLMModel,
|
|
messages: list,
|
|
funcs=None,
|
|
extra_args={},
|
|
remove_think=False,
|
|
):
|
|
import langbot_plugin.api.entities.builtin.provider.message as provider_message
|
|
return provider_message.Message(
|
|
role='assistant',
|
|
content=[provider_message.ContentElement(type='text', text='Testable response')],
|
|
)
|
|
|
|
|
|
def test_requester_base_class_is_abstract():
|
|
"""Test ProviderAPIRequester cannot be instantiated directly."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
# ProviderAPIRequester has abstract methods, but ABCMeta allows instantiation
|
|
# if you don't call the abstract methods. Test that it has abstract methods.
|
|
assert hasattr(requester.ProviderAPIRequester, 'invoke_llm')
|
|
# Check that invoke_llm is abstract
|
|
assert hasattr(requester.ProviderAPIRequester.invoke_llm, '__isabstractmethod__')
|
|
|
|
|
|
def test_requester_default_config_merged():
|
|
"""Test requester merges default config with provided config."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {'base_url': 'https://custom.example.com', 'custom_key': 'custom_value'})
|
|
|
|
assert inst.requester_cfg['base_url'] == 'https://custom.example.com'
|
|
assert inst.requester_cfg['timeout'] == 60 # from default
|
|
assert inst.requester_cfg['max_retries'] == 3 # from default
|
|
assert inst.requester_cfg['custom_key'] == 'custom_value' # custom added
|
|
|
|
|
|
def test_requester_default_config_not_modified():
|
|
"""Test that default_config dict is not modified when merging."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {'base_url': 'https://override.example.com'})
|
|
|
|
assert TestableRequester.default_config['base_url'] == 'https://default.example.com'
|
|
assert inst.requester_cfg['base_url'] == 'https://override.example.com'
|
|
|
|
|
|
def test_requester_empty_config_uses_defaults():
|
|
"""Test requester uses defaults when empty config provided."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {})
|
|
|
|
assert inst.requester_cfg == inst.default_config
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_requester_initialize_is_callable():
|
|
"""Test requester initialize method is callable (default is pass)."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {})
|
|
await inst.initialize()
|
|
|
|
# No exception should occur
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_requester_scan_models_not_implemented():
|
|
"""Test scan_models raises NotImplementedError by default."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {})
|
|
await inst.initialize()
|
|
|
|
with pytest.raises(NotImplementedError) as exc_info:
|
|
await inst.scan_models()
|
|
|
|
assert 'does not support model scanning' in str(exc_info.value)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_requester_invoke_rerank_not_implemented():
|
|
"""Test invoke_rerank raises NotImplementedError by default."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {})
|
|
await inst.initialize()
|
|
|
|
# Create fake model
|
|
fake_provider_entity = persistence_model.ModelProvider(
|
|
uuid='provider-uuid',
|
|
name='Provider',
|
|
requester='test',
|
|
base_url='https://test.com',
|
|
api_keys=[],
|
|
)
|
|
fake_token_mgr = token.TokenManager(name='test', tokens=[])
|
|
fake_requester = inst
|
|
fake_provider = requester.RuntimeProvider(
|
|
provider_entity=fake_provider_entity,
|
|
token_mgr=fake_token_mgr,
|
|
requester=fake_requester,
|
|
)
|
|
fake_model_entity = persistence_model.RerankModel(
|
|
uuid='model-uuid',
|
|
name='Model',
|
|
provider_uuid='provider-uuid',
|
|
extra_args={},
|
|
)
|
|
fake_model = requester.RuntimeRerankModel(
|
|
model_entity=fake_model_entity,
|
|
provider=fake_provider,
|
|
)
|
|
|
|
with pytest.raises(NotImplementedError) as exc_info:
|
|
await inst.invoke_rerank(fake_model, 'query', ['doc1', 'doc2'])
|
|
|
|
assert 'does not support rerank' in str(exc_info.value)
|
|
|
|
|
|
# ============================================================================
|
|
# TokenManager Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_token_manager_initial_state():
|
|
"""Test TokenManager initial state."""
|
|
mgr = token.TokenManager(name='test-manager', tokens=['key1', 'key2', 'key3'])
|
|
|
|
assert mgr.name == 'test-manager'
|
|
assert mgr.tokens == ['key1', 'key2', 'key3']
|
|
assert mgr.using_token_index == 0
|
|
|
|
|
|
def test_token_manager_get_token():
|
|
"""Test TokenManager.get_token returns current token."""
|
|
mgr = token.TokenManager(name='test', tokens=['key1', 'key2'])
|
|
|
|
assert mgr.get_token() == 'key1'
|
|
|
|
|
|
def test_token_manager_get_token_empty():
|
|
"""Test TokenManager.get_token returns empty string when no tokens."""
|
|
mgr = token.TokenManager(name='test', tokens=[])
|
|
|
|
assert mgr.get_token() == ''
|
|
|
|
|
|
def test_token_manager_next_token_cycles():
|
|
"""Test TokenManager.next_token cycles through tokens."""
|
|
mgr = token.TokenManager(name='test', tokens=['key1', 'key2', 'key3'])
|
|
|
|
assert mgr.get_token() == 'key1'
|
|
|
|
mgr.next_token()
|
|
assert mgr.get_token() == 'key2'
|
|
|
|
mgr.next_token()
|
|
assert mgr.get_token() == 'key3'
|
|
|
|
# Should cycle back to first
|
|
mgr.next_token()
|
|
assert mgr.get_token() == 'key1'
|
|
|
|
|
|
def test_token_manager_next_token_single():
|
|
"""Test TokenManager.next_token with single token."""
|
|
mgr = token.TokenManager(name='test', tokens=['single-key'])
|
|
|
|
mgr.next_token()
|
|
assert mgr.get_token() == 'single-key'
|
|
|
|
mgr.next_token()
|
|
assert mgr.get_token() == 'single-key'
|
|
|
|
|
|
def test_token_manager_next_token_empty():
|
|
"""Test TokenManager.next_token with empty tokens doesn't error."""
|
|
mgr = token.TokenManager(name='test', tokens=[])
|
|
|
|
assert mgr.next_token() is None
|
|
assert mgr.get_token() == ''
|
|
|
|
|
|
# ============================================================================
|
|
# RuntimeProvider Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_runtime_provider_initialization(runtime_provider, fake_persistence_data):
|
|
"""Test RuntimeProvider initialization."""
|
|
provider = runtime_provider
|
|
provider_entity = fake_persistence_data['providers'][0]
|
|
|
|
assert provider.provider_entity.uuid == provider_entity.uuid
|
|
assert provider.provider_entity.name == provider_entity.name
|
|
assert provider.token_mgr.name == provider_entity.uuid
|
|
assert provider.token_mgr.tokens == provider_entity.api_keys
|
|
assert isinstance(provider.requester, requester.ProviderAPIRequester)
|
|
|
|
|
|
def test_runtime_provider_has_invoke_methods(runtime_provider):
|
|
"""Test RuntimeProvider has invoke methods that delegate to requester."""
|
|
provider = runtime_provider
|
|
|
|
assert hasattr(provider, 'invoke_llm')
|
|
assert hasattr(provider, 'invoke_llm_stream')
|
|
assert hasattr(provider, 'invoke_embedding')
|
|
assert hasattr(provider, 'invoke_rerank')
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_runtime_provider_invoke_llm_delegates(runtime_provider, runtime_llm_model):
|
|
"""Test RuntimeProvider.invoke_llm delegates to requester."""
|
|
provider = runtime_provider
|
|
|
|
# Track that requester was called
|
|
provider.requester._invoke_count = 0
|
|
|
|
import langbot_plugin.api.entities.builtin.provider.message as provider_message
|
|
import langbot_plugin.api.entities.builtin.pipeline.query as pipeline_query
|
|
|
|
# Create minimal query for testing (bypass validation)
|
|
query = pipeline_query.Query.model_construct(
|
|
query_id='test-query',
|
|
launcher_type='person',
|
|
launcher_id=12345,
|
|
sender_id=12345,
|
|
message_chain=None,
|
|
message_event=None,
|
|
adapter=None,
|
|
pipeline_uuid='pipeline-uuid',
|
|
bot_uuid='bot-uuid',
|
|
pipeline_config={'ai': {}, 'output': {}, 'trigger': {}},
|
|
session=None,
|
|
prompt=None,
|
|
messages=[],
|
|
user_message=None,
|
|
use_funcs=[],
|
|
use_llm_model_uuid=None,
|
|
variables={},
|
|
resp_messages=[],
|
|
resp_message_chain=None,
|
|
current_stage_name=None,
|
|
)
|
|
|
|
messages = [provider_message.Message(role='user', content=[provider_message.ContentElement(type='text', text='Hello')])]
|
|
|
|
result = await provider.invoke_llm(query, runtime_llm_model, messages)
|
|
|
|
assert provider.requester._invoke_count == 1
|
|
assert provider.requester._last_messages == messages
|
|
assert provider.requester._last_model == runtime_llm_model
|
|
assert result.role == 'assistant'
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_runtime_provider_invoke_llm_stream_yields_chunks(runtime_provider, runtime_llm_model):
|
|
"""Test RuntimeProvider.invoke_llm_stream yields chunks from requester."""
|
|
provider = runtime_provider
|
|
|
|
import langbot_plugin.api.entities.builtin.provider.message as provider_message
|
|
import langbot_plugin.api.entities.builtin.pipeline.query as pipeline_query
|
|
|
|
query = pipeline_query.Query.model_construct(
|
|
query_id='test-stream',
|
|
launcher_type='person',
|
|
launcher_id=12345,
|
|
sender_id=12345,
|
|
message_chain=None,
|
|
message_event=None,
|
|
adapter=None,
|
|
pipeline_uuid='pipeline-uuid',
|
|
bot_uuid='bot-uuid',
|
|
pipeline_config={'ai': {}, 'output': {}, 'trigger': {}},
|
|
session=None,
|
|
prompt=None,
|
|
messages=[],
|
|
user_message=None,
|
|
use_funcs=[],
|
|
use_llm_model_uuid=None,
|
|
variables={},
|
|
resp_messages=[],
|
|
resp_message_chain=None,
|
|
current_stage_name=None,
|
|
)
|
|
|
|
messages = [provider_message.Message(role='user', content=[provider_message.ContentElement(type='text', text='Hello')])]
|
|
|
|
chunks = []
|
|
async for chunk in provider.invoke_llm_stream(query, runtime_llm_model, messages):
|
|
chunks.append(chunk)
|
|
|
|
assert len(chunks) == 1
|
|
assert chunks[0].role == 'assistant'
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_runtime_provider_invoke_embedding_returns_vectors(runtime_provider, runtime_embedding_model):
|
|
"""Test RuntimeProvider.invoke_embedding returns embedding vectors."""
|
|
provider = runtime_provider
|
|
|
|
result = await provider.invoke_embedding(runtime_embedding_model, ['text1', 'text2'])
|
|
|
|
assert len(result) == 2
|
|
assert result[0] == [0.1, 0.2, 0.3]
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_runtime_provider_invoke_rerank_returns_scores(runtime_provider, runtime_rerank_model):
|
|
"""Test RuntimeProvider.invoke_rerank returns relevance scores."""
|
|
# Need to use the correct provider for rerank model
|
|
provider = runtime_rerank_model.provider
|
|
|
|
result = await provider.invoke_rerank(runtime_rerank_model, 'query', ['doc1', 'doc2', 'doc3'])
|
|
|
|
assert len(result) == 3
|
|
assert result[0]['index'] == 0
|
|
assert result[0]['relevance_score'] == 0.9
|
|
|
|
|
|
# ============================================================================
|
|
# RuntimeLLMModel Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_runtime_llm_model_initialization(runtime_llm_model, fake_persistence_data):
|
|
"""Test RuntimeLLMModel initialization."""
|
|
model = runtime_llm_model
|
|
model_entity = fake_persistence_data['llm_models'][0]
|
|
|
|
assert model.model_entity.uuid == model_entity.uuid
|
|
assert model.model_entity.name == model_entity.name
|
|
assert model.model_entity.abilities == model_entity.abilities
|
|
assert model.model_entity.extra_args == model_entity.extra_args
|
|
assert model.provider is not None
|
|
|
|
|
|
def test_runtime_llm_model_provider_ref(runtime_llm_model):
|
|
"""Test RuntimeLLMModel has correct provider reference."""
|
|
model = runtime_llm_model
|
|
|
|
assert model.provider.provider_entity is not None
|
|
assert model.provider.token_mgr is not None
|
|
assert model.provider.requester is not None
|
|
|
|
|
|
# ============================================================================
|
|
# RuntimeEmbeddingModel Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_runtime_embedding_model_initialization(runtime_embedding_model, fake_persistence_data):
|
|
"""Test RuntimeEmbeddingModel initialization."""
|
|
model = runtime_embedding_model
|
|
model_entity = fake_persistence_data['embedding_models'][0]
|
|
|
|
assert model.model_entity.uuid == model_entity.uuid
|
|
assert model.model_entity.name == model_entity.name
|
|
assert model.model_entity.extra_args == model_entity.extra_args
|
|
assert model.provider is not None
|
|
|
|
|
|
# ============================================================================
|
|
# RuntimeRerankModel Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_runtime_rerank_model_initialization(runtime_rerank_model, fake_persistence_data):
|
|
"""Test RuntimeRerankModel initialization."""
|
|
model = runtime_rerank_model
|
|
model_entity = fake_persistence_data['rerank_models'][0]
|
|
|
|
assert model.model_entity.uuid == model_entity.uuid
|
|
assert model.model_entity.name == model_entity.name
|
|
assert model.model_entity.extra_args == model_entity.extra_args
|
|
assert model.provider is not None
|
|
|
|
|
|
# ============================================================================
|
|
# RequesterError Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_requester_error_message_format():
|
|
"""Test RequesterError message format."""
|
|
error = RequesterError('API returned 500')
|
|
|
|
assert '模型请求失败' in str(error)
|
|
assert 'API returned 500' in str(error)
|
|
|
|
|
|
def test_requester_error_is_exception():
|
|
"""Test RequesterError is Exception subclass."""
|
|
error = RequesterError('test')
|
|
|
|
assert isinstance(error, Exception)
|
|
|
|
|
|
# ============================================================================
|
|
# ProviderAPIRequester Config Validation Tests
|
|
# ============================================================================
|
|
|
|
|
|
def test_requester_with_missing_base_url():
|
|
"""Test requester handles missing base_url in config."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
# If base_url is in default_config, it will be used
|
|
inst = TestableRequester(mock_app, {'timeout': 30})
|
|
|
|
assert inst.requester_cfg['base_url'] == 'https://default.example.com'
|
|
|
|
|
|
def test_requester_with_none_values():
|
|
"""Test requester handles None values in config."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = TestableRequester(mock_app, {'timeout': None, 'base_url': 'https://test.com'})
|
|
|
|
# None values are kept in the merged config
|
|
assert inst.requester_cfg['timeout'] is None
|
|
|
|
|
|
class RequesterWithNoDefaults(requester.ProviderAPIRequester):
|
|
"""Requester with empty defaults for testing."""
|
|
|
|
name = 'no-defaults-requester'
|
|
default_config = {}
|
|
|
|
async def invoke_llm(self, query, model, messages, funcs=None, extra_args={}, remove_think=False):
|
|
pass
|
|
|
|
|
|
def test_requester_empty_defaults_with_empty_config():
|
|
"""Test requester with empty defaults and empty config."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = RequesterWithNoDefaults(mock_app, {})
|
|
|
|
assert inst.requester_cfg == {}
|
|
|
|
|
|
def test_requester_empty_defaults_with_values():
|
|
"""Test requester with empty defaults receives config values."""
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
inst = RequesterWithNoDefaults(mock_app, {'base_url': 'https://custom.com', 'api_key': 'key'})
|
|
|
|
assert inst.requester_cfg['base_url'] == 'https://custom.com'
|
|
assert inst.requester_cfg['api_key'] == 'key'
|
|
|
|
|
|
# ============================================================================
|
|
# RuntimeProvider Error Handling Tests
|
|
# ============================================================================
|
|
|
|
|
|
class ErrorThrowingRequester(requester.ProviderAPIRequester):
|
|
"""Requester that throws errors for testing."""
|
|
|
|
name = 'error-requester'
|
|
default_config = {}
|
|
|
|
async def invoke_llm(self, query, model, messages, funcs=None, extra_args={}, remove_think=False):
|
|
raise RequesterError('Simulated API error')
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_runtime_provider_invoke_llm_propagates_error(mock_app_for_modelmgr):
|
|
"""Test RuntimeProvider.invoke_llm propagates requester errors."""
|
|
mock_app = mock_app_for_modelmgr
|
|
|
|
# Add monitoring_service for error handling path
|
|
mock_app.monitoring_service = AsyncMock()
|
|
|
|
requester_inst = ErrorThrowingRequester(mock_app, {})
|
|
await requester_inst.initialize()
|
|
|
|
provider_entity = persistence_model.ModelProvider(
|
|
uuid='error-provider',
|
|
name='Error Provider',
|
|
requester='error-requester',
|
|
base_url='https://error.com',
|
|
api_keys=['error-key'],
|
|
)
|
|
token_mgr = token.TokenManager(name='error-provider', tokens=['error-key'])
|
|
|
|
provider = requester.RuntimeProvider(
|
|
provider_entity=provider_entity,
|
|
token_mgr=token_mgr,
|
|
requester=requester_inst,
|
|
)
|
|
|
|
model_entity = persistence_model.LLMModel(
|
|
uuid='error-model',
|
|
name='Error Model',
|
|
provider_uuid='error-provider',
|
|
abilities=[],
|
|
extra_args={},
|
|
)
|
|
model = requester.RuntimeLLMModel(model_entity=model_entity, provider=provider)
|
|
|
|
import langbot_plugin.api.entities.builtin.provider.message as provider_message
|
|
import langbot_plugin.api.entities.builtin.pipeline.query as pipeline_query
|
|
|
|
query = pipeline_query.Query.model_construct(
|
|
query_id='error-query',
|
|
launcher_type='person',
|
|
launcher_id=12345,
|
|
sender_id=12345,
|
|
message_chain=None,
|
|
message_event=None,
|
|
adapter=None,
|
|
pipeline_uuid='pipeline-uuid',
|
|
bot_uuid='bot-uuid',
|
|
pipeline_config={'ai': {}, 'output': {}, 'trigger': {}},
|
|
session=None,
|
|
prompt=None,
|
|
messages=[],
|
|
user_message=None,
|
|
use_funcs=[],
|
|
use_llm_model_uuid=None,
|
|
variables={},
|
|
resp_messages=[],
|
|
resp_message_chain=None,
|
|
current_stage_name=None,
|
|
)
|
|
|
|
messages = [provider_message.Message(role='user', content=[provider_message.ContentElement(type='text', text='Hello')])]
|
|
|
|
with pytest.raises(RequesterError):
|
|
await provider.invoke_llm(query, model, messages)
|
|
|
|
|
|
# ============================================================================
|
|
# LLMModelInfo Tests (from entities.py)
|
|
# ============================================================================
|
|
|
|
|
|
def test_llm_model_info_basic():
|
|
"""Test LLMModelInfo basic structure."""
|
|
from langbot.pkg.provider.modelmgr.entities import LLMModelInfo
|
|
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
fake_requester = TestableRequester(mock_app, {})
|
|
fake_token_mgr = token.TokenManager(name='test', tokens=['key'])
|
|
|
|
info = LLMModelInfo(
|
|
name='test-model',
|
|
model_name='gpt-4',
|
|
token_mgr=fake_token_mgr,
|
|
requester=fake_requester,
|
|
tool_call_supported=True,
|
|
vision_supported=False,
|
|
)
|
|
|
|
assert info.name == 'test-model'
|
|
assert info.model_name == 'gpt-4'
|
|
assert info.tool_call_supported == True
|
|
assert info.vision_supported == False
|
|
|
|
|
|
def test_llm_model_info_optional_fields():
|
|
"""Test LLMModelInfo optional fields default values."""
|
|
from langbot.pkg.provider.modelmgr.entities import LLMModelInfo
|
|
|
|
mock_app = SimpleNamespace()
|
|
mock_app.logger = Mock()
|
|
|
|
fake_requester = TestableRequester(mock_app, {})
|
|
fake_token_mgr = token.TokenManager(name='test', tokens=['key'])
|
|
|
|
info = LLMModelInfo(
|
|
name='minimal-model',
|
|
token_mgr=fake_token_mgr,
|
|
requester=fake_requester,
|
|
)
|
|
|
|
assert info.model_name is None
|
|
assert info.tool_call_supported == False # default
|
|
assert info.vision_supported == False # default
|