-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Conversation
} | ||
|
||
@Test | ||
public void requestScopes_ResultErrorIfAccountIsNull() { |
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.
These tests were copied and pasted from the tests in example/android/app/src/test....
|
||
@RunWith(RobolectricTestRunner.class) | ||
@Config(sdk = Build.VERSION_CODES.P) |
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.
This was added due to an error that complained about missing sdk 20. I was only able to get it working after looking at https://stackoverflow.com/questions/56808485/robolectric-and-android-sdk-29/57261194#57261194. I'm assuming this could also be increased depending oon what version we want to run it on
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.
LGTM! And thanks to your recent change to the test config, we can even see that they are actually running on the bots :)
Fixes flutter/flutter#82470
Pre-launch Checklist
dart format
.)[shared_preferences]
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.