fix(model): deserialize camelCase Slack Lists link fields

Fixes #1679

Author: wn1331

slack-api-client/src/test/java/test_locally/api/util/json/GsonListRecordLinkFieldTest.java

@@ -0,0 +1,51 @@
+package test_locally.api.util.json;
+
+import com.google.gson.Gson;
+import com.google.gson.JsonObject;
+import com.slack.api.methods.response.slack_lists.SlackListsItemsListResponse;
+import com.slack.api.model.list.ListRecord;
+import com.slack.api.util.json.GsonFactory;
+import org.junit.Test;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.MatcherAssert.assertThat;
+
+public class GsonListRecordLinkFieldTest {
+
+    private final Gson gson = GsonFactory.createSnakeCase();
+
+    @Test
+    public void deserializeCamelCaseLinkField() {
+        String json = "{\"ok\":true,\"items\":[{\"fields\":[{" +
+                "\"link\":[{" +
+                "\"originalUrl\":\"https://github.com/org/repo/pull/7\"," +
+                "\"displayAsUrl\":false," +
+                "\"displayName\":\"repo#7\"" +
+                "}]}]}]}";
+
+        SlackListsItemsListResponse response = gson.fromJson(json, SlackListsItemsListResponse.class);
+        ListRecord.LinkField linkField = response.getItems().get(0).getFields().get(0).getLink().get(0);
+
+        assertThat(linkField.getOriginalUrl(), is("https://github.com/org/repo/pull/7"));
+        assertThat(linkField.getDisplayAsUrl(), is(false));
+        assertThat(linkField.getDisplayName(), is("repo#7"));
+    }
+
+    @Test
+    public void serializeLinkFieldAsSnakeCase() {
+        ListRecord.LinkField linkField = ListRecord.LinkField.builder()
+                .originalUrl("https://github.com/org/repo/pull/7")
+                .displayAsUrl(false)
+                .displayName("repo#7")
+                .build();
+
+        JsonObject json = gson.toJsonTree(linkField).getAsJsonObject();
+
+        assertThat(json.get("original_url").getAsString(), is("https://github.com/org/repo/pull/7"));
+        assertThat(json.get("display_as_url").getAsBoolean(), is(false));
+        assertThat(json.get("display_name").getAsString(), is("repo#7"));
+        assertThat(json.has("originalUrl"), is(false));
+        assertThat(json.has("displayAsUrl"), is(false));
+        assertThat(json.has("displayName"), is(false));
+    }
+}

slack-api-model/src/main/java/com/slack/api/model/list/ListRecord.java

@@ -87,11 +87,11 @@ public void setMessages(List<Message> messages) {
     @NoArgsConstructor
     @AllArgsConstructor
     public static class LinkField {
-        @SerializedName("original_url")
+        @SerializedName(value = "original_url", alternate = "originalUrl")
         private String originalUrl;
-        @SerializedName("display_as_url")
+        @SerializedName(value = "display_as_url", alternate = "displayAsUrl")
         private Boolean displayAsUrl;
-        @SerializedName("display_name")
+        @SerializedName(value = "display_name", alternate = "displayName")
         private String displayName;
     }