chore: send author ids also for github and gitlab. (#2020)

## What this PR does / why we need it: Note: The sync of reviewers is still not sent for gitlab. ## Which issue(s) this PR fixes: none ## Special notes for your reviewer: ## Does this PR introduce a user-facing change? ``` no ```
terramate-io · Dec 20, 2024 · d277299 · d277299
2 parents 218cc71 + 7e5a8cd
commit d277299
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 9 deletions.
diff --git a/cmd/terramate/cli/cloud.go b/cmd/terramate/cli/cloud.go
@@ -1105,8 +1105,8 @@ func (c *cli) newBitbucketReviewRequest(pr *bitbucket.PR) *cloud.ReviewRequest {
 		UpdatedAt:   &updatedAt,
 		Status:      pr.State,
 		Author: cloud.Author{
-			Login:     pr.Author.DisplayName,
 			ID:        pr.Author.UUID,
+			Login:     pr.Author.DisplayName,
 			AvatarURL: avatarURL,
 		},
 		Branch:                c.cloud.run.metadata.BitbucketPipelinesBranch,
@@ -1384,6 +1384,7 @@ func (c *cli) newGithubReviewRequest(
 ) *cloud.ReviewRequest {
 	author := cloud.Author{}
 	if user := pull.GetUser(); user != nil {
+		author.ID = strconv.Itoa64(int64(user.GetID()))
 		author.Login = user.GetLogin()
 		author.AvatarURL = user.GetAvatarURL()
 	}
@@ -1434,14 +1435,19 @@ func (c *cli) newGithubReviewRequest(
 			rr.ApprovedCount++
 		}
 
-		login := review.GetUser().GetLogin()
+		user := review.GetUser()
+		if user == nil {
+			continue
+		}
 
+		login := user.GetLogin()
 		if _, found := uniqueReviewers[login]; found {
 			continue
 		}
 		uniqueReviewers[login] = struct{}{}
 
 		rr.Reviewers = append(rr.Reviewers, cloud.Reviewer{
+			ID:        strconv.Itoa64(user.GetID()),
 			Login:     login,
 			AvatarURL: review.GetUser().GetAvatarURL(),
 		})
@@ -1487,6 +1493,7 @@ func (c *cli) newGitlabReviewRequest(mr gitlab.MR) *cloud.ReviewRequest {
 		UpdatedAt:   &mrUpdatedAt,
 		Status:      mr.State,
 		Author: cloud.Author{
+			ID:        strconv.Itoa64(int64(mr.Author.ID)),
 			Login:     mr.Author.Username,
 			AvatarURL: mr.Author.AvatarURL,
 		},
@@ -1500,6 +1507,9 @@ func (c *cli) newGitlabReviewRequest(mr gitlab.MR) *cloud.ReviewRequest {
 			Name: l,
 		})
 	}
+
+	// TODO(i4k): implement reviewers for Gitlab
+
 	return rr
 }
 

diff --git a/e2etests/cloud/run_cloud_sync_preview_test.go b/e2etests/cloud/run_cloud_sync_preview_test.go
@@ -157,6 +157,7 @@ func TestCLIRunWithCloudSyncPreview(t *testing.T) {
 						UpdatedAt:   updatedAt,
 						PushedAt:    &pushedAt,
 						Author: cloud.Author{
+							ID:        "1",
 							Login:     "octocat",
 							AvatarURL: "https://github.com/images/error/octocat_happy.gif",
 						},
@@ -254,6 +255,7 @@ func TestCLIRunWithCloudSyncPreview(t *testing.T) {
 						UpdatedAt:   updatedAt,
 						PushedAt:    &pushedAt,
 						Author: cloud.Author{
+							ID:        "1",
 							Login:     "octocat",
 							AvatarURL: "https://github.com/images/error/octocat_happy.gif",
 						},
@@ -316,13 +318,17 @@ func TestCLIRunWithCloudSyncPreview(t *testing.T) {
 						Description: "Please pull these awesome changes in!",
 						URL:         "https://github.com/octocat/Hello-World/pull/1347",
 						Labels:      []cloud.Label{{Name: "bug", Color: "f29513", Description: "Something isn't working"}},
-						Author:      cloud.Author{Login: "octocat", AvatarURL: "https://github.com/images/error/octocat_happy.gif"},
-						Status:      "open",
-						CreatedAt:   createdAt,
-						UpdatedAt:   updatedAt,
-						PushedAt:    &pushedAt,
-						Branch:      "new-topic",
-						BaseBranch:  "master",
+						Author: cloud.Author{
+							ID:        "1",
+							Login:     "octocat",
+							AvatarURL: "https://github.com/images/error/octocat_happy.gif",
+						},
+						Status:     "open",
+						CreatedAt:  createdAt,
+						UpdatedAt:  updatedAt,
+						PushedAt:   &pushedAt,
+						Branch:     "new-topic",
+						BaseBranch: "master",
 					},
 					StackPreviews: []*cloudstore.StackPreview{
 						{

diff --git a/e2etests/cloud/run_script_cloud_preview_test.go b/e2etests/cloud/run_script_cloud_preview_test.go
@@ -189,6 +189,7 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 						UpdatedAt:   updatedAt,
 						PushedAt:    &pushedAt,
 						Author: cloud.Author{
+							ID:        "1",
 							Login:     "octocat",
 							AvatarURL: "https://github.com/images/error/octocat_happy.gif",
 						},
@@ -295,6 +296,7 @@ func TestScriptRunWithCloudSyncPreview(t *testing.T) {
 						UpdatedAt:   updatedAt,
 						PushedAt:    &pushedAt,
 						Author: cloud.Author{
+							ID:        "1",
 							Login:     "octocat",
 							AvatarURL: "https://github.com/images/error/octocat_happy.gif",
 						},