Skip to content

Add Stellar Burgers UI tests - #1

Open
supogolik-code wants to merge 3 commits into
mainfrom
develop3
Open

supogolik-code wants to merge 3 commits into
mainfrom
develop3

Conversation

@supogolik-code

Copy link
Copy Markdown
Owner

No description provided.

}

private String sendForToken(String path, TestUser user) {
String body = String.format(

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⛔️Нужно исправить.В теле передавать данные нужно через сериализацию. Это более продвинутый вариант, в теории он описан. Поправь, пожалуйста, этот момент во всем коде.

return sendForToken("/auth/login", user);
}

public void delete(String accessToken) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⛔️Нужно исправить. Для всех методов, описывающих шаги теста, необходимо использовать аннотацию @step

click(LOGIN_BUTTON);
}

public boolean isOrderButtonVisible() {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⛔️Нужно исправить. Для всех методов, описывающих шаги теста, необходимо использовать аннотацию @step

});
}

@Test

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️Можно улучшить. В allure есть аннотации для описания теста Description и DisplayName. Добавь их, пожалуйста, во всех тестах проекта

Comment thread .gitignore
@@ -0,0 +1,5 @@
target/
.allure/

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⛔️Нужно исправить. Нет аллюр отчета

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Добавь папку allure-results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants