2014-04-28 11:45:59 -04:00
|
|
|
SandboxedModule = require('sandboxed-module')
|
2014-05-20 08:04:33 -04:00
|
|
|
assert = require("chai").assert
|
2014-04-28 11:45:59 -04:00
|
|
|
sinon = require('sinon')
|
|
|
|
chai = require('chai')
|
|
|
|
chai.should()
|
|
|
|
expect = chai.expect
|
|
|
|
modulePath = require('path').join __dirname, '../../../app/js/HttpController'
|
2014-04-30 08:06:12 -04:00
|
|
|
ObjectId = require("mongojs").ObjectId
|
2014-04-28 11:45:59 -04:00
|
|
|
|
|
|
|
describe "HttpController", ->
|
|
|
|
beforeEach ->
|
|
|
|
@HttpController = SandboxedModule.require modulePath, requires:
|
|
|
|
"./DocManager": @DocManager = {}
|
2015-06-02 18:24:45 -04:00
|
|
|
"./DocArchiveManager": @DocArchiveManager = {}
|
2014-04-29 06:49:09 -04:00
|
|
|
"logger-sharelatex": @logger = { log: sinon.stub(), error: sinon.stub() }
|
2016-11-28 09:55:16 -05:00
|
|
|
"./HealthChecker": {}
|
2014-05-20 08:04:33 -04:00
|
|
|
@res = { send: sinon.stub(), json: sinon.stub(), setHeader:sinon.stub() }
|
2015-02-03 09:05:08 -05:00
|
|
|
@req = { query:{}}
|
2014-04-28 11:45:59 -04:00
|
|
|
@next = sinon.stub()
|
|
|
|
@project_id = "mock-project-id"
|
|
|
|
@doc_id = "mock-doc-id"
|
|
|
|
@doc = {
|
|
|
|
_id: @doc_id
|
2014-05-20 08:04:33 -04:00
|
|
|
lines: ["mock", "lines", " here", "", "", " spaces "]
|
2014-04-30 08:06:12 -04:00
|
|
|
version: 42
|
|
|
|
rev: 5
|
2014-04-28 11:45:59 -04:00
|
|
|
}
|
2015-02-03 09:05:08 -05:00
|
|
|
@deletedDoc = {
|
|
|
|
deleted:true
|
|
|
|
_id: @doc_id
|
|
|
|
lines: ["mock", "lines", " here", "", "", " spaces "]
|
|
|
|
version: 42
|
|
|
|
rev: 5
|
|
|
|
}
|
2014-04-28 11:45:59 -04:00
|
|
|
|
|
|
|
describe "getDoc", ->
|
2014-04-29 06:49:09 -04:00
|
|
|
|
2014-06-06 07:37:42 -04:00
|
|
|
describe "without deleted docs", ->
|
|
|
|
beforeEach ->
|
2015-02-03 09:05:08 -05:00
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
doc_id: @doc_id
|
2016-11-30 10:08:56 -05:00
|
|
|
@DocManager.getDoc = sinon.stub().callsArgWith(3, null, @doc)
|
2014-06-06 07:37:42 -04:00
|
|
|
@HttpController.getDoc @req, @res, @next
|
|
|
|
|
2016-11-30 10:08:56 -05:00
|
|
|
it "should get the document with the version (including deleted)", ->
|
2014-06-06 07:37:42 -04:00
|
|
|
@DocManager.getDoc
|
2016-11-30 10:08:56 -05:00
|
|
|
.calledWith(@project_id, @doc_id, {version: true})
|
2014-06-06 07:37:42 -04:00
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should return the doc as JSON", ->
|
|
|
|
@res.json
|
|
|
|
.calledWith({
|
|
|
|
_id: @doc_id
|
|
|
|
lines: @doc.lines
|
|
|
|
rev: @doc.rev
|
|
|
|
deleted: false
|
2016-11-28 09:55:16 -05:00
|
|
|
version: @doc.version
|
2014-06-06 07:37:42 -04:00
|
|
|
})
|
|
|
|
.should.equal true
|
|
|
|
|
2015-02-03 09:05:08 -05:00
|
|
|
describe "which is deleted", ->
|
2014-06-06 07:37:42 -04:00
|
|
|
beforeEach ->
|
2015-02-03 09:05:08 -05:00
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
doc_id: @doc_id
|
2016-11-30 10:08:56 -05:00
|
|
|
@DocManager.getDoc = sinon.stub().callsArgWith(3, null, @deletedDoc)
|
2015-02-03 09:05:08 -05:00
|
|
|
|
|
|
|
it "should get the doc from the doc manager", ->
|
2014-06-06 07:37:42 -04:00
|
|
|
@HttpController.getDoc @req, @res, @next
|
2016-11-30 10:08:56 -05:00
|
|
|
@DocManager.getDoc.calledWith(@project_id, @doc_id, {version: true}).should.equal true
|
2014-06-06 07:37:42 -04:00
|
|
|
|
2015-02-03 09:05:08 -05:00
|
|
|
it "should return 404 if the query string delete is not set ", ->
|
|
|
|
@HttpController.getDoc @req, @res, @next
|
|
|
|
@res.send.calledWith(404).should.equal true
|
|
|
|
|
|
|
|
it "should return the doc as JSON if include_deleted is set to true", ->
|
|
|
|
@req.query.include_deleted = "true"
|
|
|
|
@HttpController.getDoc @req, @res, @next
|
|
|
|
@res.json
|
|
|
|
.calledWith({
|
|
|
|
_id: @doc_id
|
|
|
|
lines: @doc.lines
|
|
|
|
rev: @doc.rev
|
|
|
|
deleted: true
|
2016-11-28 09:55:16 -05:00
|
|
|
version: @doc.version
|
2015-02-03 09:05:08 -05:00
|
|
|
})
|
2014-06-06 07:37:42 -04:00
|
|
|
.should.equal true
|
2014-04-30 08:06:12 -04:00
|
|
|
|
2014-05-20 08:04:33 -04:00
|
|
|
describe "getRawDoc", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
doc_id: @doc_id
|
2016-11-30 10:08:56 -05:00
|
|
|
@DocManager.getDoc = sinon.stub().callsArgWith(3, null, @doc)
|
2014-05-20 08:04:33 -04:00
|
|
|
@HttpController.getRawDoc @req, @res, @next
|
|
|
|
|
2016-11-30 10:08:56 -05:00
|
|
|
it "should get the document without the version", ->
|
2014-05-20 08:04:33 -04:00
|
|
|
@DocManager.getDoc
|
2016-11-30 10:08:56 -05:00
|
|
|
.calledWith(@project_id, @doc_id, {version: false})
|
2014-05-20 08:04:33 -04:00
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the content type header", ->
|
|
|
|
@res.setHeader.calledWith('content-type', 'text/plain').should.equal true
|
|
|
|
|
|
|
|
it "should send the raw version of the doc", ->
|
|
|
|
assert.deepEqual @res.send.args[0][0], "#{@doc.lines[0]}\n#{@doc.lines[1]}\n#{@doc.lines[2]}\n#{@doc.lines[3]}\n#{@doc.lines[4]}\n#{@doc.lines[5]}"
|
|
|
|
|
2014-04-30 08:06:12 -04:00
|
|
|
describe "getAllDocs", ->
|
2014-05-07 11:38:10 -04:00
|
|
|
describe "normally", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
@docs = [{
|
|
|
|
_id: ObjectId()
|
|
|
|
lines: ["mock", "lines", "one"]
|
|
|
|
rev: 2
|
|
|
|
}, {
|
|
|
|
_id: ObjectId()
|
|
|
|
lines: ["mock", "lines", "two"]
|
|
|
|
rev: 4
|
|
|
|
}]
|
|
|
|
@DocManager.getAllDocs = sinon.stub().callsArgWith(1, null, @docs)
|
|
|
|
@HttpController.getAllDocs @req, @res, @next
|
2014-04-30 08:06:12 -04:00
|
|
|
|
2014-05-07 11:38:10 -04:00
|
|
|
it "should get all the docs", ->
|
|
|
|
@DocManager.getAllDocs
|
|
|
|
.calledWith(@project_id)
|
|
|
|
.should.equal true
|
2014-04-30 08:06:12 -04:00
|
|
|
|
2014-05-07 11:38:10 -04:00
|
|
|
it "should return the doc as JSON", ->
|
|
|
|
@res.json
|
|
|
|
.calledWith([{
|
|
|
|
_id: @docs[0]._id.toString()
|
|
|
|
lines: @docs[0].lines
|
|
|
|
rev: @docs[0].rev
|
2014-06-05 08:29:50 -04:00
|
|
|
deleted: false
|
2014-05-07 11:38:10 -04:00
|
|
|
}, {
|
|
|
|
_id: @docs[1]._id.toString()
|
|
|
|
lines: @docs[1].lines
|
|
|
|
rev: @docs[1].rev
|
2014-06-05 08:29:50 -04:00
|
|
|
deleted: false
|
2014-05-07 11:38:10 -04:00
|
|
|
}])
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
describe "with a null doc", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
@docs = [{
|
|
|
|
_id: ObjectId()
|
|
|
|
lines: ["mock", "lines", "one"]
|
|
|
|
rev: 2
|
|
|
|
},
|
|
|
|
null,
|
|
|
|
{
|
|
|
|
_id: ObjectId()
|
|
|
|
lines: ["mock", "lines", "two"]
|
|
|
|
rev: 4
|
|
|
|
}]
|
|
|
|
@DocManager.getAllDocs = sinon.stub().callsArgWith(1, null, @docs)
|
|
|
|
@HttpController.getAllDocs @req, @res, @next
|
|
|
|
|
|
|
|
it "should return the non null docs as JSON", ->
|
|
|
|
@res.json
|
|
|
|
.calledWith([{
|
|
|
|
_id: @docs[0]._id.toString()
|
|
|
|
lines: @docs[0].lines
|
|
|
|
rev: @docs[0].rev
|
2014-06-05 08:29:50 -04:00
|
|
|
deleted: false
|
2014-05-07 11:38:10 -04:00
|
|
|
}, {
|
|
|
|
_id: @docs[2]._id.toString()
|
|
|
|
lines: @docs[2].lines
|
|
|
|
rev: @docs[2].rev
|
2014-06-05 08:29:50 -04:00
|
|
|
deleted: false
|
2014-05-07 11:38:10 -04:00
|
|
|
}])
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should log out an error", ->
|
|
|
|
@logger.error
|
|
|
|
.calledWith(
|
|
|
|
err: new Error("null doc")
|
|
|
|
project_id: @project_id
|
|
|
|
"encountered null doc"
|
|
|
|
)
|
|
|
|
.should.equal true
|
2014-04-29 06:49:09 -04:00
|
|
|
|
|
|
|
describe "updateDoc", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
doc_id: @doc_id
|
|
|
|
|
|
|
|
describe "when the doc lines exist and were updated", ->
|
2014-04-28 11:45:59 -04:00
|
|
|
beforeEach ->
|
2014-04-29 06:49:09 -04:00
|
|
|
@req.body =
|
|
|
|
lines: @lines = ["hello", "world"]
|
2016-11-28 09:55:16 -05:00
|
|
|
@DocManager.updateDoc = sinon.stub().yields(null, true, @rev = 5)
|
2014-04-29 06:49:09 -04:00
|
|
|
@HttpController.updateDoc @req, @res, @next
|
|
|
|
|
|
|
|
it "should update the document", ->
|
|
|
|
@DocManager.updateDoc
|
2016-11-28 09:55:16 -05:00
|
|
|
.calledWith(@project_id, @doc_id, @lines, undefined)
|
2014-04-28 11:45:59 -04:00
|
|
|
.should.equal true
|
|
|
|
|
2014-04-29 06:49:09 -04:00
|
|
|
it "should return a modified status", ->
|
|
|
|
@res.json
|
2014-05-08 10:43:08 -04:00
|
|
|
.calledWith(modified: true, rev: @rev)
|
2014-04-28 11:45:59 -04:00
|
|
|
.should.equal true
|
|
|
|
|
2014-04-29 06:49:09 -04:00
|
|
|
describe "when the doc lines exist and were not updated", ->
|
2014-04-28 11:45:59 -04:00
|
|
|
beforeEach ->
|
2014-04-29 06:49:09 -04:00
|
|
|
@req.body =
|
|
|
|
lines: @lines = ["hello", "world"]
|
2016-11-28 09:55:16 -05:00
|
|
|
@DocManager.updateDoc = sinon.stub().yields(null, false, @rev = 5)
|
2014-04-29 06:49:09 -04:00
|
|
|
@HttpController.updateDoc @req, @res, @next
|
2014-04-28 11:45:59 -04:00
|
|
|
|
2014-04-29 06:49:09 -04:00
|
|
|
it "should return a modified status", ->
|
|
|
|
@res.json
|
2014-05-08 10:43:08 -04:00
|
|
|
.calledWith(modified: false, rev: @rev)
|
2014-04-29 06:49:09 -04:00
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
describe "when the doc lines are not provided", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.body = {}
|
2016-11-28 09:55:16 -05:00
|
|
|
@DocManager.updateDoc = sinon.stub().yields(null, false)
|
2014-04-29 06:49:09 -04:00
|
|
|
@HttpController.updateDoc @req, @res, @next
|
|
|
|
|
|
|
|
it "should not update the document", ->
|
|
|
|
@DocManager.updateDoc.called.should.equal false
|
|
|
|
|
|
|
|
it "should return a 400 (bad request) response", ->
|
2014-04-28 11:45:59 -04:00
|
|
|
@res.send
|
2014-04-29 06:49:09 -04:00
|
|
|
.calledWith(400)
|
|
|
|
.should.equal true
|
2014-04-29 11:36:10 -04:00
|
|
|
|
2016-11-28 09:55:16 -05:00
|
|
|
describe "when the doc version is provided", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.body =
|
|
|
|
lines: @lines = ["hello", "world"]
|
|
|
|
version: @version = 42
|
|
|
|
@DocManager.updateDoc = sinon.stub().yields(null, true, @rev = 5)
|
|
|
|
@HttpController.updateDoc @req, @res, @next
|
|
|
|
|
|
|
|
it "should update the document with the lines and version", ->
|
|
|
|
@DocManager.updateDoc
|
|
|
|
.calledWith(@project_id, @doc_id, @lines, @version)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should return a modified status", ->
|
|
|
|
@res.json
|
|
|
|
.calledWith(modified: true, rev: @rev)
|
|
|
|
.should.equal true
|
|
|
|
|
2014-04-29 11:36:10 -04:00
|
|
|
describe "deleteDoc", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
doc_id: @doc_id
|
|
|
|
@DocManager.deleteDoc = sinon.stub().callsArg(2)
|
|
|
|
@HttpController.deleteDoc @req, @res, @next
|
|
|
|
|
|
|
|
it "should delete the document", ->
|
|
|
|
@DocManager.deleteDoc
|
|
|
|
.calledWith(@project_id, @doc_id)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should return a 204 (No Content)", ->
|
|
|
|
@res.send
|
|
|
|
.calledWith(204)
|
|
|
|
.should.equal true
|
2015-06-02 18:24:45 -04:00
|
|
|
|
|
|
|
describe "archiveAllDocs", ->
|
|
|
|
beforeEach ->
|
|
|
|
@req.params =
|
|
|
|
project_id: @project_id
|
|
|
|
@DocArchiveManager.archiveAllDocs = sinon.stub().callsArg(1)
|
|
|
|
@HttpController.archiveAllDocs @req, @res, @next
|
|
|
|
|
|
|
|
it "should archive the project", ->
|
|
|
|
@DocArchiveManager.archiveAllDocs
|
|
|
|
.calledWith(@project_id)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should return a 204 (No Content)", ->
|
|
|
|
@res.send
|
|
|
|
.calledWith(204)
|
|
|
|
.should.equal true
|