diff options
| author | Jenkins <jenkins@review.openstack.org> | 2016-02-04 22:56:57 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2016-02-04 22:56:57 +0000 |
| commit | 42b607edf117e4a3f421a554308330409c88dbb1 (patch) | |
| tree | 00cbc0f9e524dea4ba561575ffc05c0ffffc488f /openstackclient | |
| parent | f4ad41761adddbb5f9125e97d657b202924181fb (diff) | |
| parent | 38ff51baef6127975d6248783347eb4ef06a0d6f (diff) | |
| download | python-openstackclient-42b607edf117e4a3f421a554308330409c88dbb1.tar.gz | |
Merge "Use assert_not_called() in common tests"
Diffstat (limited to 'openstackclient')
| -rw-r--r-- | openstackclient/tests/common/test_utils.py | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/openstackclient/tests/common/test_utils.py b/openstackclient/tests/common/test_utils.py index 09545079..da7ce063 100644 --- a/openstackclient/tests/common/test_utils.py +++ b/openstackclient/tests/common/test_utils.py @@ -143,7 +143,7 @@ class TestUtils(test_utils.TestCase): status_f = mock.Mock(return_value=resource) res_id = str(uuid.uuid4()) self.assertTrue(utils.wait_for_status(status_f, res_id,)) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() @mock.patch.object(time, 'sleep') def test_wait_for_status_ok__with_overrides(self, mock_sleep): @@ -154,7 +154,7 @@ class TestUtils(test_utils.TestCase): self.assertTrue(utils.wait_for_status(status_f, res_id, status_field='my_status', success_status=['complete'])) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() @mock.patch.object(time, 'sleep') def test_wait_for_status_error(self, mock_sleep): @@ -163,7 +163,7 @@ class TestUtils(test_utils.TestCase): status_f = mock.Mock(return_value=resource) res_id = str(uuid.uuid4()) self.assertFalse(utils.wait_for_status(status_f, res_id)) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() @mock.patch.object(time, 'sleep') def test_wait_for_status_error_with_overrides(self, mock_sleep): @@ -174,7 +174,7 @@ class TestUtils(test_utils.TestCase): self.assertFalse(utils.wait_for_status(status_f, res_id, status_field='my_status', error_status=['failed'])) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() @mock.patch.object(time, 'sleep') def test_wait_for_delete_ok(self, mock_sleep): @@ -210,7 +210,7 @@ class TestUtils(test_utils.TestCase): manager = mock.MagicMock(get=mock_get) res_id = str(uuid.uuid4()) self.assertFalse(utils.wait_for_delete(manager, res_id)) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() @mock.patch.object(time, 'sleep') def test_wait_for_delete_error_with_overrides(self, mock_sleep): @@ -222,7 +222,7 @@ class TestUtils(test_utils.TestCase): self.assertFalse(utils.wait_for_delete(manager, res_id, status_field='my_status', error_status=['failed'])) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() @mock.patch.object(time, 'sleep') def test_wait_for_delete_error_with_overrides_exception(self, mock_sleep): @@ -232,7 +232,7 @@ class TestUtils(test_utils.TestCase): res_id = str(uuid.uuid4()) self.assertTrue(utils.wait_for_delete(manager, res_id, exception_name=['Exception'])) - self.assertFalse(mock_sleep.called) + mock_sleep.assert_not_called() def test_build_kwargs_dict_value_set(self): self.assertEqual({'arg_bla': 'bla'}, |