-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DS-4486] Tests for the Comm/Coll admins group edits #2770
[DS-4486] Tests for the Comm/Coll admins group edits #2770
Conversation
… regards to the Admin groups
} | ||
|
||
@Test | ||
public void communityAdminAddMembersToCommunityAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 2108 in 34b61f3
public void commAdminManageOwnerAdminGroupTest() throws Exception { |
} | ||
|
||
@Test | ||
public void communityAdminRemoveMembersFromCommunityAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 2108 in 34b61f3
public void commAdminManageOwnerAdminGroupTest() throws Exception { |
} | ||
|
||
@Test | ||
public void communityAdminAddChildGroupToCommunityAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 626 in 34b61f3
public void addChildGroupCommunityAdminTest() throws Exception { |
} | ||
|
||
@Test | ||
public void communityAdminRemoveChildGroupFromCommunityAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 1236 in 34b61f3
public void removeChildGroupCommunityAdminTest() throws Exception { |
|
||
|
||
@Test | ||
public void communityAdminAddMembersToCollectionAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will rename my test to communityAdminCanManageCollectionSubmittersGroupAndAdminsGroupsTest and add
this test inside my test
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 2323 in 34b61f3
public void commAdminManageSunCollectionOfSubmittersAndAdminsTest() throws Exception { |
} | ||
|
||
@Test | ||
public void communityAdminRemoveMembersFromCollectionAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 2323 in 34b61f3
public void commAdminManageSunCollectionOfSubmittersAndAdminsTest() throws Exception { |
|
||
|
||
@Test | ||
public void collectionAdminAddMembersToCollectionAdminGroupSuccess() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DSpace/dspace-server-webapp/src/test/java/org/dspace/app/rest/GroupRestRepositoryIT.java
Line 2159 in 34b61f3
public void colAdminManageSubmitterGroupAndAdminGroupTest() throws Exception { |
hi @KevinVdV |
Closing this PR as it has be integrated into #2774. |
References
Description
This PR includes tests for the edits of the community / collection roles by community / collection admin.
This PR does NOT include a fix where community/collection admins can search for groups/epeople to add.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!
400 Bad Request
,401 Unauthorized
,403 Forbidden
,404 Not Found
, etc)pom.xml
), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.