Skip to content

Conversation

@vadimnesterov
Copy link

Краткое резюме проделанной работы (с указанием классов)

Создана фикстура burger в conftest.py, предоставляющая новый экземпляр класса Burger.

Протестированы все методы класса Burger:

set_buns

add_ingredient

remove_ingredient

move_ingredient

get_price

get_receipt

# - test_set_buns_sets_bun: проверяет базовый сценарий первого вызова метода.
# - test_set_buns_uses_latest_bun: проверяет сценарий повторного вызова метода(обновление состояния)

def test_set_buns_sets_bun(burger):

Choose a reason for hiding this comment

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

Нужно исправить здесь и далее: Для корректного запуска тестов необходимо код с шагами теста поместить в тестовый метод (нейминг начинается с test_), а метод - в тестовый класс (нейминг начинается с Test). Необходимо привести к такому формату все модули внутри пакета tests

Comment on lines 145 to 150
assert lines[0] == "(==== black bun ====)"
assert lines[1] == "= sauce hot sauce ="
assert lines[2] == "(==== black bun ====)"
assert lines[3] == "" # пустая строка
assert lines[4].startswith("Price: ") # цена начинается с Price

Choose a reason for hiding this comment

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

Необходимо исправить: стоит сравнивать весь вывод от get_receipt целиком, а не построчно

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.

2 participants