openstack/deb-python-proliantutils
Code Issues Proposed changes

Using assertIsNone() instead of assertEqual(None)

Browse Source 
Instead of using assertEqual(None, ***), developers should
use assertIsNone(***).

Change-Id: If91658da3acc621fd5e57b7d11e4f9aebeab50e7
This commit is contained in:
melissaml
2017-07-04 15:14:13 +08:00
parent e39fd72626
commit 34503e780f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
proliantutils/tests/ilo/test_ris.py
View File

@@ -800,7 +800,7 @@ class IloRisTestCase(testtools.TestCase):
_uefi_boot_mode_mock.return_value = False
ret = self.client.get_persistent_boot_device()
get_host_details_mock.assert_called_once_with()
self.assertEqual(ret, None)
self.assertIsNone(ret)
@mock.patch.object(ris.RISOperations, '_get_persistent_boot_devices')
@mock.patch.object(ris.RISOperations, '_is_boot_mode_uefi')