Skip to content

Commit 665ad1d

Browse files
author
Fernando Ojeda
committed
Refactor object storage credentials.
1 parent 1bd5deb commit 665ad1d

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

tests/CLI/modules/object_storage_tests.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,10 +67,10 @@ def test_delete_credential(self):
6767
accounts = self.set_mock('SoftLayer_Network_Storage_Hub_Cleversafe_Account', 'credentialDelete')
6868
accounts.return_value = True
6969

70-
result = self.run_command(['object-storage', 'credential', 'delete', '-c=100', '100'])
70+
result = self.run_command(['object-storage', 'credential', 'delete', '-c', 100, '100'])
7171

7272
self.assert_no_fail(result)
73-
self.assertEqual(json.loads(result.output), True)
73+
self.assertEqual(result.output, 'True\n')
7474

7575
def test_limit_credential(self):
7676
accounts = self.set_mock('SoftLayer_Network_Storage_Hub_Cleversafe_Account', 'getCredentialLimit')
@@ -95,7 +95,6 @@ def test_list_credential(self):
9595
result = self.run_command(['object-storage', 'credential', 'list', '100'])
9696

9797
self.assert_no_fail(result)
98-
print(json.loads(result.output))
9998
self.assertEqual(json.loads(result.output),
10099
[{'id': 1103123,
101100
'password': 'nwUEUsx6PiEoN0B1Xe9z9hUCyXM',

tests/managers/object_storage_tests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ def test_create_credential(self):
6666

6767
def test_delete_credential(self):
6868
accounts = self.set_mock('SoftLayer_Network_Storage_Hub_Cleversafe_Account', 'credentialDelete')
69-
accounts.return_value = 'The credential was deleted successful'
69+
accounts.return_value = True
7070

7171
credential = self.object_storage.delete_credential(100)
7272
self.assertEqual(credential, True)

0 commit comments

Comments
 (0)