replace make_string_to_single_line with json dumps and loads

Author: KhushbooMer

tests/helpers.py

@@ -1,4 +1,5 @@
 import base64
+import json
 import re
 import unittest
 from unittest.mock import patch
@@ -60,11 +61,3 @@ def get_auth_headers_for_test():
         (ClientTestCase.private_key + ":").encode()
     ).decode("utf-8")
     return {"Authorization": "Basic {}".format(encoded_private_key)}
-
-
-def make_string_to_single_line(multiline_string):
-    return (
-        re.sub(r"\s(?=\s)", "", re.sub(r"\s", " ", multiline_string))
-        .replace("{ ", "{")
-        .replace(" }", "}")
-    )

tests/test_custom_metadata_fields_ops.py

@@ -1,3 +1,5 @@
+import json
+
 import responses
 from responses import matchers
 
@@ -18,7 +20,6 @@
 from tests.helpers import (
     ClientTestCase,
     create_headers_for_test,
-    make_string_to_single_line,
 )
 
 
@@ -345,17 +346,17 @@ def test_create_custom_metadata_fields_succeeds_with_type_number(self):
             },
         }
 
-        request_body = make_string_to_single_line(
+        request_body = json.dumps(json.loads(
             """{
-            "name": "test",
-            "label": "test",
-            "schema": {
-                "type": "Number",
-                "minValue": 100,
-                "maxValue": 200
-            }
-        }"""
-        )
+                "name": "test",
+                "label": "test",
+                "schema": {
+                    "type": "Number",
+                    "minValue": 100,
+                    "maxValue": 200
+                }
+            }"""
+        ))
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),
             resp.response_metadata.__dict__,
@@ -427,19 +428,19 @@ def test_create_custom_metadata_fields_succeeds_with_type_textarea(self):
             },
         }
 
-        request_body = make_string_to_single_line(
+        request_body = json.dumps(json.loads(
             """{
-            "name": "test",
-            "label": "test",
-            "schema": {
-                "type": "Textarea",
-                "defaultValue": "The",
-                "isValueRequired": true,
-                "minLength": 3,
-                "maxLength": 200
-            }
-        }"""
-        )
+                "name": "test",
+                "label": "test",
+                "schema": {
+                    "type": "Textarea",
+                    "defaultValue": "The",
+                    "isValueRequired": true,
+                    "minLength": 3,
+                    "maxLength": 200
+                }
+            }"""
+        ))
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),
             resp.response_metadata.__dict__,
@@ -505,17 +506,17 @@ def test_create_custom_metadata_fields_succeeds_with_type_date(self):
             },
         }
 
-        request_body = make_string_to_single_line(
+        request_body = json.dumps(json.loads(
             """{
-            "name": "test-date",
-            "label": "test-date",
-            "schema": {
-                "type": "Date",
-                "minValue": "2022-11-29T10:11:10+00:00",
-                "maxValue": "2022-11-30T10:11:10+00:00"
-            }
-        }"""
-        )
+                "name": "test-date",
+                "label": "test-date",
+                "schema": {
+                    "type": "Date",
+                    "minValue": "2022-11-29T10:11:10+00:00",
+                    "maxValue": "2022-11-30T10:11:10+00:00"
+                }
+            }"""
+        ))
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),
             resp.response_metadata.__dict__,
@@ -581,17 +582,17 @@ def test_create_custom_metadata_fields_succeeds_with_type_boolean(self):
             },
         }
 
-        request_body = make_string_to_single_line(
+        request_body = json.dumps(json.loads(
             """{
-            "name": "test-boolean",
-            "label": "test-boolean",
-            "schema": {
-                "type": "Boolean",
-                "defaultValue": true,
-                "isValueRequired": true
-            }
-        }"""
-        )
+                "name": "test-boolean",
+                "label": "test-boolean",
+                "schema": {
+                    "type": "Boolean",
+                    "defaultValue": true,
+                    "isValueRequired": true
+                }
+            }"""
+        ))
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),
             resp.response_metadata.__dict__,
@@ -654,17 +655,18 @@ def test_create_custom_metadata_fields_succeeds_with_type_single_select(self):
             },
         }
 
-        request_body = make_string_to_single_line(
-            """{"name": "test",
-                                   "label": "test",
-                                   "schema":
-                                       {
-                                           "type": "SingleSelect",
-                                           "selectOptions": ["small", "medium", "large", 30, 40,
-                                                             true]
-                                       }
-                                   }"""
-        )
+        request_body = json.dumps(json.loads(
+            """{
+                "name": "test",
+               "label": "test",
+               "schema":
+                   {
+                       "type": "SingleSelect",
+                       "selectOptions": ["small", "medium", "large", 30, 40,
+                                         true]
+                   }
+               }"""
+        ))
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),
             resp.response_metadata.__dict__,
@@ -733,18 +735,18 @@ def test_create_custom_metadata_fields_succeeds_with_type_multi_select(self):
             },
         }
 
-        request_body = make_string_to_single_line(
+        request_body = json.dumps(json.loads(
             """{
-            "name": "test",
-            "label": "test",
-            "schema": {
-                "type": "MultiSelect",
-                "selectOptions": ["small", "medium", "large", 30, 40, true],
-                "defaultValue": ["small", 30, true],
-                "isValueRequired": true
-            }
-        }"""
-        )
+                "name": "test",
+                "label": "test",
+                "schema": {
+                    "type": "MultiSelect",
+                    "selectOptions": ["small", "medium", "large", 30, 40, true],
+                    "defaultValue": ["small", 30, true],
+                    "isValueRequired": true
+                }
+            }"""
+        ))
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),
             resp.response_metadata.__dict__,
@@ -802,15 +804,15 @@ def test_update_custom_metadata_fields_succeeds(self):
             },
         }
 
-        request_body = make_string_to_single_line(
+        request_body = json.dumps(json.loads(
             """{
-            "label": "test-update",
-            "schema": {
-                "minValue": 100,
-                "maxValue": 200
-            }
-        }"""
-        )
+                "label": "test-update",
+                "schema": {
+                    "minValue": 100,
+                    "maxValue": 200
+                }
+            }"""
+        ))
 
         self.assertEqual(
             camel_dict_to_snake_dict(mock_response_metadata),