-
Notifications
You must be signed in to change notification settings - Fork 180
Run UI tests on API 34 #3723
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Run UI tests on API 34 #3723
Conversation
@@ -32,11 +34,21 @@ class ContextMenusTest { | |||
dispatcher = AndroidAssetDispatcher() | |||
start() | |||
} | |||
|
|||
// Prevent the System UI from reading the clipboard content | |||
if (Build.VERSION.SDK_INT == Build.VERSION_CODES.UPSIDE_DOWN_CAKE) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should these checks be >=
if not because we don't know, we should add a comment as the known tested version up to Android 14
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure, since we didn't verify all APIs and it was only required for 34 we are only checking for it.
fun clickSystemHomeScreenShortcutAddButton() { | ||
when (Build.VERSION.SDK_INT) { | ||
in Build.VERSION_CODES.O..Build.VERSION_CODES.R -> clickAddAutomaticallyButton() | ||
in Build.VERSION_CODES.S..Build.VERSION_CODES.UPSIDE_DOWN_CAKE -> clickAddToHomeScreenButton() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just add a comment stating that this is up to Android 14 as a known tested version unless we know Android 15 is the same, we could just extend to VANILLA_ICE_CREAM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done !
25053b5
to
dfb912a
Compare
Notes
All UI tests successfully passed 2x on Firebase ✅
Pull Request checklist