add UserGroupRelation

packages/app/test/integration/service/v5.user-groups.test.ts

@@ -6,10 +6,18 @@ import { getInstance } from '../setup-crowi';
 describe('UserGroupService', () => {
   let crowi;
   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 userGroupRelationId1 = new mongoose.Types.ObjectId();
+  const userGroupRelationId4 = new mongoose.Types.ObjectId();
+
+  const userId1 = new mongoose.Types.ObjectId();
+  const userId4 = new mongoose.Types.ObjectId();
 
 
   beforeAll(async() => {
@@ -17,7 +25,7 @@ describe('UserGroupService', () => {
     await crowi.configManager.updateConfigsInTheSameNamespace('crowi', { 'app:isV5Compatible': true });
 
     UserGroup = mongoose.model('UserGroup');
-
+    UserGroupRelation = mongoose.model('UserGroupRelation');
 
     // Create Groups
     await UserGroup.insertMany([
@@ -33,26 +41,56 @@ describe('UserGroupService', () => {
         name: 'v5_group2',
         description: 'description2',
       },
+      // No parent
       {
         _id: groupId3,
         name: 'v5_group3',
-        parent: groupId1,
         description: 'description3',
       },
+      {
+        _id: groupId4,
+        name: 'v5_group4',
+        parent: groupId4,
+        description: 'description4',
+      },
     ]);
+
+    // Create UserGroupRelations
+    await UserGroupRelation.insertMany([
+      // Parent
+      {
+        _id: userGroupRelationId1,
+        relatedGroup: groupId1,
+        relatedUser: userId1,
+      },
+      {
+        _id: userGroupRelationId4,
+        relatedGroup: groupId4,
+        relatedUser: userId4,
+      },
+      // Child
+      // {
+      //   _id: userGroupRelationId2,
+      //   relatedGroup: groupId2,
+      //   relatedUser: userId1,
+      // },
+    ]);
+
+
   });
 
   /*
     * Update UserGroup
     */
   test('Updated values should be reflected. (name, description, parent)', async() => {
-    const userGroup = await UserGroup.findOne({ _id: groupId1 });
+    const userGroup1 = await UserGroup.findOne({ _id: groupId1 });
+    const userGroup2 = await UserGroup.findOne({ _id: groupId2 });
 
-    const newGroupName = 'v5_group1_new';
-    const newGroupDescription = 'description1_new';
-    const newParentId = groupId2;
+    const newGroupName = 'v5_group2_new';
+    const newGroupDescription = 'description2_new';
+    const newParentId = userGroup1._id;
 
-    const updatedUserGroup = await crowi.userGroupService.updateGroup(userGroup._id, newGroupName, newGroupDescription, newParentId);
+    const updatedUserGroup = await crowi.userGroupService.updateGroup(userGroup2._id, newGroupName, newGroupDescription, newParentId);
 
     expect(updatedUserGroup.name).toBe(newGroupName);
     expect(updatedUserGroup.description).toBe(newGroupDescription);
@@ -68,6 +106,15 @@ describe('UserGroupService', () => {
     await expect(result).rejects.toThrow('The group name is already taken');
   });
 
+  test('Should throw an error when users in clild group do not include in users in parent group', async() => {
+    const userGroup1 = await UserGroup.findOne({ _id: groupId1 });
+    const userGroup4 = await UserGroup.findOne({ _id: groupId4 });
+
+    const result = crowi.userGroupService.updateGroup(userGroup1._id, userGroup1.name, userGroup1.description, userGroup4._id);
+
+    await expect(result).rejects.toThrow('The parent group does not contain the users in this group.');
+  });
+
   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);