Merge pull request #2420 from san70sh/develop

Fix 2419 - Multipart form fields ignoring numeric data
karatelabs · Oct 21, 2023 · 6f40066 · 6f40066
2 parents 83392da + 8c673f2
commit 6f40066
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 0 deletions.
diff --git a/karate-core/src/main/java/com/intuit/karate/core/ScenarioEngine.java b/karate-core/src/main/java/com/intuit/karate/core/ScenarioEngine.java
@@ -491,6 +491,9 @@ private void multiPartInternal(String name, Object value) {
         if (name != null) {
             map.put("name", name);
         }
+        if(value instanceof Number) {
+            value = value.toString();
+        }
         if (value instanceof Map) {
             map.putAll((Map) value);
             String toRead = (String) map.get("read");

diff --git a/karate-core/src/test/java/com/intuit/karate/core/KarateMockHandlerTest.java b/karate-core/src/test/java/com/intuit/karate/core/KarateMockHandlerTest.java
@@ -390,6 +390,36 @@ void testMultiPartField() {
         matchVar("response", "{ foo: ['bar'] }");
     }
 
+    @Test
+    void testMultiPartFieldWithInteger() {
+        background().scenario(
+                "pathMatches('/hello')",
+                "def response = requestParams");
+        run(
+                URL_STEP,
+                "def data = { foo: 'a', bar: 1}",
+                "multipart fields data",
+                "path 'hello'",
+                "method post"
+        );
+        matchVar("response", "{  foo: ['a'], bar: ['1'] } }");
+    }
+
+    @Test
+    void testMultiPartFieldWithFloat() {
+        background().scenario(
+                "pathMatches('/hello')",
+                "def response = requestParams");
+        run(
+                URL_STEP,
+                "def data = { foo: 1, bar: 2.0}",
+                "multipart fields data",
+                "path 'hello'",
+                "method post"
+        );
+        matchVar("response", "{ foo: ['1'], bar: ['2.0'] } }");
+    }
+
     @Test
     void testMultiPartFile() {
         background().scenario(