Idrinth / habit-evaluator / 22040173239

40%

push

github

Add comprehensive test coverage for controllers and repositories (#398)

* Improve desktop module test coverage with 50+ new test methods

Add comprehensive test coverage across the desktop module:

- MainController: Tests for applyFilter, populateCategoryComboBoxes,
  clearPointCharts, updatePointCharts (week/month), getClientVersion,
  handleSaveEditedHabits (target, maxEntries, positiveScoring,
  thresholds with validation), updateDiarySuggestions, loadDiaryEntries,
  loadCategories, handleAddDiaryEntry edge cases, shutdown, and more
- AddHabitController: Tests for buildTranslationFields, createCategory
  (local repo), populateCategoryComboBox ID mapping
- StatsController: Tests for multiple habits, multiple sleep entries,
  midnight-crossing sleep, multiple emotion pairs, empty emotion data,
  multiple diary entries
- PdfExportController: Tests for date swap logic, valid date pass-through,
  null repository handling
- H2DiaryEntryRepository: Tests for findEntriesNeedingMigration
  (legacy entries, null descriptions, non-existent user, multiple entries)
- HabitEvaluatorDesktopApp: New test class covering shouldUseDarkMode,
  isSystemDarkMode, stop with null/initialized services, config path

All 349 desktop tests pass.

https://claude.ai/code/session_01HhzprETTZTjXhux2nr1XWz

* Fix assertions in testHandleExportWithValidDatesPassesValidation to always run

Move assertNotEquals checks from inside the catch block to after the
try-catch so they execute regardless of whether an exception is thrown.

https://claude.ai/code/session_01HhzprETTZTjXhux2nr1XWz

---------

Co-authored-by: Claude <noreply@anthropic.com>

2702 of 6597 branches covered (40.96%)

Branch coverage included in aggregate %.

8963 of 18905 relevant lines covered (47.41%)

2.2 hits per line