Poprawne sprzątanie po patchowaniu funkcji za pomocą modułu mock w TestCase.setUp()

[dybi]

Actually there is one pitfall patching this way. If for some reasonsetUp fails, tearDown method will not be called for it, thus patch will remian 'unstopped'. To overcame this, you could rewrite the setup to sth like:

def setUp(self):
        super().setUp()
        patcher_in_message_handlers = mock.patch('core.message_handlers.calculate_subtask_verification_time', return_value=1800)
        patcher_in_transfer_operations = mock.patch('core.transfer_operations.calculate_subtask_verification_time', return_value=1800)
        self.addCleanup(patcher_in_message_handlers.stop)
        self.addCleanup(patcher_in_transfer_operations.stop)
        patcher_in_message_handlers.start()
        patcher_in_transfer_operations.start()

[rwrzesien]

👍

[cameel]

@dybi @rwrzesien Moglibyście podać konkretny tekst jaki powinien trafić do Coding Style?