From 3ea166607c0f9b902ca1021e4bcd9c45d4b4758d Mon Sep 17 00:00:00 2001 From: Jason Ross Date: Wed, 1 Jul 2026 08:41:01 -0500 Subject: [PATCH] test(reporting): compile instrumented SettingsScreen test with onReportProblem The #32/#33 change added a required onReportProblem parameter to SettingsScreen but left the existing instrumented SettingsScreenTest calling the old signature, so :app:compileDebugAndroidTestKotlin failed in every E2E job (the local fast gate never compiles the androidTest variant). Pass onReportProblem = {} in the test. Verified with :app:compileDebugAndroidTestKotlin on JDK 21. Co-Authored-By: Claude Opus 4.8 --- .../org/libremail/ui/settings/SettingsScreenTest.kt | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/app/src/androidTest/kotlin/org/libremail/ui/settings/SettingsScreenTest.kt b/app/src/androidTest/kotlin/org/libremail/ui/settings/SettingsScreenTest.kt index c3b6061..e3cdef4 100644 --- a/app/src/androidTest/kotlin/org/libremail/ui/settings/SettingsScreenTest.kt +++ b/app/src/androidTest/kotlin/org/libremail/ui/settings/SettingsScreenTest.kt @@ -39,7 +39,13 @@ class SettingsScreenTest { composeTestRule.setContent { LibreMailTheme(darkTheme = false, dynamicColor = false) { - SettingsScreen(onAddAccount = {}, onOpenAccount = {}, onSelectTab = {}, viewModel = viewModel) + SettingsScreen( + onAddAccount = {}, + onOpenAccount = {}, + onSelectTab = {}, + onReportProblem = {}, + viewModel = viewModel, + ) } }