| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272 |
- /* eslint-disable
- max-len,
- no-return-assign,
- no-unused-vars,
- */
- // TODO: This file was created by bulk-decaffeinate.
- // Fix any style issues and re-enable lint.
- /*
- * decaffeinate suggestions:
- * DS102: Remove unnecessary code created because of implicit returns
- * DS206: Consider reworking classes to avoid initClass
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
- const sinon = require('sinon')
- const chai = require('chai')
- const should = chai.should()
- const { expect } = chai
- const modulePath =
- '../../../../app/src/Features/Uploads/ProjectUploadController.js'
- const SandboxedModule = require('sandboxed-module')
- const MockRequest = require('../helpers/MockRequest')
- const MockResponse = require('../helpers/MockResponse')
- const ArchiveErrors = require('../../../../app/src/Features/Uploads/ArchiveErrors')
- describe('ProjectUploadController', function() {
- beforeEach(function() {
- let Timer
- this.req = new MockRequest()
- this.res = new MockResponse()
- this.user_id = 'user-id-123'
- this.metrics = {
- Timer: (Timer = (function() {
- Timer = class Timer {
- static initClass() {
- this.prototype.done = sinon.stub()
- }
- }
- Timer.initClass()
- return Timer
- })())
- }
- this.AuthenticationController = {
- getLoggedInUserId: sinon.stub().returns(this.user_id)
- }
- return (this.ProjectUploadController = SandboxedModule.require(modulePath, {
- globals: {
- console: console
- },
- requires: {
- './ProjectUploadManager': (this.ProjectUploadManager = {}),
- './FileSystemImportManager': (this.FileSystemImportManager = {}),
- 'logger-sharelatex': (this.logger = {
- log: sinon.stub(),
- error: sinon.stub(),
- err() {}
- }),
- '@overleaf/metrics': this.metrics,
- '../Authentication/AuthenticationController': this
- .AuthenticationController,
- './ArchiveErrors': ArchiveErrors,
- fs: (this.fs = {})
- }
- }))
- })
- describe('uploadProject', function() {
- beforeEach(function() {
- this.path = '/path/to/file/on/disk.zip'
- this.name = 'filename.zip'
- this.req.file = {
- path: this.path,
- originalname: this.name
- }
- this.req.session = {
- user: {
- _id: this.user_id
- }
- }
- this.project = { _id: (this.project_id = 'project-id-123') }
- return (this.fs.unlink = sinon.stub())
- })
- describe('successfully', function() {
- beforeEach(function() {
- this.ProjectUploadManager.createProjectFromZipArchive = sinon
- .stub()
- .callsArgWith(3, null, this.project)
- return this.ProjectUploadController.uploadProject(this.req, this.res)
- })
- it('should create a project owned by the logged in user', function() {
- return this.ProjectUploadManager.createProjectFromZipArchive
- .calledWith(this.user_id)
- .should.equal(true)
- })
- it('should create a project with the same name as the zip archive', function() {
- return this.ProjectUploadManager.createProjectFromZipArchive
- .calledWith(sinon.match.any, 'filename', sinon.match.any)
- .should.equal(true)
- })
- it('should create a project from the zip archive', function() {
- return this.ProjectUploadManager.createProjectFromZipArchive
- .calledWith(sinon.match.any, sinon.match.any, this.path)
- .should.equal(true)
- })
- it('should return a successful response to the FileUploader client', function() {
- return expect(this.res.body).to.deep.equal({
- success: true,
- project_id: this.project_id
- })
- })
- it('should record the time taken to do the upload', function() {
- return this.metrics.Timer.prototype.done.called.should.equal(true)
- })
- it('should remove the uploaded file', function() {
- return this.fs.unlink.calledWith(this.path).should.equal(true)
- })
- })
- describe('when ProjectUploadManager.createProjectFromZipArchive fails', function() {
- beforeEach(function() {
- this.ProjectUploadManager.createProjectFromZipArchive = sinon
- .stub()
- .callsArgWith(3, new Error('Something went wrong'), this.project)
- return this.ProjectUploadController.uploadProject(this.req, this.res)
- })
- it('should return a failed response to the FileUploader client', function() {
- return expect(this.res.body).to.deep.equal(
- JSON.stringify({ success: false, error: 'upload_failed' })
- )
- })
- })
- describe('when ProjectUploadManager.createProjectFromZipArchive reports the file as invalid', function() {
- beforeEach(function() {
- this.ProjectUploadManager.createProjectFromZipArchive = sinon
- .stub()
- .callsArgWith(
- 3,
- new ArchiveErrors.ZipContentsTooLargeError(),
- this.project
- )
- return this.ProjectUploadController.uploadProject(this.req, this.res)
- })
- it('should return the reported error to the FileUploader client', function() {
- expect(JSON.parse(this.res.body)).to.deep.equal({
- success: false,
- error: 'zip_contents_too_large'
- })
- })
- it("should return an 'unprocessable entity' status code", function() {
- return expect(this.res.statusCode).to.equal(422)
- })
- })
- })
- describe('uploadFile', function() {
- beforeEach(function() {
- this.project_id = 'project-id-123'
- this.folder_id = 'folder-id-123'
- this.path = '/path/to/file/on/disk.png'
- this.name = 'filename.png'
- this.req.file = {
- path: this.path,
- originalname: this.name
- }
- this.req.session = {
- user: {
- _id: this.user_id
- }
- }
- this.req.params = { Project_id: this.project_id }
- this.req.query = { folder_id: this.folder_id }
- return (this.fs.unlink = sinon.stub())
- })
- describe('successfully', function() {
- beforeEach(function() {
- this.entity = {
- _id: '1234',
- type: 'file'
- }
- this.FileSystemImportManager.addEntity = sinon
- .stub()
- .callsArgWith(6, null, this.entity)
- return this.ProjectUploadController.uploadFile(this.req, this.res)
- })
- it('should insert the file', function() {
- return this.FileSystemImportManager.addEntity
- .calledWith(
- this.user_id,
- this.project_id,
- this.folder_id,
- this.name,
- this.path
- )
- .should.equal(true)
- })
- it('should return a successful response to the FileUploader client', function() {
- return expect(this.res.body).to.deep.equal({
- success: true,
- entity_id: this.entity._id,
- entity_type: 'file'
- })
- })
- it('should time the request', function() {
- return this.metrics.Timer.prototype.done.called.should.equal(true)
- })
- it('should remove the uploaded file', function() {
- return this.fs.unlink.calledWith(this.path).should.equal(true)
- })
- })
- describe('when FileSystemImportManager.addEntity returns a generic error', function() {
- beforeEach(function() {
- this.FileSystemImportManager.addEntity = sinon
- .stub()
- .callsArgWith(6, new Error('Sorry something went wrong'))
- return this.ProjectUploadController.uploadFile(this.req, this.res)
- })
- it('should return an unsuccessful response to the FileUploader client', function() {
- return expect(this.res.body).to.deep.equal({
- success: false
- })
- })
- })
- describe('when FileSystemImportManager.addEntity returns a too many files error', function() {
- beforeEach(function() {
- this.FileSystemImportManager.addEntity = sinon
- .stub()
- .callsArgWith(6, new Error('project_has_too_many_files'))
- return this.ProjectUploadController.uploadFile(this.req, this.res)
- })
- it('should return an unsuccessful response to the FileUploader client', function() {
- return expect(this.res.body).to.deep.equal({
- success: false,
- error: 'project_has_too_many_files'
- })
- })
- })
- describe('with a bad request', function() {
- beforeEach(function() {
- this.req.file.originalname = ''
- return this.ProjectUploadController.uploadFile(this.req, this.res)
- })
- it('should return a a non success response', function() {
- return expect(this.res.body).to.deep.equal({
- success: false
- })
- })
- })
- })
- })
|