From 713854adc3440629a83ab770489a62131f56ebcf Mon Sep 17 00:00:00 2001
From: Felix Mulder <felix.mulder@gmail.com>
Date: Sat, 6 May 2017 10:41:46 +0200
Subject: [PATCH 1/2] Make the bot check contributors before sending greeting

---
 .../dotty/tools/bot/PullRequestService.scala  | 36 +++++++++++++++----
 1 file changed, 29 insertions(+), 7 deletions(-)

diff --git a/bot/src/dotty/tools/bot/PullRequestService.scala b/bot/src/dotty/tools/bot/PullRequestService.scala
index 87b05133e583..943cf21303a4 100644
--- a/bot/src/dotty/tools/bot/PullRequestService.scala
+++ b/bot/src/dotty/tools/bot/PullRequestService.scala
@@ -103,6 +103,9 @@ trait PullRequestService {
   def reviewUrl(issueNbr: Int): String =
     withGithubSecret(s"$githubUrl/repos/lampepfl/dotty/pulls/$issueNbr/reviews")
 
+  def contributorsUrl: String =
+    withGithubSecret("https://api.github.com/repos/lampepfl/dotty/contributors")
+
   sealed trait CommitStatus {
     def commit: Commit
     def isValid: Boolean
@@ -183,6 +186,13 @@ trait PullRequestService {
       .headOption
   }
 
+  /** Get all contributors from GitHub */
+  def getContributors(implicit client: Client): Task[Set[String]] =
+    for {
+      authors <- client.expect(get(contributorsUrl))(jsonOf[List[Author]])
+      logins  =  authors.map(_.login).flatten
+    } yield logins.toSet
+
   /** Ordered from earliest to latest */
   def getCommits(issueNbr: Int)(implicit httpClient: Client): Task[List[Commit]] = {
     def makeRequest(url: String): Task[List[Commit]] =
@@ -226,8 +236,11 @@ trait PullRequestService {
       wrongTense || firstLine.last == '.' || firstLine.length > 80
     }
 
-  def sendInitialComment(issueNbr: Int, invalidUsers: List[String], commits: List[Commit], client: Client): Task[ReviewResponse] = {
-
+  /** Returns the body of a `ReviewResponse` */
+  def sendInitialComment(issueNbr: Int,
+                         invalidUsers: List[String],
+                         commits: List[Commit],
+                         newContributors: Boolean)(implicit client: Client): Task[String] = {
     val cla = if (invalidUsers.nonEmpty) {
       s"""|## CLA ##
           |In order for us to be able to accept your contribution, all users
@@ -272,9 +285,16 @@ trait PullRequestService {
 
     val review = Review.comment(body)
 
+    val shouldPost = newContributors || commitRules.nonEmpty || invalidUsers.nonEmpty
+
     for {
       req <- post(reviewUrl(issueNbr)).withAuth(githubUser, githubToken)
-      res <- client.expect(req.withBody(review.asJson))(jsonOf[ReviewResponse])
+      res <- {
+        if (shouldPost)
+          client.expect(req.withBody(review.asJson))(jsonOf[ReviewResponse]).map(_.body)
+        else
+          Task.now("")
+      }
     } yield res
   }
 
@@ -299,10 +319,12 @@ trait PullRequestService {
           setStatus(statuses.last, httpClient)
       }
 
-      // Send positive comment:
-      _    <- sendInitialComment(issue.number, invalidUsers, commits, httpClient)
-      _    <- httpClient.shutdown
-      resp <- Ok("Fresh PR checked")
+      authors  =  commits.map(_.author.login).flatten.toSet
+      contribs <- getContributors
+      newContr =  !authors.forall(contribs.contains)
+      _        <- sendInitialComment(issue.number, invalidUsers, commits, newContr)
+      _        <- httpClient.shutdown
+      resp     <- Ok("Fresh PR checked")
     } yield resp
 
   }

From 6f49d6f2313d5e7d60ce2b4a7f8d1291f541fdd8 Mon Sep 17 00:00:00 2001
From: Felix Mulder <felix.mulder@gmail.com>
Date: Sat, 6 May 2017 10:56:04 +0200
Subject: [PATCH 2/2] Add test for getting contributors

---
 bot/test/PRServiceTests.scala | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/bot/test/PRServiceTests.scala b/bot/test/PRServiceTests.scala
index c94c553224c3..29f08dddf1cd 100644
--- a/bot/test/PRServiceTests.scala
+++ b/bot/test/PRServiceTests.scala
@@ -123,13 +123,14 @@ class PRServiceTests extends PullRequestService {
   @Test def canPostReview = {
     val invalidUsers = "felixmulder" :: "smarter" :: Nil
     val commit = Commit("", Author(Some("smarter")), Author(Some("smarter")), CommitInfo("Added stuff"))
-    val res = withClient(sendInitialComment(2281, invalidUsers, commit :: Nil, _))
+    val resBody =
+      withClient(sendInitialComment(2281, invalidUsers, commit :: Nil, false)(_))
 
     assert(
-      res.body.contains("We want to keep history") &&
-      res.body.contains("Could you folks please sign the CLA?") &&
-      res.body.contains("Have an awesome day!"),
-      s"Body of review was not as expected:\n${res.body}"
+      resBody.contains("We want to keep history") &&
+      resBody.contains("Could you folks please sign the CLA?") &&
+      resBody.contains("Have an awesome day!"),
+      s"Body of review was not as expected:\n${resBody}"
     )
   }
 
@@ -159,4 +160,9 @@ class PRServiceTests extends PullRequestService {
 
     assert(respondToComment(issueComment).run.status.code == 200)
   }
+
+  @Test def canGetContributors = {
+    val contributors = withClient(getContributors(_))
+    assert(contributors.contains("felixmulder"))
+  }
 }