8000 #111 Just method renaming · seleniumQuery/seleniumQuery@9e59db6 · GitHub
[go: up one dir, main page]

Skip to content

Commit 9e59db6

Browse files
committed
#111 Just method renaming
1 parent d687d9f commit 9e59db6

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

src/test/java/io/github/seleniumquery/functions/jquery/events/ClickFunctionTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@
3030
import static org.mockito.Mockito.*;
3131
import static testinfrastructure.testdouble.io.github.seleniumquery.SeleniumQueryObjectMother.createStubSeleniumQueryObjectWithElements;
3232
import static testinfrastructure.testdouble.org.openqa.selenium.WebElementClickSpy.createWebElementClickSpy;
33-
import static testinfrastructure.testdouble.org.openqa.selenium.WebElementMother.createClickableWebElement;
34-
import static testinfrastructure.testdouble.org.openqa.selenium.WebElementMother.createUnclickableHiddenWebElement;
33+
import static testinfrastructure.testdouble.org.openqa.selenium.WebElementMother.createWebElementClickable;
34+
import static testinfrastructure.testdouble.org.openqa.selenium.WebElementMother.createWebElementUnclickableHidden;
3535

3636
public class ClickFunctionTest {
3737

@@ -61,7 +61,7 @@ public void click__shouldCallClickOnEveryElementIfNoneThrowsException() {
6161
@Test
6262
public void click__shouldCallClickOnEveryElementEvenIfSomeThrowExceptions() {
6363
// given
64-
WebElement unclickableHiddenWebElement = createUnclickableHiddenWebElement();
64+
WebElement unclickableHiddenWebElement = createWebElementUnclickableHidden();
6565
WebElementClickSpy webElementClickSpy = createWebElementClickSpy();
6666
SeleniumQueryObject sqo = createStubSeleniumQueryObjectWithElements(unclickableHiddenWebElement, webElementClickSpy);
6767
// when
@@ -74,7 +74,7 @@ public void click__shouldCallClickOnEveryElementEvenIfSomeThrowExceptions() {
7474
public void click__shouldLogInfoIfAnyElementThrewException() {
7575
// given
7676
Log logSpy = LogInjector.injectLogSpy(ClickFunction.class);
77-
SeleniumQueryObject sqo = createStubSeleniumQueryObjectWithElements(createUnclickableHiddenWebElement(), createClickableWebElement());
77+
SeleniumQueryObject sqo = createStubSeleniumQueryObjectWithElements(createWebElementUnclickableHidden(), createWebElementClickable());
7878
// when
7979
ClickFunction.click(sqo);
8080
// then
@@ -84,8 +84,8 @@ public void click__shouldLogInfoIfAnyElementThrewException() {
8484
@Test(expected = SeleniumQueryException.class)
8585
public void click__shouldThrowExceptionIfNoElementIsClickable() {
8686
// given
87-
WebElement unclickableHiddenWebElement = createUnclickableHiddenWebElement();
88-
WebElement unclickableHiddenWebElementTwo = createUnclickableHiddenWebElement();
87+
WebElement unclickableHiddenWebElement = createWebElementUnclickableHidden();
88+
WebElement unclickableHiddenWebElementTwo = createWebElementUnclickableHidden();
8989
SeleniumQueryObject sqo = createStubSeleniumQueryObjectWithElements(unclickableHiddenWebElement, unclickableHiddenWebElementTwo);
9090
// when
9191
ClickFunction.click(sqo);

src/test/java/testinfrastructure/testdouble/org/openqa/selenium/WebElementMother.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
public class WebElementMother {
2323

24-
public static WebElement createUnclickableHiddenWebElement() {
24+
public static WebElement createWebElementUnclickableHidden() {
2525
return new WebElementStub() {
2626
// this has to be a STUB and not a DUMMY because when the $().click() function fails, it
2727
// call a series of other functions on WebElement just to print a more complete error message
@@ -32,7 +32,7 @@ public void click() {
3232
};
3333
}
3434

35-
public static WebElement createClickableWebElement() {
35+
public static WebElement createWebElementClickable() {
3636
return new WebElementDummy() {
3737
@Override
3838
public void click() { }

0 commit comments

Comments
 (0)
0