enable fix

server/store/datastore/repo.go

@@ -99,11 +99,17 @@ func (s storage) RepoBatch(repos []*model.Repo) error {
 		if exist {
 			if _, err := sess.
 				Where("repo_owner = ? AND repo_name = ?", repos[i].Owner, repos[i].Name).
-				// Cols("repo_avatar", "repo_link", "repo_private", "repo_clone", "repo_branch"). // FIX
-				AllCols().
+				Cols("repo_scm", "repo_avatar", "repo_link", "repo_private", "repo_clone", "repo_branch").
 				Update(repos[i]); err != nil {
 				return err
 			}
+
+			_, err := sess.
+				Where("repo_owner = ? AND repo_name = ?", repos[i].Owner, repos[i].Name).
+				Get(repos[i])
+			if err != nil {
+				return err
+			}
 		} else {
 			// only Insert on single object ref set auto created ID back to object
 			if _, err := sess.Insert(repos[i]); err != nil {

server/store/datastore/repo_test.go

@@ -337,11 +337,12 @@ func TestRepoBatch(t *testing.T) {
 	}
 	assert.NoError(t, store.RepoBatch([]*model.Repo{repo}))
 	assert.EqualValues(t, repos[0].ID, repo.ID)
+	_, err := store.engine.ID(repo.ID).Get(repo)
+	assert.NoError(t, err)
 	assert.True(t, repo.IsActive)
 
 	allRepos := make([]*model.Repo, 0, 4)
-	err := store.engine.Find(&allRepos)
-	assert.NoError(t, err)
+	assert.NoError(t, store.engine.Find(&allRepos))
 	assert.Len(t, allRepos, 4)
 
 	count, err := store.GetRepoCount()