Merge pull request #9 from meetsmore/fmy/assign-reviewers

assign pull requests' owners as reviewers

Author: uiur

index.js

@@ -13,6 +13,7 @@ module.exports = function createReleasePR (config) {
       var template = fs.readFileSync(templatePath, 'utf8')
       var message = releaseMessage(template, prs)
 
+      client.assignReviewers(releasePR, prs)
       return client.updatePR(releasePR, message)
     })
   })

lib/github-client.js

@@ -172,6 +172,17 @@ GithubClient.prototype.collectReleasePRs = function (releasePR) {
   })
 }
 
+GithubClient.prototype.assignReviewers = function (pr, prs) {
+  var reviewers = prs
+    .map(pr => pr.assignee ? pr.assignee : pr.user)
+    .filter(user => user.type === 'User')
+    .map(user => user.login)
+
+  return this.post(this.pullRequestEndpoint() + '/' + pr.number + '/requested_reviewers', { reviewers }).then(function (res) {
+    return res.body
+  })
+}
+
 GithubClient.prototype.updatePR = function (pr, data) {
   return this.patch(this.pullRequestEndpoint() + '/' + pr.number, data).then(function (res) {
     return res.body

test/github-client.js

@@ -163,6 +163,34 @@ describe('GithubClient', function () {
     })
   })
 
+  describe('#assignReviewers()', function () {
+    const USER1 = 'pr1-owner'
+    const USER2 = 'pr2-owner'
+    const BOT = 'bot'
+    nock('https://api.github.com')
+      .post('/repos/uiureo/awesome-app/pulls/42/requested_reviewers')
+      .query(true)
+      .reply(200, (_, requestBody) => ({
+        requested_reviewers: requestBody.reviewers.map(login => ({ login }))
+      }))
+
+    it('returns pr that has reviewers', function (done) {
+      const prs = [
+        { assignee: { login: USER1, type: 'User' } },
+        { user: { login: USER2, type: 'User' } },
+        { user: { login: BOT, type: 'Bot' } }
+      ]
+      this.client.assignReviewers({ number: 42 }, prs)
+        .then(function (pr) {
+          assert(pr.requested_reviewers.length === 2)
+          assert(pr.requested_reviewers[0].login === USER1)
+          assert(pr.requested_reviewers[1].login === USER2)
+
+          done()
+        }).catch(done)
+    })
+  })
+
   describe('#updatePR()', function () {
     nock('https://api.github.com/')
       .patch('/repos/uiureo/awesome-app/pulls/42')

test/index.js

@@ -16,6 +16,9 @@ function MockGithubClient () {
     },
     updatePR: function (releasePR, message) {
       return Promise.resolve({ pr: releasePR, message: message })
+    },
+    assignReviewers: function (releasePR, prs) {
+      return Promise.resolve({ requested_reviewers: [] })
     }
   }
 }