From 9434d70306979b7559e7f1b3c14547a56cbeff1c Mon Sep 17 00:00:00 2001 From: Tom Barron Date: Fri, 12 Feb 2016 11:48:16 -0500 Subject: [PATCH] Fix 'asert' typo in unit test The dellscapi unit tests contain two occurences of an apparent typo, 'mock_get_volume_list.asert_called_once_with'. Because mock will magically create an 'asert_called_once_with' method, this typo masks a problem with the test_manage_unmanage test where the wrong set of arguments is expected. This commit substitutes 'assert' for 'asert' in these occurences and fixes up the test_manage_unmanage test. Change-Id: I0f1b370ef61b28a46ab48d40595c79ec13e359f8 --- cinder/tests/unit/test_dellscapi.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cinder/tests/unit/test_dellscapi.py b/cinder/tests/unit/test_dellscapi.py index c553bc4ed..aead70fc0 100644 --- a/cinder/tests/unit/test_dellscapi.py +++ b/cinder/tests/unit/test_dellscapi.py @@ -5533,7 +5533,8 @@ class DellSCSanAPITestCase(test.TestCase): newname = 'guid' existing = {'source-name': 'scvolname'} self.scapi.manage_existing(newname, existing) - mock_get_volume_list.asert_called_once_with(existing, False) + mock_get_volume_list.assert_called_once_with( + existing.get('source-name'), None, False) self.assertTrue(mock_find_mappings.called) self.assertTrue(mock_size_to_gb.called) @@ -5553,7 +5554,7 @@ class DellSCSanAPITestCase(test.TestCase): self.scapi.manage_existing, newname, existing) - mock_get_volume_list.asert_called_once_with( + mock_get_volume_list.assert_called_once_with( existing.get('source-name'), existing.get('source-id'), False) -- 2.45.2