test(tradein): fix sweep-endpoint тесты после single-run guard (#874 follow-up) #876

Merged

bot-reviewer merged 1 commit from fix/sweep-guard-test-mocks into main

2026-05-31 09:07:41 +00:00

lekss361 commented

2026-05-31 09:04:21 +00:00

Owner

Summary

Hotfix к #874 (squash-merged): single-run guard добавил 409 на /scrape/avito-city-sweep, но 2 существующих теста POST'ят туда с mock-db (has_running_run→truthy) → ждут 200, получают 409. Тест-фикс пришёл после merge → на main сейчас 2 красных теста (CI).

test_city_sweep.py::test_start_endpoint_ok
test_sweep_imv_phase.py::test_sweep_endpoint_passes_enrich_imv

Оба мокают app.api.v1.admin.has_running_run→False (патчим где используется). + ruff autofix import-sort, убран unused mock_create.

Test plan

ruff clean; целевые тесты 8 passed
оба ранее-падавших → green

NB: 2 другие full-suite ошибки локально (test_cian_valuation, test_search_api) — pre-existing env-isolation (psycopg2 scheme), падают идентично на чистом main, в CI проходят.

## Summary Hotfix к #874 (squash-merged): single-run guard добавил 409 на `/scrape/avito-city-sweep`, но 2 существующих теста POST'ят туда с mock-db (`has_running_run`→truthy) → ждут 200, получают 409. Тест-фикс пришёл после merge → на main сейчас 2 красных теста (CI). - `test_city_sweep.py::test_start_endpoint_ok` - `test_sweep_imv_phase.py::test_sweep_endpoint_passes_enrich_imv` Оба мокают `app.api.v1.admin.has_running_run`→False (патчим где используется). + ruff autofix import-sort, убран unused `mock_create`. ## Test plan - [x] ruff clean; целевые тесты 8 passed - [x] оба ранее-падавших → green NB: 2 другие full-suite ошибки локально (`test_cian_valuation`, `test_search_api`) — pre-existing env-isolation (psycopg2 scheme), падают идентично на чистом main, в CI проходят.

lekss361 added 1 commit 2026-05-31 09:04:21 +00:00

test(tradein): mock has_running_run в sweep-endpoint тестах после single-run guard b12d7140d0

test_start_endpoint_ok / test_sweep_endpoint_passes_enrich_imv POST'ят на
/scrape/avito-city-sweep с mock-db (has_running_run → truthy) → новый guard
давал 409 вместо 200. Мокаем app.api.v1.admin.has_running_run→False (патчим
где используется). + ruff autofix import-sort, убран unused mock_create.

bot-reviewer approved these changes 2026-05-31 09:07:12 +00:00

bot-reviewer left a comment

Collaborator

✅ APPROVE — test-fix для #874 single-run guard (2 красных теста на main).

Чистый test-only diff. #874 (squash-merged мной) добавил 409-guard на /scrape/avito-city-sweep → 2 существующих endpoint-теста, POST'ящих туда, стали ждать 200 но получать 409 → main CI красный. Фикс: patch("app.api.v1.admin.has_running_run", return_value=False) в обоих (test_start_endpoint_ok, test_sweep_endpoint_passes_enrich_imv).

Patch-target верный: admin биндит имя через from app.services.scheduler import has_running_run → патч app.api.v1.admin.has_running_run перекрывает в неймспейсе admin → guard видит False → proceed к (mocked) create_run → 200. Совпадает с моим review #874.

Прочее — ruff autofix (UTC-импорт, import-sort, убран unused mock_create). Source не тронут, секретов/self-extending нет. Целевые 8 тестов pass.

Мержу первым (гринит main), затем #875 поверх. (Hotfix, без linked issue.)

✅ **APPROVE** — test-fix для #874 single-run guard (2 красных теста на main). Чистый test-only diff. #874 (squash-merged мной) добавил 409-guard на `/scrape/avito-city-sweep` → 2 существующих endpoint-теста, POST'ящих туда, стали ждать 200 но получать 409 → main CI красный. Фикс: `patch("app.api.v1.admin.has_running_run", return_value=False)` в обоих (`test_start_endpoint_ok`, `test_sweep_endpoint_passes_enrich_imv`). **Patch-target верный:** admin биндит имя через `from app.services.scheduler import has_running_run` → патч `app.api.v1.admin.has_running_run` перекрывает в неймспейсе admin → guard видит False → proceed к (mocked) create_run → 200. Совпадает с моим review #874. Прочее — ruff autofix (UTC-импорт, import-sort, убран unused `mock_create`). Source не тронут, секретов/self-extending нет. Целевые 8 тестов pass. Мержу первым (гринит main), затем #875 поверх. (Hotfix, без linked issue.)