Skip to content

Update sprint.json#49

Open
DunaevDS wants to merge 6 commits intoyandex-praktikum:add-bookingsfrom
DunaevDS:patch-1
Open

Update sprint.json#49
DunaevDS wants to merge 6 commits intoyandex-praktikum:add-bookingsfrom
DunaevDS:patch-1

Conversation

@DunaevDS
Copy link
Collaborator

@DunaevDS DunaevDS commented Oct 5, 2023

No description provided.

"response": []
},
{
"name": "Item 6 get by user 4 (owner) without comments",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

В описании теста Id вещи и Id владельца перепутаны

Copy link
Collaborator

Choose a reason for hiding this comment

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

Если приглядеться, там очень многое сделано копипастом, в js тестах одни и те же сообщения, а проверяют совершенно другое, взять например Item 2 get by user 4 (owner) with bookings, поэтому если кто-то отреагирует - я удивлюсь

pm.test("Test item 'nextBooking.id' field", function () {
    var jsonData = pm.response.json();
    pm.expect(jsonData).to.have.property('nextBooking');
    pm.expect(jsonData.nextBooking).to.have.property('id');
    pm.expect(jsonData.nextBooking.id, '"nextBooking.id" must be "1"').to.eql(4);
});
pm.test("Test item 'nextBooking.bookerId' field", function () {
    var jsonData = pm.response.json();
    pm.expect(jsonData).to.have.property('nextBooking');
    pm.expect(jsonData.nextBooking).to.have.property('bookerId');
    pm.expect(jsonData.nextBooking.bookerId, '"nextBooking.bookerId" must be "1"').to.eql(5);
});

@yandex-praktikum yandex-praktikum force-pushed the add-bookings branch 10 times, most recently from d3e322e to aea3842 Compare August 14, 2024 09:15
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.

3 participants