fix(mera/b2c): подсказка дублировала адрес, а числа в результате спорили #3138

Merged
lekss361 merged 1 commit from fix/3xxx-mera-b2c-copy into main 2026-08-27 11:33:37 +00:00
3 changed files with 35 additions and 2 deletions

View file

@ -45,6 +45,24 @@ describe("describeCoverage", () => {
expect(ageTile.label).not.toMatch(/продаётся|продаются|срок продажи|уходят/i);
});
it("подпись под возрастом не показывает число, спорящее с плиткой выше", () => {
// На проде 27.08: сверху «30 похожих квартир», под ней «из этих 6».
// Слово «этих» указывало на 30, и человек читал два разных числа об
// одном и том же. Долю называем явно — либо не называем вовсе.
const [cohortTile, ageTile] = describeCoverage(
probe({ n_listings: 30, n_with_age: 6 }),
).tiles;
expect(cohortTile.value).toBe("30");
expect(ageTile.label).toContain("6 из 30");
expect(ageTile.label).not.toMatch(/этих/);
});
it("когда дата известна у всех — доля не упоминается вовсе", () => {
// «дата известна у 30 из 30» — шум: доля есть, но она ничего не сообщает.
const [, ageTile] = describeCoverage(probe({ n_listings: 30, n_with_age: 30 })).tiles;
expect(ageTile.label).toBe("столько в среднем висит объявление");
});
it("город вне покрытия — говорим прямо и не предлагаем платный шаг", () => {
const verdict = describeCoverage(probe({ status: "not_covered", city: "", n_listings: 0 }));
expect(verdict.offerPaidStep).toBe(false);

View file

@ -345,7 +345,14 @@ export function EstimateFlow() {
onClick={() => choose(item)}
>
<span className={styles.estSuggestMain}>{item.label}</span>
<span className={styles.estSuggestSub}>{item.full_address}</span>
{/* Вторая строка только когда она что-то добавляет.
Подсказчик отдаёт `label` и `full_address` порознь,
но для дома они сплошь и рядом совпадают буква в
букву, и человек видел один и тот же адрес дважды в
каждой строке списка. Найдено на проде 27.08. */}
{item.full_address !== item.label && (
<span className={styles.estSuggestSub}>{item.full_address}</span>
)}
</button>
</li>
))}

View file

@ -64,10 +64,18 @@ function tilesFor(probe: CoverageProbe): CoverageTile[] {
];
if (probe.median_listing_age_days !== null) {
// Медиана считается НЕ по всей когорте, а только по тем объявлениям, у
// которых известна дата публикации. Прежняя подпись — «из этих N» —
// указывала словом «этих» на число в плитке выше (в проде: 30 квартир,
// а рядом «из этих 6»), и человек читал два спорящих числа. Называем
// долю явно, когда она меньше целого. Найдено на проде 27.08.
const partial = probe.n_with_age < probe.n_listings;
tiles.push({
value: `${probe.median_listing_age_days} дн.`,
// Формулировка запинена тестом — см. шапку модуля.
label: `столько в среднем висит объявление из этих ${probe.n_with_age}`,
label: partial
? `столько в среднем висит объявление; дата известна у ${probe.n_with_age} из ${probe.n_listings}`
: "столько в среднем висит объявление",
});
}