Idrinth / habit-evaluator / 22279806139 / 6

40%

the-one: 40%

push

github

Migrate Android app from Java/XML to Kotlin/Compose (#559)

* Migrate Android module from Java/XML to Kotlin/Jetpack Compose

Replace all Java activities, fragments, adapters, custom views, and SQLite
persistence with Kotlin equivalents using Jetpack Compose UI, Room database,
and MVVM architecture with ViewModels. Remove all XML layouts and menu
resources in favor of composable screens.

https://claude.ai/code/session_013xoEN7WmkP2zc7YUowuUZp

* Raise lollipop flavor minSdk from 21 to 23 for Compose compatibility

Jetpack Compose Material requires minSdk 23. The lollipop flavor was
still set to 21, causing manifest merger failures during build.

https://claude.ai/code/session_013xoEN7WmkP2zc7YUowuUZp

* Fix Kotlin compilation warnings in Android Compose screens

- Remove unnecessary non-null assertion in EmotionLineChart.kt
- Replace deprecated MenuAnchorType with ExposedDropdownMenuAnchorType
  in AddHabitScreen, DiaryScreen, EditHabitsScreen, HomeScreen
- Replace deprecated menuAnchor() with menuAnchor(ExposedDropdownMenuAnchorType.PrimaryNotEditable)
  in CorrelationScreen, MedicationListScreen, MedicationLogScreen
- Remove always-true null check on primitive double in SportLogScreen

https://claude.ai/code/session_013xoEN7WmkP2zc7YUowuUZp

* Rewrite Room persistence tests as Mockito-based unit tests

The Room repository tests used AndroidJUnit4, ApplicationProvider, and
in-memory Room databases (androidTest dependencies) but were placed in
src/test/ where only unit test deps are available. Rewritten all 16
persistence tests to use JUnit 5 with Mockito DAO mocking.

Also removed invalid null-context test from PreApi33PermissionHandlerTest
since Context is non-null in the NotificationPermissionHandler interface.

https://claude.ai/code/session_013xoEN7WmkP2zc7YUowuUZp

* Fix JVM signature clash in EmotionScatterChartTest ScatterPair

The secondary constructor taking List<ScatterEntry>? had the same
JVM signature as the primary constructor taking Mu... (continued)

3 of 7 branches covered (42.86%)

Branch coverage included in aggregate %.

36 of 45 relevant lines covered (80.0%)

3.02 hits per line