Tests for searchCode,searchIssues,searchUser

Author: sukeshni

Diff for: src/main/scala/codecheck/github/models/Issue.scala

@@ -104,6 +104,7 @@ case class Issue(value: JValue) extends AbstractJson(value) {
   lazy val assignee = objectOpt("assignee")(v => User(v))
   lazy val milestone = objectOpt("milestone")(v => Milestone(v))
 
+  val state = get("state")
   lazy val user = new User(value \ "user")
   lazy val labels = (value \ "labels") match {
     case JArray(arr) => arr.map(new Label(_))
@@ -115,4 +116,4 @@ case class Issue(value: JValue) extends AbstractJson(value) {
   def created_at = date("created_at")
   def updated_at = date("updated_at")
   def closed_at = dateOpt("closed_at")
-}
+}

Diff for: src/main/scala/codecheck/github/models/Repository.scala

@@ -53,6 +53,8 @@ case class Repository(value: JValue) extends AbstractJson(value) {
   def name = get("name")
   def full_name = get("full_name")
   def url = get("url")
+  def language = get("language")
+  def stargazers_count = get("stargazers_count")
 
   def description = opt("description")
   def open_issues_count = get("open_issues_count").toInt
@@ -65,4 +67,4 @@ case class Permissions(value: JValue) extends AbstractJson(value) {
   def admin = boolean("admin")
   def push = boolean("push")
   def pull = boolean("pull")
-}
+}

Diff for: src/main/scala/codecheck/github/operations/SearchOp.scala

@@ -45,7 +45,7 @@ trait SearchOp {
   }
 
   def searchUser(input: SearchInput): Future[Option[SearchUserResult]] = {
-    val path = s"/search/user?q=${input.q}&sort=${input.sort}&order=${input.order}"
+    val path = s"/search/users?q=${input.q}&sort=${input.sort}&order=${input.order}"
     exec("GET", path ).map { res =>
       res.statusCode match {
         case 200 => Some(SearchUserResult(res.body))

Diff for: src/test/scala/SearchOpSpec.scala

@@ -5,6 +5,9 @@ import codecheck.github.models.SortDirection
 import codecheck.github.models.SearchInput
 import codecheck.github.models.SearchSort
 import codecheck.github.models.SearchRepositoryResult
+import codecheck.github.models.SearchCodeResult
+import codecheck.github.models.searchCodeItems
+import codecheck.github.exceptions.GitHubAPIException
 
 class SearchOpSpec extends FunSpec
   with Constants
@@ -22,7 +25,8 @@ class SearchOpSpec extends FunSpec
         assert(res.items(0).full_name.length >= 1)
         assert(res.items(0).description.isDefined)
         assert(res.items(0).open_issues_count >= 0)
-        println("RESULT" + res)
+        assert(res.items(0).language == "Assembly")
+        assert(res.items(0).stargazers_count > res.items(1).stargazers_count)
       }
     }
     it("with valid changed query(q) SearchInput should succeed") {
@@ -36,7 +40,62 @@ class SearchOpSpec extends FunSpec
         assert(res.items(0).full_name.length >= 1)
         assert(res.items(0).description.isDefined)
         assert(res.items(0).open_issues_count >= 0)
-        println("RESULT" + res)
+      }
+    }
+  }
+  describe("searchCode") {
+    it("with valid SearchInput q,no SortOrder should succeed") {
+      var q = "addClass in:file language:js repo:jquery/jquery"
+      val q1 = q.trim.replaceAll(" ","+");
+      val input = SearchInput(q1,sort=None,order=SortDirection.desc)
+      Await.result(api.searchCode(input), TIMEOUT).map { res =>
+        assert(res.total_count >= 1)
+        assert(res.items(0).Repo.id >= 1 )
+        assert(res.items(0).Repo.full_name == "jquery/jquery")
+      }
+    }
+    //Following test results in error:
+    //  "message" : "Validation Failed",
+    //  "errors" : [ {
+    //  "message" : "Must include at least one user, organization, or repository"
+    it("with valid SearchInput it should succeed") {
+      var q = "function size:10000 language:python"
+      val q1 = q.trim.replaceAll(" ","+");
+      val input = SearchInput(q1,sort=Some(SearchSort.indexed),order=SortDirection.desc)
+      try {
+        val res = Await.result(api.searchCode(input), TIMEOUT)
+        fail
+      } catch {
+        case e: GitHubAPIException =>
+          assert(e.error.errors.length == 1)
+          assert(e.error.message == "Validation Failed")
+      }
+    }
+  }
+  describe("searchIssues") {
+    it("with valid SearchInput should succeed") {
+      var q = "windows label:bug language:python state:open"
+      val q1 = q.trim.replaceAll(" ","+");
+      val input = SearchInput(q1,sort=Some(SearchSort.created),order=SortDirection.asc)
+      Await.result(api.searchIssues(input), TIMEOUT).map { res =>
+        assert(res.total_count >= 1)
+        assert(res.items(0).labels(0).name == "bug" )
+        assert(res.items(0).state == "open")
+        assert(((res.items(0).created_at).compareTo(res.items(1).created_at)) > 0)
+      }
+    }
+  }
+  describe("searchUser") {
+    it("with valid SearchInput should succeed") {
+      var q = "tom repos:>42 followers:>1000"
+      q = q.trim.replaceAll(" ","+")
+      val q1 = q.replaceAll(">","%3E")
+      println("QUERY: searchUser" + q1)
+      val input = SearchInput(q1,sort=None,order=SortDirection.desc)
+      Await.result(api.searchUser(input), TIMEOUT).map { res =>
+        assert(res.total_count >= 1)
+        assert(res.items(0).login.length >= 1)
+        assert(res.items(0).id >= 1)
       }
     }
   }