| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533 |
- /* eslint-disable
- no-return-assign,
- */
- // TODO: This file was created by bulk-decaffeinate.
- // Fix any style issues and re-enable lint.
- /*
- * decaffeinate suggestions:
- * DS101: Remove unnecessary use of Array.from
- * 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 SandboxedModule = require('sandboxed-module')
- const sinon = require('sinon')
- const { expect } = require('chai')
- const modulePath = require('node:path').join(
- __dirname,
- '../../../app/js/ResourceWriter'
- )
- const path = require('node:path')
- describe('ResourceWriter', function () {
- beforeEach(function () {
- let Timer
- this.ResourceWriter = SandboxedModule.require(modulePath, {
- singleOnly: true,
- requires: {
- fs: (this.fs = {
- mkdir: sinon.stub().callsArg(1),
- unlink: sinon.stub().callsArg(1),
- }),
- './ResourceStateManager': (this.ResourceStateManager = {}),
- './UrlCache': (this.UrlCache = {
- createProjectDir: sinon.stub().yields(),
- }),
- './OutputFileFinder': (this.OutputFileFinder = {}),
- '@overleaf/metrics': (this.Metrics = {
- inc: sinon.stub(),
- Timer: (Timer = (function () {
- Timer = class Timer {
- static initClass() {
- this.prototype.done = sinon.stub()
- }
- }
- Timer.initClass()
- return Timer
- })()),
- }),
- },
- })
- this.project_id = 'project-id-123'
- this.basePath = '/path/to/write/files/to'
- return (this.callback = sinon.stub())
- })
- describe('syncResourcesToDisk on a full request', function () {
- beforeEach(function () {
- this.resources = ['resource-1-mock', 'resource-2-mock', 'resource-3-mock']
- this.request = {
- project_id: this.project_id,
- syncState: (this.syncState = '0123456789abcdef'),
- resources: this.resources,
- }
- this.ResourceWriter._writeResourceToDisk = sinon.stub().callsArg(3)
- this.ResourceWriter._removeExtraneousFiles = sinon.stub().yields(null)
- this.ResourceStateManager.saveProjectState = sinon.stub().callsArg(3)
- return this.ResourceWriter.syncResourcesToDisk(
- this.request,
- this.basePath,
- this.callback
- )
- })
- it('should remove old files', function () {
- return this.ResourceWriter._removeExtraneousFiles
- .calledWith(this.request, this.resources, this.basePath)
- .should.equal(true)
- })
- it('should write each resource to disk', function () {
- return Array.from(this.resources).map(resource =>
- this.ResourceWriter._writeResourceToDisk
- .calledWith(this.project_id, resource, this.basePath)
- .should.equal(true)
- )
- })
- it('should store the sync state and resource list', function () {
- return this.ResourceStateManager.saveProjectState
- .calledWith(this.syncState, this.resources, this.basePath)
- .should.equal(true)
- })
- return it('should call the callback', function () {
- return this.callback.called.should.equal(true)
- })
- })
- describe('syncResourcesToDisk on an incremental update', function () {
- beforeEach(function () {
- this.resources = ['resource-1-mock']
- this.request = {
- project_id: this.project_id,
- syncType: 'incremental',
- syncState: (this.syncState = '1234567890abcdef'),
- resources: this.resources,
- }
- this.fullResources = this.resources.concat(['file-1'])
- this.ResourceWriter._writeResourceToDisk = sinon.stub().callsArg(3)
- this.ResourceWriter._removeExtraneousFiles = sinon
- .stub()
- .yields(null, (this.outputFiles = []), (this.allFiles = []))
- this.ResourceStateManager.checkProjectStateMatches = sinon
- .stub()
- .callsArgWith(2, null, this.fullResources)
- this.ResourceStateManager.saveProjectState = sinon.stub().callsArg(3)
- this.ResourceStateManager.checkResourceFiles = sinon.stub().callsArg(3)
- return this.ResourceWriter.syncResourcesToDisk(
- this.request,
- this.basePath,
- this.callback
- )
- })
- it('should check the sync state matches', function () {
- return this.ResourceStateManager.checkProjectStateMatches
- .calledWith(this.syncState, this.basePath)
- .should.equal(true)
- })
- it('should remove old files', function () {
- return this.ResourceWriter._removeExtraneousFiles
- .calledWith(this.request, this.fullResources, this.basePath)
- .should.equal(true)
- })
- it('should check each resource exists', function () {
- return this.ResourceStateManager.checkResourceFiles
- .calledWith(this.fullResources, this.allFiles, this.basePath)
- .should.equal(true)
- })
- it('should write each resource to disk', function () {
- return Array.from(this.resources).map(resource =>
- this.ResourceWriter._writeResourceToDisk
- .calledWith(this.project_id, resource, this.basePath)
- .should.equal(true)
- )
- })
- return it('should call the callback', function () {
- return this.callback.called.should.equal(true)
- })
- })
- describe('syncResourcesToDisk on an incremental update when the state does not match', function () {
- beforeEach(function () {
- this.resources = ['resource-1-mock']
- this.request = {
- project_id: this.project_id,
- syncType: 'incremental',
- syncState: (this.syncState = '1234567890abcdef'),
- resources: this.resources,
- }
- this.ResourceStateManager.checkProjectStateMatches = sinon
- .stub()
- .callsArgWith(2, (this.error = new Error()))
- return this.ResourceWriter.syncResourcesToDisk(
- this.request,
- this.basePath,
- this.callback
- )
- })
- it('should check whether the sync state matches', function () {
- return this.ResourceStateManager.checkProjectStateMatches
- .calledWith(this.syncState, this.basePath)
- .should.equal(true)
- })
- return it('should call the callback with an error', function () {
- return this.callback.calledWith(this.error).should.equal(true)
- })
- })
- describe('_removeExtraneousFiles', function () {
- beforeEach(function () {
- this.output_files = [
- {
- path: 'output.pdf',
- type: 'pdf',
- },
- {
- path: 'extra/file.tex',
- type: 'tex',
- },
- {
- path: 'extra.aux',
- type: 'aux',
- },
- {
- path: 'cache/_chunk1',
- },
- {
- path: 'figures/image-eps-converted-to.pdf',
- type: 'pdf',
- },
- {
- path: 'foo/main-figure0.md5',
- type: 'md5',
- },
- {
- path: 'foo/main-figure0.dpth',
- type: 'dpth',
- },
- {
- path: 'foo/main-figure0.pdf',
- type: 'pdf',
- },
- {
- path: '_minted-main/default-pyg-prefix.pygstyle',
- type: 'pygstyle',
- },
- {
- path: '_minted-main/default.pygstyle',
- type: 'pygstyle',
- },
- {
- path: '_minted-main/35E248B60965545BD232AE9F0FE9750D504A7AF0CD3BAA7542030FC560DFCC45.pygtex',
- type: 'pygtex',
- },
- {
- path: '_markdown_main/30893013dec5d869a415610079774c2f.md.tex',
- type: 'tex',
- },
- {
- path: 'output.stdout',
- },
- {
- path: 'output.stderr',
- },
- ]
- this.resources = 'mock-resources'
- this.request = {
- project_id: this.project_id,
- syncType: 'incremental',
- syncState: (this.syncState = '1234567890abcdef'),
- resources: this.resources,
- metricsOpts: { path: 'foo' },
- }
- this.OutputFileFinder.findOutputFiles = sinon
- .stub()
- .callsArgWith(2, null, this.output_files)
- this.ResourceWriter._deleteFileIfNotDirectory = sinon.stub().callsArg(1)
- return this.ResourceWriter._removeExtraneousFiles(
- this.request,
- this.resources,
- this.basePath,
- this.callback
- )
- })
- it('should find the existing output files', function () {
- return this.OutputFileFinder.findOutputFiles
- .calledWith(this.resources, this.basePath)
- .should.equal(true)
- })
- it('should delete the output files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'output.pdf'))
- .should.equal(true)
- })
- it('should delete the stdout log file', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'output.stdout'))
- .should.equal(true)
- })
- it('should delete the stderr log file', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'output.stderr'))
- .should.equal(true)
- })
- it('should delete the extra files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'extra/file.tex'))
- .should.equal(true)
- })
- it('should not delete the extra aux files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'extra.aux'))
- .should.equal(false)
- })
- it('should not delete the knitr cache file', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'cache/_chunk1'))
- .should.equal(false)
- })
- it('should not delete the epstopdf converted files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(
- path.join(this.basePath, 'figures/image-eps-converted-to.pdf')
- )
- .should.equal(false)
- })
- it('should not delete the tikz md5 files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'foo/main-figure0.md5'))
- .should.equal(false)
- })
- it('should not delete the tikz dpth files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'foo/main-figure0.dpth'))
- .should.equal(false)
- })
- it('should not delete the tikz pdf files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, 'foo/main-figure0.pdf'))
- .should.equal(false)
- })
- it('should not delete the minted pygstyle files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(
- path.join(this.basePath, '_minted-main/default-pyg-prefix.pygstyle')
- )
- .should.equal(false)
- })
- it('should not delete the minted default pygstyle files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(path.join(this.basePath, '_minted-main/default.pygstyle'))
- .should.equal(false)
- })
- it('should not delete the minted default pygtex files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(
- path.join(
- this.basePath,
- '_minted-main/35E248B60965545BD232AE9F0FE9750D504A7AF0CD3BAA7542030FC560DFCC45.pygtex'
- )
- )
- .should.equal(false)
- })
- it('should not delete the markdown md.tex files', function () {
- return this.ResourceWriter._deleteFileIfNotDirectory
- .calledWith(
- path.join(
- this.basePath,
- '_markdown_main/30893013dec5d869a415610079774c2f.md.tex'
- )
- )
- .should.equal(false)
- })
- it('should call the callback', function () {
- return this.callback.called.should.equal(true)
- })
- return it('should time the request', function () {
- return this.Metrics.Timer.prototype.done.called.should.equal(true)
- })
- })
- describe('_writeResourceToDisk', function () {
- describe('with a url based resource', function () {
- beforeEach(function () {
- this.fs.mkdir = sinon.stub().callsArg(2)
- this.resource = {
- path: 'main.tex',
- url: 'http://www.example.com/primary/main.tex',
- fallbackURL: 'http://fallback.example.com/fallback/main.tex',
- modified: Date.now(),
- }
- this.UrlCache.downloadUrlToFile = sinon
- .stub()
- .callsArgWith(5, 'fake error downloading file')
- return this.ResourceWriter._writeResourceToDisk(
- this.project_id,
- this.resource,
- this.basePath,
- this.callback
- )
- })
- it('should ensure the directory exists', function () {
- this.fs.mkdir
- .calledWith(
- path.dirname(path.join(this.basePath, this.resource.path))
- )
- .should.equal(true)
- })
- it('should write the URL from the cache', function () {
- return this.UrlCache.downloadUrlToFile
- .calledWith(
- this.project_id,
- this.resource.url,
- this.resource.fallbackURL,
- path.join(this.basePath, this.resource.path),
- this.resource.modified
- )
- .should.equal(true)
- })
- it('should call the callback', function () {
- return this.callback.called.should.equal(true)
- })
- return it('should not return an error if the resource writer errored', function () {
- return expect(this.callback.args[0][0]).not.to.exist
- })
- })
- describe('with a content based resource', function () {
- beforeEach(function () {
- this.resource = {
- path: 'main.tex',
- content: 'Hello world',
- }
- this.fs.writeFile = sinon.stub().callsArg(2)
- this.fs.mkdir = sinon.stub().callsArg(2)
- return this.ResourceWriter._writeResourceToDisk(
- this.project_id,
- this.resource,
- this.basePath,
- this.callback
- )
- })
- it('should ensure the directory exists', function () {
- return this.fs.mkdir
- .calledWith(
- path.dirname(path.join(this.basePath, this.resource.path))
- )
- .should.equal(true)
- })
- it('should write the contents to disk', function () {
- return this.fs.writeFile
- .calledWith(
- path.join(this.basePath, this.resource.path),
- this.resource.content
- )
- .should.equal(true)
- })
- return it('should call the callback', function () {
- return this.callback.called.should.equal(true)
- })
- })
- return describe('with a file path that breaks out of the root folder', function () {
- beforeEach(function () {
- this.resource = {
- path: '../../main.tex',
- content: 'Hello world',
- }
- this.fs.writeFile = sinon.stub().callsArg(2)
- return this.ResourceWriter._writeResourceToDisk(
- this.project_id,
- this.resource,
- this.basePath,
- this.callback
- )
- })
- it('should not write to disk', function () {
- return this.fs.writeFile.called.should.equal(false)
- })
- it('should return an error', function () {
- this.callback.calledWith(sinon.match(Error)).should.equal(true)
- const message = this.callback.args[0][0].message
- expect(message).to.include('resource path is outside root directory')
- })
- })
- })
- return describe('checkPath', function () {
- describe('with a valid path', function () {
- beforeEach(function () {
- return this.ResourceWriter.checkPath('foo', 'bar', this.callback)
- })
- return it('should return the joined path', function () {
- return this.callback.calledWith(null, 'foo/bar').should.equal(true)
- })
- })
- describe('with an invalid path', function () {
- beforeEach(function () {
- this.ResourceWriter.checkPath('foo', 'baz/../../bar', this.callback)
- })
- it('should return an error', function () {
- this.callback.calledWith(sinon.match(Error)).should.equal(true)
- const message = this.callback.args[0][0].message
- expect(message).to.include('resource path is outside root directory')
- })
- })
- describe('with another invalid path matching on a prefix', function () {
- beforeEach(function () {
- return this.ResourceWriter.checkPath(
- 'foo',
- '../foobar/baz',
- this.callback
- )
- })
- it('should return an error', function () {
- this.callback.calledWith(sinon.match(Error)).should.equal(true)
- const message = this.callback.args[0][0].message
- expect(message).to.include('resource path is outside root directory')
- })
- })
- })
- })
|