Fix test

Author: lunny

services/repository/adopt_test.go

@@ -115,9 +115,9 @@ func TestAdoptRepository(t *testing.T) {
 	assert.Error(t, err)
 	assert.Nil(t, adoptedRepo)
 
-	unittest.AssertNotExistsBean(t, &repo_model.Repository{OwnerName: "user2", Name: "test-adopt"})
+	unittest.AssertNotExistsBean(t, &repo_model.Repository{OwnerName: user2.Name, Name: "test-adopt"})
 
-	exist, err := util.IsExist(repo_model.RepoPath("user2", "test-adopt"))
+	exist, err := util.IsExist(repo_model.RepoPath(user2.Name, "test-adopt"))
 	assert.NoError(t, err)
 	assert.True(t, exist) // the repository should be still in the disk
 }

services/repository/create.go

@@ -456,9 +456,9 @@ func createRepositoryInDB(ctx context.Context, doer, u *user_model.User, repo *r
 
 func cleanupRepository(doer *user_model.User, repoID int64) {
 	if errDelete := DeleteRepositoryDirectly(db.DefaultContext, doer, repoID); errDelete != nil {
-		log.Error("Rollback deleteRepository: %v", errDelete)
+		log.Error("cleanupRepository failed: %v", errDelete)
 		// add system notice
-		if err := system_model.CreateRepositoryNotice("DeleteRepositoryDirectly failed when create repository: %v", errDelete); err != nil {
+		if err := system_model.CreateRepositoryNotice("DeleteRepositoryDirectly failed when cleanup repository: %v", errDelete); err != nil {
 			log.Error("CreateRepositoryNotice: %v", err)
 		}
 	}

services/repository/fork_test.go

@@ -83,9 +83,9 @@ func TestForkRepositoryCleanup(t *testing.T) {
 	assert.Error(t, err)
 
 	// assert the cleanup is successful
-	unittest.AssertNotExistsBean(t, &repo_model.Repository{OwnerName: "test", Name: "test"})
+	unittest.AssertNotExistsBean(t, &repo_model.Repository{OwnerName: user2.Name, Name: "test"})
 
-	exist, err = util.IsExist(repo_model.RepoPath("test", "test"))
+	exist, err = util.IsExist(repo_model.RepoPath(user2.Name, "test"))
 	assert.NoError(t, err)
 	assert.False(t, exist)
 }