Files
houseplan-card/test/space-dialog.test.mjs
T
Claude 91a008db9c feat(space): диалог «Пространство» по референсу — серия 1 из четырёх (#600)
§4 SPEC.md референса, четыре карточки: Basics (имя и масштаб в сетке 1fr/175,
подложка двумя choice-карточками, callout о пересчёте масштаба), Appearance
(строка-тумблер границ с иконкой и подписью, сегмент стиля стен с образцами
линии, плашка цвета с hex/прозрачностью/Reset, сегмент заливки с деталями
режима, компактный список «Видимые слои» с логикой «включено = видно» — К2),
Room cards (строка-тумблер названий, четыре плитки значений с отключением и
callout-ссылкой, слайдер размера с числом, концами и «Сбросить до 100%», образец
карточки внутри блока на тинте — Q2), Sun & light (сегмент фона, плашка цвета
фона, радио «свет через окна» в строку, север селектом с компасом 44 px и полем
градусов, строка-тумблер свечения).

Одна форма на два рантайма. `renderSpaceForm` в `editors/space-form.ts`
собирает тело и футер; редакторский вход (`space-settings-dialog.ts`) и
онбординг подают порт с тем, что у них различается, и сами рисуют
`<hp-dialog data-kind=…>` — контракт #489 требует литеральный kind. Копия
разметки онбординга на 300 строк удалена; локальные дубли CELL_CM_* и
strictNumber там же — тоже («одно число — один источник»).

К10 (решение владельца Q1): Save активен только при изменениях и без ошибок,
в футере «Есть несохранённые изменения» либо ссылка «Проверьте поля: N» к
первому ошибочному; «✕», Cancel и Escape с изменениями спрашивают через
`_confirmDanger`, без изменений закрывают сразу. Снимок делается при открытии
(`editors/space-form-state.ts`, WeakMap по хосту) — `SpaceDialogState` не
получил ни одного ключа. Валидация: имя, подложка без изображения, диапазон
температур, север 0–359 — с `aria-invalid` и текстом под полем.

Строки. 51 новый ключ живёт в ленивом словаре `i18n/settings` (приём
#423/#459), не в первом кадре: стартовый граф 291 260 при потолке 292 500.
Отдельный словарь, а не `support`, — иначе онбординг тянул бы весь словарь
редактора ради одной формы. 11 существующих ключей переименованы по Q6
(«Space name», «Appearance», «Room cards», «Draw it myself», «Zigbee signal
next to devices»…), 9 мёртвых удалены из четырёх локалей. `hp-color-opacity`
получил `hide-label`: подпись рисует плашка, `label` остаётся для title/aria и
`smoke_color_picker_consumers` проходит без правок (AC5).

Смоки. Переписаны осознанно (К6 снят), каждый — с пометкой #600 у правки:
smoke_hide_layers (компактные строки, инверсия К2 проверяется кликом),
smoke_ux_fixes (сегмент, строка-тумблер, поля °C, ширина 560 — Q7),
smoke_dialog_footer_width (560 для form-shell), smoke_temp_fill (два поля по
id), smoke_settings_dialog_cards (три сегмента), smoke_space_settings и
smoke_space_create_display_defaults и smoke_lazy_editor_chunk (choice-карточка
по смыслу, не по индексу — порядок §4.2 draw → file), smoke_space_scale_defaults
(поле с единицей). Новый фокусный свидетель demo/smoke_space_settings_form.mjs:
состав карточек и отсутствие старой разметки, оболочка 560 и один скроллер,
цели 44, запись каждого нового контрола в свой ключ, Save/dirty/вопрос при
закрытии, семь контрольных ширин (320…1920).

Мутанты: space-fill-segment-writes-a-neighbour-key,
space-save-enabled-without-changes, space-discard-without-asking,
space-layer-toggle-loses-inversion.

Контрактные тесты: `houseplan-source.mjs` дописывает общие модули диалогов к
логическому исходнику; коллектор i18n видит `help(...)` порта и `shelp(...)`
(заодно встал на учёт `radar.help`, проходивший мимо счётчика — 30 → 35).

Потолок ленивого редакторского графа 225 900 → 236 300 (замер 235 560) с
разбором в bundle-budget.mjs: словарь на четыре локали и подписи — большая
часть роста; промежуточно, итог в конце серий.

Гейты: typecheck, build, bundle:sync, npm test 2792 pass (красным был только
потолок — поднят с доводом), 10 смоков пространства зелёные,
smoke_color_picker_consumers без правок, mutation-gate --check.

Issue: #600
User-Visible: no
2026-09-20 10:28:51 +03:00

125 lines
6.0 KiB
JavaScript

import test from 'node:test';
import assert from 'node:assert/strict';
import { readFileSync } from 'node:fs';
import {
createEmptySpaceConfig,
initialSpaceDisplayDraft,
switchSpacePlanSource,
touchSpaceDisplay,
applyRoomTempThresholdDraft,
roomTempRangeFromDraft,
roomTempThresholdDraft,
roomTempThresholdInputValues,
} from '../test-build/space-dialog.js';
test('room temperature draft distinguishes inheritance, zero, reversal and invalid text (#487)', () => {
assert.deepEqual(roomTempThresholdDraft('', ' '), { valid: true, min: null, max: null });
assert.deepEqual(roomTempThresholdDraft('0', '25'), { valid: true, min: 0, max: 25 });
assert.deepEqual(roomTempThresholdDraft('24,5', '18,5'), { valid: true, min: 18.5, max: 24.5 });
assert.deepEqual(roomTempThresholdDraft('', '19'), { valid: true, min: null, max: 19 });
assert.deepEqual(roomTempThresholdDraft('oops', '25'), { valid: false, min: null, max: 25 });
});
test('room temperature settings writer preserves unknown fields and removes only cleared sides (#487)', () => {
const settings = { future: 'keep', temp_min: 18, temp_max: 24 };
assert.equal(applyRoomTempThresholdDraft(settings, '', '0'), true);
assert.deepEqual(settings, { future: 'keep', temp_max: 0 });
assert.deepEqual(roomTempThresholdInputValues(settings), ['', '0']);
assert.equal(applyRoomTempThresholdDraft(settings, 'bad', '25'), false);
assert.deepEqual(settings, { future: 'keep', temp_max: 0 });
assert.deepEqual(roomTempThresholdInputValues({ temp_min: Infinity, temp_max: '20' }), ['', '']);
});
test('room temperature preview uses valid drafts and falls back on invalid drafts (#487)', () => {
const room = { settings: { temp_min: 18, temp_max: 22 } };
assert.deepEqual(roomTempRangeFromDraft(20, 25, room, '19', '21', true), { min: 19, max: 21 });
assert.deepEqual(roomTempRangeFromDraft(20, 25, room, 'bad', '21', true), { min: 18, max: 22 });
assert.deepEqual(roomTempRangeFromDraft(20, 25, room, '', '19', true), { min: 19, max: 20 });
assert.deepEqual(roomTempRangeFromDraft(20, 25, room, '19', '21', false), { min: 18, max: 22 });
});
test('a fresh space is a complete empty v8/v9 wall-model document (#324)', () => {
assert.deepEqual(createEmptySpaceConfig('floor', 'Ground floor'), {
id: 'floor',
title: 'Ground floor',
plan_url: null,
view_box: [0, 0, 1, 1],
rooms: [],
wall_segments: [],
});
});
test('fresh create projects honest File and Draw display defaults', () => {
const file = initialSpaceDisplayDraft();
assert.deepEqual(file, {
source: 'file', showBorders: false, showNames: false, displayTouched: false,
});
const draw = switchSpacePlanSource(file, 'draw');
assert.deepEqual(draw, {
source: 'draw', showBorders: true, showNames: true, displayTouched: false,
});
assert.deepEqual(switchSpacePlanSource(draw, 'file'), file);
});
test('touching either display switch preserves the complete mixed pair', () => {
const draw = switchSpacePlanSource(initialSpaceDisplayDraft(), 'draw');
const mixed = touchSpaceDisplay(draw, 'showBorders', false);
assert.deepEqual(mixed, {
source: 'draw', showBorders: false, showNames: true, displayTouched: true,
});
assert.deepEqual(switchSpacePlanSource(mixed, 'file'), {
...mixed, source: 'file',
});
assert.deepEqual(switchSpacePlanSource(mixed, 'draw'), mixed);
});
test('every supported Draw pair remains exact after a touched source round-trip', () => {
for (const [showBorders, showNames] of [
[false, false], [true, false], [false, true],
]) {
let draft = switchSpacePlanSource(initialSpaceDisplayDraft(), 'draw');
draft = touchSpaceDisplay(draft, 'showBorders', showBorders);
draft = touchSpaceDisplay(draft, 'showNames', showNames);
const roundTrip = switchSpacePlanSource(switchSpacePlanSource(draft, 'file'), 'draw');
assert.deepEqual(roundTrip, {
source: 'draw', showBorders, showNames, displayTouched: true,
});
}
});
test('source and display transitions are immutable and preserve unrelated fields', () => {
const draft = { ...initialSpaceDisplayDraft(), title: 'Ground', cellCm: 7.5 };
const switched = switchSpacePlanSource(draft, 'draw');
const touched = touchSpaceDisplay(switched, 'showNames', false);
assert.notEqual(switched, draft);
assert.notEqual(touched, switched);
assert.deepEqual(draft, {
source: 'file', showBorders: false, showNames: false, displayTouched: false,
title: 'Ground', cellCm: 7.5,
});
assert.equal(touched.title, 'Ground');
assert.equal(touched.cellCm, 7.5);
});
test('#456 Copy is an edit-settings action and is absent from onboarding', () => {
// #592: разметка диалога пространства уехала в свой модуль целиком и
// побайтово. Утверждения прежние — сменился только файл, в котором они
// ищутся; регулярки намеренно оставлены дословными, чтобы перенос не смог
// проскочить под видом «поправили тест».
// #600: форма общая (`space-form.ts`), Copy рисуется только если порт подал
// `copySpace`, а подаёт его один редакторский вход. Онбординг, как и прежде,
// о копировании не знает ни словом.
const entry = readFileSync(
new URL('../src/editors/space-settings-dialog.ts', import.meta.url), 'utf8',
);
const form = readFileSync(new URL('../src/editors/space-form.ts', import.meta.url), 'utf8');
const onboarding = readFileSync(new URL('../src/houseplan-onboarding-runtime.ts', import.meta.url), 'utf8');
assert.match(entry, /copySpace: \(\) => openSpaceCopyDialog\(this\.host\)/);
assert.match(form, /d\.mode === 'edit' && port\.copySpace[\s\S]*?<div class="dialog-action-group">[\s\S]*?btn\.copy/);
assert.doesNotMatch(form, /dialog-action-danger[\s\S]{0,300}btn\.copy/);
assert.doesNotMatch(onboarding, /btn\.copy|openSpaceCopyDialog|space\.copy_|copySpace/);
});