forked from Shiloh/githaven
48d4580dd5
Follow #30495 "HasAccess" behavior wasn't clear, to make it clear: * Use a new name `HasAnyUnitAccess`, it will be easier to review related code and permission problems. * Separate everyone access mode to a separate field, then all calls to HasAccess are reverted to old behavior before #30495. * Add new tests. --------- Co-authored-by: Giteabot <teabot@gitea.io>
124 lines
4.3 KiB
Go
124 lines
4.3 KiB
Go
// Copyright 2019 The Gitea Authors. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package repository
|
|
|
|
import (
|
|
"sync"
|
|
"testing"
|
|
|
|
"code.gitea.io/gitea/models"
|
|
activities_model "code.gitea.io/gitea/models/activities"
|
|
"code.gitea.io/gitea/models/db"
|
|
"code.gitea.io/gitea/models/organization"
|
|
access_model "code.gitea.io/gitea/models/perm/access"
|
|
repo_model "code.gitea.io/gitea/models/repo"
|
|
"code.gitea.io/gitea/models/unittest"
|
|
user_model "code.gitea.io/gitea/models/user"
|
|
"code.gitea.io/gitea/modules/util"
|
|
"code.gitea.io/gitea/services/feed"
|
|
notify_service "code.gitea.io/gitea/services/notify"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
var notifySync sync.Once
|
|
|
|
func registerNotifier() {
|
|
notifySync.Do(func() {
|
|
notify_service.RegisterNotifier(feed.NewNotifier())
|
|
})
|
|
}
|
|
|
|
func TestTransferOwnership(t *testing.T) {
|
|
registerNotifier()
|
|
|
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
|
|
|
doer := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 3})
|
|
repo.Owner = unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repo.OwnerID})
|
|
assert.NoError(t, TransferOwnership(db.DefaultContext, doer, doer, repo, nil))
|
|
|
|
transferredRepo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 3})
|
|
assert.EqualValues(t, 2, transferredRepo.OwnerID)
|
|
|
|
exist, err := util.IsExist(repo_model.RepoPath("org3", "repo3"))
|
|
assert.NoError(t, err)
|
|
assert.False(t, exist)
|
|
exist, err = util.IsExist(repo_model.RepoPath("user2", "repo3"))
|
|
assert.NoError(t, err)
|
|
assert.True(t, exist)
|
|
unittest.AssertExistsAndLoadBean(t, &activities_model.Action{
|
|
OpType: activities_model.ActionTransferRepo,
|
|
ActUserID: 2,
|
|
RepoID: 3,
|
|
Content: "org3/repo3",
|
|
})
|
|
|
|
unittest.CheckConsistencyFor(t, &repo_model.Repository{}, &user_model.User{}, &organization.Team{})
|
|
}
|
|
|
|
func TestStartRepositoryTransferSetPermission(t *testing.T) {
|
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
|
|
|
doer := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 3})
|
|
recipient := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 5})
|
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 3})
|
|
repo.Owner = unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repo.OwnerID})
|
|
|
|
hasAccess, err := access_model.HasAnyUnitAccess(db.DefaultContext, recipient.ID, repo)
|
|
assert.NoError(t, err)
|
|
assert.False(t, hasAccess)
|
|
|
|
assert.NoError(t, StartRepositoryTransfer(db.DefaultContext, doer, recipient, repo, nil))
|
|
|
|
hasAccess, err = access_model.HasAnyUnitAccess(db.DefaultContext, recipient.ID, repo)
|
|
assert.NoError(t, err)
|
|
assert.True(t, hasAccess)
|
|
|
|
unittest.CheckConsistencyFor(t, &repo_model.Repository{}, &user_model.User{}, &organization.Team{})
|
|
}
|
|
|
|
func TestRepositoryTransfer(t *testing.T) {
|
|
assert.NoError(t, unittest.PrepareTestDatabase())
|
|
|
|
doer := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 3})
|
|
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 3})
|
|
|
|
transfer, err := models.GetPendingRepositoryTransfer(db.DefaultContext, repo)
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, transfer)
|
|
|
|
// Cancel transfer
|
|
assert.NoError(t, CancelRepositoryTransfer(db.DefaultContext, repo))
|
|
|
|
transfer, err = models.GetPendingRepositoryTransfer(db.DefaultContext, repo)
|
|
assert.Error(t, err)
|
|
assert.Nil(t, transfer)
|
|
assert.True(t, models.IsErrNoPendingTransfer(err))
|
|
|
|
user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
|
|
|
assert.NoError(t, models.CreatePendingRepositoryTransfer(db.DefaultContext, doer, user2, repo.ID, nil))
|
|
|
|
transfer, err = models.GetPendingRepositoryTransfer(db.DefaultContext, repo)
|
|
assert.Nil(t, err)
|
|
assert.NoError(t, transfer.LoadAttributes(db.DefaultContext))
|
|
assert.Equal(t, "user2", transfer.Recipient.Name)
|
|
|
|
org6 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
|
|
|
|
// Only transfer can be started at any given time
|
|
err = models.CreatePendingRepositoryTransfer(db.DefaultContext, doer, org6, repo.ID, nil)
|
|
assert.Error(t, err)
|
|
assert.True(t, models.IsErrRepoTransferInProgress(err))
|
|
|
|
// Unknown user
|
|
err = models.CreatePendingRepositoryTransfer(db.DefaultContext, doer, &user_model.User{ID: 1000, LowerName: "user1000"}, repo.ID, nil)
|
|
assert.Error(t, err)
|
|
|
|
// Cancel transfer
|
|
assert.NoError(t, CancelRepositoryTransfer(db.DefaultContext, repo))
|
|
}
|