diff --git a/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/Element.java b/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/Element.java index 0953b47..cf33782 100644 --- a/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/Element.java +++ b/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/Element.java @@ -757,7 +757,7 @@ public void clickLink(String locator) { public ArrayList getAllLinks() { ArrayList ret = new ArrayList(); - List elements = elementFind("tag=a", false, false, "a"); + List elements = elementFind("tag:a", false, false, "a"); for (WebElement element : elements) { ret.add(element.getAttribute("id")); } @@ -861,8 +861,8 @@ public void pageShouldNotContainImage(String locator, String...params) { "If you wish to assert the number of located elements, use `Xpath Should Match X Times`.") @ArgumentNames({ "xpath" }) public int getMatchingXpathCount(String xpath) { - if (!xpath.startsWith("xpath=")) { - xpath = "xpath=" + xpath; + if (!xpath.startsWith("xpath=") && !xpath.startsWith("xpath:")) { + xpath = "xpath:" + xpath; } List elements = elementFind(xpath, false, false); @@ -874,8 +874,8 @@ public int getMatchingXpathCount(String xpath) { public void xpathShouldMatchXTimes(String xpath, int expectedXpathCount, String...params) { String message = robot.getParamsValue(params, 0, ""); String logLevel = robot.getParamsValue(params, 1, "INFO"); - if (!xpath.startsWith("xpath=")) { - xpath = "xpath=" + xpath; + if (!xpath.startsWith("xpath=") && !xpath.startsWith("xpath:")) { + xpath = "xpath:" + xpath; } List elements = elementFind(xpath, false, false); int actualXpathCount = elements.size(); @@ -932,7 +932,7 @@ protected boolean frameContains(String locator, String text) { } protected boolean isTextPresent(String text) { - String locator = String.format("xpath=//*[contains(., %s)]", escapeXpathValue(text)); + String locator = String.format("xpath://*[contains(., %s)]", escapeXpathValue(text)); return isElementPresent(locator); } @@ -1015,7 +1015,7 @@ protected boolean pageContains(String text) { return true; } - List elements = elementFind("xpath=//frame|//iframe", false, false); + List elements = elementFind("xpath://frame|//iframe", false, false); Iterator it = elements.iterator(); while (it.hasNext()) { current.switchTo().frame(it.next()); diff --git a/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/FormElement.java b/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/FormElement.java index a2eda16..af1dc14 100644 --- a/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/FormElement.java +++ b/src/main/java/com/github/markusbernhardt/seleniumlibrary/keywords/FormElement.java @@ -49,7 +49,7 @@ public void submitForm() { public void submitForm(String locator) { logging.info(String.format("Submitting form '%s'.", locator)); if (locator == null) { - locator = "xpath=//form"; + locator = "xpath://form"; } List webElements = element.elementFind(locator, true, true, "form"); webElements.get(0).submit(); @@ -428,13 +428,13 @@ protected WebElement getCheckbox(String locator) { } protected List getRadioButtons(String groupName) { - String xpath = String.format("xpath=//input[@type='radio' and @name='%s']", groupName); + String xpath = String.format("xpath://input[@type='radio' and @name='%s']", groupName); logging.debug("Radio group locator: " + xpath); return element.elementFind(xpath, false, true); } protected WebElement getRadioButtonWithValue(String groupName, String value) { - String xpath = String.format("xpath=//input[@type='radio' and @name='%s' and (@value='%s' or @id='%s')]", + String xpath = String.format("xpath://input[@type='radio' and @name='%s' and (@value='%s' or @id='%s')]", groupName, value, value); logging.debug("Radio group locator: " + xpath); return element.elementFind(xpath, true, true).get(0); diff --git a/src/main/java/com/github/markusbernhardt/seleniumlibrary/locators/TableElementFinder.java b/src/main/java/com/github/markusbernhardt/seleniumlibrary/locators/TableElementFinder.java index 4c9bc03..61b4143 100644 --- a/src/main/java/com/github/markusbernhardt/seleniumlibrary/locators/TableElementFinder.java +++ b/src/main/java/com/github/markusbernhardt/seleniumlibrary/locators/TableElementFinder.java @@ -85,13 +85,14 @@ protected static void addLocatorSuffix(Map> locatorSuffixes protected static List parseTableLocator(String tableLocator, String locationMethod) { String tableLocatorType = null; - if (tableLocator.startsWith("xpath=")) { + if (tableLocator.startsWith("xpath=") || tableLocator.startsWith("xpath:")) { tableLocatorType = "xpath."; - } else if (tableLocator.startsWith("jquery=") || tableLocator.startsWith("sizzle=")) { + } else if (tableLocator.startsWith("jquery=") || tableLocator.startsWith("jquery:") || + tableLocator.startsWith("sizzle=") || tableLocator.startsWith("sizzle:")) { tableLocatorType = "sizzle."; } else { - if (!tableLocator.startsWith("css=")) { - tableLocator = String.format("css=table#%s", tableLocator); + if (!tableLocator.startsWith("css=") && !tableLocator.startsWith("css:")) { + tableLocator = String.format("css:table#%s", tableLocator); } tableLocatorType = "css."; }