bittabola/tarjimon

Fork 0

Auto: test/integration-suite #6

Merged

claude-bot merged 13 commits from test/integration-suite into main

2026-02-25 23:19:11 -07:00

claude-bot commented

2026-02-25 23:17:38 -07:00

Collaborator

Automated PR for branch test/integration-suite.

Automated PR for branch `test/integration-suite`.

claude-bot added 13 commits

2026-02-25 23:17:38 -07:00

test: add integration test helpers for building mock Telegram objects ae142bfa5c

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add integration conftest with tmp_db and patch_gemini fixtures 0c5776ba42

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add health endpoint integration tests 81c8ca1e3f

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add webhook security integration tests 810a06dd84

Add 6 integration tests covering secret token validation for /webhook
and /feedback_webhook endpoints (no-secret, wrong-secret, correct-secret).
Update root conftest to prefer real httpx when installed (needed for
ASGITransport) and add uvicorn stub.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add subscription and callback handler integration tests ec5432421f

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add translation pipeline integration tests 8dc3183cf8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add YouTube pipeline integration tests 801b00e27f

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add admin dashboard integration tests 1a738ae426

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add feedback flow integration tests ef72a7b7d8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add edge case integration tests 43446fe134

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: configure pytest with asyncio auto mode and smoke marker a9b4f1b98b

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: add smoke tests for real Gemini and Supadata APIs c29da75ca0

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: fix lint errors and remove duplicate pytest.ini

Auto PR Review / review (push) Successful in 2m8s

Details

d1974fecce

Remove unused imports flagged by ruff and consolidate pytest
config into pyproject.toml only.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude-bot commented

2026-02-25 23:19:11 -07:00

Author

Collaborator

Approved

Well-structured addition of integration and smoke tests. The test fixtures correctly target verified module attributes (DatabaseManager._instance, _feedback_pending_users, user_manager.sessions). Mock builders faithfully reproduce Telegram API objects. Webhook security, admin auth, translation pipeline, subscription flow, feedback, and YouTube summarization are all covered with proper isolation. No critical or warning-level issues found.

Suggestions

tests/integration/test_feedback_flow.py: Inconsistent use of @pytest.mark.asyncio: test_feedback_flow.py applies it to every test while all other integration test files omit it. With asyncio_mode = "auto" in pyproject.toml, the decorator is redundant. Consider removing it from test_feedback_flow.py for consistency.
tests/integration/test_health.py: _get_real_httpx() is now unnecessary. The root conftest.py was updated to preserve the real httpx when installed (try/except pattern at line 62). Since integration tests require httpx to be installed, sys.modules['httpx'] will already be the real module. This function adds complexity for no benefit in the current setup.
tests/integration/conftest.py: The tmp_db fixture resets DatabaseManager._instance = None but doesn't explicitly close the old singleton's SQLite connection before discarding it. While not a real problem in tests (connections are to tmp files that get cleaned up), adding db_manager.close() or similar cleanup before resetting would be more thorough.

## Approved Well-structured addition of integration and smoke tests. The test fixtures correctly target verified module attributes (DatabaseManager._instance, _feedback_pending_users, user_manager.sessions). Mock builders faithfully reproduce Telegram API objects. Webhook security, admin auth, translation pipeline, subscription flow, feedback, and YouTube summarization are all covered with proper isolation. No critical or warning-level issues found. ### Suggestions - **tests/integration/test_feedback_flow.py**: Inconsistent use of @pytest.mark.asyncio: test_feedback_flow.py applies it to every test while all other integration test files omit it. With asyncio_mode = "auto" in pyproject.toml, the decorator is redundant. Consider removing it from test_feedback_flow.py for consistency. - **tests/integration/test_health.py**: _get_real_httpx() is now unnecessary. The root conftest.py was updated to preserve the real httpx when installed (try/except pattern at line 62). Since integration tests require httpx to be installed, sys.modules['httpx'] will already be the real module. This function adds complexity for no benefit in the current setup. - **tests/integration/conftest.py**: The tmp_db fixture resets DatabaseManager._instance = None but doesn't explicitly close the old singleton's SQLite connection before discarding it. While not a real problem in tests (connections are to tmp files that get cleaned up), adding db_manager.close() or similar cleanup before resetting would be more thorough.

claude-bot merged commit bf125c9a24 into main

2026-02-25 23:19:11 -07:00

claude-bot deleted branch test/integration-suite

2026-02-25 23:19:11 -07:00

claude-bot referenced this pull request from a commit

2026-02-25 23:19:13 -07:00

Auto: test/integration-suite (#6)