github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

improve doc manager so it calls get doc recursivly

Browse source
This commit is contained in:
Henry Oswald 2015-08-13 22:47:33 +01:00
parent 6ea284b17b
commit ef4e437c3f
2 changed files with 18 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
services/docstore/app/coffee/DocManager.coffee
View file

@ -22,7 +22,7 @@ module.exports = DocManager =
if err?
logger.err err:err, project_id:project_id, doc_id:doc_id, "error unarchiving doc"
return callback(err)
MongoManager.findDoc doc_id, callback
DocManager.getDoc project_id, doc_id, callback
else
callback err, doc

27
services/docstore/test/unit/coffee/DocManagerTests.coffee
View file

@ -45,21 +45,28 @@ describe "DocManager", ->
done()
describe "when the doc is deleted", ->
beforeEach ->
@doc = { _id: @doc_id, lines: ["mock-lines"] }
@s3doc = { _id: @doc_id, inS3: true }
@MongoManager.findDoc = sinon.stub()
@MongoManager.findDoc.callsArgWith(1, null, @s3doc)
@MongoManager.findDoc.callsArgWith(1, null, @doc)
spy = sinon.spy @DocManager.getDoc
@DocArchiveManager.unarchiveDoc = sinon.stub().callsArgWith(2)
@DocManager.getDoc @project_id, @doc_id, @callback
it "should call the DocArchive to unarchive the doc", ->
@DocArchiveManager.unarchiveDoc.calledWith(@project_id, @doc_id).should.equal true
it "should return the doc", ->
@callback.calledWith(null, @doc).should.equal true
describe "when the doc is in s3", ->
beforeEach ->
@MongoManager.findDoc = sinon.stub().callsArgWith(1, null, @doc)
@DocManager.getDoc @project_id, @doc_id, @callback
it "should try to find the doc in the docs collection", ->
@MongoManager.findDoc
.calledWith(@doc_id)
.should.equal true
it "should return the doc", ->
@callback
.calledWith(null, @doc)
.should.equal true
describe "when the doc does not exist anywhere", ->
beforeEach ->
@MongoManager.findDoc = sinon.stub().callsArgWith(1, null, null)