Update dependencies

Author: guywithnose

Gopkg.lock

@@ -706,7 +706,7 @@ func TestCmdAutoRebaseBadAPIURL(t *testing.T) {
 	app := cli.NewApp()
 	cb := &runner.Test{}
 	err := command.CmdAutoRebase(cb)(cli.NewContext(app, set, nil))
-	assert.EqualError(t, err, "parse %s/mockApi: invalid URL escape \"%s/\"")
+	assert.EqualError(t, err, "parse %s/mockApi/: invalid URL escape \"%s/\"")
 }
 
 func TestCmdAutoRebaseUserFailure(t *testing.T) {

command/autoRebase_test.go

@@ -193,7 +193,7 @@ func TestCmdParseBadApiUrl(t *testing.T) {
 	set := getBaseFlagSet(configFileName)
 	app := cli.NewApp()
 	err := command.CmdParse(cli.NewContext(app, set, nil))
-	assert.EqualError(t, err, "parse %s/mockApi: invalid URL escape \"%s/\"")
+	assert.EqualError(t, err, "parse %s/mockApi/: invalid URL escape \"%s/\"")
 }
 
 func TestCmdParseUserFailure(t *testing.T) {
@@ -326,7 +326,7 @@ func TestCmdParseCommitCompareFailure(t *testing.T) {
 }
 
 func TestCmdParseReviewFailure(t *testing.T) {
-	ts := getParseTestServer("/repos/foo/bar/pulls/1/reviews")
+	ts := getParseTestServer("/repos/foo/bar/pulls/1/reviews?per_page=100")
 	defer ts.Close()
 	_, configFileName := getConfigWithAPIURL(t, ts.URL)
 	defer removeFile(t, configFileName)

command/parse_test.go

@@ -104,16 +104,25 @@ func (pr pullRequest) getComments() <-chan *github.IssueComment {
 }
 
 func (pr pullRequest) getReviews() <-chan *github.PullRequestReview {
+	opt := &github.ListOptions{PerPage: 100}
 	allReviews := make(chan *github.PullRequestReview)
 	go func() {
 		defer close(allReviews)
-		reviews, _, err := pr.client.PullRequests.ListReviews(context.Background(), pr.Repo.Owner, pr.Repo.Name, pr.PullRequestID)
-		if err != nil {
-			return
-		}
+		for {
+			reviews, resp, err := pr.client.PullRequests.ListReviews(context.Background(), pr.Repo.Owner, pr.Repo.Name, pr.PullRequestID, opt)
+			if err != nil {
+				return
+			}
+
+			for _, review := range reviews {
+				allReviews <- review
+			}
+
+			if resp.NextPage == 0 {
+				return
+			}
 
-		for _, review := range reviews {
-			allReviews <- review
+			opt.Page = resp.NextPage
 		}
 	}()
 	return allReviews

command/pullRequest.go

@@ -102,7 +102,7 @@ func getConfigWithAPIURL(t *testing.T, url string) (config.PrpConfig, string) {
 	t.Helper()
 	conf, configFileName := getConfigWithIgnoredBuild(t)
 	profile := conf.Profiles["foo"]
-	profile.APIURL = url
+	profile.APIURL = fmt.Sprintf("%s/", url)
 	profile.Token = "abc"
 	conf.Profiles["foo"] = profile
 	assert.Nil(t, conf.Write(configFileName))
@@ -221,15 +221,15 @@ func handleCommentRequests(r *http.Request, w http.ResponseWriter, server *httpt
 }
 
 func handleReviewRequests(r *http.Request, w http.ResponseWriter, server *httptest.Server) *string {
-	if r.URL.String() == "/repos/own/rep/pulls/1/reviews" {
+	if r.URL.String() == "/repos/own/rep/pulls/1/reviews?per_page=100" {
 		bytes, _ := json.Marshal([]*github.PullRequestReview{
 			newReview("fooGuy"),
 		})
 		w.Header().Set(
 			"Link",
 			fmt.Sprintf(
-				`<%s/mockApi/repos/own/rep/issues/1/comments?per_page=100&page=2>; rel="next", `+
-					`<%s/mockApi/repos/own/rep/issues/1/comments?per_page=100&page=2>; rel="last"`,
+				`<%s/mockApi/repos/own/rep/pulls/1/reviews?per_page=100&page=2>; rel="next", `+
+					`<%s/mockApi/repos/own/rep/pulls/1/reviews?per_page=100&page=2>; rel="last"`,
 				server.URL,
 				server.URL,
 			),
@@ -238,7 +238,16 @@ func handleReviewRequests(r *http.Request, w http.ResponseWriter, server *httpte
 		return &response
 	}
 
-	if r.URL.String() == "/repos/foo/bar/pulls/1/reviews" {
+	if r.URL.String() == "/repos/own/rep/pulls/1/reviews?page=2&per_page=100" {
+
+		bytes, _ := json.Marshal([]*github.PullRequestReview{
+			newReview("fooGuy"),
+		})
+		response := string(bytes)
+		return &response
+	}
+
+	if r.URL.String() == "/repos/foo/bar/pulls/1/reviews?per_page=100" {
 		bytes, _ := json.Marshal([]*github.PullRequestReview{
 			newReview("guy"),
 			newReview("own"),
@@ -249,15 +258,15 @@ func handleReviewRequests(r *http.Request, w http.ResponseWriter, server *httpte
 		return &response
 	}
 
-	if r.URL.String() == "/repos/own/rep/pulls/2/reviews" {
+	if r.URL.String() == "/repos/own/rep/pulls/2/reviews?per_page=100" {
 		bytes, _ := json.Marshal([]*github.PullRequestReview{
 			newReview("guy"),
 		})
 		response := string(bytes)
 		return &response
 	}
 
-	if r.URL.String() == "/repos/foo/bar/pulls/2/reviews" {
+	if r.URL.String() == "/repos/foo/bar/pulls/2/reviews?per_page=100" {
 		bytes, _ := json.Marshal([]*github.PullRequestReview{
 			newReview("guy2"),
 		})