Conversation
locators/locators.py
Outdated
tests/test_bun.py
Outdated
| bun = Bun(name, price) | ||
| assert bun.get_name() == name | ||
|
|
||
| @pytest.mark.parametrize('name, price', [ |
There was a problem hiding this comment.
Можно лучше: надежнее тестировать параметр при остальных неизменных. Например тут тестируем цену, значит в параметризации должна меняться только цена, а имя быть всегда одинаковым. в других случаях - аналогично
tests/test_burger.py
Outdated
| bun = MagicMock(spec=Bun) | ||
| bun.get_name.return_value = 'black bun' | ||
| bun.get_price.return_value = 100 | ||
| return bun |
There was a problem hiding this comment.
Нужно исправить: фикстуры нужно расположить в файле conftest.py
tests/test_burger.py
Outdated
| class TestBurgerGetPrice: | ||
|
|
||
| def test_get_price_bun_only(self, burger_with_bun): | ||
| assert burger_with_bun.get_price() == 200 |
There was a problem hiding this comment.
Нужно исправить: не стоит хранить одно и то же тестовое значение отдельно - если что-то было создано в фикстуре, то тут стоит вызвать это значение, а не дублировать его. Иначе тесты станут неподдерживаемыми - легко изменить что то в фикстуре (например) и не заметить что и в тесте тоже нужно поменять. Так как сейчас эти данные не связаны, но должны быть. Удачное решение - создать файл с тестовыми данными и обращаться к нему и в фикстуре и в тестах
tests/test_burger.py
Outdated
| class TestBurgerGetReceipt: | ||
|
|
||
| def test_get_receipt_contains_bun_name(self, burger_with_bun): | ||
| assert 'black bun' in burger_with_bun.get_receipt() |
There was a problem hiding this comment.
Нужно исправить: не стоит хранить одно и то же тестовое значение отдельно - если что-то было создано в фикстуре, то тут стоит вызвать это значение, а не дублировать его. Иначе тесты станут неподдерживаемыми - легко изменить что то в фикстуре (например) и не заметить что и в тесте тоже нужно поменять. Так как сейчас эти данные не связаны, но должны быть. Удачное решение - создать файл с тестовыми данными и обращаться к нему и в фикстуре и в тестах
tests/test_constructor.py
Outdated
locators/__init__.py
Outdated
tests/test_login.py
Outdated
tests/test_personal_account.py
Outdated
tests/test_registration.py
Outdated
helpers.py
Outdated
2 participants
No description provided.