GrottoCenter / grottocenter-api / 32745063391

87%

develop: 91%

Build Type

Pull #1779

github

Committed by ClemRz

Commit Message

fix(document): address code review findings

- P1: add DocumentService.clearParentIfTypeDisallows() helper and call
  it in update-with-new-entities.js so both update paths enforce the
  same type-vs-parent policy (stale parent cleared on type change)
- P2: migration now uses DROP CONSTRAINT IF EXISTS + ADD NOT VALID +
  VALIDATE CONSTRAINT for idempotency and safe application over
  pre-existing corrupt rows
- P2: guard checkParentCycle() call sites with Number.isFinite() to
  prevent a NaN false-negative when ids are non-numeric
- P3: TYPES_REQUIRING_PARENT is now an alias of TYPES_ALLOWING_PARENT
  with a comment explaining the equivalence
- P3: rename misleading test title to reflect actual assertion

Pull Request Pull Request #1779: fix(document): prevent cyclic parent relationships

Coverage Stats

5193 of 6240 branches covered (83.22%)

Branch coverage included in aggregate %.

33 of 40 new or added lines in 5 files covered. (82.5%)

2 existing lines in 1 file now uncovered.

9558 of 10656 relevant lines covered (89.7%)

142.35 hits per line