mirror of
https://github.com/langbot-app/LangBot.git
synced 2026-06-02 12:05:54 +00:00
* Initial plan * Add package structure and resource path utilities - Created langbot/ package with __init__.py and __main__.py entry point - Added paths utility to find frontend and resource files from package installation - Updated config loading to use resource paths - Updated frontend serving to use resource paths - Added MANIFEST.in for package data inclusion - Updated pyproject.toml with build system and entry points Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> * Add PyPI publishing workflow and update license - Created GitHub Actions workflow to build frontend and publish to PyPI - Added license field to pyproject.toml to fix deprecation warning - Updated .gitignore to exclude build artifacts - Tested package building successfully Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> * Add PyPI installation documentation - Created PYPI_INSTALLATION.md with detailed installation and usage instructions - Updated README.md to feature uvx/pip installation as recommended method - Updated README_EN.md with same changes for English documentation Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> * Address code review feedback - Made package-data configuration more specific to langbot package only - Improved path detection with caching to avoid repeated file I/O - Removed sys.path searching which was incorrect for package data - Removed interactive input() call for non-interactive environment compatibility - Simplified error messages for version check Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> * Fix code review issues - Use specific exception types instead of bare except - Fix misleading comments about directory levels - Remove redundant existence check before makedirs with exist_ok=True - Use context manager for file opening to ensure proper cleanup Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> * Simplify package configuration and document behavioral differences - Removed redundant package-data configuration, relying on MANIFEST.in - Added documentation about behavioral differences between package and source installation - Clarified that include-package-data=true uses MANIFEST.in for data files Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> * chore: update pyproject.toml * chore: try pack templates in langbot/ * chore: update * chore: update * chore: update * chore: update * chore: update * chore: adjust dir structure * chore: fix imports * fix: read default-pipeline-config.json * fix: read default-pipeline-config.json * fix: tests * ci: publish pypi * chore: bump version 4.6.0-beta.1 for testing * chore: add templates/** * fix: send adapters and requesters icons * chore: bump version 4.6.0b2 for testing * chore: add platform field for docker-compose.yaml --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: RockChinQ <45992437+RockChinQ@users.noreply.github.com> Co-authored-by: Junyan Qin <rockchinq@gmail.com>
101 lines
3.6 KiB
Python
101 lines
3.6 KiB
Python
"""
|
|
RateLimit stage unit tests
|
|
|
|
Tests the actual RateLimit implementation from pkg.pipeline.ratelimit
|
|
"""
|
|
|
|
import pytest
|
|
from unittest.mock import AsyncMock, Mock, patch
|
|
from importlib import import_module
|
|
import langbot_plugin.api.entities.builtin.provider.session as provider_session
|
|
|
|
|
|
def get_modules():
|
|
"""Lazy import to ensure proper initialization order"""
|
|
# Import pipelinemgr first to trigger proper stage registration
|
|
ratelimit = import_module('langbot.pkg.pipeline.ratelimit.ratelimit')
|
|
entities = import_module('langbot.pkg.pipeline.entities')
|
|
algo_module = import_module('langbot.pkg.pipeline.ratelimit.algo')
|
|
return ratelimit, entities, algo_module
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_require_access_allowed(mock_app, sample_query):
|
|
"""Test RequireRateLimitOccupancy allows access when rate limit is not exceeded"""
|
|
ratelimit, entities, algo_module = get_modules()
|
|
|
|
sample_query.launcher_type = provider_session.LauncherTypes.PERSON
|
|
sample_query.launcher_id = '12345'
|
|
sample_query.pipeline_config = {}
|
|
|
|
# Create mock algorithm that allows access
|
|
mock_algo = Mock(spec=algo_module.ReteLimitAlgo)
|
|
mock_algo.require_access = AsyncMock(return_value=True)
|
|
mock_algo.initialize = AsyncMock()
|
|
|
|
stage = ratelimit.RateLimit(mock_app)
|
|
|
|
# Patch the algorithm selection to use our mock
|
|
with patch.object(algo_module, 'preregistered_algos', []):
|
|
stage.algo = mock_algo
|
|
|
|
result = await stage.process(sample_query, 'RequireRateLimitOccupancy')
|
|
|
|
assert result.result_type == entities.ResultType.CONTINUE
|
|
assert result.new_query == sample_query
|
|
mock_algo.require_access.assert_called_once_with(sample_query, 'person', '12345')
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_require_access_denied(mock_app, sample_query):
|
|
"""Test RequireRateLimitOccupancy denies access when rate limit is exceeded"""
|
|
ratelimit, entities, algo_module = get_modules()
|
|
|
|
sample_query.launcher_type = provider_session.LauncherTypes.PERSON
|
|
sample_query.launcher_id = '12345'
|
|
sample_query.pipeline_config = {}
|
|
|
|
# Create mock algorithm that denies access
|
|
mock_algo = Mock(spec=algo_module.ReteLimitAlgo)
|
|
mock_algo.require_access = AsyncMock(return_value=False)
|
|
mock_algo.initialize = AsyncMock()
|
|
|
|
stage = ratelimit.RateLimit(mock_app)
|
|
|
|
# Patch the algorithm selection to use our mock
|
|
with patch.object(algo_module, 'preregistered_algos', []):
|
|
stage.algo = mock_algo
|
|
|
|
result = await stage.process(sample_query, 'RequireRateLimitOccupancy')
|
|
|
|
assert result.result_type == entities.ResultType.INTERRUPT
|
|
assert result.user_notice != ''
|
|
mock_algo.require_access.assert_called_once()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_release_access(mock_app, sample_query):
|
|
"""Test ReleaseRateLimitOccupancy releases rate limit occupancy"""
|
|
ratelimit, entities, algo_module = get_modules()
|
|
|
|
sample_query.launcher_type = provider_session.LauncherTypes.PERSON
|
|
sample_query.launcher_id = '12345'
|
|
sample_query.pipeline_config = {}
|
|
|
|
# Create mock algorithm
|
|
mock_algo = Mock(spec=algo_module.ReteLimitAlgo)
|
|
mock_algo.release_access = AsyncMock()
|
|
mock_algo.initialize = AsyncMock()
|
|
|
|
stage = ratelimit.RateLimit(mock_app)
|
|
|
|
# Patch the algorithm selection to use our mock
|
|
with patch.object(algo_module, 'preregistered_algos', []):
|
|
stage.algo = mock_algo
|
|
|
|
result = await stage.process(sample_query, 'ReleaseRateLimitOccupancy')
|
|
|
|
assert result.result_type == entities.ResultType.CONTINUE
|
|
assert result.new_query == sample_query
|
|
mock_algo.release_access.assert_called_once_with(sample_query, 'person', '12345')
|