Skip to content

Commit f98447b

Browse files
author
Erick Sapp
committed
Fixed style and unused mock objects.
1 parent 9aaeec8 commit f98447b

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

tests/CLI/modules/vs_tests.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ def test_list_vs(self):
153153
'action': None,
154154
'id': 104,
155155
'backend_ip': '10.45.19.35'}])
156-
156+
157157
def test_detail_vs(self):
158158
result = self.run_command(['vs', 'detail', '100',
159159
'--passwords', '--price'])
@@ -788,6 +788,7 @@ def test_going_ready(self, _sleep):
788788
def test_reload(self, confirm_mock):
789789
mock = self.set_mock('SoftLayer_Virtual_Guest', 'reloadCurrentOperatingSystemConfguration')
790790
confirm_mock.return_value = True
791+
mock.return_value = 'true'
791792

792793
result = self.run_command(['vs', 'reload', '--postinstall', '100', '--key', '100', '--image', '100', '100'])
793794
self.assert_no_fail(result)
@@ -796,6 +797,7 @@ def test_reload(self, confirm_mock):
796797
def test_reload_no_confirm(self, confirm_mock):
797798
mock = self.set_mock('SoftLayer_Virtual_Guest', 'reloadCurrentOperatingSystemConfiguration')
798799
confirm_mock.return_value = False
800+
mock.return_value = 'false'
799801

800802
result = self.run_command(['vs', 'reload', '--postinstall', '100', '--key', '100', '--image', '100', '100'])
801803
self.assertEqual(result.exit_code, 2)

0 commit comments

Comments
 (0)