migration-script-runner / msr-core / bd1928d0-f777-4eca-ba76-b48fa2b40b79

100%

Build Type

push

circleci

Committed by Vova Lavrynovych

Commit Message

#41: Remove process.exit() from MigrationScriptExecutor

Major improvement to enable library usage in long-running applications.

Changes:
- Added IMigrationResult interface with success, executed, migrated, ignored, and errors fields
- Updated migrate() to return Promise<IMigrationResult> instead of Promise<void>
- Removed exit() method that called process.exit()
- Added getIgnored() helper method to track skipped migrations
- Updated all 16 MigrationScriptExecutor tests to validate result objects
- Updated documentation (README, API Reference, Getting Started guide)
- Added comprehensive library usage examples

Benefits:
- MSR can now be used as a library without killing the host process
- Better testability - no need to mock process.exit
- Structured error handling with detailed migration results
- Safe integration into Express, NestJS, and other web frameworks
- Process lifecycle control is caller's responsibility
- Backward compatible - CLI wrappers can still use process.exit based on result.success

Run Details

35 of 38 branches covered (92.11%)

Branch coverage included in aggregate %.

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

238 of 238 relevant lines covered (100.0%)

70.87 hits per line