Adding new tests for date picker #1
Conversation
| beforeEach(() => { | ||
| // guess, there is a bug here, cause 6 month corresponds to June, however cy.clock is | ||
| // defined as July. | ||
| const now = new Date(2020, 7, 3).getTime() |
There was a problem hiding this comment.
тут я брала август, как 7 месяц)
| .contains("Check-in") | ||
| .parent() |
There was a problem hiding this comment.
Я бы сделал cy.get("input").eq(0) – потому что название вот этих лейблочек оно не постоянное и может легко сломать тесты. А первый инпут всегда будет отвечать за старт
| .findByLabelText("Aug 3, 2020") | ||
| .should("be.enabled") | ||
| .should("have.attr", "tabindex", "-1") | ||
| .should("have.attr", "data-mui-test", "DateRangeDay"); |
There was a problem hiding this comment.
data-mui-test это аттрибут который предназначен как-раз для тестирования, чтобы находить элемент. Чтобы проверить что день выбран в принципе достаточно tabIndex, но можно еще проверить на класс .Mui-selected
| cy.get("@dateRangePicker") | ||
| .contains("Check-in") | ||
| .parent() | ||
| .find("input") | ||
| .should("have.value", "07/31/2020"); | ||
|
|
||
| cy.get("@dateRangePicker") | ||
| .contains("Check-out") | ||
| .parent() | ||
| .find("input") | ||
| .should("have.value", "08/28/2020"); |
There was a problem hiding this comment.
тут бы я проверил что нужный отрезок был выбран в самом календаре.
потому что получается что тут мы сначала вводим текст в инпуты и потом сразу проверяем что этот же текст в инпутах остался 🤷♂️
There was a problem hiding this comment.
вот тут согласна, но я не додумалась как проверить именно что отрезок выбран, по каким атрибутам.
| cy.executeInDatePicker() | ||
| .findByLabelText("Aug 28, 2020") | ||
| .should("have.focus") | ||
| .should("be.focused"); |
There was a problem hiding this comment.
Тут явно не хватает проверки что между 4 и 24 августа даты подсвечены
There was a problem hiding this comment.
я подумаю как это реализовать.
| // guess, there is a bug here, cause 6 month corresponds to June, however cy.clock is | ||
| // defined as July. |
There was a problem hiding this comment.
все правильно :)
new Date(year, monthIndex [, day [, hours [, minutes [, seconds [, milliseconds]]]]])
monthIndex
Integer value representing the month, beginning with 0 for January to 11 for December.
from: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/Date
| .click(); | ||
|
|
||
| cy.get("@datepicker") | ||
| .findByPlaceholderText("mm/dd/yyyy") |
There was a problem hiding this comment.
текстбокс лучше было бы найти с помощью просто .find("input") или еще лучше .findByRole("textbox")
| .findByLabelText("Jul 26, 2020") | ||
| .should("be.focused"); | ||
| }); | ||
| it.only("End", () => { |
|
Спасибо за воркшоп, это был мой первый опыт с Сайпресом и получилось очень интересно, отдельное спасибо за практическую часть и домашку, извините что не сразу отписала, как так закрутилась и забыла. Ф вообще больше Java человек и все эти JS штучки иногда вводят в заблуждение. |
No description provided.