-
-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update syntax for "When I go to the relative URL '$relativeURL'" step #3516
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -237,9 +237,24 @@ public void isPageScrolledToAnElement(Locator locator) | |
} | ||
|
||
/** | ||
* Goes to the relative URL | ||
* Navigates the browser to the specific path in the current host | ||
* defined in the <i>relative URL</i> step variable | ||
* <br> | ||
* A <b>relative URL</b> - points to a file within a web site (like <i>'about.html'</i> or <i>'/products'</i>)<br> | ||
* A <b>relative URL</b> - points to a file within a web site | ||
* (like <i>'about.html'</i> or <i>'/products'</i>)<br> | ||
* <p> | ||
* <table border="1"> | ||
* <caption>Examples:</caption> | ||
* <tr> | ||
* <td>Current page</td><td>Relative URL parameter</td><td>Opened page</td> | ||
* </tr> | ||
* <tr> | ||
* <td>https://mysite.com/path/foo</td><td>stats</td><td>https://mysite.com/path/stats</td> | ||
* </tr> | ||
* <tr> | ||
* <td>https://mysite.com/path/foo</td><td>/documents</td><td>https://mysite.com/documents</td> | ||
* </tr> | ||
* </table> | ||
* <p> | ||
* Actions performed at this step: | ||
* <ul> | ||
|
@@ -249,8 +264,8 @@ public void isPageScrolledToAnElement(Locator locator) | |
* <p> | ||
* @param relativeURL A string value of the relative URL | ||
*/ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. please don't forget to sync javadoc with main docs There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. updated |
||
@When("I go to the relative URL '$relativeURL'") | ||
public void iGoTo(String relativeURL) | ||
@When("I go to relative URL `$relativeURL`") | ||
public void openRelativeUrl(String relativeURL) | ||
{ | ||
setContextSteps.switchingToDefault(); | ||
URI currentURI = UriUtils.createUri(getWebDriver().getCurrentUrl()); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -292,3 +292,7 @@ Given I am on main application page | |
Composite: Given I am on a page with the URL '$pageURL' | ||
!-- WARNING: The step "Given I am on a page with the URL '$pageURL'" is deprecated and will be removed in VIVIDUS 0.7.0 | ||
Given I am on page with URL `<pageURL>` | ||
|
||
Composite: When I go to the relative URL '$relativeURL' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we need to have tests for both new and deprecated steps, so please either add new test or change some existing one to check behaviour of the new step There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. added |
||
!-- WARNING: The step "When I go to the relative URL '$relativeURL'" is deprecated and will be removed in VIVIDUS 0.7.0 | ||
When I go to relative URL `<relativeURL>` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also please add example demoing step usage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still looking for examples
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added