Merge pull request #16 from KhushbooMer/pending-comment-fix

added fixes for pending comments

Author: kishanio

imagekitio/file.py

@@ -460,8 +460,9 @@ def create_folder(
         """Create folder by provided folderName and parentFolderPath as an options"""
         url = "{}/v1/folder".format(URL.API_BASE_URL)
         headers = self.request.create_headers()
+        headers.update({"Content-Type": "application/json"})
         formatted_data = (
-            request_formatter(options.__dict__) if options is not None else dict()
+            dumps(request_formatter(options.__dict__)) if options is not None else dict()
         )
         resp = self.request.request(
             method="Post", url=url, headers=headers, data=formatted_data
@@ -484,8 +485,9 @@ def delete_folder(
         """Delete folder by provided folderPath as an options"""
         url = "{}/v1/folder".format(URL.API_BASE_URL)
         headers = self.request.create_headers()
+        headers.update({"Content-Type": "application/json"})
         formatted_data = (
-            request_formatter(options.__dict__) if options is not None else dict()
+            dumps(request_formatter(options.__dict__)) if options is not None else dict()
         )
         resp = self.request.request(
             method="Delete", url=url, headers=headers, data=formatted_data

tests/test_folder_ops.py

@@ -87,7 +87,7 @@ def test_create_folder_succeeds(self):
         )
         self.assertEqual("http://test.com/v1/folder", responses.calls[0].request.url)
         self.assertEqual(
-            "folderName=folder_name&parentFolderPath=%2Ftest",
+            '{"folderName": "folder_name", "parentFolderPath": "/test"}',
             responses.calls[0].request.body,
         )
 
@@ -169,7 +169,7 @@ def test_delete_folder_succeeds(self):
             resp.response_metadata.__dict__,
         )
         self.assertEqual("http://test.com/v1/folder", responses.calls[0].request.url)
-        self.assertEqual("folderPath=%2FfolderName", responses.calls[0].request.body)
+        self.assertEqual('{"folderPath": "/folderName"}', responses.calls[0].request.body)
 
 
 class TestCopyFolder(ClientTestCase):