docs(sdd): record WU9's realized diff and size:exception
Same estimating lesson as every prior WU: the 350-450 forecast covered the state/model/UI work, not the matching test files, the 14 regenerated lib/l10n/gen/ files, or dart format correcting pre-existing drift in a touched test file (Dart SDK 3.12.0 skew, not new logic). Realized 1,822 changed lines; recorded as an accepted size:exception with a breakdown, not a scope-creep surprise.
This commit is contained in:
@@ -550,6 +550,19 @@ with a skip pill but cards still showing the old button row).
|
||||
**Spec refs**: `alarm-vacation-ranges` — Active Range Detection, Per-Alarm Pause-Impact Computation, Upcoming Ranges
|
||||
Query, Past-Ranges History Query, Vacation Summary Row Replaces the Inline Panel (destination screen)
|
||||
**Verify**: `flutter test test/estado/estado_alarmas_test.dart test/pantallas/pantalla_vacaciones_test.dart && flutter analyze && dart format --set-exit-if-changed $(git diff --name-only --diff-filter=ACM HEAD -- '*.dart')`
|
||||
|
||||
**`size:exception` — realized diff exceeded the ~350-450 forecast.** Commit `9dfcf0b`: **1,822 changed lines**
|
||||
(1,432+/390-) across 23 files. Breakdown: `estado_alarmas_test.dart` 558 (**mostly `dart format` reformatting
|
||||
pre-existing, unrelated test blocks it happened to touch** — the same Dart-SDK-3.12.0 skew as Engram
|
||||
`reference/dart-format-scope-hazard` id 2511, here as an unavoidable side effect of the mandatory scoped-format
|
||||
gate on a file this WU legitimately edits, not new test logic; the actual new WU9 tests are ~215 of those lines),
|
||||
new `pantalla_vacaciones.dart` 353, 14 regenerated `lib/l10n/gen/*.dart` files 334, new `pantalla_vacaciones_test.dart`
|
||||
224, `pantalla_alarmas.dart` 197 (net code MOVED OUT to the new screen file, not new logic), `estado_alarmas.dart`
|
||||
63, `alarma_musical.dart` 14, ARB sources 32, this docs file 47. Same "test files + generated l10n + reformatting
|
||||
of touched files aren't counted by a lib/-only estimate" pattern as every prior WU (Engram
|
||||
`reference/estimating-strict-tdd-diffs`, id 2514). Justification: the 4 query methods, the model addition, and the
|
||||
new screen are one cohesive vertical slice (state + model + UI) that WU8 explicitly deferred to this commit;
|
||||
splitting further would leave either dead query methods with no UI consumer or a UI screen with no queries to call.
|
||||
**New tests**: `pantalla_vacaciones_test.dart`
|
||||
**Modified tests**: `estado_alarmas_test.dart` (additions only)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user