The tests from jbisss

Ревью

1

Тесты всегда начинаются с анонимной страницы ОК, предлагаю здесь сделать постоянную ссылку на ок, и добавить аннотации Before и After чтобы методы отрабатывали автоматически перед и после каждого теста

Tests/src/test/java/org/login_tests/BaseTest.java

Lines 6 to 7 in 480b7c9

 public void open(String s){ 

 com.codeborne.selenide.Selenide.open(s);

т.е. вот это перенести в бейзтест

Tests/src/test/java/org/login_tests/MainTest.java

Lines 13 to 16 in 480b7c9

 @BeforeAll 

 static void openPage(){ 

 base.open("https://ok.ru"); 

 }

Tests/src/test/java/org/login_tests/MainTest.java

Lines 39 to 42 in 480b7c9

 @AfterAll 

 static void clearCookies(){ 

 base.close(); 

 }

2

Локаторы выносим в константы
Проверяем видимость элемента перед действием с ним .shouldBe(visible.because("Нет кнопки ...").click()

Tests/src/test/java/org/login_tests/LoginPage.java

Line 12 in 480b7c9

$(By.xpath("//*[@id='field_email']")).setValue(email);

3

Не оборачивать проверки, ассерты могут быть в тестах
Добавлять сообщения в ассерты

Tests/src/test/java/org/login_tests/LoginPage.java

Lines 32 to 36 in 480b7c9

 public void checkUsername(String expectedUsername){ 

 assertEquals(expectedUsername, this.takeUserName()); 

 } 

 public void checkNotLogged(boolean expectedResult){ 

 assertFalse(expectedResult);

4

Оформить комментарии как джавадоки

Tests/src/test/java/org/login_tests/LoginPage.java

Line 20 in 480b7c9

// action on page

5

Использовать наследование

Tests/src/test/java/org/login_tests/MainTest.java

Line 11 in 480b7c9

base = new BaseTest();

6

Логины, пароли и тд в константы

Tests/src/test/java/org/login_tests/MainTest.java

Lines 23 to 24 in 480b7c9

 .setEmail("technoPol10") 

 .setPwd("zoloto1234")

7

Зачем два BaseTest? Можно переделать структуру проекта - все тесты в одном пакете, все страницы в другом пакете, внутри пакетов можно еще сделать деление по пакетам - логины, группы и тд, а Base классы оставить без дальнейшего разделения

8

Создаем страницы по мере надобности

Tests/src/test/java/org/enter_the_group/MainTest.java

Line 15 in 480b7c9

BASE_GROUP_PAGE = new BaseGroupPage();

9

Убираем в методы в пейджах, уносим из теста (PageObject!!)

Tests/src/test/java/org/enter_the_group/MainTest.java

Line 28 in 480b7c9

$(By.xpath("//div[@class=\"tico ellip\"]"));

Локаторы в пейджи

Tests/src/test/java/org/enter_the_group/MainTest.java

Lines 27 to 28 in 480b7c9

 private static final SelenideElement PROFILE_PHOTO = 

 $(By.xpath("//div[@class=\"tico ellip\"]"));

10

Не раскрыта суть аннотации, тут стоило показать как можно интересно использовать @beforeeach, добавляя условия на каждом уровне
https://junit.org/junit5/docs/current/user-guide/#writing-tests-nested

Tests/src/test/java/org/enter_the_group/MainTest.java

Line 25 in 480b7c9

@Nested

Тесты должны быть независимыми!!!

Tests/src/test/java/org/enter_the_group/MainTest.java

Line 33 in 480b7c9

$(PROFILE_PHOTO).shouldBe(Condition.visible);

11

Проверки делаем через ассерты

Tests/src/test/java/org/enter_the_group/MainTest.java

Line 33 in 480b7c9

$(PROFILE_PHOTO).shouldBe(Condition.visible);

12

Используем ожидания каких то конкретный событий, всегда лучше ждать какое то событие конкретное, давай разберемся почему тут нужна пауза и чего нам ждать вместе если что
https://viclovsky.github.io/разное/2019/02/06/thread-sleep/

Tests/src/test/java/org/enter_the_group/GroupPage.java

Line 26 in 480b7c9

sleep(2000);

13

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

Tests/src/test/java/org/enter_the_group/MainTest.java

Lines 48 to 54 in 480b7c9

 @AfterAll 

 public static void clearCookies(){ 

 BASE_GROUP_PAGE 

 .openGroupPage() 

 .dropDownGroup(); 

 BASE_TEST.close(); 

 }

	public void open(String s){
	com.codeborne.selenide.Selenide.open(s);

	@BeforeAll
	static void openPage(){
	base.open("https://ok.ru");
	}

	public void checkUsername(String expectedUsername){
	assertEquals(expectedUsername, this.takeUserName());
	}
	public void checkNotLogged(boolean expectedResult){
	assertFalse(expectedResult);

	private static final SelenideElement PROFILE_PHOTO =
	$(By.xpath("//div[@class=\"tico ellip\"]"));

	@AfterAll
	public static void clearCookies(){
	BASE_GROUP_PAGE
	.openGroupPage()
	.dropDownGroup();
	BASE_TEST.close();
	}

jbisss / tests Goto Github PK

tests's People

Contributors

Watchers

tests's Issues

1

2

3

4

5

6

7

8

9

10

11

12

13

Recommend Projects

Recommend Topics

Recommend Org

Jobs