Unverified Commit ff2fdf40 authored by Greg Spencer's avatar Greg Spencer Committed by GitHub

Fix semantics testing now that dropdown can take a11y focus again. (#42475)

Fixes the android_semantics_testing integration test to have the right criteria now that ce150971 has landed.
parent ffcd0cd1
......@@ -369,10 +369,9 @@ void main() {
isEnabled: true,
isFocusable: true,
actions: <AndroidSemanticsAction>[
// TODO(gspencergoog): This should really be clearAccessibilityFocus,
// but TalkBack doesn't find it for some reason.
// https://github.com/flutter/flutter/issues/40101
AndroidSemanticsAction.accessibilityFocus,
if (item == popupItems.first) AndroidSemanticsAction.clearAccessibilityFocus,
if (item != popupItems.first) AndroidSemanticsAction.accessibilityFocus,
AndroidSemanticsAction.click,
],
),
reason: "Popup $item doesn't have the right semantics");
......@@ -395,9 +394,10 @@ void main() {
isFocusable: true,
actions: <AndroidSemanticsAction>[
// TODO(gspencergoog): This should really be clearAccessibilityFocus,
// but TalkBack doesn't find it for some reason.
// but TalkBack doesn't focus it the second time for some reason.
// https://github.com/flutter/flutter/issues/40101
AndroidSemanticsAction.accessibilityFocus,
AndroidSemanticsAction.click,
],
),
reason: "Popup $item doesn't have the right semantics the second time");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment