diff options
author | Qianbiao NG <iampurse@vip.qq.com> | 2019-03-18 15:31:58 +0800 |
---|---|---|
committer | Qianbiao NG <iampurse@vip.qq.com> | 2019-03-18 15:56:48 +0800 |
commit | da0de5c301d4e5fa36d1a76e4c32131ce080dda0 (patch) | |
tree | 11514ed10c9753af3bf21bf56416efe850d75e8a /ironic/tests/unit/drivers/modules/ibmc | |
parent | f944f60041dfbf19547d35f4abf3f18001a05930 (diff) | |
download | ironic-da0de5c301d4e5fa36d1a76e4c32131ce080dda0.tar.gz |
Optimize: HUAWEI iBMC driver utils
Shorter the validation logic of `ibmc_verify_ca` driver info property.
Change-Id: I779bfe1b535bce57ec437ea044fee00c9f9d9a22
Diffstat (limited to 'ironic/tests/unit/drivers/modules/ibmc')
-rw-r--r-- | ironic/tests/unit/drivers/modules/ibmc/test_utils.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ironic/tests/unit/drivers/modules/ibmc/test_utils.py b/ironic/tests/unit/drivers/modules/ibmc/test_utils.py index 7863ced5b..87df0e6e0 100644 --- a/ironic/tests/unit/drivers/modules/ibmc/test_utils.py +++ b/ironic/tests/unit/drivers/modules/ibmc/test_utils.py @@ -70,7 +70,7 @@ class IBMCUtilsTestCase(base.IBMCTestCase): 'Invalid iBMC address', utils.parse_driver_info, self.node) - @mock.patch.object(os.path, 'isdir', autospec=True) + @mock.patch.object(os.path, 'exists', autospec=True) def test_parse_driver_info_path_verify_ca(self, mock_isdir): mock_isdir.return_value = True @@ -82,7 +82,7 @@ class IBMCUtilsTestCase(base.IBMCTestCase): self.assertEqual(self.parsed_driver_info, response) mock_isdir.assert_called_once_with(fake_path) - @mock.patch.object(os.path, 'isfile', autospec=True) + @mock.patch.object(os.path, 'exists', autospec=True) def test_parse_driver_info_valid_capath(self, mock_isfile): mock_isfile.return_value = True fake_path = '/path/to/a/valid/CA.pem' |