Update Date handling.

Author: gk-brown

README.md

@@ -106,6 +106,7 @@ Method parameters may be any of the following types:
 * `java.time.LocalDateTime`
 * `java.time.Duration`
 * `java.time.Period`
+* `java.util.Date`
 * `java.util.UUID`
 
 The following multi-value types are also supported:
@@ -118,9 +119,7 @@ Additionally, `java.util.Map`, bean, record, and `org.w3c.dom.Document` types ar
 
 The `FormData` annotation can be used to indicate that a handler method accepts [form data](https://www.w3.org/TR/html401/interact/forms.html#h-17.13.4). Arguments of type `jakarta.servlet.http.Part` may be used with requests submitted as [multi-part](https://jakarta.ee/specifications/servlet/6.1/jakarta-servlet-spec-6.1#_MultipartConfig) form data.
 
-Argument values are parsed from their string representations. Unspecified values are automatically converted to 0, `false`, or the null character for primitive types. 
-
-`List`, `Set`, and array elements are converted to their declared types. If no values are provided for a list, set, or array parameter, an empty instance (not `null`) will be passed to the method.
+Unspecified values are automatically converted to 0, `false`, or the null character for primitive types. `List`, `Set`, and array elements are converted to their declared types. If no values are provided for a list, set, or array parameter, an empty instance (not `null`) will be passed to the method.
 
 If a provided value cannot be coerced to the expected type, an HTTP 403 (forbidden) response will be returned. If no method is found that matches the provided arguments, HTTP 405 (method not allowed) will be returned.
 

kilo-client/src/main/java/org/httprpc/kilo/beans/BeanAdapter.java

@@ -822,7 +822,7 @@ private static Object toRawType(Object value, Class<?> type) {
                 if (value instanceof Number number) {
                     return new Date(number.longValue());
                 } else {
-                    throw new IllegalArgumentException("Value is not a number.");
+                    return new Date(Long.parseLong(value.toString()));
                 }
             } else if (type == Instant.class) {
                 if (value instanceof Number number) {

kilo-client/src/main/java/org/httprpc/kilo/io/JSONEncoder.java

@@ -177,6 +177,10 @@ private void encode(Map<?, ?> map, Writer writer) throws IOException {
                 throw new IllegalArgumentException("Missing key.");
             }
 
+            if (key instanceof Date date) {
+                key = date.getTime();
+            }
+
             if (i > 0) {
                 writer.write(",");
             }

kilo-client/src/test/java/org/httprpc/kilo/io/JSONEncoderTest.java

@@ -169,23 +169,26 @@ public void testUUID() throws IOException {
 
     @Test
     public void testKeys() throws IOException {
-        var instant = Instant.now();
+        var date = new Date();
+        var instant = date.toInstant();
         var localDate = LocalDate.now();
         var localTime = LocalTime.now();
         var localDateTime = LocalDateTime.now();
 
         var expected = "{\n"
-            + "  \"" + instant + "\": 1,\n"
-            + "  \"" + localDate + "\": 2,\n"
-            + "  \"" + localTime + "\": 3,\n"
-            + "  \"" + localDateTime + "\": 4\n"
+            + "  \"" + date.getTime() + "\": 1,\n"
+            + "  \"" + instant + "\": 2,\n"
+            + "  \"" + localDate + "\": 3,\n"
+            + "  \"" + localTime + "\": 4,\n"
+            + "  \"" + localDateTime + "\": 5\n"
             + "}";
 
         var map = mapOf(
-            entry(instant, 1),
-            entry(localDate, 2),
-            entry(localTime, 3),
-            entry(localDateTime, 4)
+            entry(date, 1),
+            entry(instant, 2),
+            entry(localDate, 3),
+            entry(localTime, 4),
+            entry(localDateTime, 5)
         );
 
         var actual = encode(map);

kilo-test/src/main/java/org/httprpc/kilo/test/TestService.java

@@ -40,6 +40,7 @@
 import java.time.Period;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Date;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Iterator;
@@ -102,8 +103,8 @@ public interface Response {
         boolean getFlag();
         char getCharacter();
         DayOfWeek getDayOfWeek();
-        Long getDate();
-        List<Long> getDates();
+        Date getDate();
+        List<Date> getDates();
         Instant getInstant();
         LocalDate getLocalDate();
         LocalTime getLocalTime();
@@ -182,7 +183,7 @@ public Number next() {
     @RequestMethod("GET")
     public Response testGet(@Required String string, List<String> strings,
         Integer number, Set<Integer> numbers, boolean flag, char character, DayOfWeek dayOfWeek,
-        Long date, List<Long> dates,
+        Date date, List<Date> dates,
         Instant instant, LocalDate localDate, LocalTime localTime, LocalDateTime localDateTime,
         Duration duration, Period period,
         UUID uuid) {
@@ -317,7 +318,7 @@ public Coordinates testPostCoordinates(Coordinates coordinates) {
     @ResourcePath("form-data")
     @FormData
     public Map<String, Object> testPostFormData(@Required String string, List<String> strings,
-        Integer number, Long date,
+        Integer number, Date date,
         Part file, List<Part> files) {
         var fileSize = 0L;
 

kilo-test/src/test/java/org/httprpc/kilo/test/WebServiceProxyTest.java

@@ -102,8 +102,8 @@ public void testGet() throws IOException {
         assertTrue(result.getFlag());
         assertEquals('a', result.getCharacter());
         assertEquals(dayOfWeek, result.getDayOfWeek());
-        assertEquals(date.getTime(), result.getDate());
-        assertEquals(listOf(date.getTime()), result.getDates());
+        assertEquals(date, result.getDate());
+        assertEquals(listOf(date), result.getDates());
         assertEquals(instant, result.getInstant());
         assertEquals(localDate, result.getLocalDate());
         assertEquals(localTime, result.getLocalTime());

kilo-test/src/test/resources/org/httprpc/kilo/test/test.json

@@ -146,7 +146,7 @@
                 "iterable": false,
                 "keyType": null,
                 "map": false,
-                "name": "Long",
+                "name": "Date",
                 "valueType": null
               }
             },
@@ -161,7 +161,7 @@
                   "iterable": false,
                   "keyType": null,
                   "map": false,
-                  "name": "Long",
+                  "name": "Date",
                   "valueType": null
                 },
                 "intrinsic": true,
@@ -1099,7 +1099,7 @@
                 "iterable": false,
                 "keyType": null,
                 "map": false,
-                "name": "Long",
+                "name": "Date",
                 "valueType": null
               }
             },
@@ -2133,7 +2133,7 @@
             "iterable": false,
             "keyType": null,
             "map": false,
-            "name": "Long",
+            "name": "Date",
             "valueType": null
           }
         },
@@ -2149,7 +2149,7 @@
               "iterable": false,
               "keyType": null,
               "map": false,
-              "name": "Long",
+              "name": "Date",
               "valueType": null
             },
             "intrinsic": true,