Added hdd parameter as an optional parameter to the subscription plan id
This commit is contained in:
		
					parent
					
						
							
								487ab6b927
							
						
					
				
			
			
				commit
				
					
						9eecfbda64
					
				
			
		
					 2 changed files with 8 additions and 2 deletions
				
			
		|  | @ -220,16 +220,20 @@ class StripeUtils(object): | ||||||
|         return charge |         return charge | ||||||
| 
 | 
 | ||||||
|     @staticmethod |     @staticmethod | ||||||
|     def get_stripe_plan_id(cpu, ram, ssd, version, app): |     def get_stripe_plan_id(cpu, ram, ssd, version, app='dcl', hdd=None): | ||||||
|         """ |         """ | ||||||
|         Returns the stripe plan id string of the form `dcl-v1-cpu-2-ram-5gb-ssd-10gb` based on the input parameters |         Returns the stripe plan id string of the form `dcl-v1-cpu-2-ram-5gb-ssd-10gb` based on the input parameters | ||||||
| 
 | 
 | ||||||
|         :param cpu: The number of cores |         :param cpu: The number of cores | ||||||
|         :param ram: The size of the RAM in GB |         :param ram: The size of the RAM in GB | ||||||
|         :param ssd: The size of storage in GB |         :param ssd: The size of ssd storage in GB | ||||||
|  |         :param hdd: The size of hdd storage in GB | ||||||
|         :param version: The version of the Stripe plans |         :param version: The version of the Stripe plans | ||||||
|  |         :param app: The application to which the stripe plan belongs to. By default it is 'dcl' | ||||||
|         :return: A string of the form `dcl-v1-cpu-2-ram-5gb-ssd-10gb` |         :return: A string of the form `dcl-v1-cpu-2-ram-5gb-ssd-10gb` | ||||||
|         """ |         """ | ||||||
|         dcl_plan_string = 'cpu-{cpu}-ram-{ram}gb-ssd-{ssd}gb'.format(cpu=cpu, ram=ram, ssd=ssd) |         dcl_plan_string = 'cpu-{cpu}-ram-{ram}gb-ssd-{ssd}gb'.format(cpu=cpu, ram=ram, ssd=ssd) | ||||||
|  |         if hdd is not None: | ||||||
|  |             dcl_plan_string = '{dcl_plan_string}-hdd-{hdd}gb'.format(dcl_plan_string=dcl_plan_string, hdd=hdd) | ||||||
|         stripe_plan_id_string = '{app}-v{version}-{plan}'.format(app=app, version=version, plan=dcl_plan_string) |         stripe_plan_id_string = '{app}-v{version}-{plan}'.format(app=app, version=version, plan=dcl_plan_string) | ||||||
|         return stripe_plan_id_string |         return stripe_plan_id_string | ||||||
|  |  | ||||||
|  | @ -138,6 +138,8 @@ class StripePlanTestCase(TestStripeCustomerDescription): | ||||||
|     def test_get_stripe_plan_id_string(self): |     def test_get_stripe_plan_id_string(self): | ||||||
|         plan_id_string = StripeUtils.get_stripe_plan_id(cpu=2, ram=20, ssd=100, version=1, app='dcl') |         plan_id_string = StripeUtils.get_stripe_plan_id(cpu=2, ram=20, ssd=100, version=1, app='dcl') | ||||||
|         self.assertEqual(plan_id_string, 'dcl-v1-cpu-2-ram-20gb-ssd-100gb') |         self.assertEqual(plan_id_string, 'dcl-v1-cpu-2-ram-20gb-ssd-100gb') | ||||||
|  |         plan_id_string = StripeUtils.get_stripe_plan_id(cpu=2, ram=20, ssd=100, version=1, app='dcl', hdd=200) | ||||||
|  |         self.assertEqual(plan_id_string, 'dcl-v1-cpu-2-ram-20gb-ssd-100gb-hdd-200gb') | ||||||
| 
 | 
 | ||||||
|     def test_get_or_create_plan(self): |     def test_get_or_create_plan(self): | ||||||
|         stripe_plan = self.stripe_utils.get_or_create_stripe_plan(2000, "test plan 1", stripe_plan_id='test-plan-1') |         stripe_plan = self.stripe_utils.get_or_create_stripe_plan(2000, "test plan 1", stripe_plan_id='test-plan-1') | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue