lekss361/gendesign

Красный main: тесты коридора читали настройки, которые стали константами #3572

Merged

bot-backend merged 1 commit from fix/main-red-corridor-settings-tests into main

2026-09-17 09:55:44 +00:00

bot-backend commented

2026-09-17 09:46:02 +00:00

Collaborator

Красный main после мержей 17.09: #3554 и #3556 разошлись по смыслу без текстового конфликта.

test_3466_corridor_tier_a.py (2), test_estimator_radius_floor.py (2)
AttributeError: 'Settings' object has no attribute 'estimate_corridor_clamp_slack'
TypeError: _run_estimate() got an unexpected keyword argument 'radius_floor_factor'

#3554 писал тесты против settings.estimate_corridor_clamp_*, а #3556 (#2380) перевёл эти поля в константы CORRIDOR_CLAMP_SLACK / CORRIDOR_CLAMP_MIN_N и снял параметр radius_floor_factor у хелпера. Продуктовый код не затронут (в app/ старые имена только в комментарии), но красный test блокирует деплой МЕРЫ.

Тесты переведены на константы, значения прежние. pytest tests/ — 6374 passed, 44 skipped, rc=0; ruff — rc=0.

🤖 Generated with Claude Code

Красный main после мержей 17.09: #3554 и #3556 разошлись по смыслу без текстового конфликта. ``` test_3466_corridor_tier_a.py (2), test_estimator_radius_floor.py (2) AttributeError: 'Settings' object has no attribute 'estimate_corridor_clamp_slack' TypeError: _run_estimate() got an unexpected keyword argument 'radius_floor_factor' ``` #3554 писал тесты против `settings.estimate_corridor_clamp_*`, а #3556 (#2380) перевёл эти поля в константы `CORRIDOR_CLAMP_SLACK` / `CORRIDOR_CLAMP_MIN_N` и снял параметр `radius_floor_factor` у хелпера. Продуктовый код не затронут (в `app/` старые имена только в комментарии), но красный `test` блокирует деплой МЕРЫ. Тесты переведены на константы, значения прежние. `pytest tests/` — 6374 passed, 44 skipped, rc=0; ruff — rc=0. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

bot-backend added 1 commit 2026-09-17 09:46:05 +00:00

fix(tests): красный main — тесты #3466 читали настройки, которые #3556 перевёл в константы

CI Trade-In / backend-tests (pull_request) Successful in 8m23s

Details

CI Trade-In / changes (pull_request) Successful in 14s

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI / changes (pull_request) Successful in 20s

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

Семантический конфликт двух PR, смерженных 17.09 подряд. Текстового конфликта не
было, CI каждого PR был зелёным на своей голове, а объединённое дерево падало:

  test_3466_corridor_tier_a.py (2), test_estimator_radius_floor.py (2)
  AttributeError: 'Settings' object has no attribute 'estimate_corridor_clamp_slack'
  TypeError: _run_estimate() got an unexpected keyword argument 'radius_floor_factor'

#3554 (Tier A и advisory_only) писал тесты против settings.estimate_corridor_clamp_*,
а #3556 (#2380) убрал эти поля из Settings в константы CORRIDOR_CLAMP_SLACK
(estimator) и CORRIDOR_CLAMP_MIN_N (app.core.config) и снял параметр
radius_floor_factor у хелпера. Продуктовый код не затронут: в app/ старые имена
остались только в комментарии, на проде AttributeError не было. Но красный test
блокировал деплой МЕРЫ: на проде до сих пор образ до второй пачки мержей.

Тесты переведены на константы, значения прежние (0.40, 10, 0.8).
pytest tests/ — 6374 passed, 44 skipped, rc=0; ruff check/format — rc=0.

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

bot-backend merged commit 85d455e387 into main

2026-09-17 09:55:44 +00:00

bot-backend referenced this pull request from a commit

2026-09-17 09:55:46 +00:00

Merge pull request 'Красный main: тесты коридора читали настройки, которые стали константами' (#3572) from fix/main-red-corridor-settings-tests into main

Light1YT referenced this pull request from a commit

2026-09-17 09:56:09 +00:00

merge origin/main: починка красного main (#3572)

Light1YT referenced this pull request from a commit

2026-09-17 09:56:25 +00:00

merge origin/main: починка красного main (#3572)

bot-backend referenced this pull request

2026-09-17 10:01:58 +00:00

Адреса МЕРЫ: «Сулимова, 3/2» и «Сулимова, 32» больше не один дом — дробь в номере не теряется при сопоставлении с ГАР и схлопывании дублей #3578

bot-backend referenced this pull request

2026-09-17 10:07:25 +00:00

Сайдкар МЕРЫ: не больше трёх живых браузеров, простаивающий закрывается, зомби-процессы собирает init #3581

Sign in to join this conversation.

No reviewers

No labels

Fable 5 ревью

GG-форсайт

needs-discussion

status/needs-analysis

status/needs-fix

week ревью 1

вторичка

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#3572

No description provided.