mirror of
https://github.com/haiwen/seahub.git
synced 2025-09-01 23:20:51 +00:00
update transfer repo (#6965)
* update * optimize code * optimize * update sql * test * update * optimize * update UI * change style --------- Co-authored-by: 孙永强 <11704063+s-yongqiang@user.noreply.gitee.com> Co-authored-by: Michael An <2331806369@qq.com>
This commit is contained in:
@@ -57,29 +57,6 @@ class RepoOwnerTest(BaseTestCase):
|
||||
resp = self.client.put(url, data, 'application/x-www-form-urlencoded')
|
||||
self.assertEqual(200, resp.status_code)
|
||||
|
||||
def test_reshare_to_user_after_transfer_repo(self):
|
||||
|
||||
tmp_user = 'tmp_user@email.com'
|
||||
self.create_user(tmp_user)
|
||||
|
||||
# share user's repo to tmp_user with 'rw' permission
|
||||
seafile_api.share_repo(self.user_repo_id, self.user.username,
|
||||
tmp_user, 'rw')
|
||||
|
||||
assert seafile_api.check_permission_by_path(self.user_repo_id,
|
||||
'/', tmp_user) == 'rw'
|
||||
|
||||
self.login_as(self.user)
|
||||
|
||||
url = reverse("api2-repo-owner", args=[self.user_repo_id])
|
||||
data = 'owner=%s' % self.admin.email
|
||||
|
||||
resp = self.client.put(url, data, 'application/x-www-form-urlencoded')
|
||||
self.assertEqual(200, resp.status_code)
|
||||
|
||||
assert seafile_api.check_permission_by_path(self.user_repo_id,
|
||||
'/', tmp_user) == 'rw'
|
||||
|
||||
def test_not_reshare_to_user_after_transfer_repo(self):
|
||||
|
||||
# remove all share
|
||||
@@ -111,88 +88,6 @@ class RepoOwnerTest(BaseTestCase):
|
||||
shared_repos = seafile_api.get_share_in_repo_list(self.admin.username, -1, -1)
|
||||
assert len(shared_repos) == 0
|
||||
|
||||
def test_reshare_to_group_after_transfer_repo(self):
|
||||
# If new owner in group repo shared to, reshare to group
|
||||
|
||||
# share user's repo to group with 'r' permission
|
||||
seafile_api.set_group_repo(self.user_repo_id, self.group_id,
|
||||
self.user_name, 'r')
|
||||
|
||||
group_repos = seafile_api.get_repos_by_group(self.group_id)
|
||||
assert group_repos[0].permission == 'r'
|
||||
|
||||
# add admin user to group
|
||||
ccnet_api.group_add_member(self.group_id, self.user_name, self.admin.username)
|
||||
|
||||
self.login_as(self.user)
|
||||
|
||||
url = reverse("api2-repo-owner", args=[self.user_repo_id])
|
||||
data = 'owner=%s' % self.admin.email
|
||||
|
||||
# transfer repo to admin
|
||||
resp = self.client.put(url, data, 'application/x-www-form-urlencoded')
|
||||
self.assertEqual(200, resp.status_code)
|
||||
|
||||
group_repos = seafile_api.get_repos_by_group(self.group_id)
|
||||
assert group_repos[0].permission == 'r'
|
||||
|
||||
def test_not_reshare_to_group_after_transfer_repo(self):
|
||||
# If new owner NOT in group repo shared to, NOT reshare to group
|
||||
|
||||
# share user's repo to group with 'r' permission
|
||||
seafile_api.set_group_repo(self.user_repo_id, self.group_id,
|
||||
self.user_name, 'r')
|
||||
|
||||
group_repos = seafile_api.get_repos_by_group(self.group_id)
|
||||
assert group_repos[0].permission == 'r'
|
||||
|
||||
self.login_as(self.user)
|
||||
|
||||
url = reverse("api2-repo-owner", args=[self.user_repo_id])
|
||||
data = 'owner=%s' % self.admin.email
|
||||
|
||||
# transfer repo to admin
|
||||
resp = self.client.put(url, data, 'application/x-www-form-urlencoded')
|
||||
self.assertEqual(200, resp.status_code)
|
||||
|
||||
group_repos = seafile_api.get_repos_by_group(self.group_id)
|
||||
assert len(group_repos) == 0
|
||||
|
||||
def test_reshare_to_user_group_after_transfer_repo(self):
|
||||
|
||||
tmp_user = 'tmp_user@email.com'
|
||||
self.create_user(tmp_user)
|
||||
|
||||
# add admin user to group
|
||||
ccnet_api.group_add_member(self.group_id, self.user_name, self.admin.username)
|
||||
|
||||
# share user's repo to tmp_user with 'rw' permission
|
||||
seafile_api.share_repo(self.user_repo_id, self.user.username,
|
||||
tmp_user, 'rw')
|
||||
|
||||
# share user's repo to group with 'r' permission
|
||||
seafile_api.set_group_repo(self.user_repo_id, self.group_id,
|
||||
self.user_name, 'r')
|
||||
group_repos = seafile_api.get_repos_by_group(self.group_id)
|
||||
|
||||
assert group_repos[0].permission == 'r'
|
||||
assert seafile_api.check_permission_by_path(self.user_repo_id,
|
||||
'/', tmp_user) == 'rw'
|
||||
|
||||
self.login_as(self.user)
|
||||
|
||||
url = reverse("api2-repo-owner", args=[self.user_repo_id])
|
||||
data = 'owner=%s' % self.admin.email
|
||||
|
||||
# transfer repo to admin
|
||||
resp = self.client.put(url, data, 'application/x-www-form-urlencoded')
|
||||
self.assertEqual(200, resp.status_code)
|
||||
|
||||
group_repos = seafile_api.get_repos_by_group(self.group_id)
|
||||
assert group_repos[0].permission == 'r'
|
||||
assert seafile_api.check_permission_by_path(self.user_repo_id,
|
||||
'/', tmp_user) == 'rw'
|
||||
|
||||
def test_can_not_transfer_if_not_repo_owner(self):
|
||||
self.login_as(self.admin)
|
||||
|
||||
|
Reference in New Issue
Block a user