Add missing duration field in VideoMessageContent class. (#569)

* Add missing duration field in VideoMessageContent class.
Close #568
* Use Integer type to consistency with AudioMessageContent.
* fix javadoc

Author: tokuhirom

line-bot-model/src/main/java/com/linecorp/bot/model/event/message/VideoMessageContent.java

@@ -16,8 +16,6 @@
 
 package com.linecorp.bot.model.event.message;
 
-import java.net.URI;
-
 import com.fasterxml.jackson.annotation.JsonTypeName;
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
 import com.fasterxml.jackson.databind.annotation.JsonPOJOBuilder;
@@ -36,6 +34,9 @@ public static class VideoMessageContentBuilder {
     }
 
     String id;
-    URI url;
+    /**
+     * Length of video file (milliseconds).
+     */
+    Integer duration;
     ContentProvider contentProvider;
 }

line-bot-model/src/test/java/com/linecorp/bot/model/event/CallbackRequestTest.java

@@ -42,6 +42,7 @@
 import com.linecorp.bot.model.event.message.TextMessageContent;
 import com.linecorp.bot.model.event.message.TextMessageContent.Emoji;
 import com.linecorp.bot.model.event.message.UnknownMessageContent;
+import com.linecorp.bot.model.event.message.VideoMessageContent;
 import com.linecorp.bot.model.event.source.GroupSource;
 import com.linecorp.bot.model.event.source.Source;
 import com.linecorp.bot.model.event.source.UnknownSource;
@@ -579,6 +580,22 @@ public void testUnsend() throws IOException {
         });
     }
 
+    @Test
+    public void testVideo() throws IOException {
+        parse("callback/video.json", callbackRequest -> {
+            assertDestination(callbackRequest);
+            Event event = callbackRequest.getEvents().get(0);
+            assertThat(event.getSource()).isInstanceOf(UserSource.class);
+            assertThat(event).isInstanceOf(MessageEvent.class);
+            assertThat(event.getMode())
+                    .isEqualTo(EventMode.ACTIVE);
+
+            MessageEvent messageEvent = (MessageEvent) event;
+            VideoMessageContent videoMessageContent = (VideoMessageContent)messageEvent.getMessage();
+            assertThat(videoMessageContent.getDuration()).isEqualTo(60000L);
+        });
+    }
+
     // Event, that has brand new eventType
     @Test
     public void testUnknown() throws IOException {

line-bot-model/src/test/resources/callback/video.json

@@ -0,0 +1,25 @@
+{
+  "destination": "Uab012345678901234567890123456789",
+  "events": [
+    {
+      "replyToken": "nHuyWiB7yP5Zw52FIkcQobQuGDXCTA",
+      "type": "message",
+      "mode": "active",
+      "timestamp": 1462629479859,
+      "source": {
+        "type": "user",
+        "userId": "U4af4980629..."
+      },
+      "message": {
+        "id": "325708",
+        "type": "video",
+        "duration": 60000,
+        "contentProvider": {
+          "type": "external",
+          "originalContentUrl": "https://example.com/original.mp4",
+          "previewImageUrl": "https://example.com/preview.jpg"
+        }
+      }
+    }
+  ]
+}

sample-spring-boot-kitchensink/src/main/java/com/example/bot/spring/KitchenSinkController.java

@@ -178,6 +178,8 @@ public void handleAudioMessageEvent(MessageEvent<AudioMessageContent> event) thr
 
     @EventMapping
     public void handleVideoMessageEvent(MessageEvent<VideoMessageContent> event) throws IOException {
+        log.info("Got video message: duration={}ms", event.getMessage().getDuration());
+
         // You need to install ffmpeg and ImageMagick.
         handleHeavyContent(
                 event.getReplyToken(),