diff --git a/models/pull_test.go b/models/pull_test.go
index 325818e0bf24..153739f5401c 100644
--- a/models/pull_test.go
+++ b/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 --git a/services/pull/check.go b/services/pull/check.go
index b1b950582318..c5d8a585cb31 100644
--- a/services/pull/check.go
+++ b/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)