Merge pull request #1854 from bibonix/fix-1026-milestone-properties

#1026: expose updatedAt and closedAt on Milestone.Smart

Author: yegor256

pom.xml

@@ -304,7 +304,6 @@
           <plugin>
             <groupId>com.qulice</groupId>
             <artifactId>qulice-maven-plugin</artifactId>
-            <version>0.25.1</version>
             <configuration>
               <excludes combine.children="append">
                 <exclude>xml:/src/it/settings.xml</exclude>

src/main/java/com/jcabi/github/Milestone.java

@@ -258,6 +258,36 @@ public Date dueOn() throws IOException {
             }
         }
 
+        /**
+         * When this milestone was last updated.
+         * @return Date of last update
+         * @throws IOException If there is any I/O problem
+         */
+        public Date updatedAt() throws IOException {
+            try {
+                return new GitHub.Time(
+                    this.jsn.text("updated_at")
+                ).date();
+            } catch (final ParseException ex) {
+                throw new IllegalStateException(ex);
+            }
+        }
+
+        /**
+         * When this milestone was closed.
+         * @return Date of closure
+         * @throws IOException If there is any I/O problem
+         */
+        public Date closedAt() throws IOException {
+            try {
+                return new GitHub.Time(
+                    this.jsn.text("closed_at")
+                ).date();
+            } catch (final ParseException ex) {
+                throw new IllegalStateException(ex);
+            }
+        }
+
         /**
          * Change milestone due date.
          * @param dueon New milestone due date

src/test/java/com/jcabi/github/MilestoneTest.java

@@ -76,4 +76,34 @@ void fetchesDueOn() throws IOException {
             Matchers.notNullValue()
         );
     }
+
+    @Test
+    void fetchesUpdatedAt() throws IOException {
+        final Milestone milestone = Mockito.mock(Milestone.class);
+        Mockito.doReturn(
+            Json.createObjectBuilder()
+                .add("updated_at", "2014-03-03T18:58:10Z")
+                .build()
+        ).when(milestone).json();
+        MatcherAssert.assertThat(
+            "updatedAt() is null",
+            new Milestone.Smart(milestone).updatedAt(),
+            Matchers.notNullValue()
+        );
+    }
+
+    @Test
+    void fetchesClosedAt() throws IOException {
+        final Milestone milestone = Mockito.mock(Milestone.class);
+        Mockito.doReturn(
+            Json.createObjectBuilder()
+                .add("closed_at", "2013-02-12T13:22:01Z")
+                .build()
+        ).when(milestone).json();
+        MatcherAssert.assertThat(
+            "closedAt() is null",
+            new Milestone.Smart(milestone).closedAt(),
+            Matchers.notNullValue()
+        );
+    }
 }