From 34503e780f61eea7d65c14bc88d03aff15363dea Mon Sep 17 00:00:00 2001 From: melissaml Date: Tue, 4 Jul 2017 15:14:13 +0800 Subject: [PATCH] Using assertIsNone() instead of assertEqual(None) Instead of using assertEqual(None, ***), developers should use assertIsNone(***). Change-Id: If91658da3acc621fd5e57b7d11e4f9aebeab50e7 --- proliantutils/tests/ilo/test_ris.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/proliantutils/tests/ilo/test_ris.py b/proliantutils/tests/ilo/test_ris.py index 1097115..a969183 100755 --- a/proliantutils/tests/ilo/test_ris.py +++ b/proliantutils/tests/ilo/test_ris.py @@ -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')