Merge pull request #35 from marqeta/add-delay-between-auto-approval

Bump the delay in PR Bot

Author: JahonCodes

pullrequest/review/reviewer.go

@@ -45,7 +45,7 @@ func (r *reviewer) Approve(ctx context.Context, id id.PR, body string, opts Appr
 		return r.handleAutoMergeError(ctx, id, err)
 	}
 	oplog.Info().Msgf("enabled auto merge on PR")
-	time.Sleep(500 * time.Millisecond)
+	time.Sleep(1 * time.Second)
 	err = r.api.AddReview(ctx, id, body, gh.Approve)
 	if err != nil {
 		oplog.Err(err).Msgf("error approving PR")

pullrequest/review/reviewer_test.go

@@ -194,12 +194,12 @@ func Test_reviewer_Approve_Sleep(t *testing.T) {
 	})
 	duration := time.Since(start)
 
-	// Verify no error and sleep duration is at least 500ms
+	// Verify no error and sleep duration is at least 1 second
 	if err != nil {
 		t.Errorf("Approve() error = %v, want nil", err)
 	}
-	if duration < 500*time.Millisecond {
-		t.Errorf("Sleep duration = %v, want at least 500ms", duration)
+	if duration < 1*time.Second {
+		t.Errorf("Sleep duration = %v, want at least 1s", duration)
 	}
 }