migration-script-runner / msr-core / c50aeafa-b342-4a63-96d4-03b25a237a9a

100%

Build Type

push

circleci

Committed by Vova Lavrynovych

Commit Message

#68: Refactor MigrationScriptExecutor to reduce complexity

Extract RollbackService to isolate rollback logic following Single Responsibility Principle.

Changes:
- Created RollbackService with all 4 rollback strategies (BACKUP, DOWN, BOTH, NONE)
- Created IRollbackService interface for dependency injection
- Reduced MigrationScriptExecutor by ~15% (1,229 to ~1,050 lines)
- Removed 7 private rollback methods, replaced with rollbackService delegation

Testing:
- Added 31 tests for RollbackService
- All 565 tests passing with 100% coverage

Documentation:
- Updated architecture.md with RollbackService workflows
- Added IRollbackService API documentation with examples

Benefits:
- Better separation of concerns and testability
- No breaking changes - public API unchanged

Run Details

513 of 513 branches covered (100.0%)

Branch coverage included in aggregate %.

73 of 73 new or added lines in 4 files covered. (100.0%)

972 of 972 relevant lines covered (100.0%)

564.86 hits per line