Fix argument order in assertEqual: tests/test_nfs.py
The assertEqual in cinder/tests/test_nfs.py is using incorrect argument order (observed, expected), which causes the error message about mismatch to be reversed if the test case fails. Change it to (expected, observed). Change-Id: I7fec2e5df91369c3cae2adef3189e8bd86fe4480 Partial-Bug: #1259292
This commit is contained in:
parent
48eb05aa84
commit
d813f3f005
|
@ -165,14 +165,14 @@ class RemoteFsDriverTestCase(test.TestCase):
|
|||
secure_file_permissions,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'true')
|
||||
self.assertEqual('true', nas_option)
|
||||
|
||||
secure_file_operations = 'auto'
|
||||
nas_option = drv._determine_nas_security_option_setting(
|
||||
secure_file_operations,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'true')
|
||||
self.assertEqual('true', nas_option)
|
||||
|
||||
@mock.patch('os.path.join')
|
||||
@mock.patch('os.path.isfile')
|
||||
|
@ -201,14 +201,14 @@ class RemoteFsDriverTestCase(test.TestCase):
|
|||
secure_file_permissions,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'true')
|
||||
self.assertEqual('true', nas_option)
|
||||
|
||||
secure_file_operations = 'auto'
|
||||
nas_option = drv._determine_nas_security_option_setting(
|
||||
secure_file_operations,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'true')
|
||||
self.assertEqual('true', nas_option)
|
||||
|
||||
@mock.patch('os.path.join')
|
||||
@mock.patch('os.path.isfile')
|
||||
|
@ -236,14 +236,14 @@ class RemoteFsDriverTestCase(test.TestCase):
|
|||
secure_file_permissions,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'false')
|
||||
self.assertEqual('false', nas_option)
|
||||
|
||||
secure_file_operations = 'auto'
|
||||
nas_option = drv._determine_nas_security_option_setting(
|
||||
secure_file_operations,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'false')
|
||||
self.assertEqual('false', nas_option)
|
||||
|
||||
def test_determine_nas_security_options_when_admin_set_true(self):
|
||||
"""Test the setting of the NAS Security Option
|
||||
|
@ -263,14 +263,14 @@ class RemoteFsDriverTestCase(test.TestCase):
|
|||
secure_file_permissions,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'true')
|
||||
self.assertEqual('true', nas_option)
|
||||
|
||||
secure_file_operations = 'true'
|
||||
nas_option = drv._determine_nas_security_option_setting(
|
||||
secure_file_operations,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'true')
|
||||
self.assertEqual('true', nas_option)
|
||||
|
||||
def test_determine_nas_security_options_when_admin_set_false(self):
|
||||
"""Test the setting of the NAS Security Option
|
||||
|
@ -290,14 +290,14 @@ class RemoteFsDriverTestCase(test.TestCase):
|
|||
secure_file_permissions,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'false')
|
||||
self.assertEqual('false', nas_option)
|
||||
|
||||
secure_file_operations = 'false'
|
||||
nas_option = drv._determine_nas_security_option_setting(
|
||||
secure_file_operations,
|
||||
nas_mount, is_new_install)
|
||||
|
||||
self.assertEqual(nas_option, 'false')
|
||||
self.assertEqual('false', nas_option)
|
||||
|
||||
@mock.patch.object(remotefs, 'LOG')
|
||||
def test_set_nas_security_options(self, LOG):
|
||||
|
@ -311,9 +311,9 @@ class RemoteFsDriverTestCase(test.TestCase):
|
|||
|
||||
drv.set_nas_security_options(is_new_install)
|
||||
|
||||
self.assertEqual(drv.configuration.nas_secure_file_operations, 'false')
|
||||
self.assertEqual(drv.configuration.nas_secure_file_permissions,
|
||||
'false')
|
||||
self.assertEqual('false', drv.configuration.nas_secure_file_operations)
|
||||
self.assertEqual('false',
|
||||
drv.configuration.nas_secure_file_permissions)
|
||||
self.assertTrue(LOG.warn.called)
|
||||
|
||||
def test_secure_file_operations_enabled_true(self):
|
||||
|
@ -526,10 +526,10 @@ class NfsDriverTestCase(test.TestCase):
|
|||
|
||||
self.assertIn(self.TEST_NFS_EXPORT1, drv.shares)
|
||||
self.assertIn(self.TEST_NFS_EXPORT2, drv.shares)
|
||||
self.assertEqual(len(drv.shares), 2)
|
||||
self.assertEqual(2, len(drv.shares))
|
||||
|
||||
self.assertEqual(drv.shares[self.TEST_NFS_EXPORT2],
|
||||
self.TEST_NFS_EXPORT2_OPTIONS)
|
||||
self.assertEqual(self.TEST_NFS_EXPORT2_OPTIONS,
|
||||
drv.shares[self.TEST_NFS_EXPORT2])
|
||||
|
||||
mox.VerifyAll()
|
||||
|
||||
|
@ -835,7 +835,7 @@ class NfsDriverTestCase(test.TestCase):
|
|||
with mock.patch.object(drv, '_execute') as mock_execute:
|
||||
drv.delete_volume(volume)
|
||||
|
||||
self.assertEqual(mock_execute.call_count, 0)
|
||||
self.assertEqual(0, mock_execute.call_count)
|
||||
|
||||
def test_get_volume_stats(self):
|
||||
"""get_volume_stats must fill the correct values."""
|
||||
|
@ -859,8 +859,8 @@ class NfsDriverTestCase(test.TestCase):
|
|||
mox.ReplayAll()
|
||||
|
||||
drv.get_volume_stats()
|
||||
self.assertEqual(drv._stats['total_capacity_gb'], 30.0)
|
||||
self.assertEqual(drv._stats['free_capacity_gb'], 5.0)
|
||||
self.assertEqual(30.0, drv._stats['total_capacity_gb'])
|
||||
self.assertEqual(5.0, drv._stats['free_capacity_gb'])
|
||||
|
||||
mox.VerifyAll()
|
||||
|
||||
|
@ -1014,8 +1014,8 @@ class NfsDriverTestCase(test.TestCase):
|
|||
|
||||
drv.set_nas_security_options(is_new_install)
|
||||
|
||||
self.assertEqual(drv.configuration.nas_secure_file_operations, 'true')
|
||||
self.assertEqual(drv.configuration.nas_secure_file_permissions, 'true')
|
||||
self.assertEqual('true', drv.configuration.nas_secure_file_operations)
|
||||
self.assertEqual('true', drv.configuration.nas_secure_file_permissions)
|
||||
self.assertFalse(LOG.warn.called)
|
||||
|
||||
@mock.patch.object(nfs, 'LOG')
|
||||
|
@ -1037,9 +1037,9 @@ class NfsDriverTestCase(test.TestCase):
|
|||
|
||||
drv.set_nas_security_options(is_new_install)
|
||||
|
||||
self.assertEqual(drv.configuration.nas_secure_file_operations, 'false')
|
||||
self.assertEqual(drv.configuration.nas_secure_file_permissions,
|
||||
'false')
|
||||
self.assertEqual('false', drv.configuration.nas_secure_file_operations)
|
||||
self.assertEqual('false',
|
||||
drv.configuration.nas_secure_file_permissions)
|
||||
self.assertTrue(LOG.warn.called)
|
||||
|
||||
def test_set_nas_security_options_exception_if_no_mounted_shares(self):
|
||||
|
|
Loading…
Reference in New Issue