diff options
| author | wxiaoguang <wxiaoguang@gmail.com> | 2024-04-20 11:15:04 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-04-20 03:15:04 +0000 |
| commit | 48d4580dd5e975de2e8207bb9b9a2f258711d38c (patch) | |
| tree | 4a2fbca0b792b37f09014952a7325bbb09e7ac97 /tests/integration/api_repo_test.go | |
| parent | 89e39872fff39797107acafb984dc2dc3ec3dd6a (diff) | |
| download | gitea-48d4580dd5e975de2e8207bb9b9a2f258711d38c.tar.gz gitea-48d4580dd5e975de2e8207bb9b9a2f258711d38c.zip | |
Clarify permission "HasAccess" behavior (#30585)
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>
Diffstat (limited to 'tests/integration/api_repo_test.go')
| -rw-r--r-- | tests/integration/api_repo_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/integration/api_repo_test.go b/tests/integration/api_repo_test.go index 481732f8df..bc2720d51e 100644 --- a/tests/integration/api_repo_test.go +++ b/tests/integration/api_repo_test.go @@ -222,7 +222,7 @@ func TestAPISearchRepo(t *testing.T) { assert.Len(t, repoNames, expected.count) for _, repo := range body.Data { r := getRepo(t, repo.ID) - hasAccess, err := access_model.HasAccess(db.DefaultContext, userID, r) + hasAccess, err := access_model.HasAnyUnitAccess(db.DefaultContext, userID, r) assert.NoError(t, err, "Error when checking if User: %d has access to %s: %v", userID, repo.FullName, err) assert.True(t, hasAccess, "User: %d does not have access to %s", userID, repo.FullName) |