test: improve test coverage to 93% (Issue #16) - #18

Merged

KHolodilin merged 35 commits into

mainfrom

feature/issue-2-improve-test-coverage-95

Jan 23, 2026

Commits on Jan 21, 2026

Commits on Jan 22, 2026

Fix undefined mock_print variable in test_passwords.py
KHolodilin
committed
c1a2045
Fix linting errors: unused variables and code simplification
KHolodilin
committed
6070593
Fix linting errors: unused variable and import location
KHolodilin
committed
78766ad
Add ruff check to pre-commit hooks and ignore PLC0415 for imap.py
KHolodilin
committed
5b15898
Format test_passwords.py with ruff
KHolodilin
committed
1aaf466
docs: add mandatory pre-commit and ruff format checks before commit
KHolodilin
committed
fe6026e
docs: mark pre-commit hooks as required
KHolodilin
committed
3ae3369
docs: update README with new command structure and quick start guide
KHolodilin
committed
ad831a0
docs: fix remaining old command references in README
KHolodilin
committed
001eec2
fix: update test mocks to use correct ConfigLoader patch path
KHolodilin
committed
b67a3df
fix: improve test error messages and ensure file existence checks
KHolodilin
committed
5430955
fix: correct test assertions for UI methods and integration tests - Fix SMTP tests to check info() instead of print() when has_rich=False - Fix UI error tests to properly mock ConfigLoader.load() m…
KHolodilin
committed
925378c
fix: correct get_imap_password patch path and test assertions - Fix SMTP and UI tests to patch get_imap_password in smtp module instead of auth module - Fix password tests to patch sys.platform in …
KHolodilin
committed
ac0f003
fix: improve Path and stat.filemode patching in password permission tests - Use MockPathClass with __new__ to properly mock Path constructor - Add stat.filemode patch inside test context managers -…
KHolodilin
committed
ab2b4c5
fix: use function factory instead of class for Path patching in password tests - Replace MockPathClass with path_factory function for better compatibility - Add spec=Path to mock_path for better ty…
KHolodilin
committed
30ae30b
fix: use MagicMock with side_effect for Path patching in password tests - Ensure Path is properly mocked as callable using MagicMock(side_effect=path_factory) - Add assert_called() checks for mock_…
KHolodilin
committed
6461396
fix: correct indentation and variable name in permission warning tests - Fix indentation error in test_set_password_file_permission_warning_with_rich_console - Rename mock_filemode to mock_filemode…
KHolodilin
committed
d0249be
fix: rename mock_filemode to mock_filemode_patch in first test too - Rename mock_filemode to mock_filemode_patch in test_set_password_file_permission_warning - This ensures consistency and avoids P…
KHolodilin
committed
186f80b
fix: remove global stat.filemode patch decorator from permission tests - Remove @patch('stat.filemode') decorator that was patching global stat.filemode - Use only the local patch inside the test c…
KHolodilin
committed
8442da2
refactor: use UI instead of print for config validation warnings - Add optional ui parameter to validate_config, load_config, and ConfigLoader methods - Use ui.warn() when UI is available, fallback…
KHolodilin
committed
90d5536
fix: update tests to expect ui parameter in ConfigLoader.load() - Update test_validate_config_file_success to expect ui parameter - Update test_main_custom_config_path to check ui parameter in call…
KHolodilin
committed
3ed319e
fix: improve permission warning tests - Add explicit assertions for stat.filemode() calls - Simplify warning message checks - Ensure tests fail with clear error messages if stat.filemode() is not c…
KHolodilin
committed
4811e43

Commits on Jan 23, 2026