Add requestreviewers

github/pulls_reviewers.go

@@ -0,0 +1,83 @@
+// Copyright 2017 The go-github AUTHORS. All rights reserved.
+//
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package github
+
+import (
+	"context"
+	"fmt"
+)
+
+// RequestReviewers submits a set of logins to be potential reviewers on a PR.
+//
+// GitHub API docs: https://developer.github.com/v3/pulls/review_requests/#create-a-review-request
+func (s *PullRequestsService) RequestReviewers(ctx context.Context, owner, repo string, number int, logins []string) (*PullRequest, *Response, error) {
+	u := fmt.Sprintf("repos/%s/%s/pulls/%d/requested_reviewers", owner, repo, number)
+	reviewers := struct {
+		Reviewers []string `json:"reviewers,omitempty"`
+	}{
+		Reviewers: logins,
+	}
+	req, err := s.client.NewRequest("POST", u, &reviewers)
+	if err != nil {
+		return nil, nil, err
+	}
+
+	// TODO: remove custom Accept header when this API fully launches
+	req.Header.Set("Accept", mediaTypePullRequestReviewsPreview)
+
+	r := new(PullRequest)
+	resp, err := s.client.Do(ctx, req, r)
+	if err != nil {
+		return nil, resp, err
+	}
+
+	return r, resp, nil
+}
+
+// ListReviewers lists users whose reviews have been requested on the specified pull request.
+//
+// GitHub API docs: https://developer.github.com/v3/pulls/review_requests/#list-review-requests
+func (s *PullRequestsService) ListReviewers(ctx context.Context, owner, repo string, number int) (*[]User, *Response, error) {
+	u := fmt.Sprintf("repos/%v/%v/pulls/%d/requested_reviewers", owner, repo, number)
+
+	req, err := s.client.NewRequest("GET", u, nil)
+	if err != nil {
+		return nil, nil, err
+	}
+
+	// TODO: remove custom Accept header when this API fully launches
+	req.Header.Set("Accept", mediaTypePullRequestReviewsPreview)
+
+	var users []User
+	resp, err := s.client.Do(ctx, req, &users)
+	if err != nil {
+		return nil, resp, err
+	}
+
+	return &users, resp, nil
+}
+
+// RemoveReviewers removes the review request for the provided GitHub users for the specified pull request.
+//
+// GitHub API docs: https://developer.github.com/v3/pulls/review_requests/#delete-a-review-request
+func (s *PullRequestsService) RemoveReviewers(ctx context.Context, owner, repo string, number int, logins []string) (*Response, error) {
+	u := fmt.Sprintf("repos/%s/%s/pulls/%d/requested_reviewers", owner, repo, number)
+
+	reviewers := struct {
+		Reviewers []string `json:"reviewers,omitempty"`
+	}{
+		Reviewers: logins,
+	}
+	req, err := s.client.NewRequest("DELETE", u, &reviewers)
+	if err != nil {
+		return nil, err
+	}
+
+	// TODO: remove custom Accept header when this API fully launches
+	req.Header.Set("Accept", mediaTypePullRequestReviewsPreview)
+
+	return s.client.Do(ctx, req, reviewers)
+}

github/pulls_reviewers_test.go

@@ -0,0 +1,123 @@
+// Copyright 2017 The go-github AUTHORS. All rights reserved.
+//
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package github
+
+import (
+	"context"
+	"encoding/json"
+	"fmt"
+	"io/ioutil"
+	"net/http"
+	"reflect"
+	"testing"
+)
+
+func TestRequestReviewers(t *testing.T) {
+	setup()
+	defer teardown()
+
+	type reviewers struct {
+		Reviewers []string `json:"reviewers,omitempty"`
+	}
+	have := reviewers{}
+	mux.HandleFunc("/repos/o/r/pulls/1/requested_reviewers", func(w http.ResponseWriter, r *http.Request) {
+		testMethod(t, r, "POST")
+		testHeader(t, r, "Accept", mediaTypePullRequestReviewsPreview)
+		b, err := ioutil.ReadAll(r.Body)
+		if err != nil {
+			t.Errorf("TestReviewerRequest couldn't read request body: %v", err)
+		}
+		if err := json.Unmarshal(b, &have); err != nil {
+			return
+		}
+	})
+
+	logins := []string{"octocat", "googlebot"}
+
+	// This returns a PR, unmarshalling of which is tested elsewhere
+	_, _, err := client.PullRequests.RequestReviewers(context.Background(), "o", "r", 1, logins)
+	if err != nil {
+		t.Errorf("PullRequests.RequestReviewers returned error: %v", err)
+	}
+
+	want := reviewers{
+		Reviewers: logins,
+	}
+	if !reflect.DeepEqual(have, want) {
+		t.Errorf("PullRequests.ListReviews returned %+v, want %+v", have, want)
+	}
+}
+
+func TestRemoveReviewers(t *testing.T) {
+	setup()
+	defer teardown()
+
+	type reviewers struct {
+		Reviewers []string `json:"reviewers,omitempty"`
+	}
+	have := reviewers{}
+	mux.HandleFunc("/repos/o/r/pulls/1/requested_reviewers", func(w http.ResponseWriter, r *http.Request) {
+		testMethod(t, r, "DELETE")
+		testHeader(t, r, "Accept", mediaTypePullRequestReviewsPreview)
+		b, err := ioutil.ReadAll(r.Body)
+		if err != nil {
+			t.Errorf("TestReviewerRequest couldn't read request body: %v", err)
+		}
+		if err := json.Unmarshal(b, &have); err != nil {
+			return
+		}
+	})
+
+	logins := []string{"octocat", "googlebot"}
+
+	_, err := client.PullRequests.RemoveReviewers(context.Background(), "o", "r", 1, logins)
+	if err != nil {
+		t.Errorf("PullRequests.RequestReviewers returned error: %v", err)
+	}
+
+	want := reviewers{
+		Reviewers: logins,
+	}
+	if !reflect.DeepEqual(have, want) {
+		t.Errorf("PullRequests.ListReviews returned %+v, want %+v", have, want)
+	}
+}
+
+func TestListReviewers(t *testing.T) {
+	setup()
+	defer teardown()
+
+	sampleResponse := `[
+  {
+    "login": "octocat",
+    "id": 1
+  }
+]`
+
+	mux.HandleFunc("/repos/o/r/pulls/1/requested_reviewers", func(w http.ResponseWriter, r *http.Request) {
+		testMethod(t, r, "GET")
+		testHeader(t, r, "Accept", mediaTypePullRequestReviewsPreview)
+		fmt.Fprintf(w, sampleResponse)
+	})
+
+	// This returns a PR, unmarshalling of which is tested elsewhere
+	have, _, err := client.PullRequests.ListReviewers(context.Background(), "o", "r", 1)
+	if err != nil {
+		t.Errorf("PullRequests.RequestReviewers returned error: %v", err)
+	}
+	_login := "octocat"
+	_id := 1
+
+	want := []User{
+		{
+			Login: &_login,
+			ID:    &_id,
+		},
+	}
+	if !reflect.DeepEqual(have, &want) {
+		t.Errorf("PullRequests.ListReviews returned %+v, want %+v", have, want)
+	}
+}