Skip to content

Conversation

@nonvachara-paloit
Copy link
Contributor

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_action would be thrown from flutter_driver/lib/src/common/deserialization_factory.dart when a call to driver.sendTextInputAction(TextInputAction.done); was made despite the method sendTextInputAction is available for use since #106561.

Previous works has been done in #131776, I merely added tests.

Best regards.

@google-cla
Copy link

google-cla bot commented Nov 29, 2023

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.

@github-actions github-actions bot added a: tests "flutter test", flutter_test, or one of our tests framework flutter/packages/flutter repository. See also f: labels. labels Nov 29, 2023
@HansMuller HansMuller requested a review from chunhtai December 1, 2023 22:46
Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chunhtai chunhtai added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 7, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 7, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Dec 7, 2023

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.

  • Merge guidelines: A PR needs at least one approved review if the author is already part of flutter-hackers or two member reviews if the author is not a flutter-hacker before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@nonvachara-paloit
Copy link
Contributor Author

Hi. I'm not sure what could be the next step here?

@chunhtai
Copy link
Contributor

sorry I missed this pr. This needs a secondary review. I will ask for someone on the team to take a look

Copy link
Contributor

@gspencergoog gspencergoog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

32384589-a60f0e74-c078-11e7-9bc1-e5b5287aea9d

@chunhtai chunhtai added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 19, 2023
Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM stamp from a Japanese personal seal

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 21, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Dec 21, 2023

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.

@chunhtai chunhtai added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 21, 2023
@auto-submit auto-submit bot merged commit 90badf7 into flutter:master Dec 21, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 22, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 22, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 22, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 23, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 23, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 29, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 30, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 30, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 31, 2023
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 2, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 2, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 2, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 2, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 2, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 3, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 4, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 4, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 4, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 5, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 16, 2024
… allow sendTextInputAction usages through flutter_driver. (flutter/flutter#139197)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests "flutter test", flutter_test, or one of our tests autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants