From b3887fd9844483c850cb8b73eff0f814086a7079 Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Tue, 6 Mar 2018 09:49:54 +0000 Subject: [PATCH] update unit tests for incoming project versions --- .../ProjectManager/updateProjectTests.coffee | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/services/document-updater/test/unit/coffee/ProjectManager/updateProjectTests.coffee b/services/document-updater/test/unit/coffee/ProjectManager/updateProjectTests.coffee index b04a8c7a50..a5d3e881c6 100644 --- a/services/document-updater/test/unit/coffee/ProjectManager/updateProjectTests.coffee +++ b/services/document-updater/test/unit/coffee/ProjectManager/updateProjectTests.coffee @@ -3,6 +3,7 @@ chai = require('chai') should = chai.should() modulePath = "../../../../app/js/ProjectManager.js" SandboxedModule = require('sandboxed-module') +_ = require('underscore') describe "ProjectManager", -> beforeEach -> @@ -49,16 +50,19 @@ describe "ProjectManager", -> @ProjectManager.updateProjectWithLocks @project_id, @user_id, @docUpdates, @fileUpdates, @version, @callback it "should rename the docs in the updates", -> + firstDocUpdateWithVersion = _.extend({}, @firstDocUpdate, {version: "#{@version}.0"}) + secondDocUpdateWithVersion = _.extend({}, @secondDocUpdate, {version: "#{@version}.1"}) @DocumentManager.renameDocWithLock - .calledWith(@project_id, @firstDocUpdate.id, @user_id, @firstDocUpdate) + .calledWith(@project_id, @firstDocUpdate.id, @user_id, firstDocUpdateWithVersion) .should.equal true @DocumentManager.renameDocWithLock - .calledWith(@project_id, @secondDocUpdate.id, @user_id, @secondDocUpdate) + .calledWith(@project_id, @secondDocUpdate.id, @user_id, secondDocUpdateWithVersion) .should.equal true it "should rename the files in the updates", -> + firstFileUpdateWithVersion = _.extend({}, @firstFileUpdate, {version: "#{@version}.2"}) @ProjectHistoryRedisManager.queueRenameEntity - .calledWith(@project_id, 'file', @firstFileUpdate.id, @user_id, @firstFileUpdate) + .calledWith(@project_id, 'file', @firstFileUpdate.id, @user_id, firstFileUpdateWithVersion) .should.equal true it "should not flush the history", -> @@ -117,16 +121,19 @@ describe "ProjectManager", -> @ProjectManager.updateProjectWithLocks @project_id, @user_id, @docUpdates, @fileUpdates, @version, @callback it "should add the docs in the updates", -> + firstDocUpdateWithVersion = _.extend({}, @firstDocUpdate, {version: "#{@version}.0"}) + secondDocUpdateWithVersion = _.extend({}, @secondDocUpdate, {version: "#{@version}.1"}) @ProjectHistoryRedisManager.queueAddEntity - .calledWith(@project_id, 'doc', @firstDocUpdate.id, @user_id, @firstDocUpdate) + .calledWith(@project_id, 'doc', @firstDocUpdate.id, @user_id, firstDocUpdateWithVersion) .should.equal true @ProjectHistoryRedisManager.queueAddEntity - .calledWith(@project_id, 'doc', @secondDocUpdate.id, @user_id, @secondDocUpdate) + .calledWith(@project_id, 'doc', @secondDocUpdate.id, @user_id, secondDocUpdateWithVersion) .should.equal true it "should add the files in the updates", -> + firstFileUpdateWithVersion = _.extend({}, @firstFileUpdate, {version: "#{@version}.2"}) @ProjectHistoryRedisManager.queueAddEntity - .calledWith(@project_id, 'file', @firstFileUpdate.id, @user_id, @firstFileUpdate) + .calledWith(@project_id, 'file', @firstFileUpdate.id, @user_id, firstFileUpdateWithVersion) .should.equal true it "should not flush the history", ->