При проверке py.test --pep8 ругается на exit(10) — что не так?

Question

[sash999]

есть кусок кода:
if (len(sys.argv) != 2):
print('Usage: {} object_id'.format(sys.argv[0]))
exit(10)

При проверке py.test ругается на вот этот exit(10). Что не так и как можно переписать этот кусок чтобы тест проходил чисто?

Comments

[sash999]

сбилось немного форматирование, перед принтом и экзитом есть по 4 пробела.

[sash999]

if (len(sys.argv) != 2):
    print('Usage: {} object_id'.format(sys.argv[0]))
    exit(10)

sasha@admin ~/Py-dev/docx $ py.test --pep8 merge.py 
================================== test session starts ==================================
platform linux -- Python 3.5.2, pytest-3.8.0, py-1.6.0, pluggy-0.7.1
rootdir: /home/sasha/Py-dev/docx, inifile:
plugins: pep8-1.0.6
collected 1 item / 1 errors                                                             

======================================== ERRORS =========================================
_______________________________ ERROR collecting merge.py _______________________________
merge.py:27: in <module>
    exit(10)
/usr/lib/python3.5/_sitebuiltins.py:26: in __call__
    raise SystemExit(code)
E   SystemExit: 10
------------------------------------ Captured stdout ------------------------------------
Usage: /usr/local/bin/py.test object_id
!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!
================================ 1 error in 0.20 seconds ================================

[sash999]

Pavel Denisov, я догадываюсь. :) Вопрос-то несколько другой - можно ли сделать что-то, чтобы py.test проходил чисто? Если вопрос тупой - сразу прошу прощения, я только учусь.

[sash999]

Pavel Denisov, а как правильно проверять на pep8? Просто это я в какой-то книге вычитал и как по другому не знаю.

[sash999]

Pavel Denisov, да, решение вынести под main работает, спасибо!!!

Answer 1

[dedalqq]

Видимо не вызывать в тестах функции которые приводят к завершению всей программы. Как варинат можно обернуть exit(10) в свою функцию __exit() а в тесте ее мокать. А вообще, по хорошему. Тестами проверяется "бизнес логика" выход из программы к таковой не относится. Вы вызываете бизнес логику передавая что то на вход и получая что то на выход. По сути тесты позволяют вам запустить вашу бизнеслогику на необходимо наборе входнх данных и убедиться что на выходе получается именно то, что ожидается. А выход обыгрывается уже по другому.