| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283 |
- import mongoose from 'mongoose';
- import { getInstance } from '../setup-crowi';
- describe('UserGroupService', () => {
- let crowi;
- let User;
- let UserGroup;
- let UserGroupRelation;
- const groupId1 = new mongoose.Types.ObjectId();
- const groupId2 = new mongoose.Types.ObjectId();
- const groupId3 = new mongoose.Types.ObjectId();
- const groupId4 = new mongoose.Types.ObjectId();
- const groupId5 = new mongoose.Types.ObjectId();
- const groupId6 = new mongoose.Types.ObjectId();
- const groupId7 = new mongoose.Types.ObjectId();
- const groupId8 = new mongoose.Types.ObjectId();
- const groupId9 = new mongoose.Types.ObjectId();
- const groupId10 = new mongoose.Types.ObjectId();
- const groupId11 = new mongoose.Types.ObjectId();
- const groupId12 = new mongoose.Types.ObjectId();
- const userId1 = new mongoose.Types.ObjectId();
- beforeAll(async() => {
- crowi = await getInstance();
- User = mongoose.model('User');
- UserGroup = mongoose.model('UserGroup');
- UserGroupRelation = mongoose.model('UserGroupRelation');
- await User.insertMany([
- // ug -> User Group
- {
- _id: userId1, name: 'ug_test_user1', username: 'ug_test_user1', email: 'ug_test_user1@example.com',
- },
- ]);
- // Create Groups
- await UserGroup.insertMany([
- // No parent
- {
- _id: groupId1,
- name: 'v5_group1',
- description: 'description1',
- },
- // No parent
- {
- _id: groupId2,
- name: 'v5_group2',
- description: 'description2',
- },
- // No parent
- {
- _id: groupId3,
- name: 'v5_group3',
- description: 'description3',
- },
- // No parent
- {
- _id: groupId4,
- name: 'v5_group4',
- description: 'description4',
- },
- // No parent
- {
- _id: groupId5,
- name: 'v5_group5',
- description: 'description5',
- },
- // No parent
- {
- _id: groupId6,
- name: 'v5_group6',
- description: 'description6',
- },
- // No parent
- {
- _id: groupId7,
- name: 'v5_group7',
- description: 'description7',
- parent: groupId6,
- },
- // No parent
- {
- _id: groupId8,
- name: 'v5_group8',
- description: 'description8',
- },
- {
- _id: groupId9,
- name: 'v5_group9',
- description: 'description9',
- },
- {
- _id: groupId10,
- name: 'v5_group10',
- description: 'description10',
- parent: groupId9,
- },
- {
- _id: groupId11,
- name: 'v5_group11',
- description: 'descriptio11',
- },
- {
- _id: groupId12,
- name: 'v5_group12',
- description: 'description12',
- parent: groupId11,
- },
- ]);
- // Create UserGroupRelations
- await UserGroupRelation.insertMany([
- {
- relatedGroup: groupId4,
- relatedUser: userId1,
- },
- {
- relatedGroup: groupId6,
- relatedUser: userId1,
- },
- {
- relatedGroup: groupId8,
- relatedUser: userId1,
- },
- {
- relatedGroup: groupId9,
- relatedUser: userId1,
- },
- {
- relatedGroup: groupId10,
- relatedUser: userId1,
- },
- {
- relatedGroup: groupId11,
- relatedUser: userId1,
- },
- {
- relatedGroup: groupId12,
- relatedUser: userId1,
- },
- ]);
- });
- /*
- * Update UserGroup
- */
- test('Updated values should be reflected. (name, description, parent)', async() => {
- const userGroup2 = await UserGroup.findOne({ _id: groupId2 });
- const newGroupName = 'v5_group1_new';
- const newGroupDescription = 'description1_new';
- const newParentId = userGroup2._id;
- const updatedUserGroup = await crowi.userGroupService.updateGroup(groupId1, newGroupName, newGroupDescription, newParentId);
- expect(updatedUserGroup.name).toBe(newGroupName);
- expect(updatedUserGroup.description).toBe(newGroupDescription);
- expect(updatedUserGroup.parent).toStrictEqual(newParentId);
- });
- test('Should throw an error when trying to set existing group name', async() => {
- const userGroup2 = await UserGroup.findOne({ _id: groupId2 });
- const result = crowi.userGroupService.updateGroup(groupId1, userGroup2.name);
- await expect(result).rejects.toThrow('The group name is already taken');
- });
- test('Parent should be null when parent group is released', async() => {
- const userGroup = await UserGroup.findOne({ _id: groupId3 });
- const updatedUserGroup = await crowi.userGroupService.updateGroup(userGroup._id, userGroup.name, userGroup.description, null);
- expect(updatedUserGroup.parent).toBeNull();
- });
- /*
- * forceUpdateParents: false
- */
- test('Should throw an error when users in child group do not exist in parent group', async() => {
- const userGroup4 = await UserGroup.findOne({ _id: groupId4, parent: null });
- const result = crowi.userGroupService.updateGroup(userGroup4._id, userGroup4.name, userGroup4.description, groupId5);
- await expect(result).rejects.toThrow('The parent group does not contain the users in this group.');
- });
- /*
- * forceUpdateParents: true
- */
- test('User should be included to parent group (2 groups ver)', async() => {
- const userGroup4 = await UserGroup.findOne({ _id: groupId4, parent: null });
- const userGroup5 = await UserGroup.findOne({ _id: groupId5, parent: null });
- // userGroup4 has userId1
- const userGroupRelation4BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: userGroup4, relatedUser: userId1 });
- expect(userGroupRelation4BeforeUpdate).not.toBeNull();
- // userGroup5 has not userId1
- const userGroupRelation5BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: userGroup5, relatedUser: userId1 });
- expect(userGroupRelation5BeforeUpdate).toBeNull();
- // update userGroup4's parent with userGroup5 (forceUpdate: true)
- const forceUpdateParents = true;
- const updatedUserGroup = await crowi.userGroupService.updateGroup(
- userGroup4._id, userGroup4.name, userGroup4.description, groupId5, forceUpdateParents,
- );
- expect(updatedUserGroup.parent).toStrictEqual(groupId5);
- // userGroup5 should have userId1
- const userGroupRelation5AfterUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId5, relatedUser: userGroupRelation4BeforeUpdate.relatedUser });
- expect(userGroupRelation5AfterUpdate).not.toBeNull();
- });
- test('User should be included to parent group (3 groups ver)', async() => {
- const userGroup8 = await UserGroup.findOne({ _id: groupId8, parent: null });
- // userGroup7 has not userId1
- const userGroupRelation6BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId6, relatedUser: userId1 });
- const userGroupRelation7BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId7, relatedUser: userId1 });
- const userGroupRelation8BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId8, relatedUser: userId1 });
- expect(userGroupRelation6BeforeUpdate).not.toBeNull();
- // userGroup7 does not have userId1
- expect(userGroupRelation7BeforeUpdate).toBeNull();
- expect(userGroupRelation8BeforeUpdate).not.toBeNull();
- // update userGroup8's parent with userGroup7 (forceUpdate: true)
- const forceUpdateParents = true;
- await crowi.userGroupService.updateGroup(
- userGroup8._id, userGroup8.name, userGroup8.description, groupId7, forceUpdateParents,
- );
- const userGroupRelation6AfterUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId6, relatedUser: userId1 });
- const userGroupRelation7AfterUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId7, relatedUser: userId1 });
- const userGroupRelation8AfterUpdate = await UserGroupRelation.findOne({ relatedGroup: groupId8, relatedUser: userId1 });
- expect(userGroupRelation6AfterUpdate).not.toBeNull();
- // userGroup7 should have userId1
- expect(userGroupRelation7AfterUpdate).not.toBeNull();
- expect(userGroupRelation8AfterUpdate).not.toBeNull();
- });
- test('Should throw an error when trying to choose parent from descendant groups.', async() => {
- const userGroup9 = await UserGroup.findOne({ _id: groupId9, parent: null });
- const userGroup10 = await UserGroup.findOne({ _id: groupId10, parent: groupId9 });
- const userGroupRelation9BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: userGroup9._id, relatedUser: userId1 });
- const userGroupRelation10BeforeUpdate = await UserGroupRelation.findOne({ relatedGroup: userGroup10._id, relatedUser: userId1 });
- expect(userGroupRelation9BeforeUpdate).not.toBeNull();
- expect(userGroupRelation10BeforeUpdate).not.toBeNull();
- const result = crowi.userGroupService.updateGroup(
- userGroup9._id, userGroup9.name, userGroup9.description, userGroup10._id,
- );
- await expect(result).rejects.toThrow('It is not allowed to choose parent from descendant groups.');
- });
- test('User should be deleted from child groups when the user excluded from the parent group', async() => {
- const userGroup11 = await UserGroup.findOne({ _id: groupId11, parent: null });
- const userGroup12 = await UserGroup.findOne({ _id: groupId12, parent: groupId11 });
- // Both groups have user1
- const userGroupRelation11BeforeRemove = await UserGroupRelation.findOne({ relatedGroup: userGroup11._id, relatedUser: userId1 });
- const userGroupRelation12BeforeRemove = await UserGroupRelation.findOne({ relatedGroup: userGroup12._id, relatedUser: userId1 });
- expect(userGroupRelation11BeforeRemove).not.toBeNull();
- expect(userGroupRelation12BeforeRemove).not.toBeNull();
- // remove user1 from the parent group
- await crowi.userGroupService.removeUserByUsername(
- userGroup11._id, 'ug_test_user1',
- );
- // Both groups have not user1
- const userGroupRelation11AfterRemove = await UserGroupRelation.findOne({ relatedGroup: userGroup11._id, relatedUser: userId1 });
- const userGroupRelation12AfterRemove = await UserGroupRelation.findOne({ relatedGroup: userGroup12._id, relatedUser: userId1 });
- await expect(userGroupRelation11AfterRemove).toBeNull();
- await expect(userGroupRelation12AfterRemove).toBeNull();
- });
- });
|