| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527 |
- import { expect } from 'chai'
- import _ from 'lodash'
- import timekeeper from 'timekeeper'
- import Settings from '@overleaf/settings'
- import UserHelper from './helpers/User.mjs'
- import express from 'express'
- import { plainTextResponse } from '../../../app/src/infrastructure/Response.mjs'
- const User = UserHelper.promises
- const LinkedUrlProxy = express()
- LinkedUrlProxy.get('/', (req, res, next) => {
- if (req.query.url === 'http://example.com/foo') {
- return plainTextResponse(res, 'foo foo foo')
- } else if (req.query.url === 'http://example.com/bar') {
- return plainTextResponse(res, 'bar bar bar')
- } else if (req.query.url === 'http://example.com/large') {
- return plainTextResponse(res, 'x'.repeat(Settings.maxUploadSize + 1))
- } else {
- return res.sendStatus(404)
- }
- })
- describe('LinkedFiles', function () {
- before(function () {
- timekeeper.freeze(new Date())
- })
- after(function () {
- timekeeper.reset()
- })
- let projectOne, projectOneId, projectOneRootFolderId
- let projectTwo, projectTwoId, projectTwoRootFolderId
- const sourceDocName = 'test.txt'
- let sourceDocId
- let owner
- let server
- before(function (done) {
- server = LinkedUrlProxy.listen(6543, done)
- })
- after(function (done) {
- server.close(done)
- })
- beforeEach(async function () {
- owner = new User()
- await owner.login()
- })
- describe('creating a project linked file', function () {
- beforeEach(async function () {
- projectOneId = await owner.createProject('plf-test-one', {
- template: 'blank',
- })
- projectOne = await owner.getProject(projectOneId)
- projectOneRootFolderId = projectOne.rootFolder[0]._id.toString()
- projectTwoId = await owner.createProject('plf-test-two', {
- template: 'blank',
- })
- projectTwo = await owner.getProject(projectTwoId)
- projectTwoRootFolderId = projectTwo.rootFolder[0]._id.toString()
- sourceDocId = await owner.createDocInProject(
- projectTwoId,
- projectTwoRootFolderId,
- sourceDocName
- )
- await owner.createDocInProject(
- projectTwoId,
- projectTwoRootFolderId,
- 'some-harmless-doc.txt'
- )
- })
- it('should produce a list of the users projects and their entities', async function () {
- let { body } = await owner.doRequest('get', {
- url: '/user/projects',
- json: true,
- })
- expect(body).to.deep.equal({
- projects: [
- {
- _id: projectOneId,
- name: 'plf-test-one',
- accessLevel: 'owner',
- },
- {
- _id: projectTwoId,
- name: 'plf-test-two',
- accessLevel: 'owner',
- },
- ],
- })
- ;({ body } = await owner.doRequest('get', {
- url: `/project/${projectTwoId}/entities`,
- json: true,
- }))
- expect(body).to.deep.equal({
- project_id: projectTwoId,
- entities: [
- { path: '/main.tex', type: 'doc' },
- { path: '/some-harmless-doc.txt', type: 'doc' },
- { path: '/test.txt', type: 'doc' },
- ],
- })
- })
- it('should import a file and refresh it if there is no v1 id', async function () {
- // import the file from the source project
- let { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- name: 'test-link.txt',
- parent_folder_id: projectOneRootFolderId,
- provider: 'project_file',
- data: {
- source_project_id: projectTwoId,
- source_entity_path: `/${sourceDocName}`,
- },
- },
- })
- expect(response.statusCode).to.equal(200)
- const existingFileId = body.new_file_id
- expect(existingFileId).to.exist
- let updatedProjectOne = await owner.getProject(projectOneId)
- let firstFile = updatedProjectOne.rootFolder[0].fileRefs[0]
- expect(firstFile._id.toString()).to.equal(existingFileId.toString())
- expect(firstFile.linkedFileData).to.deep.equal({
- provider: 'project_file',
- source_project_id: projectTwoId,
- source_entity_path: `/${sourceDocName}`,
- importedAt: new Date().toISOString(),
- })
- expect(firstFile.name).to.equal('test-link.txt')
- // refresh the file
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file/${existingFileId}/refresh`,
- json: true,
- }))
- expect(response.statusCode).to.equal(200)
- const newFileId = body.new_file_id
- expect(newFileId).to.exist
- expect(newFileId).to.not.equal(existingFileId)
- updatedProjectOne = await owner.getProject(projectOneId)
- firstFile = updatedProjectOne.rootFolder[0].fileRefs[0]
- expect(firstFile._id.toString()).to.equal(newFileId.toString())
- expect(firstFile.name).to.equal('test-link.txt')
- // should not work if there is a v1 id
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- name: 'test-link-should-not-work.txt',
- parent_folder_id: projectOneRootFolderId,
- provider: 'project_file',
- data: {
- v1_source_doc_id: 1234,
- source_entity_path: `/${sourceDocName}`,
- },
- },
- }))
- expect(response.statusCode).to.equal(403)
- expect(body).to.equal(
- 'The project that contains this file is not shared with you'
- )
- })
- it('should generate a proper error message when the source file has been deleted', async function () {
- // import the file from the source project
- let { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- name: 'test-link.txt',
- parent_folder_id: projectOneRootFolderId,
- provider: 'project_file',
- data: {
- source_project_id: projectTwoId,
- source_entity_path: `/${sourceDocName}`,
- },
- },
- })
- expect(response.statusCode).to.equal(200)
- const existingFileId = body.new_file_id
- expect(existingFileId).to.exist
- // rename the source file
- await owner.renameItemInProject(
- projectTwoId,
- 'doc',
- sourceDocId,
- 'renamed-doc.txt'
- )
- // refresh the file
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file/${existingFileId}/refresh`,
- json: true,
- }))
- expect(response.statusCode).to.equal(404)
- expect(body).to.equal('Source file not found')
- })
- })
- describe('with a linked project_file from a v1 project that has not been imported', function () {
- beforeEach(async function () {
- projectOneId = await owner.createProject('plf-v1-test-one', {
- template: 'blank',
- })
- projectOne = await owner.getProject(projectOneId)
- projectOneRootFolderId = projectOne.rootFolder[0]._id.toString()
- projectOne.rootFolder[0].fileRefs.push({
- linkedFileData: {
- provider: 'project_file',
- v1_source_doc_id: 9999999, // We won't find this id in the database
- source_entity_path: 'example.jpeg',
- },
- _id: 'abcd',
- rev: 0,
- created: new Date(),
- name: 'example.jpeg',
- })
- await owner.saveProject(projectOne)
- })
- it('should refuse to refresh', async function () {
- const { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file/abcd/refresh`,
- json: true,
- })
- expect(response.statusCode).to.equal(409)
- expect(body).to.equal(
- 'Sorry, the source project is not yet imported to Overleaf v2. Please import it to Overleaf v2 to refresh this file'
- )
- })
- })
- describe('creating a URL based linked file', function () {
- beforeEach(async function () {
- projectOneId = await owner.createProject('url-linked-files-project', {
- template: 'blank',
- })
- projectOne = await owner.getProject(projectOneId)
- projectOneRootFolderId = projectOne.rootFolder[0]._id.toString()
- })
- it('should download, create and replace a file', async function () {
- // downloading the initial file
- let { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'http://example.com/foo',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-1',
- },
- })
- expect(response.statusCode).to.equal(200)
- let updatedProject = await owner.getProject(projectOneId)
- let file = updatedProject.rootFolder[0].fileRefs[0]
- expect(file.linkedFileData).to.deep.equal({
- provider: 'url',
- url: 'http://example.com/foo',
- importedAt: new Date().toISOString(),
- })
- ;({ response, body } = await owner.doRequest(
- 'get',
- `/project/${projectOneId}/file/${file._id}`
- ))
- expect(response.statusCode).to.equal(200)
- expect(body).to.equal('foo foo foo')
- // replacing the file
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'http://example.com/foo',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-2',
- },
- }))
- expect(response.statusCode).to.equal(200)
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'http://example.com/bar',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-2',
- },
- }))
- expect(response.statusCode).to.equal(200)
- updatedProject = await owner.getProject(projectOneId)
- file = updatedProject.rootFolder[0].fileRefs[1]
- expect(file.linkedFileData).to.deep.equal({
- provider: 'url',
- url: 'http://example.com/bar',
- importedAt: new Date().toISOString(),
- })
- ;({ response, body } = await owner.doRequest(
- 'get',
- `/project/${projectOneId}/file/${file._id}`
- ))
- expect(response.statusCode).to.equal(200)
- expect(body).to.equal('bar bar bar')
- })
- it('should return an error if the file exceeds the maximum size', async function () {
- // download does not succeed
- const { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'http://example.com/large',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-large-file-1',
- },
- })
- expect(response.statusCode).to.equal(422)
- expect(body).to.equal('File too large')
- })
- it("should return an error if the file can't be downloaded", async function () {
- // download does not succeed
- let { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'http://example.com/does-not-exist',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-3',
- },
- })
- expect(response.statusCode).to.equal(422) // unprocessable
- expect(body).to.equal(
- 'Your URL could not be reached (404 status code). Please check it and try again.'
- )
- // url is invalid
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: '!^$%',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-4',
- },
- }))
- expect(response.statusCode).to.equal(422) // unprocessable
- expect(body).to.equal(
- 'Your URL is not valid. Please check it and try again.'
- )
- // URL is non-http
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'ftp://127.0.0.1',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-5',
- },
- }))
- expect(response.statusCode).to.equal(422) // unprocessable
- expect(body).to.equal(
- 'Your URL is not valid. Please check it and try again.'
- )
- })
- it('should accept a URL withuot a leading http://, and add it', async function () {
- let { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- provider: 'url',
- data: {
- url: 'example.com/foo',
- },
- parent_folder_id: projectOneRootFolderId,
- name: 'url-test-file-6',
- },
- })
- expect(response.statusCode).to.equal(200)
- const updatedProject = await owner.getProject(projectOneId)
- const file = _.find(
- updatedProject.rootFolder[0].fileRefs,
- file => file.name === 'url-test-file-6'
- )
- expect(file.linkedFileData).to.deep.equal({
- provider: 'url',
- url: 'http://example.com/foo',
- importedAt: new Date().toISOString(),
- })
- ;({ response, body } = await owner.doRequest(
- 'get',
- `/project/${projectOneId}/file/${file._id}`
- ))
- expect(response.statusCode).to.equal(200)
- expect(body).to.equal('foo foo foo')
- })
- })
- // TODO: Add test for asking for host that return ENOTFOUND
- // (This will probably end up handled by the proxy)
- describe('creating a linked output file', function () {
- beforeEach(async function () {
- projectOneId = await owner.createProject('output-test-one', {
- template: 'blank',
- })
- projectOne = await owner.getProject(projectOneId)
- projectOneRootFolderId = projectOne.rootFolder[0]._id.toString()
- projectTwoId = await owner.createProject('output-test-two', {
- template: 'blank',
- })
- projectTwo = await owner.getProject(projectTwoId)
- projectTwoRootFolderId = projectTwo.rootFolder[0]._id.toString()
- })
- it('should import the output.pdf file from the source project and refresh it', async function () {
- // import the file
- let { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file`,
- json: {
- name: 'test.pdf',
- parent_folder_id: projectOneRootFolderId,
- provider: 'project_output_file',
- data: {
- source_project_id: projectTwoId,
- source_output_file_path: 'output.pdf',
- build_id: '1234-abcd',
- },
- },
- })
- expect(response.statusCode).to.equal(200)
- const existingFileId = body.new_file_id
- expect(existingFileId).to.exist
- const updatedProject = await owner.getProject(projectOneId)
- const firstFile = updatedProject.rootFolder[0].fileRefs[0]
- expect(firstFile._id.toString()).to.equal(existingFileId.toString())
- expect(firstFile.linkedFileData).to.deep.equal({
- provider: 'project_output_file',
- source_project_id: projectTwoId,
- source_output_file_path: 'output.pdf',
- build_id: '1234-abcd',
- importedAt: new Date().toISOString(),
- })
- expect(firstFile.name).to.equal('test.pdf')
- // refresh the file
- ;({ response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file/${existingFileId}/refresh`,
- json: true,
- }))
- expect(response.statusCode).to.equal(200)
- const refreshedFileId = body.new_file_id
- expect(refreshedFileId).to.exist
- expect(refreshedFileId).to.not.equal(existingFileId)
- const refreshedProject = await owner.getProject(projectOneId)
- const refreshedFile = refreshedProject.rootFolder[0].fileRefs[0]
- expect(refreshedFile._id.toString()).to.equal(refreshedFileId.toString())
- expect(refreshedFile.name).to.equal('test.pdf')
- })
- })
- describe('with a linked project_output_file from a v1 project that has not been imported', function () {
- beforeEach(async function () {
- projectOneId = await owner.createProject('output-v1-test-one', {
- template: 'blank',
- })
- projectOne = await owner.getProject(projectOneId)
- projectOneRootFolderId = projectOne.rootFolder[0]._id.toString()
- projectOne.rootFolder[0].fileRefs.push({
- linkedFileData: {
- provider: 'project_output_file',
- v1_source_doc_id: 9999999, // We won't find this id in the database
- source_output_file_path: 'output.pdf',
- },
- _id: 'abcdef',
- rev: 0,
- created: new Date(),
- name: 'whatever.pdf',
- })
- await owner.saveProject(projectOne)
- })
- it('should refuse to refresh', async function () {
- const { response, body } = await owner.doRequest('post', {
- url: `/project/${projectOneId}/linked_file/abcdef/refresh`,
- json: true,
- })
- expect(response.statusCode).to.equal(409)
- expect(body).to.equal(
- 'Sorry, the source project is not yet imported to Overleaf v2. Please import it to Overleaf v2 to refresh this file'
- )
- })
- })
- })
|