| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275 |
- const sinon = require('sinon')
- const chai = require('chai')
- const { expect } = chai
- const SandboxedModule = require('sandboxed-module')
- const Errors = require('../../../app/js/Errors')
- const modulePath = '../../../app/js/FileController.js'
- describe('FileController', function() {
- let PersistorManager,
- FileHandler,
- LocalFileWriter,
- FileController,
- req,
- res,
- next,
- stream
- const settings = {
- s3: {
- buckets: {
- user_files: 'user_files'
- }
- }
- }
- const fileSize = 1234
- const fileStream = 'fileStream'
- const projectId = 'projectId'
- const fileId = 'file_id'
- const bucket = 'user_files'
- const key = `${projectId}/${fileId}`
- const error = new Error('incorrect utensil')
- beforeEach(function() {
- PersistorManager = {
- sendStream: sinon.stub().yields(),
- copyFile: sinon.stub().yields(),
- deleteFile: sinon.stub().yields()
- }
- FileHandler = {
- getFile: sinon.stub().yields(null, fileStream),
- getFileSize: sinon.stub().yields(null, fileSize),
- deleteFile: sinon.stub().yields(),
- insertFile: sinon.stub().yields(),
- getDirectorySize: sinon.stub().yields(null, fileSize)
- }
- LocalFileWriter = {}
- stream = {
- pipeline: sinon.stub()
- }
- FileController = SandboxedModule.require(modulePath, {
- requires: {
- './LocalFileWriter': LocalFileWriter,
- './FileHandler': FileHandler,
- './PersistorManager': PersistorManager,
- './Errors': Errors,
- stream: stream,
- 'settings-sharelatex': settings,
- 'metrics-sharelatex': {
- inc() {}
- }
- },
- globals: { console }
- })
- req = {
- key: key,
- bucket: bucket,
- query: {},
- params: {
- project_id: projectId,
- file_id: fileId
- },
- headers: {},
- setLogMessage: sinon.stub(),
- addLogField: sinon.stub(),
- addLogFields: sinon.stub()
- }
- res = {
- set: sinon.stub().returnsThis(),
- sendStatus: sinon.stub().returnsThis(),
- status: sinon.stub().returnsThis()
- }
- next = sinon.stub()
- })
- describe('getFile', function() {
- it('should pipe the stream', function() {
- FileController.getFile(req, res, next)
- expect(stream.pipeline).to.have.been.calledWith(fileStream, res)
- })
- it('should send a 200 if the cacheWarm param is true', function(done) {
- req.query.cacheWarm = true
- res.sendStatus = statusCode => {
- statusCode.should.equal(200)
- done()
- }
- FileController.getFile(req, res, next)
- })
- it('should send an error if there is a problem', function() {
- FileHandler.getFile.yields(error)
- FileController.getFile(req, res, next)
- expect(next).to.have.been.calledWith(error)
- })
- describe('with a range header', function() {
- let expectedOptions
- beforeEach(function() {
- expectedOptions = {
- bucket,
- key,
- format: undefined,
- style: undefined
- }
- })
- it('should pass range options to FileHandler', function() {
- req.headers.range = 'bytes=0-8'
- expectedOptions.start = 0
- expectedOptions.end = 8
- FileController.getFile(req, res, next)
- expect(FileHandler.getFile).to.have.been.calledWith(
- bucket,
- key,
- expectedOptions
- )
- })
- it('should ignore an invalid range header', function() {
- req.headers.range = 'potato'
- FileController.getFile(req, res, next)
- expect(FileHandler.getFile).to.have.been.calledWith(
- bucket,
- key,
- expectedOptions
- )
- })
- it("should ignore any type other than 'bytes'", function() {
- req.headers.range = 'wombats=0-8'
- FileController.getFile(req, res, next)
- expect(FileHandler.getFile).to.have.been.calledWith(
- bucket,
- key,
- expectedOptions
- )
- })
- })
- })
- describe('getFileHead', function() {
- it('should return the file size in a Content-Length header', function(done) {
- res.end = () => {
- expect(res.status).to.have.been.calledWith(200)
- expect(res.set).to.have.been.calledWith('Content-Length', fileSize)
- done()
- }
- FileController.getFileHead(req, res, next)
- })
- it('should return a 404 is the file is not found', function(done) {
- FileHandler.getFileSize.yields(new Errors.NotFoundError())
- res.sendStatus = code => {
- expect(code).to.equal(404)
- done()
- }
- FileController.getFileHead(req, res, next)
- })
- it('should send an error on internal errors', function() {
- FileHandler.getFileSize.yields(error)
- FileController.getFileHead(req, res, next)
- expect(next).to.have.been.calledWith(error)
- })
- })
- describe('insertFile', function() {
- it('should send bucket name key and res to PersistorManager', function(done) {
- res.sendStatus = code => {
- expect(FileHandler.insertFile).to.have.been.calledWith(bucket, key, req)
- expect(code).to.equal(200)
- done()
- }
- FileController.insertFile(req, res, next)
- })
- })
- describe('copyFile', function() {
- const oldFileId = 'oldFileId'
- const oldProjectId = 'oldProjectid'
- const oldKey = `${oldProjectId}/${oldFileId}`
- beforeEach(function() {
- req.body = {
- source: {
- project_id: oldProjectId,
- file_id: oldFileId
- }
- }
- })
- it('should send bucket name and both keys to PersistorManager', function(done) {
- res.sendStatus = code => {
- code.should.equal(200)
- expect(PersistorManager.copyFile).to.have.been.calledWith(
- bucket,
- oldKey,
- key
- )
- done()
- }
- FileController.copyFile(req, res, next)
- })
- it('should send a 404 if the original file was not found', function(done) {
- PersistorManager.copyFile.yields(new Errors.NotFoundError())
- res.sendStatus = code => {
- code.should.equal(404)
- done()
- }
- FileController.copyFile(req, res, next)
- })
- it('should send an error if there was an error', function() {
- PersistorManager.copyFile.yields(error)
- FileController.copyFile(req, res, next)
- expect(next).to.have.been.calledWith(error)
- })
- })
- describe('delete file', function() {
- it('should tell the file handler', function(done) {
- res.sendStatus = code => {
- code.should.equal(204)
- expect(FileHandler.deleteFile).to.have.been.calledWith(bucket, key)
- done()
- }
- FileController.deleteFile(req, res, next)
- })
- it('should send a 500 if there was an error', function() {
- FileHandler.deleteFile.yields(error)
- FileController.deleteFile(req, res, next)
- expect(next).to.have.been.calledWith(error)
- })
- })
- describe('directorySize', function() {
- it('should return total directory size bytes', function(done) {
- FileController.directorySize(req, {
- json: result => {
- expect(result['total bytes']).to.equal(fileSize)
- done()
- }
- })
- })
- it('should send a 500 if there was an error', function() {
- FileHandler.getDirectorySize.yields(error)
- FileController.directorySize(req, res, next)
- expect(next).to.have.been.calledWith(error)
- })
- })
- })
|