get all changed files (#742)

* get all changed files in PR

Former-commit-id: b890d18

libs/orchestrator/github/github.go

@@ -13,7 +13,11 @@ import (
 )
 
 func NewGitHubService(ghToken string, repoName string, owner string) GithubService {
-	client := github.NewTokenClient(context.Background(), ghToken)
+	client := github.NewClient(nil)
+	if ghToken != "" {
+		client = client.WithAuthToken(ghToken)
+	}
+
 	return GithubService{
 		Client:   client,
 		RepoName: repoName,
@@ -47,15 +51,21 @@ func (svc *GithubService) GetUserTeams(organisation string, user string) ([]stri
 }
 
 func (svc *GithubService) GetChangedFiles(prNumber int) ([]string, error) {
-	files, _, err := svc.Client.PullRequests.ListFiles(context.Background(), svc.Owner, svc.RepoName, prNumber, nil)
-	if err != nil {
-		log.Fatalf("error getting pull request files: %v", err)
-	}
-
-	fileNames := make([]string, len(files))
+	var fileNames []string
+	opts := github.ListOptions{PerPage: 100}
+	for {
+		files, resp, err := svc.Client.PullRequests.ListFiles(context.Background(), svc.Owner, svc.RepoName, prNumber, &opts)
+		if err != nil {
+			log.Fatalf("error getting pull request files: %v", err)
+		}
 
-	for i, file := range files {
-		fileNames[i] = *file.Filename
+		for _, file := range files {
+			fileNames = append(fileNames, *file.Filename)
+		}
+		if resp.NextPage == 0 {
+			break
+		}
+		opts.Page = resp.NextPage
 	}
 	return fileNames, nil
 }

libs/orchestrator/github/github_test.go

@@ -112,3 +112,9 @@ func TestFindAllProjectsDependantOnImpactedProjects(t *testing.T) {
 	assert.NotContains(t, projectNames, "k")
 	assert.NotContains(t, projectNames, "b")
 }
+
+func TestFindAllChangedFilesOfPR(t *testing.T) {
+	githubPrService := NewGitHubService("", "digger", "diggerhq")
+	files, _ := githubPrService.GetChangedFiles(98)
+	assert.Equal(t, 45, len(files))
+}