René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Suspicious functionality change in driver.swipe() and driver.scroll() duration argument","articleBody":"## The problem\r\n\r\nBefore the ActionChains were used (Appium 1.20.x, Python Client 1.x.x) as they are now written in action_helpers.py we had used the following method to swipe the screen open from screen lock state:\r\n\r\n`driver.swipe(x1, y1, x2, y2, duration=3000)`\r\n\r\nThis gave a slow swipe that took 3 seconds to move from x1,y1 to x2,y2.\r\n\r\nWhen updating to Appium 2.0 and the latest Appium Python Client, the swipe did not work properly anymore.\r\n\r\nIf I understand the code properly in action_helpers.py, the functionality now uses the duration argument as pause between the move from x1,y1 to x2,y2:\r\n\r\n`actions.w3c_actions.pointer_action.pause(duration / 1000)`\r\n\r\nBut the actual swipe action is very fast, in my case with a three second pause before the actual fast swipe action!\r\n\r\nTo get the swipe to work as before, I made a copy of the swipe method from action_helpers.py and changed like this:\r\n\r\n    def swipe(self: T, start_x: int, start_y: int, end_x: int, end_y: int, duration: int = 0) -\u003e T:\r\n        actions = ActionChains(self)\r\n        actions.w3c_actions = ActionBuilder(self, mouse=PointerInput(interaction.POINTER_TOUCH, \"touch\"), duration=duration)\r\n        actions.w3c_actions.pointer_action.move_to_location(start_x, start_y)\r\n        actions.w3c_actions.pointer_action.pointer_down()\r\n        actions.w3c_actions.pointer_action.move_to_location(end_x, end_y)\r\n        actions.w3c_actions.pointer_action.release()\r\n        actions.perform()\r\n\r\n**Notice that I have moved the duration parameter to the ActionBuilder constructor and removed the pause completely.**\r\n\r\n\r\nSo I am proposing to implement the swipe() and scroll() like this and if someone requires an actual pause between the move, then I suggest adding an additional pause argument to the method signature.\r\n\r\nAlso I would like to see long_press(element: WebElement, timeout: int) to be added to action_helpers.py, where the pause would actually use the timeout value.\r\n\r\nThanks! \r\n\r\n## Environment\r\nAppium version: 2.0.0Beta40\r\nAppium Python Client version 2.6.0\r\nOS version: Linux Ubuntu 20.04\r\nMobile platform: Android (real device)\r\n","author":{"url":"https://github.com/jatalahd","@type":"Person","name":"jatalahd"},"datePublished":"2022-06-30T15:52:51.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/743/python-client/issues/743"}