Fix lint (#9688)

Author: lunny

Diff for: models/pull_test.go

@@ -182,7 +182,7 @@ func TestPullRequest_UpdateCols(t *testing.T) {
 		BaseBranch: "baseBranch",
 		HeadBranch: "headBranch",
 	}
-	pr.UpdateCols("head_branch")
+	assert.NoError(t, pr.UpdateCols("head_branch"))
 
 	pr = AssertExistsAndLoadBean(t, &PullRequest{ID: 1}).(*PullRequest)
 	assert.Equal(t, "master", pr.BaseBranch)

Diff for: services/pull/check.go

@@ -201,7 +201,9 @@ func TestPullRequests(ctx context.Context) {
 			} else if err = TestPatch(pr); err != nil {
 				log.Error("testPatch[%d]: %v", pr.ID, err)
 				pr.Status = models.PullRequestStatusError
-				pr.UpdateCols("status")
+				if err := pr.UpdateCols("status"); err != nil {
+					log.Error("update pr [%d] status to PullRequestStatusError failed: %v", pr.ID, err)
+				}
 				continue
 			}
 			checkAndUpdateStatus(pr)