Using Python bindings, Selenium WebDriver click() is not working sometimes.

Question

I am trying to submit an input(type= button).But I am unable to update the value. Any help is appreciated. I have attached the testcase below for your reference.

search for CLICK FAILS HERE

from selenium import webdriver
from selenium.webdriver.common.by import By
from selenium.webdriver.support.ui import Select
from selenium.common.exceptions import NoSuchElementException
import unittest, time, re,datetime,os,sys

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


def is_element_present(inst,selector,value):
    try:
        inst.find_element(by=selector, value=value)
        return True
    except:
        return False



class Testing(unittest.TestCase):
    def setUp(self):
        self.driver = webdriver.Chrome()
        self.driver.implicitly_wait(5)
        self.base_url = "http://new.ossmoketest.appspot.com/"
        self.verificationErrors = []

    def test_ing(self):
        try:

            driver = self.driver
            driver.get(self.base_url + "/Apab4b39d4_09d7_11e1_8df9_139372201eeb/1/signin?forward=/%3F")
            now = datetime.datetime.now()
            start = time.clock()

            for i in range(5000000):
                try:
                    if is_element_present(driver,By.ID,"userid"): break
                except: pass
            else: self.fail("time out")
            end = time.clock()
            diff = end - start
            print diff
            driver.find_element_by_id("userid").clear()
            driver.find_element_by_id("userid").send_keys("senthil.arumugam@orangescape.com")
            driver.find_element_by_xpath("//input[@src='/static/images/signup.png']").click()
            print 'finished'
            start = time.clock()

            for i in range(5000000):
                try:
                    if is_element_present(driver,By.LINK_TEXT,"Logout"): break
                except: pass
            else: self.fail("time out")
            end = time.clock()
            diff = end - start
            print diff
            time.sleep(5)

            start = time.clock()
            name = "smoketest"+ str(now.minute) +str(now.second)
            for i in range(5000000):
                try:
                    if is_element_present(driver,By.LINK_TEXT,"PurchaseOrder"): break
                except: pass
            else: self.fail("time out")
            end = time.clock()
            diff = end - start
            driver.find_element_by_link_text('PurchaseOrder').click()
            name = "smoketest"+ str(now.minute) +str(now.second)
            start = time.clock()
            for i in range(5000000):
                try:
                    if is_element_present(driver,By.ID,"Customer_Name"): break
                except: pass
            else: self.fail("time out")
            end = time.clock()
            diff = end - start

            newproduct = "rexona"+ str(now.minute) +str(now.second)
            newprice = str(now.minute) +str(now.second)
            newprice = float(newprice)
            print newprice
            driver.find_element_by_xpath("//input[starts-with(@id,'New_Product')]").send_keys(newproduct)
            driver.find_element_by_xpath("//input[starts-with(@id,'Price')]").clear()
            time.sleep(3)

            driver.find_element_by_xpath("//input[starts-with(@id,'Price')]").send_keys(Keys.CONTROL+'a'+Keys.NULL, str(newprice))
            Mouse_cntrl = ActionChains(driver)
            Mouse_cntrl.release(driver.find_element_by_xpath("//input[starts-with(@id,'Price')]"))
            value = newprice
            print value
            time.sleep(2)
            print 'start'
            print driver.find_element_by_xpath("//input[starts-with(@id,'NewItem_NewItem')]").get_attribute('data-id')
            # ------------------------CLICK FAILS HERE ------------------------------
#            driver.find_element_by_xpath("//input[starts-with(@id,'NewItem_NewItem')]").click()
#            driver.find_element_by_xpath("//input[starts-with(@id,'NewItem_NewItem')]").submit()
            driver.find_element_by_xpath("//input[starts-with(@id,'NewItem_NewItem')]").send_keys(keys.ENTER)
#            Mouse_cntrl.double_click(driver.find_element_by_xpath("//input[starts-with(@id,'NewItem_NewItem')]"))
            for i in range(10):
                try:
                    print driver.switch_to_alert().text
                    if driver.switch_to_alert():
                        driver.switch_to_alert().accept()
                        break
                except: pass
                time.sleep(1)
            else:
                print "alert not found"
            print 'finished  -- '
            time.sleep(8)
            driver.find_element_by_xpath("//input[starts-with(@id,'Product')]").click()
            arg = newproduct
            print 'end'
            for i in range(60):
                try:
                    if is_element_present(driver,By.LINK_TEXT,arg): break
                except: pass
                time.sleep(1)
            else: self.fail("time out")
    #        sel.mouse_over("//html/body/ul/li/a[.=\""+arg+"\"]")
            driver.find_element_by_link_text(arg).click()
            start = time.clock()
            time.sleep(25)
            for i in range(1000000):
                try:

                    if newprice == float(driver.find_element_by_id('Unit_Price').text):
                        end = time.clock()
                        diff = end - start
                        log.log(module='Smoke',testcase='Action New', result='Pass',time_taken= diff)
                        break
                except: pass
            else:
                log.log(module='Smoke',testcase='Action New', result='Fail')
                self.fail('New Failure')
            log.log(module='Smoke',testcase='On Submit', result='Pass',time_taken= diff)
            driver.find_element_by_id('Quantity').send_keys(Keys.CONTROL+'a'+Keys.NULL,"1")
            time.sleep(5)
            start = time.clock()
            for i in range(1000000):
                try:
                    if value == float(driver.find_element_by_id('Unit_Price').text):
                        end = time.clock()
                        diff = end - start
                        log.log(module='Smoke',testcase='Multiply', result='Pass',time_taken= diff)
                        break
                except: pass
            else: self.fail("time out")
            for i in range(1000000):
                try:
                    if value == float(driver.find_element_by_id('Amount').text):
                        end = time.clock()
                        diff = end - start
                        log.log(module='Smoke',testcase='DSUM with Parent', result='Pass',time_taken= diff)
                        break
                except: pass
            else:
                end = time.clock()
                diff = end - start
                log.log(module='Smoke',testcase='DSUM with Parent', result='Fail',time_taken= diff)
                self.fail("time out")

        except:
            self.driver.quit()
            e = sys.exc_info()[1]
            print str(e)

    def tearDown(self):
        self.driver.quit()
        self.assertEqual([], self.verificationErrors)

if __name__ == "__main__":
     unittest.main()

It has been a showstopper for my work. Any help is appreciated.Thanks

what about the css selector: `driver.find_element_by_css_selector("input[id*=NewItem_NewItem]").click()` Also, does this line give you the attribute? `print driver.find_element_by_xpath("//input[starts-with(@id,'NewItem_NewItem')]").get_attribute('data-id')` If so, is there javascript running after page load that may be affecting this element? — Bob Evans, Jan 24 '12 at 02:53
The element is probably not 'ready' for the click in some way as a result of page rewriting. Determine why, then loop while that condition pervails or until a time out occurs. — Martin Spamer, Feb 28 '12 at 13:20

score 30 · Answer 1 · edited Dec 08 '14 at 13:43

30

You could try substituting .click() with .send_keys("\n"), which is equivalent to "Pressing enter while focusing on an element".

So this:

driver.find_element_by_link_text('PurchaseOrder').click()

would become this:

driver.find_element_by_link_text('PurchaseOrder').send_keys("\n")

edited Dec 08 '14 at 13:43

Farvardin

5,336
5
33
54

answered Mar 13 '12 at 03:42

EwyynTomato

4,009
1
31
39

4

Just ran into this issue today, and `send_keys("\n")` didn't work. However `send_keys(Keys.ENTER)` did. – Brandon Campbell Mar 04 '20 at 19:46
1

I just ran into this issue and none of the solutions worked. .submit() worked instead of .click() – Bharath M Jun 18 '20 at 09:42

score 9 · Answer 2 · answered Oct 15 '14 at 02:16

9

In case this is still a recurring problem for anyone else, if you have confirmed your code is correct (you've reviewed it for errors etc.) and you still find the find_element_by_...('text').click() function not working properly it is often due to your code continuing to run before the JavaScript can update the page.

A simple solution is to import time then insert the below code immediately after any click() methods:

time.sleep(2)

The duration of the sleep timer can be whatever you choose. In my case I used 2 seconds. Hopefully that helps.

answered Oct 15 '14 at 02:16

BCR

960
11
27

8

don't do this. there are better ways than polluting your testcases with static sleeps – Corey Goldberg May 03 '16 at 21:00
I realize that this is not recommended, but it did work for me and, if nothing else, provided a reason for why my code wasn't working. – Matt Cremeens May 23 '17 at 17:08
Static sleeps are awful. Learn how to wait the proper way. – Moser Oct 11 '17 at 16:11
2

@Moser 'Learn how to wait the proper way' & 'There are better ways' such as...? – Rafael Moreno Jan 29 '18 at 19:05
@Moser And what is the proper way? Can you post the answer? – Blaszard Jun 14 '18 at 05:55
@RafaelMoreno, @Blaszard, you can refer to the following for `wait` ways: https://www.selenium.dev/docs/site/en/webdriver/waits/ – Rohan Chougule Mar 04 '21 at 11:26
they improved the waits over time. at the time the static sleep method was very effective. – BCR Mar 05 '21 at 14:52

score 4 · Answer 3 · edited May 23 '17 at 11:52

I had this problem as well. Sometimes, for whatever reason webdriver didn't click the button. It was able to find the button (it didn't throw a NoSuchElementException and a WebDriverWait didn't help).

The problem with clicking the button twice was that if the first click succeed, the second one would fail (or click the submit button on the next page if it found a match!). My first attempt was to put the second click in a try/except block - this is how I found out about it clicking submit on the next page. XD And it also really slowed down my test when it couldn't find the second button.

I found some good insights at Selenium 2.0b3 IE WebDriver, Click not firing. Basically, I click on a parent element first, which seemingly does nothing. Then I click on the submit button.

score 2 · Answer 4 · answered Sep 12 '17 at 09:15

2

If the element you click() is an url. I found that taking the href properties and using driver.get(elem.get_attribute('href')) being the cleanest.

answered Sep 12 '17 at 09:15

MortenB

2,749
1
31
35

score 0 · Answer 5 · answered May 03 '18 at 20:03

I had the same issue where a two-part drop down menu selection would not generate what it's supposed to generate with proper selections. It worked when I did imported time and use time.sleep(2) between the two "click"s. (For reference I used find_element_by_xpath to find an modify the options.)

score 0 · Answer 6 · answered Jun 18 '20 at 09:50

I ran into the above issue where the same .click() is working in all browsers(IE, Chrome and Firefox) but not working for Safari. So I tried all the possible solutions from this post and none of them worked and it doesn't even throw error.

I tried below by substituting .click() with .submit(), which is an equivalent method and it worked not only for Safari but also for all other browsers.

So this:

driver.find_element_by_id('login_btn').click()

Replaced as:

driver.find_element_by_id('login_btn').submit()

If above fails to work on other browsers due to any reason keep .submit() in try and except. If it fails .click() will be triggered and executed.

Hope this helps if none of the other solutions works.

score 0 · Answer 7 · answered May 17 '21 at 09:23

0

For those, click() doesn't work, use submit() if that button element(clickable element) is in a form element. Basically, in order to submit a form, we need to use submit(), click() will not work in some cases.

answered May 17 '21 at 09:23

Jay Darji

11
2

score 0 · Answer 8 · answered May 29 '22 at 16:07

0

I foolishly did elem.click with the parentheses: elem.click().

answered May 29 '22 at 16:07

Rivers Cuomo

316
2
5
11

score 0 · Answer 9 · answered Mar 06 '12 at 14:17

I would try other element finders like className, cssSelector or something. xPath sometimes doesnt provide errors if the element isn't found. So first start by finding out if the element is really found by webdriver.

You can also try to click or use the other commands two times in a row. This already solved some of such issues.

Using Python bindings, Selenium WebDriver click() is not working sometimes.

9 Answers9

Linked