Add support for Change API's "Set Ready-For-Review" endpoint

changes.go

@@ -58,6 +58,11 @@ type CommitMessageInput struct {
 	NotifyDetails []NotifyInfo `json:"notify_details"`
 }
 
+// ReadyForReviewInput entity contains information for transitioning a change from WIP to ready.
+type ReadyForReviewInput struct {
+	Message string `json:"message,omitempty"`
+}
+
 // ChangeEditInput entity contains information for restoring a path within change edit.
 type ChangeEditInput struct {
 	RestorePath string `json:"restore_path,omitempty"`
@@ -782,6 +787,24 @@ func (s *ChangesService) SetCommitMessage(changeID string, input *CommitMessageI
 	return s.client.Do(req, nil)
 }
 
+// SetReadyForReview marks the change as ready for review (set WIP property to false)
+// Changes may only be marked ready by the owner, project owners or site administrators.
+// Activates notifications of reviewer. The request body does not need to include a
+// WorkInProgressInput entity if no review comment is added.
+// Marking a change ready for review also adds all of the reviewers of the change to the attention set.
+//
+// Gerrit API docs: https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#set-ready-for-review
+func (s *ChangesService) SetReadyForReview(changeID string, input *ReadyForReviewInput) (*Response, error) {
+	u := fmt.Sprintf("changes/%s/ready", changeID)
+
+	req, err := s.client.NewRequest("POST", u, input)
+	if err != nil {
+		return nil, err
+	}
+
+	return s.client.Do(req, nil)
+}
+
 // SetTopic sets the topic of a change.
 // The new topic must be provided in the request body inside a TopicInput entity.
 //

changes_test.go

@@ -450,3 +450,52 @@ func TestChangesService_SetCommitMessage_NotFound(t *testing.T) {
 		t.Error("Expected 404 code")
 	}
 }
+
+func TestChangesService_SetReadyForReview(t *testing.T) {
+	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/changes/123/ready" {
+			t.Errorf("%s != /changes/123/ready", r.URL.Path)
+		}
+		if r.Method != "POST" {
+			t.Error("Method != POST")
+		}
+		w.WriteHeader(http.StatusOK)
+	}))
+	defer ts.Close()
+
+	client, err := gerrit.NewClient(ts.URL, nil)
+	if err != nil {
+		t.Error(err)
+	}
+	cm := &gerrit.ReadyForReviewInput{Message: "Now ready for review"}
+	_, err = client.Changes.SetReadyForReview("123", cm)
+	if err != nil {
+		t.Error(err)
+	}
+}
+
+func TestChangesService_SetReadyForReview_NotFound(t *testing.T) {
+	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/changes/123/ready" {
+			t.Errorf("%s != /changes/123/ready", r.URL.Path)
+		}
+		if r.Method != "POST" {
+			t.Error("Method != POST")
+		}
+		w.WriteHeader(http.StatusNotFound)
+	}))
+	defer ts.Close()
+
+	client, err := gerrit.NewClient(ts.URL, nil)
+	if err != nil {
+		t.Error(err)
+	}
+	cm := &gerrit.ReadyForReviewInput{Message: "Now ready for review"}
+	resp, err := client.Changes.SetReadyForReview("123", cm)
+	if err == nil {
+		t.Error("Expected error, instead nil")
+	}
+	if resp.StatusCode != http.StatusNotFound {
+		t.Error("Expected 404 code")
+	}
+}