Skip to content

Custom driver provider tests #102

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

snfrolov
Copy link
Contributor

No description provided.

snfrolov and others added 17 commits May 4, 2018 22:59
Added steps for entering random numbers (alfa-laboratory#60)
Update BlackListManager.java
setted latest version of browser by default (alfa-laboratory#76)
Добавлен headless режим для chrome и firefox 🦄 (alfa-laboratory#91)
CustomDriverProvider customDriverProvider = new CustomDriverProvider();
WebDriver currentDriver;
currentDriver = customDriverProvider.createDriver(new DesiredCapabilities());
assertThat(currentDriver.getClass().getName(), is("org.openqa.selenium.chrome.ChromeDriver"));
assertThat(currentDriver.manage().window().getSize().getWidth(), is(666));
assertThat(currentDriver.manage().window().getSize().getHeight(), is(666));
currentDriver.quit();
}

@Ignore
@Test
public void createFirefoxDriverTest() {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тесты с установкой размера окна для разного типа драйверов излишни. механизм установки одинаковый. имеет смысл проверить, что создается именно нужного типа драйвер. но для этого нужно дополнительно добавить поддержку запуска этого драйвера в travis или через контейнеры попробовать реализовать

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

установку размеров удалил, travis ковыряю

}

@Test
public void createNonexistentDriverTest() {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут лучше default термин использовать

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

поправлено

}

@Test
public void createEmptyDriverTest() {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если createNonexistentDriverTest пройдет, то этот тоже

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

удалил

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants