Fix test by making the whole ListItem clickable.
Also improve height of View to let the test see the View.
This commit is contained in:
parent
cb2007f93d
commit
223d0632cf
3 changed files with 4 additions and 3 deletions
|
|
@ -82,6 +82,7 @@ class DeveloperSettingsViewTest {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Config(qualifiers = "h1024dp")
|
||||||
@Test
|
@Test
|
||||||
fun `clicking on configure tracing invokes the expected callback`() {
|
fun `clicking on configure tracing invokes the expected callback`() {
|
||||||
val eventsRecorder = EventsRecorder<DeveloperSettingsEvents>(expectEvents = false)
|
val eventsRecorder = EventsRecorder<DeveloperSettingsEvents>(expectEvents = false)
|
||||||
|
|
@ -96,7 +97,7 @@ class DeveloperSettingsViewTest {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Config(qualifiers = "h1024dp")
|
@Config(qualifiers = "h1500dp")
|
||||||
@Test
|
@Test
|
||||||
fun `clicking on clear cache emits the expected event`() {
|
fun `clicking on clear cache emits the expected event`() {
|
||||||
val eventsRecorder = EventsRecorder<DeveloperSettingsEvents>()
|
val eventsRecorder = EventsRecorder<DeveloperSettingsEvents>()
|
||||||
|
|
|
||||||
|
|
@ -47,6 +47,7 @@ fun PreferenceCheckbox(
|
||||||
) {
|
) {
|
||||||
ListItem(
|
ListItem(
|
||||||
modifier = modifier,
|
modifier = modifier,
|
||||||
|
onClick = onCheckedChange.takeIf { enabled }?.let { { onCheckedChange(!isChecked) } },
|
||||||
leadingContent = preferenceIcon(
|
leadingContent = preferenceIcon(
|
||||||
icon = icon,
|
icon = icon,
|
||||||
iconResourceId = iconResourceId,
|
iconResourceId = iconResourceId,
|
||||||
|
|
@ -71,7 +72,6 @@ fun PreferenceCheckbox(
|
||||||
},
|
},
|
||||||
trailingContent = ListItemContent.Checkbox(
|
trailingContent = ListItemContent.Checkbox(
|
||||||
checked = isChecked,
|
checked = isChecked,
|
||||||
onChange = onCheckedChange,
|
|
||||||
enabled = enabled,
|
enabled = enabled,
|
||||||
),
|
),
|
||||||
)
|
)
|
||||||
|
|
|
||||||
|
|
@ -46,6 +46,7 @@ fun PreferenceSwitch(
|
||||||
ListItem(
|
ListItem(
|
||||||
modifier = modifier,
|
modifier = modifier,
|
||||||
enabled = enabled,
|
enabled = enabled,
|
||||||
|
onClick = onCheckedChange.takeIf { enabled }?.let { { onCheckedChange(!isChecked) } },
|
||||||
leadingContent = preferenceIcon(
|
leadingContent = preferenceIcon(
|
||||||
icon = icon,
|
icon = icon,
|
||||||
iconResourceId = iconResourceId,
|
iconResourceId = iconResourceId,
|
||||||
|
|
@ -68,7 +69,6 @@ fun PreferenceSwitch(
|
||||||
},
|
},
|
||||||
trailingContent = ListItemContent.Switch(
|
trailingContent = ListItemContent.Switch(
|
||||||
checked = isChecked,
|
checked = isChecked,
|
||||||
onChange = onCheckedChange,
|
|
||||||
enabled = enabled,
|
enabled = enabled,
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue