Skip to content

API tests#1

Open
DraicKin wants to merge 1 commit into
mainfrom
develop2
Open

API tests#1
DraicKin wants to merge 1 commit into
mainfrom
develop2

Conversation

@DraicKin
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown

@pavel-ulan pavel-ulan left a comment

Choose a reason for hiding this comment

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

Да, тут отлично. Оставил коммент на подумать, но по заданию вопросов нет.

public void cleanUp() {
if (token != null) {
ValidatableResponse deleteResponse = authUserClient.deleteUser(token);
Assert.assertEquals("User not deleted", HTTP_ACCEPTED, deleteResponse.extract().statusCode());
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

отлично
👍

public void notGetsUserOrdersWithoutAuth() {
ValidatableResponse getOrdersResponse = orderClient.getUserOrdersWithoutAuth();
Assert.assertEquals("Invalid Status Code", HTTP_UNAUTHORIZED, getOrdersResponse.extract().statusCode());
Assert.assertFalse("Did get an order", getOrdersResponse.extract().body().path("success"));
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

можно лучше
смотри в чем проблема junit.asserts. Если ты проверяешь несколько ассертов в одном тесте, то в случае если в одном из них ошибка, все следующие не отработаю, а тест просто зафейлится. Для этого существуют softAsserts, они работают немного иначе и дадут тебе информацию по всем проверкам в тесте, даже если первая упала. Правильнее пользоваться ими в таких ситуациях.
Но есть небольшое НО. Rest-assured отлично умеет делать такое и сам, но только если ты проверяешь при помощи встроенных asserts.
т.е. response().assertThat().statusCode().equals(200).and().assertThat().extract().body().path("success").equals("ok")
и т.д. отработает полностью, даже если что-то из проверок не пройдет.
Так что всегда стоит детально изучить инструмент которым ты пользуешься.

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