fix(tradein): add cian_proxy_url to test _mock_settings — unblock deploy (#923) #925

Merged

lekss361 merged 1 commit from fix/cian-proxy-test-mock into main

2026-05-31 19:34:42 +00:00

bot-backend commented

2026-05-31 19:34:33 +00:00

Collaborator

Проблема

PR #923 переключил Cian-скраперы на settings.cian_proxy_url, но общий хелпер _mock_settings в tests/test_scraper_proxy.py отдавал SimpleNamespace только с scraper_proxy_url. Поэтому пред-существующие cian proxy-wiring тесты (test_cian_newbuilding_own_session_receives_proxies и аналоги для cian/detail/valuation) упали с:

AttributeError: 'types.SimpleNamespace' object has no attribute 'cian_proxy_url'

→ pytest-гейт в deploy-tradein.yml упал → прод не передеплоился (всё ещё старый образ, CIAN_PROXY_URL не активен).

Фикс

Одна строка: _mock_settings теперь выставляет cian_proxy_url=scraper_proxy_url (зеркалит fallback реального property cian_proxy_url = CIAN_PROXY_URL or scraper_proxy_url). Существующие assertion'ы (прокси передан в AsyncSession) снова проходят; новые precedence-тесты не затронуты.

Чисто тестовый хелпер — рантайм-код не меняется.

Unblocks #923 deploy.

## Проблема PR #923 переключил Cian-скраперы на `settings.cian_proxy_url`, но общий хелпер `_mock_settings` в `tests/test_scraper_proxy.py` отдавал `SimpleNamespace` только с `scraper_proxy_url`. Поэтому **пред-существующие** cian proxy-wiring тесты (`test_cian_newbuilding_own_session_receives_proxies` и аналоги для cian/detail/valuation) упали с: ``` AttributeError: 'types.SimpleNamespace' object has no attribute 'cian_proxy_url' ``` → pytest-гейт в `deploy-tradein.yml` упал → прод **не передеплоился** (всё ещё старый образ, `CIAN_PROXY_URL` не активен). ## Фикс Одна строка: `_mock_settings` теперь выставляет `cian_proxy_url=scraper_proxy_url` (зеркалит fallback реального property `cian_proxy_url = CIAN_PROXY_URL or scraper_proxy_url`). Существующие assertion'ы (прокси передан в `AsyncSession`) снова проходят; новые precedence-тесты не затронуты. Чисто тестовый хелпер — рантайм-код не меняется. Unblocks #923 deploy.

bot-backend added 1 commit 2026-05-31 19:34:34 +00:00

fix(tradein): add cian_proxy_url to test _mock_settings — unblock deploy (#923 ) b9690d4d63

PR #923 switched Cian scrapers to settings.cian_proxy_url but the shared
_mock_settings helper in test_scraper_proxy.py still only exposed scraper_proxy_url,
so pre-existing cian proxy-wiring tests broke with AttributeError → deploy pytest gate failed.
Mirror scraper_proxy_url into cian_proxy_url in the mock.