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.

Sign up for GitHub

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

Jump to bottom

BugHunters: Регистрация и авторизация, центр коммерции, сайты #25

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

BugHunters: Регистрация и авторизация, центр коммерции, сайты #25

wants to merge 5 commits into from

Conversation

FunnyRockfish
Copy link

No description provided.

BeeHeather
BeeHeather suggested changes Dec 1, 2024
View reviewed changes
Copy link

@BeeHeather BeeHeather left a comment

Choose a reason for hiding this comment

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

Привет! Оставила несколько комментариев, посмотри пожалуйста. Еще сомневаюсь насчет автоматизации кейсов на регистрацию пользователя, у VK ID же двухфакторка стоит( уточню этот момент у преподавателей

BugHunters-Артём-Черников.md Outdated


4. **Выбор языка (Где: Приветственная страница создания ЛК):**
На приветственной странице создания ЛК доступны два языка на выбор: русский и English.
Copy link

Choose a reason for hiding this comment

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

Го тогда уж Русский тоже с большой буквы 💪🇷🇺

Copy link

Choose a reason for hiding this comment

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

Вообще этот и следующий тест я бы объединила в один. Мы в нем можем проверить наличие свитчера языка, чекнуть какой язык у нас дефолтный, переключить язык, и снова проверить текст на странице

Copy link
Author

Choose a reason for hiding this comment

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

fixed)

BugHunters-Артём-Черников.md Outdated
![image](https://github.com/user-attachments/assets/1115d7ac-7d04-46ff-b10d-feac86e5bb97)


11. **Выбор языка при создании ЛК (Где: Страница создания ЛК):**
Copy link

Choose a reason for hiding this comment

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

Мне кажется мы тут дублируем тест одного и того же функционала

Copy link
Author

Choose a reason for hiding this comment

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

fixed

BugHunters-Артём-Черников.md Outdated
4. **Обучение (Где: Центр коммерции — Главная страница):**
При клике на кнопку «Пройти обучение» появляется диалоговое окно с выбором одного из трёх способов обучения.

5. **Закрытие окна обучения (Где: Центр коммерции — Диалоговое окно «Пройти обучение»):**
Copy link

Choose a reason for hiding this comment

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

Еще стоит проверить нажатие на кнопку "Не сейчас"

Copy link
Author

Choose a reason for hiding this comment

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

fixed

BugHunters-Артём-Черников.md Outdated


3. **Приветственная страница (Где: Центр коммерции — Главная страница):**
Если у пользователя нет каталогов, отображается страница с предложением создать каталог или пройти обучение.
Copy link

Choose a reason for hiding this comment

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

А по какому принципу определяется, что отобразится - диалоговое окно об обучении или страница с созданием каталога? тут нужно больше конкретики

Copy link
Author

Choose a reason for hiding this comment

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

fixed

BugHunters-Артём-Черников.md
Если у пользователя нет каталогов, отображается страница с предложением создать каталог или пройти обучение.

4. **Обучение (Где: Центр коммерции — Главная страница):**
При клике на кнопку «Пройти обучение» появляется диалоговое окно с выбором одного из трёх способов обучения.
Copy link

Choose a reason for hiding this comment

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

Здесь скрина не хватает

Copy link
Author

Choose a reason for hiding this comment

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

fixed

BugHunters-Артём-Черников.md


9. **Проверка ссылки (Где: Окно создания каталога):**
При корректно указанной ссылке на фид или сообщество процесс каталога завершается успешно - редирект на профиль каталога.
Copy link

Choose a reason for hiding this comment

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

А при некорректной?

Copy link
Author

Choose a reason for hiding this comment

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

fixed

BugHunters-Артём-Черников.md Outdated


6. **Добавление пискеля (Где: Сайты. Окно добавления пикселя)**
Если введены правильные данные пикселя, то сайт будет добавлен в список пикселей на главной странице сайтов и высвечено уведомление об успешном создании пикселя. ![image](https://github.com/user-attachments/assets/131bd3c9-9a1a-4921-b8b8-a5b01f8fa46f)
Copy link

Choose a reason for hiding this comment

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

Мы здесь домен сайта указывали или ID?

Copy link

Choose a reason for hiding this comment

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

Еще го проверим закрытие формы создания пикселя и нажатие на кнопку "Получить код"

Copy link
Author

Choose a reason for hiding this comment

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

fixed)

BugHunters-Артём-Черников.md Outdated
![image](https://github.com/user-attachments/assets/efae12f6-2b11-4617-9902-90b5d2bdc162)


5. **Добавление пискеля (Где: Сайты. Окно добавления пикселя)**
Copy link

Choose a reason for hiding this comment

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

Стоит добавить проверки на закрытие формы добавления пикселя

Copy link
Author

Choose a reason for hiding this comment

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

Добавил, но он небольшой совсем ахах

Copy link

Choose a reason for hiding this comment

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

та сойдет, главное чтоб был)

BeeHeather
BeeHeather reviewed Dec 2, 2024
View reviewed changes
BugHunters-Артём-Черников.md Outdated
При нажатии на значок кошелька открывается [старница с реквизитами](https://ads.vk.com/hq/budget/transactions).
![image](https://github.com/user-attachments/assets/f15688ad-bcc1-4344-8e65-5693f15d3f19)

2. **Указание реквизитов (Где: [старница с реквизитами](https://ads.vk.com/hq/budget/transactions))**
Copy link

Choose a reason for hiding this comment

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

Несколько кейсов в одном

BugHunters-Артём-Черников.md

13. **Сохранение реквизитов(Где: окно добавления реквизитов)**
Если сохранение прошло успешно, появляется уведомление об этом. Его можно закрыть по кнопке "Закрыть" или по крестику.
(вообще упс, я не знал, что их будут в ручную видимо проверять ахахах)
Copy link

Choose a reason for hiding this comment

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

👀

BugHunters-Артём-Черников.md
![image](https://github.com/user-attachments/assets/4d84f13c-cfd8-46a1-8a00-ad2a5e4d3e40)

8. **Заполнение данных о Юр. Адресе - позитивный (Где: окно добавления реквизитов)**
Если заполнить все обязательные поля данными, соответствующими ограничениям к полям - а именно, индекс = 6 символам, выбрать регион из списка, город не больше 40 символов, улица не больше 60 символов, номер и корпус дома в сумме не больше 10 символов, то можно перейти на следующий шаг - кнопка "Далее" становится кликабельной.
Copy link

Choose a reason for hiding this comment

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

Го добавим негативный кейс (хотя бы с пустыми обязательными полями)

BugHunters-Артём-Черников.md Outdated
Можно оставить все поля пустыми и перейти к следующему шагу - кнопка "Далее" будет кликабельной.
![image](https://github.com/user-attachments/assets/acd0738b-8602-4a6d-a62b-9f91db515d76)

11. **Заполнение данных о Контактах (Где: окно добавления реквизитов)**
Copy link

Choose a reason for hiding this comment

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

Аналогично, го добавим негативный кейс

BugHunters-Артём-Черников.md Outdated
Если заполнить все обязательные поля данными, соответствующими ограничениям к полям - а именно, контактное лицо не более 40 символов и не пустое, телефон не больше 12 сиволов, то кнопка "Далее" становится кликабельной, и можно перейти к следующему шагу.
![image](https://github.com/user-attachments/assets/e33f6f26-b935-4a5f-b5e1-60cda1c83b51)

12. **Заполнение данных о Документах (Где: окно добавления реквизитов)**
Copy link

Choose a reason for hiding this comment

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

Аналогично, го добавим негативный кейс

BugHunters-Артём-Черников.md Outdated
![image](https://github.com/user-attachments/assets/efae12f6-2b11-4617-9902-90b5d2bdc162)


5. **Добавление пискеля (Где: Сайты. Окно добавления пикселя)**
Copy link

Choose a reason for hiding this comment

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

та сойдет, главное чтоб был)

BeeHeather
BeeHeather suggested changes Dec 2, 2024
View reviewed changes
Copy link

@BeeHeather BeeHeather left a comment

Choose a reason for hiding this comment

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

Привет! Оставила несколько комментов, посмотри пж)

BeeHeather
BeeHeather approved these changes Dec 4, 2024
View reviewed changes
Copy link

@BeeHeather BeeHeather 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
Reviewers

@BeeHeather BeeHeather BeeHeather approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FunnyRockfish @BeeHeather