mirror of
https://github.com/overleaf/overleaf.git
synced 2024-12-02 11:00:39 -05:00
Merge branch 'master' of github.com:sharelatex/web-sharelatex
This commit is contained in:
commit
77ac6c1baf
7 changed files with 95 additions and 2 deletions
|
@ -106,3 +106,18 @@ module.exports = ClsiManager =
|
||||||
resources: resources
|
resources: resources
|
||||||
}
|
}
|
||||||
|
|
||||||
|
wordCount: (project_id, file, options, callback = (error, response) ->) ->
|
||||||
|
ClsiManager._buildRequest project_id, options, (error, req) ->
|
||||||
|
compilerUrl = ClsiManager._getCompilerUrl(options?.compileGroup)
|
||||||
|
filename = file || req.compile.rootResourcePath
|
||||||
|
request.get {
|
||||||
|
url: "#{compilerUrl}/project/#{project_id}/wordcount?file=#{filename}"
|
||||||
|
}, (error, response, body) ->
|
||||||
|
return callback(error) if error?
|
||||||
|
if 200 <= response.statusCode < 300
|
||||||
|
callback null, body
|
||||||
|
else
|
||||||
|
error = new Error("CLSI returned non-success code: #{response.statusCode}")
|
||||||
|
logger.error err: error, project_id: project_id, "CLSI returned failure code"
|
||||||
|
callback error, body
|
||||||
|
|
||||||
|
|
|
@ -102,3 +102,11 @@ module.exports = CompileController =
|
||||||
proxy.pipe(res)
|
proxy.pipe(res)
|
||||||
proxy.on "error", (error) ->
|
proxy.on "error", (error) ->
|
||||||
logger.warn err: error, url: url, "CLSI proxy error"
|
logger.warn err: error, url: url, "CLSI proxy error"
|
||||||
|
|
||||||
|
wordCount: (req, res, next) ->
|
||||||
|
project_id = req.params.Project_id
|
||||||
|
file = req.query.file || false
|
||||||
|
CompileManager.wordCount project_id, file, (error, body) ->
|
||||||
|
return next(error) if error?
|
||||||
|
res.contentType("application/json")
|
||||||
|
res.send 200, body
|
||||||
|
|
|
@ -91,3 +91,7 @@ module.exports = CompileManager =
|
||||||
else
|
else
|
||||||
ProjectRootDocManager.setRootDocAutomatically project_id, callback
|
ProjectRootDocManager.setRootDocAutomatically project_id, callback
|
||||||
|
|
||||||
|
wordCount: (project_id, file, callback = (error) ->) ->
|
||||||
|
CompileManager.getProjectCompileLimits project_id, (error, limits) ->
|
||||||
|
return callback(error) if error?
|
||||||
|
ClsiManager.wordCount project_id, file, limits, callback
|
||||||
|
|
|
@ -111,6 +111,7 @@ module.exports = class Router
|
||||||
webRouter.delete "/project/:Project_id/output", SecurityManager.requestCanAccessProject, CompileController.deleteAuxFiles
|
webRouter.delete "/project/:Project_id/output", SecurityManager.requestCanAccessProject, CompileController.deleteAuxFiles
|
||||||
webRouter.get "/project/:Project_id/sync/code", SecurityManager.requestCanAccessProject, CompileController.proxySync
|
webRouter.get "/project/:Project_id/sync/code", SecurityManager.requestCanAccessProject, CompileController.proxySync
|
||||||
webRouter.get "/project/:Project_id/sync/pdf", SecurityManager.requestCanAccessProject, CompileController.proxySync
|
webRouter.get "/project/:Project_id/sync/pdf", SecurityManager.requestCanAccessProject, CompileController.proxySync
|
||||||
|
webRouter.get "/project/:Project_id/wordcount", SecurityManager.requestCanAccessProject, CompileController.wordCount
|
||||||
|
|
||||||
webRouter.delete '/Project/:Project_id', SecurityManager.requestIsOwner, ProjectController.deleteProject
|
webRouter.delete '/Project/:Project_id', SecurityManager.requestIsOwner, ProjectController.deleteProject
|
||||||
webRouter.post '/Project/:Project_id/restore', SecurityManager.requestIsOwner, ProjectController.restoreProject
|
webRouter.post '/Project/:Project_id/restore', SecurityManager.requestIsOwner, ProjectController.restoreProject
|
||||||
|
|
|
@ -260,3 +260,30 @@ describe "ClsiManager", ->
|
||||||
json: @req
|
json: @req
|
||||||
jar: false
|
jar: false
|
||||||
}).should.equal true
|
}).should.equal true
|
||||||
|
|
||||||
|
describe "wordCount", ->
|
||||||
|
beforeEach ->
|
||||||
|
@request.get = sinon.stub().callsArgWith(1, null, {statusCode: 200}, @body = { mock: "foo" })
|
||||||
|
@ClsiManager._buildRequest = sinon.stub().callsArgWith(2, null, { compile: { rootResourcePath: "rootfile.text" } })
|
||||||
|
@ClsiManager._getCompilerUrl = sinon.stub().returns "compiler.url"
|
||||||
|
|
||||||
|
describe "with root file", ->
|
||||||
|
beforeEach ->
|
||||||
|
@ClsiManager.wordCount @project_id, false, {}, @callback
|
||||||
|
|
||||||
|
it "should call wordCount with root file", ->
|
||||||
|
@request.get
|
||||||
|
.calledWith({ url: "compiler.url/project/#{@project_id}/wordcount?file=rootfile.text" })
|
||||||
|
.should.equal true
|
||||||
|
|
||||||
|
it "should call the callback", ->
|
||||||
|
@callback.called.should.equal true
|
||||||
|
|
||||||
|
describe "with param file", ->
|
||||||
|
beforeEach ->
|
||||||
|
@ClsiManager.wordCount @project_id, "main.tex", {}, @callback
|
||||||
|
|
||||||
|
it "should call wordCount with param file", ->
|
||||||
|
@request.get
|
||||||
|
.calledWith({ url: "compiler.url/project/#{@project_id}/wordcount?file=main.tex" })
|
||||||
|
.should.equal true
|
||||||
|
|
|
@ -352,3 +352,22 @@ describe "CompileController", ->
|
||||||
@CompileController.compileAndDownloadPdf @req, @res
|
@CompileController.compileAndDownloadPdf @req, @res
|
||||||
@CompileController.proxyToClsi.calledWith(@project_id, "/project/#{@project_id}/output/output.pdf", @req, @res).should.equal true
|
@CompileController.proxyToClsi.calledWith(@project_id, "/project/#{@project_id}/output/output.pdf", @req, @res).should.equal true
|
||||||
done()
|
done()
|
||||||
|
|
||||||
|
describe "wordCount", ->
|
||||||
|
beforeEach ->
|
||||||
|
@CompileManager.wordCount = sinon.stub().callsArgWith(2, null, {content:"body"})
|
||||||
|
@req.params =
|
||||||
|
Project_id: @project_id
|
||||||
|
@res.send = sinon.stub()
|
||||||
|
@res.contentType = sinon.stub()
|
||||||
|
@CompileController.wordCount @req, @res, @next
|
||||||
|
|
||||||
|
it "should proxy to the CLSI", ->
|
||||||
|
@CompileManager.wordCount
|
||||||
|
.calledWith(@project_id, false)
|
||||||
|
.should.equal true
|
||||||
|
|
||||||
|
it "should return a 200 and body", ->
|
||||||
|
@res.send
|
||||||
|
.calledWith(200, {content:"body"})
|
||||||
|
.should.equal true
|
||||||
|
|
|
@ -259,3 +259,22 @@ describe "CompileManager", ->
|
||||||
@CompileManager._checkIfAutoCompileLimitHasBeenHit true, (err, canCompile)=>
|
@CompileManager._checkIfAutoCompileLimitHasBeenHit true, (err, canCompile)=>
|
||||||
canCompile.should.equal false
|
canCompile.should.equal false
|
||||||
done()
|
done()
|
||||||
|
|
||||||
|
describe "wordCount", ->
|
||||||
|
beforeEach ->
|
||||||
|
@CompileManager.getProjectCompileLimits = sinon.stub().callsArgWith 1, null, @limits = { compileGroup: "mock-compile-group" }
|
||||||
|
@ClsiManager.wordCount = sinon.stub().callsArg(3)
|
||||||
|
@CompileManager.wordCount @project_id, false, @callback
|
||||||
|
|
||||||
|
it "should look up the compile group to use", ->
|
||||||
|
@CompileManager.getProjectCompileLimits
|
||||||
|
.calledWith(@project_id)
|
||||||
|
.should.equal true
|
||||||
|
|
||||||
|
it "should call wordCount for project", ->
|
||||||
|
@ClsiManager.wordCount
|
||||||
|
.calledWith(@project_id, false, @limits)
|
||||||
|
.should.equal true
|
||||||
|
|
||||||
|
it "should call the callback", ->
|
||||||
|
@callback.called.should.equal true
|
||||||
|
|
Loading…
Reference in a new issue