Merge branch 'imprv/create-duplicate-page-test' of https://github.com/weseek/growi into imprv/create-duplicate-page-test

src/test/service/page.test.js

@@ -281,7 +281,10 @@ describe('PageService', () => {
   describe('duplicate page', () => {
     let xssSpy;
     let duplicateDescendantsWithStreamSpy;
-    const serializePageSecurely = require('../../server/models/serializers/page-serializer');
+
+    jest.mock('../../server/models/serializers/page-serializer');
+    const { serializePageSecurely } = require('../../server/models/serializers/page-serializer');
+    serializePageSecurely.mockImplementation(page => page);
 
     beforeEach(async(done) => {
       xssSpy = jest.spyOn(crowi.xss, 'process').mockImplementation(path => path);
@@ -292,7 +295,6 @@ describe('PageService', () => {
     test('duplicate page (isRecursively: false)', async() => {
       crowi.models.Page.findRelatedTagsById = jest.fn().mockImplementation(() => { return parentTag });
       const originTagsMock = jest.spyOn(Page, 'findRelatedTagsById').mockImplementation(() => { return parentTag });
-      const serializeSpy = jest.spyOn(serializePageSecurely, 'serializePageSecurely').mockImplementation();
 
       const resultPage = await crowi.pageService.duplicate(parentForDuplicate, '/newParent', testUser1, false);
       const resultAnotherPage = await crowi.pageService.duplicate(parentForDuplicate, '/newAnotherParent', testUser2, false);
@@ -301,13 +303,14 @@ describe('PageService', () => {
 
       expect(xssSpy).toHaveBeenCalled();
       expect(duplicateDescendantsWithStreamSpy).not.toHaveBeenCalled();
+      expect(serializePageSecurely).toHaveBeenCalled();
       expect(resultPage.path).toBe('/newParent');
       expect(resultPage.lastUpdateUser._id).toEqual(testUser1._id);
       expect(resultAnotherPage.lastUpdateUser._id).toEqual(testUser2._id);
       expect(resultPage.revision).not.toEqual(parentForDuplicate.revision);
       expect(resultPage.grant).toEqual(parentForDuplicate.grant);
       expect(resultPage.tags).toEqual([originTagsMock().name]);
-      expect(serializeSpy).toHaveBeenCalled();
+
     });
 
     test('duplicateDescendants()', () => {