Get doc lines from docstore when performing batch operations

services/web/app/coffee/Features/Docstore/DocstoreManager.coffee

@@ -4,7 +4,7 @@ settings = require "settings-sharelatex"
 
 module.exports = DocstoreManager =
 	deleteDoc: (project_id, doc_id, callback = (error) ->) ->
-		logger.log project_id: project_id, "deleting doc in docstore api"
+		logger.log project_id: project_id, doc_id: doc_id, "deleting doc in docstore api"
 		url = "#{settings.apis.docstore.url}/project/#{project_id}/doc/#{doc_id}"
 		request.del url, (error, res, body) ->
 			return callback(error) if error?
@@ -14,3 +14,18 @@ module.exports = DocstoreManager =
 				error = new Error("docstore api responded with non-success code: #{res.statusCode}")
 				logger.error err: error, project_id: project_id, doc_id: doc_id, "error deleting doc in docstore"
 				callback(error)
+
+	getAllDocs: (project_id, callback = (error) ->) ->
+		logger.log project_id: project_id, "getting all docs for project in docstore api"
+		url = "#{settings.apis.docstore.url}/project/#{project_id}/doc"
+		request.get {
+			url: url
+			json: true
+		}, (error, res, docs) ->
+			return callback(error) if error?
+			if 200 <= res.statusCode < 300
+				callback(null, docs)
+			else
+				error = new Error("docstore api responded with non-success code: #{res.statusCode}")
+				logger.error err: error, project_id: project_id, "error getting all docs in docstore"
+				callback(error)

services/web/app/coffee/Features/Project/ProjectEntityHandler.coffee

@@ -33,12 +33,29 @@ module.exports = ProjectEntityHandler =
 	getAllDocs: (project_id, sl_req_id, callback) ->
 		{callback, sl_req_id} = slReqIdHelper.getCallbackAndReqId(callback, sl_req_id)
 		logger.log project_id:project_id, "getting all docs for project"
-		@getAllFolders project_id, sl_req_id, (err, folders) ->
-			return callback(err) if err?
+
+		# We get the path and name info from the project, and the lines and
+		# version info from the doc store.
+		DocstoreManager.getAllDocs project_id, (error, docContentsArray) ->
+			return callback(error) if error?
+
+			# Turn array from docstore into a dictionary based on doc id
+			docContents = {}
+			for docContent in docContentsArray
+				docContents[docContent._id] = docContent
+
+			ProjectEntityHandler.getAllFolders project_id, sl_req_id, (error, folders) ->
+				return callback(error) if error?
 				docs = {}
 				for folderPath, folder of folders
 					for doc in folder.docs
-					docs[path.join(folderPath, doc.name)] = doc
+						content = docContents[doc._id.toString()]
+						docs[path.join(folderPath, doc.name)] = {
+							_id:   doc._id
+							name:  doc.name
+							lines: content.lines
+							rev:   content.rev
+						}
 				logger.log count:_.keys(docs).length, project_id:project_id, "returning docs for project"
 				callback null, docs
 
@@ -59,17 +76,20 @@ module.exports = ProjectEntityHandler =
 		logger.log sl_req_id: sl_req_id, project_id:project_id, "flushing project to tpds"
 		documentUpdaterHandler = require('../../Features/DocumentUpdater/DocumentUpdaterHandler')
 		documentUpdaterHandler.flushProjectToMongo project_id, undefined, (error) ->
-			Project.findById project_id, (err, project) ->
+			return callback(error) if error?
+			Project.findById project_id, (error, project) ->
 				return callback(error) if error?
 				requests = []
-				self.getAllDocs project_id, (err, docs) ->
+				self.getAllDocs project_id, (error, docs) ->
+					return callback(error) if error?
 					for docPath, doc of docs
 						do (docPath, doc) ->
 							requests.push (callback) ->
 								tpdsUpdateSender.addDoc {project_id:project_id, docLines:doc.lines, path:docPath, project_name:project.name, rev:doc.rev||0},
 									sl_req_id,
 									callback
-					self.getAllFiles project_id, (err, files) ->
+					self.getAllFiles project_id, (error, files) ->
+						return callback(error) if error?
 						for filePath, file of files
 							do (filePath, file) ->
 								requests.push (callback) ->

services/web/config/settings.defaults.coffee

@@ -72,6 +72,8 @@ module.exports =
 			url :"http://localhost:3012"
 		spelling:
 			url : "http://localhost:3005"
+		trackchanges:
+			url : "http://localhost:3015"
 		docstore:
 			url : "http://localhost:3016"
 		versioning:

services/web/test/UnitTests/coffee/Docstore/DocstoreManagerTests.coffee

@@ -52,3 +52,36 @@ describe "DocstoreManager", ->
 					}, "error deleting doc in docstore")
 					.should.equal true
 
+
+	describe "getAllDocs", ->
+		describe "with a successful response code", ->
+			beforeEach ->
+				@request.get = sinon.stub().callsArgWith(1, null, statusCode: 204, @docs = [{ _id: "mock-doc-id" }])
+				@DocstoreManager.getAllDocs @project_id, @callback
+
+			it "should get all the project docs in the docstore api", ->
+				@request.get
+					.calledWith({
+						url: "#{@settings.apis.docstore.url}/project/#{@project_id}/doc"
+						json: true
+					})
+					.should.equal true
+
+			it "should call the callback with the docs", ->
+				@callback.calledWith(null, @docs).should.equal true
+
+		describe "with a failed response code", ->
+			beforeEach ->
+				@request.get = sinon.stub().callsArgWith(1, null, statusCode: 500, "")
+				@DocstoreManager.getAllDocs @project_id, @callback
+
+			it "should call the callback with an error", ->
+				@callback.calledWith(new Error("docstore api responded with non-success code: 500")).should.equal true
+
+			it "should log the error", ->
+				@logger.error
+					.calledWith({
+						err: new Error("docstore api responded with a non-success code: 500")
+						project_id: @project_id
+					}, "error getting all docs in docstore")
+					.should.equal true

services/web/test/UnitTests/coffee/Project/ProjectEntityHandlerTests.coffee

@@ -7,7 +7,7 @@ SandboxedModule = require('sandboxed-module')
 ObjectId = require("mongoose").Types.ObjectId
 tk = require 'timekeeper'
 
-describe 'project entity handler', ->
+describe 'ProjectEntityHandler', ->
 	project_id = '4eecb1c1bffa66588e0000a1'
 	folder_id = "4eecaffcbffa66588e000008"
 	rootFolderId = "4eecaffcbffa66588e000007"
@@ -481,137 +481,162 @@ describe 'project entity handler', ->
 				@projectUpdater.markAsUpdated.calledWith(project_id).should.equal true
 			@ProjectEntityHandler.updateDocLines project_id, docId, docLines, done
 
-	describe "flushing folders, docs and files", ->
+	describe "getting folders, docs and files", ->
 		beforeEach ->
 			@project.rootFolder = [
-				folders: [{
-					name    : "folder1"
-					folders : [{
-						name    : "folder2"
-						folders : [{
-							name:"folder4",
-							docs:[{name:"doc3", rev:3,lines:["doc3"]}],
-							fileRefs:[{_id:"file3_id", rev:3,name:"file3"}],
-							folders:[]
+				docs: [@doc1 = {
+					name  : "doc1"
+					_id   : "doc1_id"
 				}]
-						docs    : [{
-							rev:2
-							name: "doc2"
-							lines: ["doc2", "lines"]
-						}]
-						fileRefs   : []
-					}]
-					docs    : []
-					fileRefs   : [{
-						rev:2
-						name : "file2"
-						_id  : "file2_id"
-					}]
-				}, {
-					name    : "folder3"
-					folders : []
-					docs    : []
-					fileRefs   : []
-				}]
-				docs: [{
-					rev:1
-					name: "doc1"
-					lines: ["doc1", "lines"]
-				}]
-				fileRefs: [{
-					rev:1
+				fileRefs: [@file1 = {
+					rev  : 1
 					_id  : "file1_id"
 					name : "file1"
 				}]
+				folders: [@folder1 = {
+					name    : "folder1"
+					docs    : [@doc2 = {
+						name  : "doc2"
+						_id   : "doc2_id"
+					}]
+					fileRefs   : [@file2 = {
+						rev  : 2
+						name : "file2"
+						_id  : "file2_id"
+					}]
+					folders : []
+				}]
 			]
 
-		it "should work for a very small project", (done)->
-			@project.rootFolder[0].folders = []
-			@ProjectEntityHandler.getAllDocs project_id, (err, docs) =>
-				docs["/doc1"].name.should.equal "doc1"
-				@ProjectEntityHandler.getAllFiles project_id, (err, files) =>
-					files["/file1"].name.should.equal "file1"
-					done()
+		describe "getAllFolders", ->
+			beforeEach ->
+				@callback = sinon.stub()
+				@ProjectEntityHandler.getAllFolders project_id, @callback
 
-		it "should be able to get all folders", (done) ->
-			@ProjectEntityHandler.getAllFolders project_id, (err, folders) ->
-				should.exist folders["/"]
-				should.exist folders["/folder1"]
-				should.exist folders["/folder1/folder2"]
-				should.exist folders["/folder1/folder2/folder4"]
-				folders["/folder1/folder2/folder4"].name.should.equal "folder4"
-				should.exist folders["/folder3"]
-				done()
+			it "should call the callback with the folders", ->
+				@callback
+					.calledWith(null, {
+						"/": @project.rootFolder[0]
+						"/folder1": @folder1
+					})
+					.should.equal true
 
-		it "should be able to get all docs", (done) ->
-			@ProjectEntityHandler.getAllDocs project_id, (err, docs) ->
-				docs["/doc1"].name.should.equal "doc1"
-				docs["/folder1/folder2/doc2"].name.should.equal "doc2"
-				docs["/folder1/folder2/folder4/doc3"].lines.should.deep.equal ["doc3"]
-				done()
+		describe "getAllFiles", ->
+			beforeEach ->
+				@callback = sinon.stub()
+				@ProjectEntityHandler.getAllFiles project_id, @callback
 
-		it "should be able to get all files", (done) ->
-			@ProjectEntityHandler.getAllFiles project_id, (err, files) ->
-				files["/file1"].name.should.equal "file1"
-				files["/folder1/file2"].name.should.equal "file2"
-				files["/folder1/folder2/folder4/file3"].name.should.equal "file3"
-				done()
+			it "should call the callback with the files", ->
+				@callback
+					.calledWith(null, {
+						"/file1": @file1
+						"/folder1/file2": @file2
+					})
+					.should.equal true
+
+		describe "getAllDocs", ->
+			beforeEach ->
+				@docs = [{
+					_id:   @doc1._id
+					lines: @lines1 = ["one"]
+					rev:   @rev1 = 1
+				}, {
+					_id:   @doc2._id
+					lines: @lines2 = ["two"]
+					rev:   @rev2 = 2
+				}]
+				@DocstoreManager.getAllDocs = sinon.stub().callsArgWith(1, null, @docs)
+				@ProjectEntityHandler.getAllDocs project_id, @callback
+
+			it "should get the doc lines and rev from the docstore", ->
+				@DocstoreManager.getAllDocs
+					.calledWith(project_id)
+					.should.equal true
+
+			it "should call the callback with the docs with the lines and rev included", ->
+				@callback
+					.calledWith(null, {
+						"/doc1": {
+							_id:   @doc1._id
+							lines: @lines1
+							name:  @doc1.name
+							rev:   @rev1
+						}
+						"/folder1/doc2":  {
+							_id:   @doc2._id
+							lines: @lines2
+							name:  @doc2.name
+							rev:   @rev2
+						}
+					})
+					.should.equal true
 
 	describe "flushProjectToThirdPartyDataStore", ->
 		beforeEach (done) ->
-				@addedDocs = {}
-				@addedFiles = {}
-				@tpdsUpdateSender.addDoc = (options, _, callback) =>
-					callback()
-				sinon.spy @tpdsUpdateSender, "addDoc"
-				@tpdsUpdateSender.addFile = (options, _, callback) =>
-					callback()
-				sinon.spy @tpdsUpdateSender, "addFile"
-				@documentUpdaterHandler.flushProjectToMongo = (project_id, sl_req_id, callback) ->
-					callback()
-				sinon.spy @documentUpdaterHandler, "flushProjectToMongo"
+			@project = {
+				_id: project_id
+				name: "Mock project name"
+			}
+			@ProjectModel.findById = sinon.stub().callsArgWith(1, null, @project)
+			@documentUpdaterHandler.flushProjectToMongo = sinon.stub().callsArg(2)
+			@tpdsUpdateSender.addDoc = sinon.stub().callsArg(2)
+			@tpdsUpdateSender.addFile = sinon.stub().callsArg(2)
+			@docs = {
+				"/doc/one": @doc1 = { _id: "mock-doc-1", lines: ["one"], rev: 5 }
+				"/doc/two": @doc2 = { _id: "mock-doc-2", lines: ["two"], rev: 6 }
+			}
+			@files = {
+				"/file/one": @file1 = { _id: "mock-file-1", rev: 7 }
+				"/file/two": @file2 = { _id: "mock-file-2", rev: 8 }
+			}
+			@ProjectEntityHandler.getAllDocs = sinon.stub().callsArgWith(1, null, @docs)
+			@ProjectEntityHandler.getAllFiles = sinon.stub().callsArgWith(1, null, @files)
 
-				@ProjectEntityHandler.flushProjectToThirdPartyDataStore project_id, (err) -> done()
+			@ProjectEntityHandler.flushProjectToThirdPartyDataStore project_id, () -> done()
 
-			it "should flush the documents from the document updater", ->
+		it "should flush the project from the doc updater", ->
 			@documentUpdaterHandler.flushProjectToMongo
-					.calledWith(@project._id).should.equal true
-				@documentUpdaterHandler.flushProjectToMongo
-					.calledBefore(@tpdsUpdateSender.addDoc).should.equal true
-				@documentUpdaterHandler.flushProjectToMongo
-					.calledBefore(@tpdsUpdateSender.addFile).should.equal true
+				.calledWith(project_id)
+				.should.equal true
 
-			it "should call addDoc for each doc", ->
-				@tpdsUpdateSender.addDoc.calledWith(
-					project_id : @project._id
-					path : "/doc1"
-					docLines: ["doc1", "lines"]
-					project_name: @project.name
-					rev:1
-				).should.equal true
-				@tpdsUpdateSender.addDoc.calledWith(
-					project_id : @project._id
-					path : "/folder1/folder2/doc2"
-					docLines: ["doc2", "lines"]
-					project_name: @project.name
-					rev:2
-				).should.equal true
+		it "should look up the project in mongo", ->
+			@ProjectModel.findById
+				.calledWith(project_id)
+				.should.equal true
 
-			it "should call addFile for each file", ->
-				@tpdsUpdateSender.addFile.calledWith(
-					project_id : @project._id
-					file_id    : "file1_id"
-					path       : "/file1"
+		it "should get all the docs in the project", ->
+			@ProjectEntityHandler.getAllDocs
+				.calledWith(project_id)
+				.should.equal true
+
+		it "should get all the files in the project", ->
+			@ProjectEntityHandler.getAllFiles
+				.calledWith(project_id)
+				.should.equal true
+
+		it "should flush each doc to the TPDS", ->
+			for path, doc of @docs
+				@tpdsUpdateSender.addDoc
+					.calledWith({
+						project_id: project_id,
+						docLines: doc.lines
 						project_name: @project.name
-					rev:1
-				).should.equal true
-				@tpdsUpdateSender.addFile.calledWith(
-					project_id : @project._id
-					file_id    : "file2_id"
-					path       : "/folder1/file2"
+						rev: doc.rev
+						path: path
+					})
+					.should.equal true
+
+		it "should flush each file to the TPDS", ->
+			for path, file of @files
+				@tpdsUpdateSender.addFile
+					.calledWith({
+						project_id: project_id,
+						file_id: file._id
 						project_name: @project.name
-					rev:2
-				).should.equal true
+						rev: file.rev
+						path: path
+					})
+					.should.equal true
 	
 	describe "setRootDoc", ->
 		it "should call Project.update", ->