improvements

Author: lunny

modules/gitrepo/fetch.go

@@ -10,8 +10,7 @@ import (
 )
 
 func FetchRemoteCommit(ctx context.Context, repo, remoteRepo Repository, commitID string) error {
-	_, err := RunCmdString(ctx, repo, gitcmd.NewCommand("fetch", "--no-tags").
+	return RunCmd(ctx, repo, gitcmd.NewCommand("fetch", "--no-tags").
 		AddDynamicArguments(repoPath(remoteRepo)).
 		AddDynamicArguments(commitID))
-	return err
 }

tests/integration/pull_comment_test.go

@@ -27,7 +27,7 @@ func testWaitForPullRequestStatus(t *testing.T, prIssue *issues_model.Issue, exp
 		retIssue = unittest.AssertExistsAndLoadBean(t, &prIssueCond)
 		require.NoError(t, retIssue.LoadPullRequest(t.Context()))
 		return retIssue.PullRequest.Status == expectedStatus
-	}, 10*time.Second, 20*time.Millisecond)
+	}, 5*time.Second, 20*time.Millisecond)
 	return retIssue
 }