From a068e4b612db47f757235cf827140f17eebe6b40 Mon Sep 17 00:00:00 2001
From: bot-backend <bot-backend@gendsgn.local>
Date: Thu, 13 Aug 2026 21:05:54 +0000
Subject: [PATCH] =?UTF-8?q?test(ci):=20=D1=81=D1=82=D0=BE=D1=80=D0=BE?=
 =?UTF-8?q?=D0=B6=20=D0=BF=D1=80=D0=BE=D0=BF=D1=83=D1=81=D0=BA=D0=BE=D0=B2?=
 =?UTF-8?q?=20=D0=BA=D1=80=D0=B8=D1=87=D0=B8=D1=82=20::error::=20=D0=BF?=
 =?UTF-8?q?=D0=BE=D0=B4=20Actions=20(#2871)=20(#2876)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/tests/conftest.py             | 13 ++++++--
 backend/tests/test_skip_guard_loud.py | 44 +++++++++++++++++++++++++++
 2 files changed, 55 insertions(+), 2 deletions(-)
 create mode 100644 backend/tests/test_skip_guard_loud.py

diff --git a/backend/tests/conftest.py b/backend/tests/conftest.py
index a3266cb6..5276e7b2 100644
--- a/backend/tests/conftest.py
+++ b/backend/tests/conftest.py
@@ -96,15 +96,24 @@ def pytest_sessionfinish(session, exitstatus) -> None:
     unlisted = sorted(_observed_skips - _allowed_skips())
     if not unlisted:
         return
-    print(
-        f"\nНЕУЧТЁННЫЙ ПРОПУСК ({len(unlisted)}): проверка не исполнилась и не "
+    head = (
+        f"НЕУЧТЁННЫЙ ПРОПУСК ({len(unlisted)}): проверка не исполнилась и не "
         f"объявлена в {_SKIP_ALLOWLIST_PATH.name}:"
     )
+    print(f"\n{head}")
     for nodeid in unlisted:
         print(f"  - {nodeid}")
     print(
         "Почини тест либо внеси его в skip_allowlist.txt с причиной — "
         "пропуск без записи неотличим от пройденной проверки."
     )
+    # #2871: под Actions дублируем в ::error:: — иначе сообщение тонет.
+    # 13.08 этот сторож четыре прогона подряд ронял job'у совершенно правильно,
+    # а его строка лежала посреди тысячи других (обычный print, по-русски) —
+    # и поиск по «FAILED / ERROR» её не находил. Причину искали три часа
+    # в диске, раннере, покрытии и кэше. Сторож, который роняет прогон,
+    # обязан кричать так, чтобы его нашли.
+    if os.environ.get("GITHUB_ACTIONS") or os.environ.get("CI"):
+        print(f"::error::{head} " + "; ".join(unlisted))
     if exitstatus == 0:
         session.exitstatus = 1
diff --git a/backend/tests/test_skip_guard_loud.py b/backend/tests/test_skip_guard_loud.py
new file mode 100644
index 00000000..6f83d126
--- /dev/null
+++ b/backend/tests/test_skip_guard_loud.py
@@ -0,0 +1,44 @@
+"""Проверка, что сторож пропусков кричит под Actions (#2871)."""
+
+from __future__ import annotations
+
+import types
+
+import tests.conftest as ct
+
+
+def _run_guard(monkeypatch, capsys, *, ci: bool, observed: set[str]) -> str:
+    monkeypatch.setattr(ct, "_observed_skips", observed)
+    monkeypatch.setattr(ct, "_allowed_skips", lambda: set())
+    monkeypatch.delenv("GITHUB_ACTIONS", raising=False)
+    monkeypatch.delenv("CI", raising=False)
+    if ci:
+        monkeypatch.setenv("GITHUB_ACTIONS", "true")
+    session = types.SimpleNamespace(exitstatus=0)
+    ct.pytest_sessionfinish(session, 0)
+    return capsys.readouterr().out, session.exitstatus
+
+
+def test_guard_emits_error_annotation_under_actions(monkeypatch, capsys) -> None:
+    out, rc = _run_guard(monkeypatch, capsys, ci=True, observed={"tests/x.py::test_y"})
+    assert "::error::" in out, "под Actions сторож обязан подниматься в аннотации"
+    assert "tests/x.py::test_y" in out
+    assert rc == 1
+
+
+def test_guard_stays_quiet_locally(monkeypatch, capsys) -> None:
+    """Контроль: локально ::error:: не нужен, человеческое сообщение остаётся."""
+    out, rc = _run_guard(monkeypatch, capsys, ci=False, observed={"tests/x.py::test_y"})
+    assert "::error::" not in out
+    assert "НЕУЧТЁННЫЙ ПРОПУСК" in out
+    assert rc == 1
+
+
+def test_guard_silent_when_all_skips_declared(monkeypatch, capsys) -> None:
+    """Контроль: без незадекларированных пропусков сторож молчит и не роняет."""
+    monkeypatch.setattr(ct, "_observed_skips", set())
+    monkeypatch.setattr(ct, "_allowed_skips", lambda: set())
+    session = types.SimpleNamespace(exitstatus=0)
+    ct.pytest_sessionfinish(session, 0)
+    assert capsys.readouterr().out == ""
+    assert session.exitstatus == 0