Merge pull request #97 from ashawley/fix-null-pr-head-repo

Change head.repo on pull requests to Option

Author: shunjikonishi

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

@@ -56,7 +56,7 @@ case class PullRequestRef(value: JValue) extends AbstractJson(value) {
   def ref = get("ref")
   def sha = get("sha")
   lazy val user = User(value \ "user")
-  lazy val repo = Repository(value \ "repo")
+  lazy val repo = objectOpt("repo")(Repository(_))
 }
 
 case class PullRequest(value: JValue) extends AbstractJson(value) {

src/test/scala/PullRequestOpSpec.scala

@@ -24,9 +24,9 @@ class PullRequestOpSpec extends FunSpec with api.Constants {
       assert(list.exists(_.deletions == None))
       assert(list.exists(_.changed_files == None))
       assert(list.exists(_.maintainer_can_modify == None))
-      assert(list.exists(_.base.repo.full_name == s"$otherUser/$otherUserRepo"))
+      assert(list.exists(_.base.repo.exists(_.full_name == s"$otherUser/$otherUserRepo")))
       assert(list.exists(_.base.user.login == otherUser))
-      assert(list.exists(_.base.repo.name == otherUserRepo))
+      assert(list.exists(_.base.repo.exists(_.name == otherUserRepo)))
     }
   }