From 79ae87019a7f3e118cdd3f910b58355f103c112e Mon Sep 17 00:00:00 2001 From: bot-backend Date: Mon, 31 Aug 2026 12:59:27 +0500 Subject: [PATCH] =?UTF-8?q?=D0=B1=D1=8D=D0=BA=D1=82=D0=B5=D1=81=D1=82:=20?= =?UTF-8?q?=D0=BF=D0=BE=D0=B4=D0=BF=D0=B8=D1=81=D0=B8=20=D0=BE=D0=BA=D0=BD?= =?UTF-8?q?=D0=B0=20=D0=BF=D1=80=D0=B8=D0=B2=D0=B5=D0=B4=D0=B5=D0=BD=D1=8B?= =?UTF-8?q?=20=D0=BA=20=D0=B7=D0=B0=D0=BC=D0=B5=D1=80=D1=83,=20=D1=83?= =?UTF-8?q?=D0=B2=D0=B5=D1=80=D0=B5=D0=BD=D0=BD=D0=BE=D1=81=D1=82=D1=8C=20?= =?UTF-8?q?=E2=80=94=20=D0=BA=20=D1=82=D0=BE=D0=BC=D1=83=20=D0=B6=D0=B5=20?= =?UTF-8?q?=D0=BF=D1=80=D0=BE=D0=B3=D0=BE=D0=BD=D1=83?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Снимает три блокера ревью на этой же ветке. 1. ПОДПИСЬ ОКНА. Числа перевели на представительную выборку, а подпись осталась от прежнего прогона: «сделки II квартала 2026 года». Автор честно записал, что квартальный состав пересборки не смотрел, и оставил критерий снятия — посчитать deal_date по выборке. Посчитал: III кв 2025 — 106 (26,5 %) I кв 2026 — 88 (22,0 %) IV кв 2025 — 110 (27,5 %) II кв 2026 — 96 (24,0 %) Пересборка покрывает ЧЕТЫРЕ квартала, II квартал 2026 — четверть её. Прежняя подпись для этих чисел была бы прямым враньём. 2. ЗНАМЕНАТЕЛЬ. Был 5 954 — сделки только II квартала. Для выборки по четырём кварталам это завышало долю вчетверо: 5,5 % вместо 1,3 %. Замер тем же фильтром, что у выборки: в окне 24 333, из них II квартал 2026 — 5 954. Оба числа одним запросом. 3. УВЕРЕННОСТЬ. Плитка «325 из 327» была из прогона 29.08, а лид секции печатал «замер 31.08» — старому числу приписывалась новая дата, и 325 в лиде совпадало с 325 в плитке, читаясь как одна выборка. Заменено на «400 из 400» из ТОЙ ЖЕ пересборки. Знаменатель 400 — запрошенные сделки, а не 325 сматчившихся: уверенность считается на каждой оценке. 4. Оговорка уверенности не выводилась вовсе: AccuracyV3 собирал оговорки двух величин из трёх. Плитка стояла без единого слова о том, почему уверенность низкая. Добавлена третья. Тесты проверены фальсификацией: возврат знаменатели одного квартала даёт «expected 5954 to be 24333» и «expected 5.458 to be less than 2»; снятие оговорки из вывода — «оговорка не выводится». 157 passed. --- .../__tests__/backtest-window-labels.test.tsx | 68 ++++++++++++++++++ .../mera-public/_components/v3/AccuracyV3.tsx | 5 ++ .../src/app/mera-public/landing-facts.ts | 72 +++++++++++-------- 3 files changed, 117 insertions(+), 28 deletions(-) create mode 100644 tradein-mvp/frontend/src/app/mera-public/__tests__/backtest-window-labels.test.tsx diff --git a/tradein-mvp/frontend/src/app/mera-public/__tests__/backtest-window-labels.test.tsx b/tradein-mvp/frontend/src/app/mera-public/__tests__/backtest-window-labels.test.tsx new file mode 100644 index 00000000..27235f9f --- /dev/null +++ b/tradein-mvp/frontend/src/app/mera-public/__tests__/backtest-window-labels.test.tsx @@ -0,0 +1,68 @@ +/** + * Подписи бэктеста обязаны описывать ТО окно, по которому числа посчитаны. + * + * 31.08.2026 числа перевели на представительную выборку (медиана трёх + * пересборок), но подписи остались от прежнего прогона: «сделки II квартала + * 2026 года» и знаменатель 5 954 (только этот квартал). Замер показал, что + * пересборка покрывает ЧЕТЫРЕ квартала и II квартал 2026 — лишь 24 % её + * сделок. Доля выборки при старом знаменателе выходила 5,5 % вместо 1,3 %, + * то есть вчетверо льстила. + * + * Отдельно: плитка «уверенность низкая» показывалась БЕЗ своей оговорки — + * `AccuracyV3` собирал оговорки только двух величин из трёх. + */ +import { render, screen } from "@testing-library/react"; +import { describe, expect, it } from "vitest"; + +import { AccuracyV3 } from "../_components/v3/AccuracyV3"; +import { + BACKTEST, + BACKTEST_PERIOD_LABEL, + BACKTEST_POPULATION, + BACKTEST_SHARE_LABEL, +} from "../landing-facts"; + +describe("подписи окна бэктеста", () => { + it("окно названо целиком, а не одним кварталом из четырёх", () => { + // Прежняя подпись называла II квартал 2026 — в представительной выборке + // это 24 % сделок, а не все. + expect(BACKTEST_PERIOD_LABEL).not.toMatch(/^сделки II квартала/u); + expect(BACKTEST_PERIOD_LABEL).toMatch(/четыре квартала/u); + }); + + it("знаменатель — того же окна, что и выборка", () => { + // 5 954 — только II квартал 2026; окно целиком — 24 333 (замер 31.08.2026). + expect(BACKTEST_POPULATION).toBe(24_333); + }); + + it("доля выборки выводится и получается около 1,3 %, а не 5,5 %", () => { + const share = (BACKTEST.priceError.sampleN / BACKTEST_POPULATION) * 100; + expect(share).toBeGreaterThan(1.0); + expect(share).toBeLessThan(2.0); + expect(BACKTEST_SHARE_LABEL).toBe("1,3 %"); + }); + + it("уверенность посчитана на том же прогоне, что и остальные числа", () => { + // «325 из 327» было из прогона 29.08 и под общей подписью «замер 31.08» + // приписывало старому числу новую дату. + expect(BACKTEST.confidenceLow.text).not.toMatch(/327/u); + }); +}); + +describe("оговорки в интерфейсе", () => { + it("на экране есть оговорки ВСЕХ ТРЁХ величин блока", () => { + render(); + for (const набор of [ + BACKTEST.priceError.caveats, + BACKTEST.coverage.caveats, + BACKTEST.confidenceLow.caveats, + ]) { + for (const line of набор) { + expect( + screen.queryByText(line), + `оговорка не выводится: ${line.slice(0, 48)}…`, + ).not.toBeNull(); + } + } + }); +}); diff --git a/tradein-mvp/frontend/src/app/mera-public/_components/v3/AccuracyV3.tsx b/tradein-mvp/frontend/src/app/mera-public/_components/v3/AccuracyV3.tsx index 2be7c7d8..418fff0b 100644 --- a/tradein-mvp/frontend/src/app/mera-public/_components/v3/AccuracyV3.tsx +++ b/tradein-mvp/frontend/src/app/mera-public/_components/v3/AccuracyV3.tsx @@ -114,9 +114,14 @@ export function AccuracyV3({ }); } + // Оговорки ВСЕХ ТРЁХ величин блока, а не двух. `confidenceLow` показывается + // отдельной плиткой, но её оговорка сюда не попадала — то есть на экране + // стояло «400 из 400 — уверенность низкая» без единого слова о том, почему. + // Плитка без своей оговорки — ровно то, что этот блок и не должен делать. const caveats = [ ...BACKTEST.priceError.caveats, ...BACKTEST.coverage.caveats, + ...BACKTEST.confidenceLow.caveats, ]; const deals = showcase?.deals ?? []; diff --git a/tradein-mvp/frontend/src/app/mera-public/landing-facts.ts b/tradein-mvp/frontend/src/app/mera-public/landing-facts.ts index 9b0aef3a..d0688d7c 100644 --- a/tradein-mvp/frontend/src/app/mera-public/landing-facts.ts +++ b/tradein-mvp/frontend/src/app/mera-public/landing-facts.ts @@ -97,14 +97,20 @@ export const BACKTEST: Readonly 0, + * `deal_date >= 2025-06-01`, город Екатеринбург): **24 333** сделки. + * + * ЗНАМЕНАТЕЛЬ БЕРЁТСЯ ИЗ ТОГО ЖЕ ОКНА, ЧТО И ВЫБОРКА — правило не изменилось, + * изменилось окно. Раньше здесь стояло 5 954 (только II квартал 2026), и это + * было верно для прогона 29.08, который в один квартал и укладывался. Для + * пересборок вразброс, покрывающих четыре квартала, тот же знаменатель дал бы + * долю вчетверо больше реальной — 5,5 % вместо 1,3 %. + * + * Обе величины проверены одним запросом: в окне 24 333, из них с + * `deal_date = 2026-04-01` — 5 954. */ -export const BACKTEST_POPULATION = 5_954; +export const BACKTEST_POPULATION = 24_333; /** * Доля выборки. ВЫВОДИТСЯ, а не вписывается: разъехаться с `sampleN` не может.