fix: remove dead code and optimize cache cleanup

[bivlked]

Описание

Исправление выявленных проблем из анализа проекта и оптимизация производительности кэша.

Изменения

Исправления

• 🐛 Удален мертвый код: убраны недостижимые проверки if rate is None после вызова get_currency_rate() в currency_converter.py
• 🔧 Исправлена сигнатура функции: изменен возвращаемый тип get_currency_rate() с Optional[float] на float в cbr_parser.py
• 📝 Обновлена документация: исправлен docstring get_currency_rate() для соответствия фактическому поведению

Оптимизации

• ⚡ Оптимизирован кэш: реализована ленивая очистка кэша (только при переполнении)
• 🎯 Улучшена производительность: проверка TTL в get() теперь выполняется только для конкретной записи (O(1) вместо O(n))

Тесты

• 🧪 Обновлены тесты для соответствия новой логике
• ✅ Все тесты проходят (91 passed, 6 skipped)
• 📊 Покрытие тестами: 77.40% (выше требуемых 70%)
• 🔍 Pylint: 10.00/10

Файлы изменений

• src/currate/currency_converter.py - удален мертвый код
• src/currate/cbr_parser.py - исправлена сигнатура и docstring
• src/currate/cache.py - оптимизирована очистка кэша
• tests/test_cache_extended.py - обновлен тест
• tests/test_currency_converter_extended.py - обновлен тест
• CHANGELOG.md - обновлена документация изменений

Проверка

• Все тесты проходят
• Покрытие тестами ≥ 70% (77.40%)
• Pylint проверка пройдена (10.00/10)
• Документация обновлена
• Код соответствует стандартам проекта

Связанные задачи

Исправление проблем, выявленных в анализе проекта CurRate.

---

Note

Aligns get_currency_rate() to always return float, removes unreachable None checks, optimizes cache with lazy cleanup and per-key TTL checks, and updates tests/changelog accordingly.

• Core changes:
  • Cache (src/currate/cache.py): Lazy cleanup on set() only when capacity is hit; TTL checked per entry in get(); preserves LRU behavior.
  • Parser (src/currate/cbr_parser.py): get_currency_rate() return type changed from Optional[float] to float; docstring clarified (no None, explicit exceptions).
  • Converter (src/currate/currency_converter.py): Removed dead if rate is None branches after get_currency_rate(); error handling via exceptions only.
• Tests:
  • Updated to reflect non-None rate behavior and lazy cache cleanup (patched datetime, explicit cleanup_expired() where needed).
• Docs:
  • CHANGELOG.md updated with fixes, performance notes, and test/quality metrics.

^{Written by Cursor Bugbot for commit 2e2d0ff. This will update automatically on new commits. Configure here.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!