939019 - [UITest][UI] Merge textinput mode with Keyboard

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Tom Jao [:mbrsl]]

There two tests "text inputmode" and "Keyboard" in UITest app which provide rather similar functionality. We should merge them into one test.

Note that Keyboard test is used by gaia-ui-test and should be modified with care.

Comment 1

[Tom Jao [:mbrsl]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/14005

Zac, would you please verify the code change of test_keyboard_predictive_key.py for me?
I made some changes to the layout of this test and broke test_keyboard_predictive_key.py. The reason is that the second keyboard_page.tap_text_input() will click at the "center" of the text field. If the text field is too short (in this case), it will click on the characters and caret will not be at the end of the string. This makes predicative words disappear.

For example, we expected "keyboard|" but the result is "keyboa|rd"
| is the position of caret.

I don't know why would we needs to switch to system frame and then switch back. So I just remove them and we will not face the problem of second keyboard_page.tap_text_input()

Comment 2

[Zac C (:zac)]

Hey Tom, yes you are right, that excess frame switching was a bit of a fudge around because of the behaviour of the keyboard app object. We actually addressed that last week but I have not got to removing the redundant frame switching from tests yet. Thanks for doing it for me on this test!

Comment 3

[Zac C (:zac)]

Comment on attachment 8337643 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/14005

The python test changes are r+!

Comment 4

[[e-mail]]

Comment on attachment 8337643 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/14005

Looks better, thanks!

Comment 5

[[e-mail]]

merged in gaia-master https://github.com/mozilla-b2g/gaia/commit/f97aa61b01cadbeca0baf986a8acc9e4ecec2932