Skip to content

Commit

Permalink
[py] Fix inner test classes
Browse files Browse the repository at this point in the history
  • Loading branch information
davehunt committed Nov 14, 2016
1 parent 708c8d9 commit d29bac2
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 15 deletions.
4 changes: 2 additions & 2 deletions py/test/selenium/webdriver/common/interactions_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,9 +81,9 @@ def _isElementAvailable(driver, id):
def testDragAndDrop(driver, pages):
"""Copied from org.openqa.selenium.interactions.TestBasicMouseInterface."""
element_available_timeout = 15
wait = WebDriverWait(element_available_timeout)
wait = WebDriverWait(driver, element_available_timeout)
pages.load("droppableItems.html")
wait.until(lambda dr: dr._isElementAvailable(driver, "draggable"))
wait.until(lambda dr: _isElementAvailable(driver, "draggable"))

if not _isElementAvailable(driver, "draggable"):
raise AssertionError("Could not find draggable element after 15 seconds.")
Expand Down
26 changes: 13 additions & 13 deletions py/test/selenium/webdriver/support/event_firing_webdriver_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,22 +38,22 @@ def test_should_fire_navigation_events(driver, log, pages):

class EventListener(AbstractEventListener):

def before_navigate_to(url, driver):
def before_navigate_to(self, url, driver):
log.write(("before_navigate_to %s" % url.split("/")[-1]).encode())

def after_navigate_to(url, driver):
def after_navigate_to(self, url, driver):
log.write(("after_navigate_to %s" % url.split("/")[-1]).encode())

def before_navigate_back(driver):
def before_navigate_back(self, driver):
log.write(b"before_navigate_back")

def after_navigate_back(driver):
def after_navigate_back(self, driver):
log.write(b"after_navigate_back")

def before_navigate_forward(driver):
def before_navigate_forward(self, driver):
log.write(b"before_navigate_forward")

def after_navigate_forward(driver):
def after_navigate_forward(self, driver):
log.write(b"after_navigate_forward")

ef_driver = EventFiringWebDriver(driver, EventListener())
Expand All @@ -79,10 +79,10 @@ def test_should_fire_click_event(driver, log, pages):

class EventListener(AbstractEventListener):

def before_click(element, driver):
def before_click(self, element, driver):
log.write(b"before_click")

def after_click(element, driver):
def after_click(self, element, driver):
log.write(b"after_click")

ef_driver = EventFiringWebDriver(driver, EventListener())
Expand All @@ -97,10 +97,10 @@ def test_should_fire_change_value_event(driver, log, pages):

class EventListener(AbstractEventListener):

def before_change_value_of(element, driver):
def before_change_value_of(self, element, driver):
log.write(b"before_change_value_of")

def after_change_value_of(element, driver):
def after_change_value_of(self, element, driver):
log.write(b"after_change_value_of")

ef_driver = EventFiringWebDriver(driver, EventListener())
Expand All @@ -124,10 +124,10 @@ def test_should_fire_find_event(driver, log, pages):

class EventListener(AbstractEventListener):

def before_find(by, value, driver):
def before_find(self, by, value, driver):
log.write(("before_find by %s %s" % (by, value)).encode())

def after_find(by, value, driver):
def after_find(self, by, value, driver):
log.write(("after_find by %s %s" % (by, value)).encode())

ef_driver = EventFiringWebDriver(driver, EventListener())
Expand Down Expand Up @@ -155,7 +155,7 @@ def after_find(by, value, driver):
def test_should_call_listener_when_an_exception_is_thrown(driver, log, pages):

class EventListener(AbstractEventListener):
def on_exception(exception, driver):
def on_exception(self, exception, driver):
if isinstance(exception, NoSuchElementException):
log.write(b"NoSuchElementException is thrown")

Expand Down

0 comments on commit d29bac2

Please sign in to comment.