ActionChains does not reset actions

[bayilyas]

🐛 Bug Report

ActionChains does not reset actions.

To Reproduce

Drag and drop

Detailed steps to reproduce the behavior:

All elements drag and drop every loop iteration

Expected behavior

Reset actions and do one drag and drop for each iteration

Test script or set of commands reproducing this issue

from selenium import webdriver
from selenium.webdriver.common.action_chains import ActionChains

driver = webdriver.Chrome()
actions = ActionChains(driver)

driver.get('http://www.dhtmlgoodies.com/scripts/drag-drop-custom/demo-drag-drop-3.html')

countries = {
    "Italy": "Rome",
    "Spain": "Madrid",
    "Norway": "Oslo",
    "Denmark": "Copenhagen",
    "South Korea": "Seoul",
    "Sweden": "Stockholm",
    "United States": "Washington",
}

for country in countries:
    country_box = driver.find_element_by_xpath(f"//div[.='{country}']")
    capital_box = driver.find_element_by_xpath(f"//div[.='{countries[country]}' and contains(@id, 'box')]")
    actions.drag_and_drop(capital_box, country_box).perform()
    actions.reset_actions()

Environment

OS: macOS Catalina Version 10.15.2
Language Bindings version: Python 3.141.0

Browser: Chrome
Browser version: 79.0.3945.88 (Official Build) (64-bit)
Browser Driver version: ChromeDriver 79.0.3945.36

Browser: Firefox
Browser version: 72.0.1 (64-bit)
Browser Driver version: geckodriver v0.26.0, v0.21.0

[isaulv]

@bayilyas Using the latest public release of the python bindings, this worked pretty well.
Can you verify that you're using the latest stable python release (3.141.0)?

[bayilyas]

Yes, I'm using 3.141.0
You can see difference using ActionChains(driver) in every iteration:

for country in countries:
    country_box = driver.find_element_by_xpath(f"//div[.='{country}']")
    capital_box = driver.find_element_by_xpath(f"//div[.='{countries[country]}' and contains(@id, 'box')]")
    ActionChains(driver).drag_and_drop(capital_box, country_box).perform()
    # actions.drag_and_drop(capital_box, country_box).perform()
    # actions.reset_actions()

[isaulv]

@bayilyas I have confirmed this is a bug. It seems that the driver builds up a list internally instead of the code itself so this may be driver bug. :/

[bayilyas]

@Dude-x Same behavior with geckodriver 0.26.0 and 0.21.0 versions

[isaulv]

@bayilyas It is possible that the command to reset actions is flawed in the Python driver. Can you test another language binding like Java?

[bayilyas]

@Dude-x No issue with Java

[isaulv]

@bayilyas I've have studied the Java source code and it does not have a method to reset actions, despite building up a list. Also the list it uses internally is "ummodifiable".

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.