Closed Bug 939019 Opened 8 years ago Closed 8 years ago

[UITest][UI] Merge textinput mode with Keyboard


(Firefox OS Graveyard :: Gaia, defect)

Gonk (Firefox OS)
Not set


(Not tracked)



(Reporter: tjao, Assigned: tjao)




(1 file)

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.
Assignee: nobody → tjao
Blocks: eng-mode
Zac, would you please verify the code change of for me?
I made some changes to the layout of this test and broke 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()
Attachment #8337643 - Flags: review?(zcampbell)
Attachment #8337643 - Flags: review?(gasolin)
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 on attachment 8337643 [details] [review]

The python test changes are r+!
Attachment #8337643 - Flags: review?(zcampbell) → review+
Comment on attachment 8337643 [details] [review]

Looks better, thanks!
Attachment #8337643 - Flags: review?(gasolin) → review+
merged in gaia-master
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.