Skip to content

Commit 39f82b1

Browse files
caberoscaberos
authored andcommitted
fix the team code review and unit test
1 parent 3de562b commit 39f82b1

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

SoftLayer/CLI/order/place.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,6 @@ def cli(env, package_keyname, location, preset, verify, billing, complex_type,
9595
])
9696

9797
else:
98-
print(args)
9998
for pod in pods:
10099
closure.append(pod['name'])
101100
click.secho(click.style('Warning: Closed soon: %s' % (', '.join(closure)), fg='yellow'))

tests/CLI/modules/order_tests.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,7 @@ def test_place(self):
141141

142142
self.assert_no_fail(result)
143143
self.assert_called_with('SoftLayer_Product_Order', 'placeOrder')
144-
self.assertEqual("('package', 'DALLAS13', ('ITEM1',))\n"
145-
'Warning: Closed soon: ams01.pod01, wdc07.pod01, TEST00.pod2\n'
144+
self.assertEqual('Warning: Closed soon: ams01.pod01, wdc07.pod01, TEST00.pod2\n'
146145
'{\n "id": 1234,\n "created": "2017-04-04 07:39:20",\n "status": "APPROVED"\n}\n',
147146
str(result.output))
148147

@@ -162,8 +161,7 @@ def test_place_with_quantity(self):
162161

163162
self.assert_no_fail(result)
164163
self.assert_called_with('SoftLayer_Product_Order', 'placeOrder')
165-
self.assertEqual("('package', 'DALLAS13', ('ITEM1',))\n"
166-
'Warning: Closed soon: ams01.pod01, wdc07.pod01, TEST00.pod2\n'
164+
self.assertEqual('Warning: Closed soon: ams01.pod01, wdc07.pod01, TEST00.pod2\n'
167165
'{\n "id": 1234,\n "created": "2017-04-04 07:39:20",\n "status": "APPROVED"\n}\n',
168166
str(result.output))
169167

0 commit comments

Comments
 (0)