Fixed #200

 o Added getAllBuilds() and getAllBuilds(Range range)
   in MavenJobWithDetails class.

Author: khmarbaise

ReleaseNotes.md

@@ -80,6 +80,17 @@ public class QueueItem {
 }
 ```
 
+ * [Fixed Issue 200][issue-200]
+
+ Added two methods to get all builds and a range of builds
+ so we are more in line wiht JobWithDetails.
+
+```java
+public class MavenJobWithDetails { 
+    public List<MavenBuild> getAllBuilds();
+    public List<MavenBuild> getAllBuilds(Range range);
+}
+```
 
 ## Release 0.3.6
 
@@ -672,6 +683,7 @@ TestReport testReport = mavenJob.getLastSuccessfulBuild().getTestReport();
 [issue-182]: https://github.com/jenkinsci/java-client-api/issues/182
 [issue-186]: https://github.com/jenkinsci/java-client-api/issues/186
 [issue-198]: https://github.com/jenkinsci/java-client-api/issues/198
+[issue-200]: https://github.com/jenkinsci/java-client-api/issues/200
 [issue-201]: https://github.com/jenkinsci/java-client-api/issues/201
 [issue-202]: https://github.com/jenkinsci/java-client-api/issues/202
 [pull-123]: https://github.com/jenkinsci/java-client-api/pull/123

jenkins-client/src/main/java/com/offbytwo/jenkins/model/AllMavenBuilds.java

@@ -0,0 +1,28 @@
+/*
+ * Copyright (c) 2013 Cosmin Stejerean, Karl Heinz Marbaise, and contributors.
+ *
+ * Distributed under the MIT license: http://opensource.org/licenses/MIT
+ */
+
+package com.offbytwo.jenkins.model;
+
+import java.util.List;
+
+/**
+ * This class is only needed to get all builds in
+ * {@link MavenJobWithDetails#getAllBuilds()}.
+ * 
+ * @author Karl Heinz Marbaise
+ *
+ *         NOTE: This class is not part of any public API
+ */
+class AllMavenBuilds extends BaseModel {
+    private List<MavenBuild> allBuilds;
+
+    public AllMavenBuilds() {
+    }
+
+    public List<MavenBuild> getAllBuilds() {
+        return this.allBuilds;
+    }
+}

jenkins-client/src/main/java/com/offbytwo/jenkins/model/MavenJobWithDetails.java

@@ -1,10 +1,21 @@
 package com.offbytwo.jenkins.model;
 
+import static com.google.common.collect.Lists.transform;
+
+import java.io.IOException;
 import java.util.Collections;
 import java.util.List;
 
+import org.apache.http.HttpStatus;
+import org.apache.http.client.HttpResponseException;
+
 import com.google.common.base.Function;
+import com.google.common.base.Optional;
+import com.google.common.base.Predicate;
+import com.google.common.collect.Iterables;
 import com.google.common.collect.Lists;
+import com.offbytwo.jenkins.client.util.EncodingUtils;
+import com.offbytwo.jenkins.helper.Range;
 
 public class MavenJobWithDetails extends MavenJob {
 
@@ -35,6 +46,12 @@ public boolean isBuildable() {
     }
 
     /**
+     * This method will give you back the builds of a particular job.<br/>
+     * <b>Note: Jenkins limits the number of results to a maximum of 100 builds
+     * which you will get back.</b>. In case you have more than 100 build you
+     * won't get back all builds via this method. In such cases you need to use
+     * {@link #getAllBuilds()}.
+     * 
      * @return the list of {@link MavenBuild}. In case of no builds have been
      *         executed yet {@link Collections#emptyList()} will be returned.
      */
@@ -51,6 +68,102 @@ public MavenBuild apply(MavenBuild from) {
         }
     }
 
+    /**
+     * This method will give you back all builds which exists independent of the
+     * number. You should be aware that this can be much in some cases if you
+     * have more than 100 builds which is by default limited by Jenkins
+     * {@link #getBuilds()}. This method limits it to particular information
+     * which can be later used to get supplemental information about a
+     * particular build {@link Build#details()} to reduce the amount of data
+     * which needed to be transfered.
+     * 
+     * @return the list of {@link Build}. In case of no builds have been
+     *         executed yet return {@link Collections#emptyList()}.
+     * @throws IOException
+     *             In case of failure.
+     * @see <a href="https://issues.jenkins-ci.org/browse/JENKINS-30238">Jenkins
+     *      Issue</a>
+     */
+    public List<MavenBuild> getAllBuilds() throws IOException {
+        String path = "/";
+
+        try {
+            List<MavenBuild> builds = client.get(path + "job/" + EncodingUtils.encode(this.getName())
+                    + "?tree=allBuilds[number[*],url[*],queueId[*]]", AllMavenBuilds.class).getAllBuilds();
+
+            if (builds == null) {
+                return Collections.emptyList();
+            } else {
+                return transform(builds, new Function<MavenBuild, MavenBuild>() {
+                    @Override
+                    public MavenBuild apply(MavenBuild from) {
+                        return buildWithClient(from);
+                    }
+                });
+            }
+        } catch (HttpResponseException e) {
+            // TODO: Thinks about a better handling if the job does not exist?
+            if (e.getStatusCode() == HttpStatus.SC_NOT_FOUND) {
+                // TODO: Check this if this is necessary or a good idea?
+
+                return null;
+            }
+            throw e;
+        }
+
+    }
+    
+    /**
+    *
+    * <ul>
+    * <li>{M,N}: From the M-th element (inclusive) to the N-th element
+    * (exclusive).</li>
+    * <li>{M,}: From the M-th element (inclusive) to the end.</li>
+    * <li>{,N}: From the first element (inclusive) to the N-th element
+    * (exclusive). The same as {0,N}.</li>
+    * <li>{N}: Just retrieve the N-th element. The same as {N,N+1}.</li>
+    * </ul>
+    * 
+    * <b>Note: At the moment there seemed to be no option to get the number of
+    * existing builds for a job. The only option is to get all builds via
+    * {@link #getAllBuilds()}.</b>
+    * 
+    * @param range
+    *            {@link Range}
+    * @return the list of {@link Build}. In case of no builds have been
+    *         executed yet return {@link Collections#emptyList()}.
+    * @throws IOException
+    *             in case of an error.
+    */
+   public List<MavenBuild> getAllBuilds(Range range) throws IOException {
+       String path = "/" + "job/" + EncodingUtils.encode(this.getName())
+               + "?tree=allBuilds[number[*],url[*],queueId[*]]";
+
+       try {
+           List<MavenBuild> builds = client.get(path + range.getRangeString(), AllMavenBuilds.class).getAllBuilds();
+
+           if (builds == null) {
+               return Collections.emptyList();
+           } else {
+               return transform(builds, new Function<MavenBuild, MavenBuild>() {
+                   @Override
+                   public MavenBuild apply(MavenBuild from) {
+                       return buildWithClient(from);
+                   }
+               });
+           }
+       } catch (HttpResponseException e) {
+           // TODO: Thinks about a better handline if the job does not exist?
+           if (e.getStatusCode() == HttpStatus.SC_NOT_FOUND) {
+               // TODO: Check this if this is necessary or a good idea?
+
+               return null;
+           }
+           throw e;
+       }
+   }
+    
+    
     /**
      * @return The firstBuild. If {@link #firstBuild} has never been run
      *         {@link MavenBuild#BUILD_HAS_NEVER_RUN} will be returned.
@@ -169,6 +282,21 @@ public List<Job> getUpstreamProjects() {
         }
     }
 
+    public MavenBuild getBuildByNumber(final int buildNumber) {
+
+        Predicate<MavenBuild> isMatchingBuildNumber = new Predicate<MavenBuild>() {
+
+            @Override
+            public boolean apply(MavenBuild input) {
+                return input.getNumber() == buildNumber;
+            }
+        };
+
+        Optional<MavenBuild> optionalBuild = Iterables.tryFind(builds, isMatchingBuildNumber);
+        // TODO: Check if we could use Build#NO...instead of Null?
+        return optionalBuild.orNull() == null ? null : buildWithClient(optionalBuild.orNull());
+    }
+    
     private MavenBuild buildWithClient(MavenBuild from) {
         MavenBuild ret = new MavenBuild(from);
         ret.setClient(client);