Skip to content

Add new tests for input "item name", valid and invalid#390

Open
NikolaHello wants to merge 6 commits intomainfrom
vm/add-jenkins-tests
Open

Add new tests for input "item name", valid and invalid#390
NikolaHello wants to merge 6 commits intomainfrom
vm/add-jenkins-tests

Conversation

@NikolaHello
Copy link
Copy Markdown
Collaborator

Add new tests for input "item name", valid and invalid

@igolinskaya
Copy link
Copy Markdown
Collaborator

CI / Run tests (pull_request) не прошел. можно искать ошибку другого теста или ставить тэг. в python projects выше писали как починить https://redroverschool.slack.com/archives/C05US8RLPFU/p1776891733864619 tests/test_nikola_input_item_name.py::test_invalid_input_item_name[my item]

@StasAxlS
Copy link
Copy Markdown
Collaborator

CI / Run tests (pull_request) не прошел. можно искать ошибку другого теста или ставить тэг. в python projects выше писали как починить https://redroverschool.slack.com/archives/C05US8RLPFU/p1776891733864619 tests/test_nikola_input_item_name.py::test_invalid_input_item_name[my item]

Так не проходит на CI именно тест который она добавляет же. - test_invalid_input_item_name С ожиданиями надо попробовать поиграть, возможно какой то элемент на CI смещен(запустить у себя в докере Дженкинс и попробовать там потестить).

@NikolaHello
Copy link
Copy Markdown
Collaborator Author

NikolaHello commented Apr 24, 2026

тесты в test_nikola_input_item_name.py поправила, теперь они все проходят. CI падает из-за другого теста test_create_new_item.py

@StasAxlS
Copy link
Copy Markdown
Collaborator

Долго PR провисел, что теперь имя файла не очень - test_nikola_input_item_name.py
Раз по Юзер Сторис New Item то, либо перенеси свой тест в test_new_item.py либо создай test_new_item2.py

@StasAxlS
Copy link
Copy Markdown
Collaborator

StasAxlS commented Apr 25, 2026

И когда скипаем тест, надо карточку ER завести, со скринами ошибки скипнутого теста на CI. Что бы было легче его исправлять потом.

@NikolaHello
Copy link
Copy Markdown
Collaborator Author

@StasAxlS просмотрела лекцию, поправила имя файла, удалила лишние ассерты и создала ER
надеюсь на этот раз оформила все правильно)

@StasAxlS
Copy link
Copy Markdown
Collaborator

StasAxlS commented Apr 28, 2026

@StasAxlS просмотрела лекцию, поправила имя файла, удалила лишние ассерты и создала ER надеюсь на этот раз оформила все правильно)

Вопрос а зачем тут оставила файл test_nikola_input_item_name.py‎ ? Тут разные тесты? Или просто забыла убрать?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants