Skip to content

Commit 6025c3e

Browse files
author
Fernando Ojeda
committed
Fix quote file storage.
1 parent 4c341bf commit 6025c3e

File tree

2 files changed

+24
-4
lines changed

2 files changed

+24
-4
lines changed

SoftLayer/managers/ordering.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,9 @@ def get_item_price_id(core, prices):
405405
price_id = price['id']
406406
# this check is mostly to work nicely with preset configs
407407
elif capacity_min <= int(core) <= capacity_max:
408-
price_id = price['id']
408+
if "STORAGE" in price.get("capacityRestrictionType") or "CORE" in price.get(
409+
"capacityRestrictionType"):
410+
price_id = price['id']
409411
return price_id
410412

411413
def get_item_capacity(self, items, item_keynames):
@@ -417,6 +419,9 @@ def get_item_capacity(self, items, item_keynames):
417419
if "GUEST_CORE" in item["keyName"]:
418420
item_capacity = item['capacity']
419421
break
422+
if "TIER" in item["keyName"]:
423+
item_capacity = item['capacity']
424+
break
420425
return item_capacity
421426

422427
def get_preset_prices(self, preset):

tests/managers/ordering_tests.py

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -640,12 +640,27 @@ def test_get_item_price_id_without_capacity_restriction(self):
640640

641641
self.assertEqual(1234, price_id)
642642

643-
def test_get_item_price_id_with_capacity_restriction(self):
643+
def test_get_item_price_id_core_with_capacity_restriction(self):
644644
category1 = {'categoryCode': 'cat1'}
645645
price1 = [{'id': 1234, 'locationGroupId': '', "capacityRestrictionMaximum": "16",
646-
"capacityRestrictionMinimum": "1", 'categories': [category1]},
646+
"capacityRestrictionMinimum": "1", "capacityRestrictionType": "CORE",
647+
'categories': [category1]},
647648
{'id': 2222, 'locationGroupId': '', "capacityRestrictionMaximum": "56",
648-
"capacityRestrictionMinimum": "36", 'categories': [category1]}]
649+
"capacityRestrictionMinimum": "36", "capacityRestrictionType": "CORE",
650+
'categories': [category1]}]
651+
652+
price_id = self.ordering.get_item_price_id("8", price1)
653+
654+
self.assertEqual(1234, price_id)
655+
656+
def test_get_item_price_id_storage_with_capacity_restriction(self):
657+
category1 = {'categoryCode': 'cat1'}
658+
price1 = [{'id': 1234, 'locationGroupId': '', "capacityRestrictionMaximum": "16",
659+
"capacityRestrictionMinimum": "1", "capacityRestrictionType": "STORAGE_SPACE",
660+
'categories': [category1]},
661+
{'id': 2222, 'locationGroupId': '', "capacityRestrictionMaximum": "56",
662+
"capacityRestrictionMinimum": "36", "capacityRestrictionType": "STORAGE_SPACE",
663+
'categories': [category1]}]
649664

650665
price_id = self.ordering.get_item_price_id("8", price1)
651666

0 commit comments

Comments
 (0)