Skip to content

Commit 2c430e6

Browse files
added py37 to test list, ignored depreciated testing messages
1 parent 72ce028 commit 2c430e6

File tree

3 files changed

+23
-17
lines changed

3 files changed

+23
-17
lines changed

setup.cfg

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
[tool:pytest]
22
python_files = *_tests.py
3+
filterwarnings =
4+
ignore::DeprecationWarning
5+
ignore::PendingDeprecationWarning
36

47
[wheel]
58
universal=1

tests/CLI/modules/order_tests.py

+18-16
Original file line numberDiff line numberDiff line change
@@ -47,50 +47,43 @@ def test_item_list(self):
4747
self.assertEqual(expected_results, json.loads(result.output))
4848

4949
def test_package_list(self):
50-
item1 = {'name': 'package1', 'keyName': 'PACKAGE1', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 1}
51-
item2 = {'name': 'package2', 'keyName': 'PACKAGE2', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 1}
52-
item3 = {'name': 'package2', 'keyName': 'PACKAGE2', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 0}
5350
p_mock = self.set_mock('SoftLayer_Product_Package', 'getAllObjects')
54-
p_mock.return_value = [item1, item2, item3]
51+
p_mock.return_value = _get_all_packages()
5552
_filter = {'type': {'keyName': {'operation': '!= BLUEMIX_SERVICE'}}}
5653

5754
result = self.run_command(['order', 'package-list'])
5855

5956
self.assert_no_fail(result)
6057
self.assert_called_with('SoftLayer_Product_Package', 'getAllObjects', filter=_filter)
61-
expected_results = [{'name': 'package1', 'keyName': 'PACKAGE1', 'type': 'BARE_METAL_CPU'},
62-
{'name': 'package2', 'keyName': 'PACKAGE2', 'type': 'BARE_METAL_CPU'}]
58+
expected_results = [{'id': 1, 'name': 'package1', 'keyName': 'PACKAGE1', 'type': 'BARE_METAL_CPU'},
59+
{'id': 2, 'name': 'package2', 'keyName': 'PACKAGE2', 'type': 'BARE_METAL_CPU'}]
6360
self.assertEqual(expected_results, json.loads(result.output))
6461

6562
def test_package_list_keyword(self):
66-
item1 = {'name': 'package1', 'keyName': 'PACKAGE1', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 1}
67-
item2 = {'name': 'package2', 'keyName': 'PACKAGE2', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 1}
6863
p_mock = self.set_mock('SoftLayer_Product_Package', 'getAllObjects')
69-
p_mock.return_value = [item1, item2]
64+
p_mock.return_value = _get_all_packages()
7065

7166
_filter = {'type': {'keyName': {'operation': '!= BLUEMIX_SERVICE'}}}
7267
_filter['name'] = {'operation': '*= package1'}
7368
result = self.run_command(['order', 'package-list', '--keyword', 'package1'])
7469

7570
self.assert_no_fail(result)
7671
self.assert_called_with('SoftLayer_Product_Package', 'getAllObjects', filter=_filter)
77-
expected_results = [{'name': 'package1', 'keyName': 'PACKAGE1', 'type': 'BARE_METAL_CPU'},
78-
{'name': 'package2', 'keyName': 'PACKAGE2', 'type': 'BARE_METAL_CPU'}]
72+
expected_results = [{'id': 1, 'name': 'package1', 'keyName': 'PACKAGE1', 'type': 'BARE_METAL_CPU'},
73+
{'id': 2, 'name': 'package2', 'keyName': 'PACKAGE2', 'type': 'BARE_METAL_CPU'}]
7974
self.assertEqual(expected_results, json.loads(result.output))
8075

8176
def test_package_list_type(self):
82-
item1 = {'name': 'package1', 'keyName': 'PACKAGE1', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 1}
83-
item2 = {'name': 'package2', 'keyName': 'PACKAGE2', 'type': {'keyName': 'BARE_METAL_CPU'}, 'isActive': 1}
8477
p_mock = self.set_mock('SoftLayer_Product_Package', 'getAllObjects')
85-
p_mock.return_value = [item1, item2]
78+
p_mock.return_value = _get_all_packages()
8679

8780
_filter = {'type': {'keyName': {'operation': 'BARE_METAL_CPU'}}}
8881
result = self.run_command(['order', 'package-list', '--package_type', 'BARE_METAL_CPU'])
8982

9083
self.assert_no_fail(result)
9184
self.assert_called_with('SoftLayer_Product_Package', 'getAllObjects', filter=_filter)
92-
expected_results = [{'name': 'package1', 'keyName': 'PACKAGE1', 'type': 'BARE_METAL_CPU'},
93-
{'name': 'package2', 'keyName': 'PACKAGE2', 'type': 'BARE_METAL_CPU'}]
85+
expected_results = [{'id': 1, 'name': 'package1', 'keyName': 'PACKAGE1', 'type': 'BARE_METAL_CPU'},
86+
{'id': 2, 'name': 'package2', 'keyName': 'PACKAGE2', 'type': 'BARE_METAL_CPU'}]
9487
self.assertEqual(expected_results, json.loads(result.output))
9588

9689
def test_place(self):
@@ -227,3 +220,12 @@ def _get_verified_order_return(self):
227220
price2 = {'item': item2, 'hourlyRecurringFee': '0.05',
228221
'recurringFee': '150'}
229222
return {'orderContainers': [{'prices': [price1, price2]}]}
223+
224+
def _get_all_packages():
225+
package_type = {'keyName': 'BARE_METAL_CPU'}
226+
all_packages = [
227+
{'id': 1, 'name': 'package1', 'keyName': 'PACKAGE1', 'type': package_type, 'isActive': 1},
228+
{'id': 2, 'name': 'package2', 'keyName': 'PACKAGE2', 'type': package_type, 'isActive': 1},
229+
{'id': 3, 'name': 'package2', 'keyName': 'PACKAGE2', 'type': package_type, 'isActive': 0}
230+
]
231+
return all_packages

tox.ini

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
[tox]
2-
envlist = py27,py35,py36,pypy,analysis,coverage
2+
envlist = py27,py35,py36,py37,pypy,analysis,coverage
3+
34

45
[flake8]
56
max-line-length=120

0 commit comments

Comments
 (0)