-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Add send_text_input_action case to deserialization_factory to allow sendTextInputAction usages through flutter_driver. #139197
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
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
chunhtai
left a comment
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
|
auto label is removed for flutter/flutter/139197, due to This PR has not met approval requirements for merging. The PR author is not a member of flutter-hackers and needs 1 more review(s) in order to merge this PR.
|
|
Hi. I'm not sure what could be the next step here? |
|
sorry I missed this pr. This needs a secondary review. I will ask for someone on the team to take a look |
gspencergoog
left a comment
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.
cbracken
left a comment
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.
|
auto label is removed for flutter/flutter/139197, due to - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label. |
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)


As a follow up to #131776.
Summary:
Previously in #106561, SendTextInputAction was added to Flutter Driver.
But it still cannot be used from flutter_driver tests. This PR intends to resolve that issue.
Issue:
An
DriverError: Unsupported command kind send_text_input_actionwould be thrown fromflutter_driver/lib/src/common/deserialization_factory.dartwhen a call todriver.sendTextInputAction(TextInputAction.done);was made despite the methodsendTextInputActionis available for use since #106561.Previous works has been done in #131776, I merely added tests.
Best regards.