Przeglądaj źródła

add test for duplicate multiple pages with GRANT_RESTRICTED

yohei0125 4 lat temu
rodzic
commit
3642aef433

+ 64 - 1
packages/app/test/integration/service/v5.non-public-page.test.ts

@@ -60,11 +60,14 @@ describe('PageService page operations with non-public pages', () => {
   const pageIdDuplicate1 = new mongoose.Types.ObjectId();
   const pageIdDuplicate2 = new mongoose.Types.ObjectId();
   const pageIdDuplicate3 = new mongoose.Types.ObjectId();
+  const pageIdDuplicate4 = new mongoose.Types.ObjectId();
+  const pageIdDuplicate5 = new mongoose.Types.ObjectId();
   // revision id
   const revisionIdDuplicate1 = new mongoose.Types.ObjectId();
   const revisionIdDuplicate2 = new mongoose.Types.ObjectId();
   const revisionIdDuplicate3 = new mongoose.Types.ObjectId();
-
+  const revisionIdDuplicate4 = new mongoose.Types.ObjectId();
+  const revisionIdDuplicate5 = new mongoose.Types.ObjectId();
 
   /**
    * Revert
@@ -320,6 +323,24 @@ describe('PageService page operations with non-public pages', () => {
         revision: revisionIdDuplicate3,
         parent: pageIdDuplicate2,
       },
+      {
+        _id: pageIdDuplicate4,
+        path: '/np_duplicate4',
+        grant: Page.GRANT_USER_GROUP,
+        grantedGroup: groupIdA,
+        creator: npDummyUser1._id,
+        lastUpdateUser: npDummyUser1._id,
+        revision: revisionIdDuplicate4,
+        parent: rootPage._id,
+      },
+      {
+        _id: pageIdDuplicate5,
+        path: '/np_duplicate4/np_duplicate5',
+        grant: Page.GRANT_RESTRICTED,
+        creator: npDummyUser1._id,
+        lastUpdateUser: npDummyUser1._id,
+        revision: revisionIdDuplicate5,
+      },
     ]);
     await Revision.insertMany([
       {
@@ -343,6 +364,20 @@ describe('PageService page operations with non-public pages', () => {
         pageId: pageIdDuplicate3,
         author: npDummyUser2._id,
       },
+      {
+        _id: revisionIdDuplicate4,
+        body: 'np_duplicate4',
+        format: 'markdown',
+        pageId: pageIdDuplicate4,
+        author: npDummyUser2._id,
+      },
+      {
+        _id: revisionIdDuplicate5,
+        body: 'np_duplicate5',
+        format: 'markdown',
+        pageId: pageIdDuplicate5,
+        author: npDummyUser2._id,
+      },
     ]);
 
     /**
@@ -627,6 +662,7 @@ describe('PageService page operations with non-public pages', () => {
 
       expect(duplicatedRevision.body).toBe(baseRevision.body);
     });
+
     test('Should duplicate multiple pages with GRANT_USER_GROUP', async() => {
       const basePath1 = '/np_duplicate2';
       const basePath2 = '/np_duplicate2/np_duplicate3';
@@ -658,6 +694,33 @@ describe('PageService page operations with non-public pages', () => {
       expect(duplicatedRevision1.pageId).toStrictEqual(duplicatedPage1._id);
       expect(duplicatedRevision2.pageId).toStrictEqual(duplicatedPage2._id);
     });
+    test('Should duplicate multiple pages with GRANT_RESTRICTED', async() => {
+      const basePath1 = '/np_duplicate4';
+      const basePath2 = '/np_duplicate4/np_duplicate5';
+      const basePage1 = await Page.findOne({ path: basePath1, parent: rootPage._id })
+        .populate({ path: 'revision', model: 'Revision', grantedPage: groupIdA._id });
+      const basePage2 = await Page.findOne({ path: basePath2 }).populate({ path: 'revision', model: 'Revision', grantedPage: groupIdB._id });
+      const baseRevision1 = basePage1.revision;
+      const baseRevision2 = basePage2.revision;
+      expectAllToBeTruthy([basePage1, basePage2, baseRevision1, baseRevision2]);
+
+      const newPagePath = '/dup_np_duplicate4';
+      await duplicate(basePage1, newPagePath, npDummyUser1, true);
+
+      const duplicatedPage1 = await Page.findOne({ path: newPagePath }).populate({ path: 'revision', model: 'Revision' });
+      const duplicatedPage2 = await Page.findOne({ path: '/dup_np_duplicate4/np_duplicate5' }).populate({ path: 'revision', model: 'Revision' });
+      const duplicatedRevision1 = duplicatedPage1.revision;
+      // AD => After Duplicate
+      expectAllToBeTruthy([duplicatedPage1, duplicatedRevision1]);
+      expect(duplicatedPage2).toBeNull();
+
+      expect(xssSpy).toHaveBeenCalled();
+      expect(duplicatedPage1.grantedGroup).toStrictEqual(groupIdA._id);
+      expect(duplicatedPage1.parent).toStrictEqual(basePage1.parent);
+
+      expect(duplicatedRevision1.body).toBe(baseRevision1.body);
+      expect(duplicatedRevision1.pageId).toStrictEqual(duplicatedPage1._id);
+    });
 
   });
   describe('Delete', () => {