Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Волков Никита #36

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

VolkovNik
Copy link

Проверки:

  1. Авторизация
  2. Страница фильмов. Переход на страницу фильма при нажатии на карточку фильма/сериала
  3. Страница фильмов. Переход на страницу жанра при нажатии на карточку жанра
    (ссылка на чек-листы: http://jira.bmstu.cloud/browse/QA-1357)

@VolkovNik VolkovNik changed the title Семинар 2: E2E тесты Волков Никита Oct 21, 2021
@KoroLion
Copy link

KoroLion commented Oct 22, 2021

остался тест из примера (c mail.ru, cases/example/login_test.js), который запускается и заваливает проверку

UPD: пофикшено

@VolkovNik
Copy link
Author

Снимок экрана 2021-10-22 в 20 52 21

@VolkovNik
Copy link
Author

остался тест из примера (c mail.ru, cases/example/login_test.js), который запускается и заваливает проверку

убрал тест из примера

cases/login_test.js Outdated Show resolved Hide resolved
cases/login_test.js Outdated Show resolved Hide resolved
Copy link

@moroz-matros moroz-matros left a comment

Choose a reason for hiding this comment

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

Файл с результатами тестирования вроде как надо закинуть в репу, да. И можно еще ридми пофиксить на предмет строки запуска тестов, а то искать EMAIL было не оч удобно :D

@VolkovNik
Copy link
Author

test_results

@VolkovNik
Copy link
Author

Файл с результатами тестирования вроде как надо закинуть в репу, да. И можно еще ридми пофиксить на предмет строки запуска тестов, а то искать EMAIL было не оч удобно :D

Сорри, я чет не подумал про изменение ридми. Я закинул и добавил EMAIL. Спасибо за ревью

Copy link

@moroz-matros moroz-matros left a comment

Choose a reason for hiding this comment

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

красотень, аппрув!

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