Skip to content

Remove duplicate logging from navigation #31 #35

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.

Sign up for GitHub

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

Merged
merged 10 commits into from
Aug 22, 2019
Merged

Remove duplicate logging from navigation #31 #35

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7125a1a
refactored state of elements
Aug 8, 2019
0c16676
re-check waitForNotExists methods for correct work. split 1 test to 2…
Aug 9, 2019
b0567a6
removed IElementWithState interface
Aug 9, 2019
9fc21fd
refactored conditional wait
Aug 9, 2019
59f2641
merged with the master branch
Aug 9, 2019
6e6fe54
Merge branch 'master' of https://github.com/aquality-automation/aqual…
Aug 13, 2019
f1ca748
Merge branch 'master' of https://github.com/aquality-automation/aqual…
Aug 16, 2019
7f03255
Merge branch 'master' of https://github.com/aquality-automation/aqual…
Aug 21, 2019
1dc49f6
Merge branch 'master' of https://github.com/aquality-automation/aqual…
Aug 21, 2019
6a98363
removed duplicate logging from Navigation #31
Aug 21, 2019
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions src/main/java/aquality/selenium/browser/Browser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,23 +60,20 @@ public RemoteWebDriver getDriver() {
* @param url URL where you wish to navigate
*/
public void goTo(String url) {
logger.info(String.format(getLocManager().getValue("loc.browser.goTo"), url));
navigate().to(url);
}

/**
* Executes navigating back
*/
public void goBack() {
logger.info(getLocManager().getValue("loc.browser.back"));
navigate().back();
}

/**
* Executes navigating forward
*/
public void goForward() {
logger.info(getLocManager().getValue("loc.browser.forward"));
navigate().forward();
}

Expand All @@ -101,7 +98,6 @@ public String getCurrentUrl() {
* Executes refreshing of current page
*/
public void refresh() {
logger.info(getLocManager().getValue("loc.browser.refresh"));
navigate().refresh();
}

Expand Down
1 change: 0 additions & 1 deletion src/main/resources/localization/en.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
{
"loc.browser.arguments.setting" : "Setting browser start arguments from JSON file: %s",
"loc.browser.goTo" : "Go to URL '%1$s'",
"loc.browser.back" : "Return to previous page",
"loc.browser.forward" : "Proceed to the next page",
"loc.browser.caps.setting" : "Setting browser capabilities from JSON file: %s",
Expand Down
1 change: 0 additions & 1 deletion src/main/resources/localization/ru.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
{
"loc.browser.arguments.setting" : "Установка стартовых аргументов браузера из JSON файла: %s",
"loc.browser.goTo" : "Переход по URL '%1$s'",
"loc.browser.back" : "Возврат на предыдущую страницу",
"loc.browser.forward" : "Перейти на следующую страницу",
"loc.browser.caps.setting" : "Установка capabilities браузера из JSON файла: %s",
Expand Down