Fix docker-java#682 - Add LogPath to docker inspect response (docker-java#683)

jkubrynski · KostyaSha · commit f44616f0808e · 2016-08-28T18:43:31.000+03:00
* Fix docker-java#682 - Add LogPath to docker inspect response * Fix docker-java#682 - Add LogPath to docker inspect response - fixes after code review * Fix docker-java#682 - Add LogPath to docker inspect response - add javadoc * Fix docker-java#682 - Add LogPath to docker inspect response - clean test assertion
diff --git a/src/main/java/com/github/dockerjava/api/command/InspectContainerResponse.java b/src/main/java/com/github/dockerjava/api/command/InspectContainerResponse.java
@@ -53,6 +53,12 @@ public class InspectContainerResponse {
     @JsonProperty("HostsPath")
     private String hostsPath;
 
+    /**
+     * @since {@link RemoteApiVersion#VERSION_1_17}
+     */
+    @JsonProperty("LogPath")
+    private String logPath;
+
     @JsonProperty("Id")
     private String id;
 
@@ -168,6 +174,11 @@ public String getHostsPath() {
         return hostsPath;
     }
 
+    @CheckForNull
+    public String getLogPath() {
+        return logPath;
+    }
+
     public String getName() {
         return name;
     }
diff --git a/src/test/java/com/github/dockerjava/api/command/InspectContainerResponseTest.java b/src/test/java/com/github/dockerjava/api/command/InspectContainerResponseTest.java
@@ -22,6 +22,7 @@
 import static com.github.dockerjava.test.serdes.JSONTestHelper.testRoundTrip;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.containsString;
+import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.isEmptyString;
 import static org.hamcrest.Matchers.nullValue;
 import static org.hamcrest.core.IsNot.not;
@@ -51,6 +52,7 @@ public void roundTrip_full() throws IOException {
         assertEquals(response.getVolumesRW()[1].getVolume().getPath(), "/bar/foo/myvol2");
         assertFalse(response.getVolumesRW()[1].getAccessMode().toBoolean());
         assertTrue(response.getVolumesRW()[0].getAccessMode().toBoolean());
+        assertThat(response.getLogPath(), is("/mnt/sda1/var/lib/docker/containers/469e5edd8d5b33e3c905a7ffc97360ec6ee211d6782815fbcd144568045819e1/469e5edd8d5b33e3c905a7ffc97360ec6ee211d6782815fbcd144568045819e1-json.log"));
     }
 
     @Test
