test(pipeline-status): regression guard for duplicate Closes mentions #86

New issue

Closed

opened 2026-08-16 20:11:22 +03:00 by slaid098 · 0 comments

slaid098 commented

2026-08-16 20:11:22 +03:00

Owner

Контекст

Урок PR#55: pipeline-status падал в AMBIGUOUS, когда в body PR один и тот же номер issue упоминался несколько раз (Closes #55 + Closes #55). Сейчас это защищено структурой кода (check_issue() дедупит через set()), но выделенного регрессионного теста нет — защита живёт только на чтении кода и может быть случайно потеряна рефакторингом.

Задача

В tests/test_pipeline_status.py добавить тест: check_issue на body с одним и тем же номером дважды (Closes #45 … Closes #45, а также вариант разными ключевыми словами Closes #45 + Fixes #45) → фаза ISSUE DONE, НЕ AMBIGUOUS.
Рядом комментарием сослаться на существующий тест-контраст разных номеров (test_check_issue_ambiguous_multiple: Closes #45 + Fixes #47 → AMBIGUOUS); дублировать его не нужно.

Контракты

Изменения только в tests/; продакшн-код pipeline-status.py не трогаем.

Инварианты

Весь набор pytest tests/test_pipeline_status*.py зелёный до и после.

Граничные случаи

Учесть формат, который уже поддерживает CLOSURE_RE (Closes/Fixes/Resolves + #N).

Влияние на связанные компоненты

Нет (тестовая добавка).

Вне scope

Любые изменения поведения оракула.

Критерии приемки

Новый тест присутствует, имя явно говорит о сценарии (например test_check_issue_duplicate_same_number_not_ambiguous).
pytest tests/test_pipeline_status*.py зелёный.

## Контекст Урок PR#55: `pipeline-status` падал в AMBIGUOUS, когда в body PR один и тот же номер issue упоминался несколько раз (`Closes #55` + `Closes #55`). Сейчас это защищено структурой кода (`check_issue()` дедупит через `set()`), но выделенного регрессионного теста нет — защита живёт только на чтении кода и может быть случайно потеряна рефакторингом. ## Задача 1. В `tests/test_pipeline_status.py` добавить тест: `check_issue` на body с одним и тем же номером дважды (`Closes #45` … `Closes #45`, а также вариант разными ключевыми словами `Closes #45` + `Fixes #45`) → фаза ISSUE DONE, НЕ AMBIGUOUS. 2. Рядом комментарием сослаться на существующий тест-контраст разных номеров (`test_check_issue_ambiguous_multiple`: `Closes #45` + `Fixes #47` → AMBIGUOUS); дублировать его не нужно. ## Контракты - Изменения только в tests/; продакшн-код pipeline-status.py не трогаем. ## Инварианты - Весь набор `pytest tests/test_pipeline_status*.py` зелёный до и после. ## Граничные случаи - Учесть формат, который уже поддерживает CLOSURE_RE (Closes/Fixes/Resolves + `#N`). ## Влияние на связанные компоненты - Нет (тестовая добавка). ## Вне scope - Любые изменения поведения оракула. ## Критерии приемки - Новый тест присутствует, имя явно говорит о сценарии (например `test_check_issue_duplicate_same_number_not_ambiguous`). - `pytest tests/test_pipeline_status*.py` зелёный.

slaid098 added the

enhancement

label

2026-08-16 20:11:22 +03:00

slaid098 referenced this issue from a pull request that will close it,

2026-08-16 20:49:22 +03:00

test(pipeline-status): regression guard for duplicate Closes mentions #92

slaid098 referenced this issue

2026-08-16 20:57:27 +03:00

fix(pipeline-status): ignore issue refs inside code spans in PR body #93

slaid098 closed this issue

2026-08-16 20:59:29 +03:00