Стыковка двух функций с патчем для unit test

У меня есть функция, которую я хочу unit test содержит вызовы двух других функций. Я не уверен, как я могу одновременно фальсифицировать обе функции, используя исправление. Я привел пример того, что я имею в виду ниже. Когда я запускаю nosetests, тесты проходят, но я чувствую, что должен быть более чистый способ сделать это, и я действительно не понимаю, что касается f.close()...

Структура каталогов выглядит следующим образом:

program/
  program/
    data.py
  tests/
    data_test.py

data.py:

import cPickle

def write_out(file_path, data):
    f = open(file_path, 'wb')
    cPickle.dump(data, f)
    f.close()

data_test.py:

from mock import MagicMock, patch

def test_write_out():
    path = '~/collection'
    mock_open = MagicMock()
    mock_pickle = MagicMock()
    f_mock = MagicMock()
    with patch('__builtin__.open', mock_open):
        f = mock_open.return_value
        f.method.return_value = path
        with patch('cPickle.dump', mock_pickle):
            write_out(path, 'data')
            mock_open.assert_called_once_with('~/collection', 'wb')
            f.close.assert_any_call()
            mock_pickle.assert_called_once_with('data', f)

Результаты:

$ nosetests
.
----------------------------------------------------------------------
Ran 1 test in 0.008s
OK

Ответы

Ответ 1

Вы можете упростить свой тест, используя декоратор патча и вложив их так (они по умолчанию являются MagicMock):

@patch('cPickle.dump')
@patch('__builtin__.open')
def test_write_out(mock_open, mock_pickle):
    path = '~/collection'
    f = mock_open.return_value
    f.method.return_value = path

    write_out(path, 'data')

    mock_open.assert_called_once_with('~/collection', 'wb')
    mock_pickle.assert_called_once_with('data', f)
    f.close.assert_any_call()

Вызов экземпляра MagicMock возвращает новый экземпляр MagicMock, поэтому вы можете проверить, что возвращаемое значение вызывалось так же, как и любой другой издевавшийся объект. В этом случае f является MagicMock с именем 'open()' (попробуйте распечатать f).

Ответ 2

В дополнение к ответу @Matti John вы также можете использовать patch внутри функции test_write_out:

from mock import MagicMock, patch

def test_write_out():
    path = '~/collection'
    with patch('__builtin__.open') as mock_open, \
            patch('cPickle.dump') as mock_pickle:

        f = mock_open.return_value
        ...

Ответ 3

Вот простой пример того, как тестировать функцию повышения ConflictError в create_collection, используя mock:

import os
from unittest import TestCase
from mock import patch
from ..program.data import ConflictError, create_collection


class TestCreateCollection(TestCase):
    def test_path_exists(self):
        with patch.object(os.path, 'exists') as mock_method:
            mock_method.return_value = True

            self.assertRaises(ConflictError, create_collection, 'test')

Также см. mock docs и Michael Foord awesome введение в mock.