First attempt#2
Conversation
| if module_name == '_pytest': | ||
| sys_modules[module_name].outcomes.Skipped = sys_modules[module_name].SkipTest | ||
|
|
||
| assert skip_exceptions_to_reraise() == (sys_modules[module_name].SkipTest,) |
There was a problem hiding this comment.
suggestion (non-blocking): Думаю, можно еще добавить случай, когда в sys.modules несколько ключей
There was a problem hiding this comment.
Решили пока не добавлять этот тест, все ломается и ничего не работает, записал себе и попробую еще на днях
| with pytest.raises(ColumnError): | ||
| assert processor._get_datetime_from_string('18 9 2018') == expected | ||
| else: | ||
| assert processor._get_datetime_from_string('18 9 2018') == expected |
There was a problem hiding this comment.
nitpick: Ты же задаешь в параметрах value, в функцию тоже его можешь передавать
| processor = DateTimeProcessor(timezone=timezone) | ||
| if expected: | ||
| mocker.patch('main.DateTimeProcessor._get_datetime_from_string', return_value=expected) | ||
| assert processor.process_value(value) == expected |
There was a problem hiding this comment.
suggestion (non-blocking): Кажется, что здесь везде можно задавать expected, ведь случай с exception ты рассматриваешь в тесте ниже
There was a problem hiding this comment.
Там проверяются разные ошибки в разных методах, я бы не стал их смешивать)
| 'value, timezone, exception', | ||
| [ | ||
| (123, None, ColumnError), | ||
| (123, 'UBC', ValueError), |
There was a problem hiding this comment.
question: А разве в методе process_value возникает ValueError?
There was a problem hiding this comment.
ValueError вызывается в init :) Но я отдельно решил не писать тест для init
2 participants
No description provided.