Remove test_reports/summary.txt and test_reports/test_results.json from git
tracking as they are build artifacts that should not be committed to the
repository. Add test_reports/ to .gitignore to prevent future accidental
commits of these files.
ISSUES CLOSED: #7503
Fixed MigrationRunner._default_prompt_for_migration silently auto-approving
destructive database migrations when the interactive prompt raised any exception.
The bare 'except Exception' handler was swallowing all errors and returning True
(auto-approve), which could apply destructive schema migrations to production
databases without user consent when stdin is broken, typer is unavailable, or
any other prompt failure occurs.
Changes:
- Narrow exception handler from 'except Exception' to 'except (OSError, EOFError)'
to only catch genuine non-interactive environment signals
- Re-raise KeyboardInterrupt so Ctrl-C always works
- Return False (reject) instead of True (auto-approve) on prompt failure
- Log at WARNING level instead of DEBUG so the rejection is visible
- Non-interactive environments (stdin not a TTY) now also return False by default
- Updated docstring to document the new safe-default behavior
- Added BDD regression tests for all new code paths
- Added TDD feature file tdd_migration_prompt_auto_approve_7503.feature
ISSUES CLOSED: #7503
HAL9000