Conversation
gennady-bars
left a comment
gennady-bars
left a comment
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Работа проделана огромная
- Все юнит-тесты (72 шт) проходят успешно
- Код тестов аккуратный и понятный
но есть некоторые недочеты:
- Нужно задеплоить приложение и показать ссылку в
Readme - в
README.mdтеперь нужно добавить описание проекта, технологий, как скачать и запустить, как задеплоить и ссылку на задеплоенный сайт. http://localhost:3000будет повторяться несколько раз в коде тестов. Нужно вынести в константуtestUrl, чтобы можно было одним движением изменить урл для тестов
А можно в документации посмотреть, как настроить быстроbaseUrlдля cypress https://docs.cypress.io/api/commands/visit#Visit-is-automatically-prefixed-with-baseUrl- скрин https://disk.yandex.ru/i/7K8npr8qDG4D5g тесты
cypressне проходят "Краторная булка N-200i"дублируется 3 раза"[class^=modal-header_buttonClose__]"дублируется 4 раза- Если строка-селектор повторяется более 2х раз в коде, значит, этот селектор нужно выносить в константу, чтобы можно было одним движением изменить его и не искать по всему коду дублирования этого селектора.
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Удачного рефакторинга кода.
| describe("service is available", function () { | ||
| beforeEach(() => { | ||
| cy.viewport(1280, 720); | ||
| cy.visit("http://localhost:3000"); |
There was a problem hiding this comment.
http://localhost:3000 будет повторяться несколько раз в коде тестов. Нужно вынести в константу testUrl, чтобы можно было одним движением изменить урл для тестов
А можно в документации посмотреть, как настроить быстро baseUrl для cypress https://docs.cypress.io/api/commands/visit#Visit-is-automatically-prefixed-with-baseUrl
| @@ -0,0 +1,65 @@ | |||
| describe("service is available", function () { | |||
There was a problem hiding this comment.
- скрин https://disk.yandex.ru/i/7K8npr8qDG4D5g тесты
cypressне проходят
| cy.wait(3000); | ||
| }); | ||
| it("should open burger constructor page by default and test modal", function () { | ||
| cy.contains("Краторная булка N-200i").click(); |
There was a problem hiding this comment.
"Краторная булка N-200i" дублируется 3 раза
| cy.contains("Краторная булка N-200i").click(); | ||
| cy.get("[class^=modal-header_buttonClose__]").click(); | ||
| cy.contains("Сыр с астероидной плесенью").click(); | ||
| cy.get("[class^=modal-header_buttonClose__]").click(); |
There was a problem hiding this comment.
"[class^=modal-header_buttonClose__]" дублируется 4 раза
There was a problem hiding this comment.
Если строка-селектор повторяется более 2х раз в коде, значит, этот селектор нужно выносить в константу, чтобы можно было одним движением изменить его и не искать по всему коду дублирования этого селектора.
| cy.get('img[alt*="Краторная булка N-200i"]') | ||
| .trigger("dragstart") | ||
| .trigger("dragleave"); | ||
| cy.get("[class^=burger-constructor_constructor__container]") |
There was a problem hiding this comment.
"[class^=burger-constructor_constructor__container]" 4 раза повторяется
и так далее
gennady-bars
left a comment
gennady-bars
left a comment
There was a problem hiding this comment.
Уже почти все отлично, но:
- в задеплоенном приложении просто черный экран https://disk.yandex.ru/i/78GIs7OZt8OoVQ
- Скрин https://disk.yandex.ru/i/E3aPQdmeC32a1g не проходит 1 тест
|
|
||
| Graduation project for Yandex.Praktikum React-developer course | ||
|
|
||
| «[Stellar Burgers on GH-Pages](https://mrhalless.github.io/react-burger/)» |
There was a problem hiding this comment.
в задеплоенном приложении просто черный экран https://disk.yandex.ru/i/78GIs7OZt8OoVQ
gennady-bars
left a comment
gennady-bars
left a comment
There was a problem hiding this comment.
Поздравляю! Ваша работа принята.
Вы отлично потрудились.
Удачи в Вашей новой профессии.
Несколько советов для дальнейшего развития:
- сверстайте несколько небольших проектов для себя, для закрепления знаний, и чтобы можно добавить их в портфолио
- ходите на собеседования: опыт прохождения собеседований не менее важен знаний теории, и Вы поймете, какие знания требуются на рынке, и где Вы проседаете
- читайте habr и слушаете подкаксты: они очень хорошо расширяют кругозор и позволяют узнать, что происходит в отрасли:
https://soundcloud.com/everyonecan
https://soundcloud.com/podlodka
https://soundcloud.com/devschacht
https://soundcloud.com/begebot
2 participants
No description provided.