Added by zluo over 6 years ago. Updated over 5 years ago.
0%
Description
we have atm mouse_set using coordinators, it will be helpful directly to use mouse_set by using menu label (id).
For multiple menu levels this is a good option.
Discussed with zlou in person - he mixed a mouse clicking problem in yast widgets with https://hackweek.suse.com/16/projects/make-yast-testing-independent-of-keyboard-shortcuts
From what I understand the real problem is that assert_and_click moves the mouse to a hidden place which in zlou's use case hides the menu he wants to click in a 2nd step.
so only give an option to leave mouse? i don't get fully understand, could you provide an instance to show the case or any other suggestion?
@mitiao Are you still working on this?
As I understand the issue, it would be about conditionally skipping
if ($old_mouse_coords->{x} > -1 && $old_mouse_coords->{y} > -1) {
return mouse_set($old_mouse_coords->{x}, $old_mouse_coords->{y});
}
else {
return mouse_hide();
}
in testapi::assert_and_click. So that would be another optional parameter to keep the mouse where the click was done.
the PRs has been approved and waited for merge.
@mkittler i am re-assigning this ticket to you since i am leaving today
could you help to take over the PRs and push them to repo? thanks.
Ok. So what's currently blocking this is that we have to sync os-autoinst and test code deployment. I'll also resolve the conflicts on the test code PR.
This is blocked by merging/deploying both PRs at the same time.
Both PRs are finally merged now.