Skip to content
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

moveCursorTo не идентичен moveTo в hermione 7 (move target out of bounds) #878

Closed
1 task
hheimerd opened this issue Mar 21, 2024 · 5 comments
Closed
1 task

Comments

@hheimerd
Copy link

Verify latest release

  • I verified that the issue exists in the latest Hermione release

Hermione version

8.5.0

Which area(s) of Hermione are affected? (leave empty if unsure)

No response

Link to the code that reproduces this issue or a replay of the bug

https://github.com/gemini-testing/hermione/blob/4dc6c2fd812e6a3c9d4d82671d59187049d52761/src/browser/commands/moveCursorTo.ts#L27

Reproduction steps

  1. создать окно браузера высотой в 900px
  2. сделать body высотой 2000px
  3. прокрутить окно вниз на 200px
    4.1. вызвать browser.$('body').moveCursorTo()
    4.2 вызвать browser.$('body').moveCursorTo({yOffset: 1200})

Describe the Bug

В обоих случаях произойдет ошибка "move target out of bounds"

Предполагаю это из-за того, что вьюпорт высотой 900px, а move происходит за его пределы (-200px и 1200px) соответственно.

Expected Behavior

Должен происходить сролл до нужного участка и мышь должна встать в указанную точку вне зависимости от размера вьюпорта.

Which Node.js version are you using?

20.10.0

@hheimerd
Copy link
Author

Еще в отличии от оригинальной реализации, в этой реализации проверка на typeof xOffset === 'number' не имеет смысла, так как в определении функции ставятся нули
https://github.com/gemini-testing/hermione/blob/4dc6c2fd812e6a3c9d4d82671d59187049d52761/src/browser/commands/moveCursorTo.ts#L14

@hheimerd
Copy link
Author

hheimerd commented Mar 21, 2024

moveTo({xOffset: -width / 2, yOffset: -height / 2 }) тоже не работает, так что возможно снова стал актуален этот баг
webdriverio/webdriverio#4206

@shadowusr
Copy link
Member

Thanks for your report. We'll look at this problem as soon as possible — the next responsible will pick it up. It's weird it doesn't match wdio7 behaviour, since we used almost the same code to implement this.


Спасибо за репорт, посмотрим как только будет возможность — следующий ответственный подхватит эту проблему. Странно, что поведение не совпадает с wdio7, т.к. мы брали код почти без изменений.

@KuznetsovRoman
Copy link
Member

@hheimerd, Действительно, у меня получилось воспроизвести ошибку с moveCursorTo. Однако та же самая ошибка получается от использования чистого moveTo с webdriver@8.20. Мы бы предпочли тут сохранить оригинальное поведение moveTo в webdriverio@8.20 и кидать ошибку в тех же случаях, в которых ее кидает webdriverio.

Причем интересно, что эта ошибка летит только при использовании webdriver протокола автоматизации. С devtools ошибок нет.

Если есть какой-нибудь пример, в котором hermione@7 в таком случае не кидает ошибку - приносите, посмотрим.

@DudaGod
Copy link
Member

DudaGod commented May 13, 2024

Closed due to inactivity.

@DudaGod DudaGod closed this as completed May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants