migration-script-runner / msr-core / 6d07eb39-49dd-4778-831b-dae92b0a808b

100%

Build Type

push

circleci

Committed by Vova Lavrynovych

Commit Message

#66: Add validation documentation and achieve 100% branch coverage

Added comprehensive validation documentation covering built-in rules, custom validators, and configuration options. Refactored monolithic configuration.md into organized sections for better navigation.

Documentation Changes:
- Added validation/ directory with 3 comprehensive guides:
  * index.md - Validation overview and quick start
  * built-in-validation.md - All 8 validation rules with examples
  * custom-validation.md - Custom validator guide with 7 examples
- Split configuration.md (721 lines) into 5 focused documents:
  * index.md, migration-settings.md, validation-settings.md, rollback-settings.md, backup-settings.md
- Updated guides/hooks.md with backup/restore monitoring example
- Updated index.md with validation features

Code Changes:
- Refactored MigrationScriptExecutor.ts lines 671-681 to use explicit if checks instead of optional chaining for restore hooks
- Matches existing pattern at lines 790-793 (onBeforeMigrate hook)
- Added unit tests for restore hooks (4 comprehensive tests)
- Properly typed mocks with no unsafe casts

No breaking changes

Run Details

386 of 386 branches covered (100.0%)

Branch coverage included in aggregate %.

276 of 276 new or added lines in 16 files covered. (100.0%)

843 of 843 relevant lines covered (100.0%)

641.2 hits per line