Как протестировать блок except если он не поднимает исключение?

Question

[Сергей Ы]

Доброго дня, нужна помощь с вопросом тестирования блока except в джанге, который не поднимает исключение

@receiver(post_save, sender=AgentRequest)
def action_after_save(instance, **kwargs):
    try:
        send_email(instance)
    except Exception as e:
        logger = logging.getLogger("Send mail")
        logger.error(f"Ошибка отправки {str(e)}")
        #raise e   # ЕСЛИ РАССКОМЕНТИРУЮ ТЕСТ ПРОЙДЁТ

class AgentTest(TestCase):
    def test_action_after_save(self):
        """Тест action_after_save"""
        with self.assertRaises(Exception):
            action_after_save(instance=None)

Creating test database for alias 'default'...
System check identified no issues (0 silenced).
Ошибка отправки 'NoneType' object has no attribute 'email_text'
F...
======================================================================
FAIL: test_action_after_save (agent.tests.AgentTest.test_action_after_save)
Тест action_after_save
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/app/web/agent/tests.py", line 57, in test_action_after_save
with self.assertRaises(Exception):
AssertionError: Exception not raised

----------------------------------------------------------------------

Answer 1

[Everything_is_bad]

Так на assertRaises надо тестить send_email, а action_after_save надо тестить на запись в logger, если вообще такое надо тестить

Comments

[Сергей Ы]

Я это понимаю, так и сделал только в другом модуле, может не совсем удачный пример, написал чтобы по-миниму но понятно было, если блоке exept , больше кода а не только логер

[Сергей П]

Сергей Ы, вы тестите action_after_save, который штатно не бросает эксепшн (гасит его). Вы не можете проверять это требуя наличия исключения. Его там нет, требуйте, чтобы не было исключения. Если понятно выразился.

[Everything_is_bad]

Сергей Ы, у тебя action_after_save наружу не райзит исключение, так что чекай был ли вызов logger.error

[Сергей Ы]

Ок, ладно суть впринципе понятна, думал может упустил что то и есть вариант без подъема исключения его в тесте отловить, спасибо!

[Сергей Ы]

Everything_is_bad, "так что чекай был ли вызов logger.error" --- буду благодарен примеру

[Everything_is_bad]

Сергей Ы, https://docs.python.org/3/library/unittest.mock.html ну и я бы в добавок перешел бы на pytest

[Сергей Ы]

Everything_is_bad, Спасибо за наводку, поковырял TestCase нашел assertLogs, пока достаточно или для таких пустых блоков, которые чисто логируют, как то так, можно и текст из лога вытащить и его сравнить

def test_action_after_save(self):
        """Тест action_after_save"""
        
        with self.assertLogs('Send mail', level='ERROR'):
            action_after_save(instance=None)

[chemdev]

Если ты мокаешь место куда отправляется емаил, то можно по идее просто смотреть отправился или нет

Answer 2

[Сергей Ы]

В данном случае достаточно, кто помогал спасибо!

def test_action_after_save(self):
        """Тест action_after_save"""
        with self.assertLogs('Send mail', level='ERROR'):
            action_after_save(instance=None)