Merge pull request code-check#64 from code-check/list-prs

List prs

Author: shunjikonishi

src/main/scala/codecheck/github/models/PullRequest.scala

@@ -38,6 +38,14 @@ object PullRequestAction {
   def fromString(str: String) = values.filter(_.name == str).head
 }
 
+case class PullRequestListOption(
+  state: IssueState = IssueState.open,
+  head: Option[String] = None,
+  base: Option[String] = None,
+  sort: IssueSort = IssueSort.created,
+  direction: SortDirection = SortDirection.desc
+)
+
 case class PullRequestRef(value: JValue) extends AbstractJson(value) {
   def label = get("label")
   def ref = get("ref")

src/main/scala/codecheck/github/operations/PullRequestOp.scala

@@ -2,15 +2,37 @@ package codecheck.github.operations
 
 import scala.concurrent.Future
 import scala.concurrent.ExecutionContext.Implicits.global
-import org.json4s.{JObject, JString}
+import org.json4s.JArray
+import org.json4s.JObject
+import org.json4s.JString
 
 import codecheck.github.api.GitHubAPI
 import codecheck.github.models.PullRequestInput
+import codecheck.github.models.PullRequestListOption
 import codecheck.github.models.PullRequest
 
 trait PullRequestOp {
   self: GitHubAPI =>
 
+  def listPullRequests(
+    owner: String,
+    repo: String,
+    option: PullRequestListOption = PullRequestListOption()
+  ): Future[List[PullRequest]] = {
+    val q = s"?state=${option.state}" +
+      s"&sort=${option.sort}" +
+      s"&direction=${option.direction}" +
+      option.head.map("&head=" + _).getOrElse("") +
+      option.base.map("&base=" + _).getOrElse("")
+
+    exec("GET", s"/repos/$owner/$repo/pulls$q").map(
+      _.body match {
+        case JArray(arr) => arr.map(v => PullRequest(v))
+        case _ => throw new IllegalStateException()
+      }
+    )
+  }
+
   def createPullRequest(owner: String, repo: String, input: PullRequestInput): Future[PullRequest] = {
     val path = s"/repos/$owner/$repo/pulls"
     exec("POST", path, input.value).map { result =>

src/test/scala/PullRequestOpSpec.scala

@@ -6,6 +6,17 @@ import java.util.Date
 
 class PullRequestOpSpec extends FunSpec with Constants {
 
+  describe("listPullRequests") {
+    it("with valid repo should succeed") {
+      val list = Await.result(api.listPullRequests(otherUser, otherUserRepo), TIMEOUT)
+      assert(list.length >= 0)
+      assert(list.exists(_.state == "open"))
+      assert(list.exists(_.base.repo.full_name == s"$otherUser/$otherUserRepo"))
+      assert(list.exists(_.base.user.login == otherUser))
+      assert(list.exists(_.base.repo.name == otherUserRepo))
+    }
+  }
+
   describe("createPullRequest(owner, repo, input)") {
     val username = otherUser
     val reponame = otherUserRepo