status-im / desktop-qa-automation

Legacy desktop tests
6 stars 12 forks source link

Rework scroll method #571

Closed anastasiyaig closed 1 month ago

anastasiyaig commented 7 months ago

Scroll method can lead to infinite loop

https://ci.status.im/job/status-desktop/job/e2e/job/prs/1983/allure/#testresult/23f01f6862da577

Screenshot 2024-03-06 at 12 21 02 Screenshot 2024-03-06 at 12 22 08

We need to rework this somehow so this is not happening anymore

We have 2 scroll methods and they both work weird

self._scroll.vertical_down_to
self._scroll.vertical_scroll_to
anastasiyaig commented 4 months ago

talked to @caybro and there is a suggestion to use this within StatusScrollView:

    function scrollHome() {
        flickable.contentY = 0
    }

    function scrollEnd() {
        flickable.contentY = flickable.contentHeight - flickable.height
    }

    function scrollPageUp() {
        root.ScrollBar.vertical.decrease()
    }

    function scrollPageDown() {
        root.ScrollBar.vertical.increase()
    }