openstack/ironic
Code Issues Proposed changes

Merge "Use more granular mocking in test_utils"

Browse Source
This commit is contained in:
Zuul 2018-03-12 14:06:17 +00:00 committed by Gerrit Code Review
commit 49946235be
1 changed files with 16 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
ironic/tests/unit/common/test_utils.py
View File

@ -418,25 +418,29 @@ class TempFilesTestCase(base.TestCase):
mock_dir_writable.assert_called_once_with('/fake/path') mock_dir_writable.assert_called_once_with('/fake/path')
mock_free_space.assert_called_once_with('/fake/path', 5) mock_free_space.assert_called_once_with('/fake/path', 5)
@mock.patch.object(os.path, 'exists', autospec=True)
@mock.patch.object(utils, '_check_dir_writable', autospec=True) @mock.patch.object(utils, '_check_dir_writable', autospec=True)
@mock.patch.object(utils, '_check_dir_free_space', autospec=True) @mock.patch.object(utils, '_check_dir_free_space', autospec=True)
def test_check_dir_no_dir(self, mock_free_space, mock_dir_writable, def test_check_dir_no_dir(self, mock_free_space, mock_dir_writable):
mock_exists):
mock_exists.return_value = False
self.config(tempdir='/fake/path') self.config(tempdir='/fake/path')
# NOTE(dtantsur): self.config uses os.path.exists, so we cannot mock
# on the method level.
with mock.patch.object(os.path, 'exists',
autospec=True) as mock_exists:
mock_exists.return_value = False
self.assertRaises(exception.PathNotFound, utils.check_dir) self.assertRaises(exception.PathNotFound, utils.check_dir)
mock_exists.assert_called_once_with(CONF.tempdir) mock_exists.assert_called_once_with(CONF.tempdir)
self.assertFalse(mock_free_space.called) self.assertFalse(mock_free_space.called)
self.assertFalse(mock_dir_writable.called) self.assertFalse(mock_dir_writable.called)
@mock.patch.object(os.path, 'exists', autospec=True)
@mock.patch.object(utils, '_check_dir_writable', autospec=True) @mock.patch.object(utils, '_check_dir_writable', autospec=True)
@mock.patch.object(utils, '_check_dir_free_space', autospec=True) @mock.patch.object(utils, '_check_dir_free_space', autospec=True)
def test_check_dir_ok(self, mock_free_space, mock_dir_writable, def test_check_dir_ok(self, mock_free_space, mock_dir_writable):
mock_exists):
mock_exists.return_value = True
self.config(tempdir='/fake/path') self.config(tempdir='/fake/path')
# NOTE(dtantsur): self.config uses os.path.exists, so we cannot mock
# on the method level.
with mock.patch.object(os.path, 'exists',
autospec=True) as mock_exists:
mock_exists.return_value = True
utils.check_dir() utils.check_dir()
mock_exists.assert_called_once_with(CONF.tempdir) mock_exists.assert_called_once_with(CONF.tempdir)
mock_dir_writable.assert_called_once_with(CONF.tempdir) mock_dir_writable.assert_called_once_with(CONF.tempdir)