openstack
/
kayobe
Code Issues Proposed changes

Use assertIsInstance() instead of assertTrue(isinstance()) 

pep8 complained about it.

Change-Id: I09061b8a9769d737115b7677599e8796503f0725
This commit is contained in:
Mark Goddard 2023-12-20 17:09:09 +00:00
parent bb2af07567
commit 96d0daa7e6
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
kayobe/tests/unit/test_utils.py
View File

@ -162,11 +162,9 @@ key3:
mock_read.return_value = config
result = utils.read_config_dump_yaml_file("/path/to/file")
# Can't read the value without an encryption key, so just check type.
self.assertTrue(isinstance(result["key1"],
AnsibleVaultEncryptedUnicode))
self.assertIsInstance(result["key1"], AnsibleVaultEncryptedUnicode)
self.assertEqual(result["key2"], "value2")
self.assertTrue(isinstance(result["key3"][0],
AnsibleVaultEncryptedUnicode))
self.assertIsInstance(result["key3"][0], AnsibleVaultEncryptedUnicode)
mock_read.assert_called_once_with("/path/to/file")
@mock.patch.object(utils, "read_file")