mobile-dev-inc / maestro

Painless Mobile UI Automation
https://maestro.mobile.dev/
Apache License 2.0
5.88k stars 281 forks source link

Timeout not working in scrollUntilVisible #2108

Closed alecalma closed 1 week ago

alecalma commented 1 month ago

Is there an existing issue for this?

Steps to reproduce

- launchApp:
    clearState: true
- swipe:  
    direction: DOWN
- assertVisible:
    text: "Login"
    enabled: true
- tapOn:
    text: "Login"
- scrollUntilVisible:
    element:
        text: XYZ
    direction: DOWN 
    timeout: 1000

Actual results

As easy as it sounds, the timeout parameter does not work for the method scrollUntilVisible, Maestro keeps scrolling forever.

Expected results

I expect the test to fail after the timeout duration.

About app

Native app for android mobile in ReactNative.

About environment

Pixel 9 Pro API35 from AndroidStudio.

Logs

No response

Maestro version

1.39.0

How did you install Maestro?

Homebrew

Anything else?

No response

linear[bot] commented 1 month ago

MA-2386 Timeout not working in scrollUntilVisible

vibin commented 4 weeks ago

@bartekpacia fixed this in #2112.

github-actions[bot] commented 5 days ago

This issue has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar problem, please file a new issue. Make sure to follow the template and provide all the information necessary to reproduce the issue. Thank you for helping keep us our issue tracker clean!