Merge pull request #143 from razorpay/VA_changes

added changes for VA close end point

Author: neera11

razorpay/resources/virtual_account.py

@@ -53,9 +53,8 @@ def close(self, virtual_account_id, data={}, **kwargs):
             virtual_account_id :
                 Id for which Virtual Account objects has to be Closed
         """
-        url = "{}/{}".format(self.base_url, virtual_account_id)
-        data['status'] = 'closed'
-        return self.patch_url(url, data, **kwargs)
+        url = "{}/{}/close".format(self.base_url, virtual_account_id)
+        return self.post_url(url, data, **kwargs)
 
     def payments(self, virtual_account_id, data={}, **kwargs):
         """"

tests/test_client_virtual_account.py

@@ -73,8 +73,8 @@ def test_virtual_accounts_fetch(self):
     @responses.activate
     def test_virtual_accounts_close(self):
         result = mock_file('fake_virtual_accounts_closed')
-        url = '{}/{}'.format(self.base_url, self.fake_virtual_account_id)
-        responses.add(responses.PATCH,
+        url = '{}/{}/close'.format(self.base_url, self.fake_virtual_account_id)
+        responses.add(responses.POST,
                       url,
                       status=200,
                       body=result,