Modify test for Service Subscriptions

Author: josemacassan

tests/integration/test_integration_service_subscriptions.py

@@ -1,5 +1,4 @@
 
-
 from tests.integration import asserts
 
 def test_list_service_subscriptions(account):
@@ -15,21 +14,16 @@ def test_read_service_subscription(account, service_subscription, service_subscr
     asserts.assert_resource(resource)
     asserts.assert_resource_params(service_subscription,service_subscription_params)
 
-
-def test_change_plan_service_subscription(account, account_plan, service_subscription):
+def test_change_plan_service_subscription(account, account_plan, service_plan, service_subscription):
     asserts.assert_resource(account)
     asserts.assert_resource(account_plan)
-    resource = account.service_subscriptions.change_plan(service_subscription.entity_id,
-                                                         account_plan.entity_id)
-    asserts.assert_resource(resource)
-
+    account.service_subscriptions.change_plan(service_subscription.entity_id,
+                                                         service_plan.entity_id)
 
 # TODO: https://issues.redhat.com/browse/THREESCALE-11693
 def test_approve_service_subscription(account, service_subscription,service_subscription_params):
     asserts.assert_resource(service_subscription)
     service_subscription['state'] = 'pending'
-    account.service_subscriptions.update(service_subscription.entity_id,
-                                         service_subscription_params)
     resource = account.service_subscriptions.approve(service_subscription.entity_id)
     asserts.assert_resource(resource)
     asserts.assert_resource_params(service_subscription, service_subscription_params)

threescale_api/resources.py

@@ -294,7 +294,7 @@ def approve(self, entity_id: int, **kwargs):
     def change_plan(self, entity_id: int, plan_id: int, **kwargs):
         params = {"plan_id": plan_id}
         url = self.url + f"/{entity_id}/change_plan"
-        response = self.rest.put(url=url, params=params)
+        response = self.rest.put(url=url, json=params, **kwargs)
         instance = utils.extract_response(response=response)
         return instance