| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976 |
- const sinon = require('sinon')
- const SandboxedModule = require('sandboxed-module')
- const path = require('path')
- const { ObjectId } = require('mongodb-legacy')
- const modulePath = path.join(
- __dirname,
- '../../../../app/src/Features/DocumentUpdater/DocumentUpdaterHandler'
- )
- describe('DocumentUpdaterHandler', function () {
- beforeEach(function () {
- this.project_id = 'project-id-923'
- this.projectHistoryId = 'ol-project-id-1'
- this.doc_id = 'doc-id-394'
- this.lines = ['one', 'two', 'three']
- this.version = 42
- this.user_id = 'mock-user-id-123'
- this.project = { _id: this.project_id }
- this.request = sinon.stub()
- this.projectEntityHandler = {}
- this.settings = {
- apis: {
- documentupdater: {
- url: 'http://document_updater.example.com',
- },
- project_history: {
- url: 'http://project_history.example.com',
- },
- },
- moduleImportSequence: [],
- }
- this.source = 'dropbox'
- this.callback = sinon.stub()
- this.handler = SandboxedModule.require(modulePath, {
- requires: {
- request: {
- defaults: () => {
- return this.request
- },
- },
- '@overleaf/settings': this.settings,
- '../Project/ProjectEntityHandler': this.projectEntityHandler,
- '../../models/Project': {
- Project: (this.Project = {}),
- },
- '../Project/ProjectGetter': (this.ProjectGetter = {
- getProjectWithoutLock: sinon.stub(),
- }),
- '../../Features/Project/ProjectLocator': {},
- '@overleaf/metrics': {
- Timer: class {
- done() {}
- },
- },
- '../FileStore/FileStoreHandler': {
- _buildUrl: sinon.stub().callsFake((projectId, fileId) => {
- return `http://filestore/project/${projectId}/file/${fileId}`
- }),
- },
- },
- })
- this.ProjectGetter.getProjectWithoutLock
- .withArgs(this.project_id)
- .yields(null, this.project)
- })
- describe('flushProjectToMongo', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- this.handler.flushProjectToMongo(this.project_id, this.callback)
- })
- it('should flush the document from the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/flush`,
- method: 'POST',
- })
- .should.equal(true)
- })
- it('should call the callback with no error', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.flushProjectToMongo(this.project_id, this.callback)
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.flushProjectToMongo(this.project_id, this.callback)
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('flushProjectToMongoAndDelete', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- this.handler.flushProjectToMongoAndDelete(
- this.project_id,
- this.callback
- )
- })
- it('should delete the project from the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}`,
- method: 'DELETE',
- })
- .should.equal(true)
- })
- it('should call the callback with no error', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.flushProjectToMongoAndDelete(
- this.project_id,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.flushProjectToMongoAndDelete(
- this.project_id,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('flushDocToMongo', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- this.handler.flushDocToMongo(
- this.project_id,
- this.doc_id,
- this.callback
- )
- })
- it('should flush the document from the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/flush`,
- method: 'POST',
- })
- .should.equal(true)
- })
- it('should call the callback with no error', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.flushDocToMongo(
- this.project_id,
- this.doc_id,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.flushDocToMongo(
- this.project_id,
- this.doc_id,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('deleteDoc', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- this.handler.deleteDoc(this.project_id, this.doc_id, this.callback)
- })
- it('should delete the document from the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}`,
- method: 'DELETE',
- })
- .should.equal(true)
- })
- it('should call the callback with no error', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.deleteDoc(this.project_id, this.doc_id, this.callback)
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.deleteDoc(this.project_id, this.doc_id, this.callback)
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- describe("with 'ignoreFlushErrors' option", function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- })
- it('when option is true, should send a `ignore_flush_errors=true` URL query to document-updater', function () {
- this.handler.deleteDoc(
- this.project_id,
- this.doc_id,
- true,
- this.callback
- )
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}?ignore_flush_errors=true`,
- method: 'DELETE',
- })
- .should.equal(true)
- })
- it("when option is false, shouldn't send any URL query to document-updater", function () {
- this.handler.deleteDoc(
- this.project_id,
- this.doc_id,
- false,
- this.callback
- )
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}`,
- method: 'DELETE',
- })
- .should.equal(true)
- })
- })
- })
- describe('setDocument', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- this.handler.setDocument(
- this.project_id,
- this.doc_id,
- this.user_id,
- this.lines,
- this.source,
- this.callback
- )
- })
- it('should set the document in the document updater', function () {
- this.request
- .calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}`,
- json: {
- lines: this.lines,
- source: this.source,
- user_id: this.user_id,
- },
- method: 'POST',
- timeout: 30 * 1000,
- })
- .should.equal(true)
- })
- it('should call the callback with no error', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.setDocument(
- this.project_id,
- this.doc_id,
- this.user_id,
- this.lines,
- this.source,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.setDocument(
- this.project_id,
- this.doc_id,
- this.user_id,
- this.lines,
- this.source,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('getComment', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.comment = {
- id: 'mock-comment-id-1',
- }
- this.body = this.comment
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.getComment(
- this.project_id,
- this.doc_id,
- this.comment.id,
- this.callback
- )
- })
- it('should get the comment from the document updater', function () {
- const url = `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/comment/${this.comment.id}`
- this.request
- .calledWith({
- url,
- method: 'GET',
- json: true,
- timeout: 30 * 1000,
- })
- .should.equal(true)
- })
- it('should call the callback with the comment', function () {
- this.callback.calledWithExactly(null, this.comment).should.equal(true)
- })
- })
- })
- describe('getDocument', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.body = {
- lines: this.lines,
- version: this.version,
- ops: (this.ops = ['mock-op-1', 'mock-op-2']),
- ranges: (this.ranges = { mock: 'ranges' }),
- }
- this.fromVersion = 2
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.getDocument(
- this.project_id,
- this.doc_id,
- this.fromVersion,
- this.callback
- )
- })
- it('should get the document from the document updater', function () {
- this.request
- .calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}?fromVersion=${this.fromVersion}`,
- method: 'GET',
- json: true,
- timeout: 30 * 1000,
- })
- .should.equal(true)
- })
- it('should call the callback with the lines and version', function () {
- this.callback
- .calledWith(null, this.lines, this.version, this.ranges, this.ops)
- .should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.getDocument(
- this.project_id,
- this.doc_id,
- this.fromVersion,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.getDocument(
- this.project_id,
- this.doc_id,
- this.fromVersion,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('getProjectDocsIfMatch', function () {
- beforeEach(function () {
- this.project_state_hash = '1234567890abcdef'
- })
- describe('successfully', function () {
- beforeEach(function () {
- this.doc0 = {
- _id: this.doc_id,
- lines: this.lines,
- v: this.version,
- }
- this.docs = [this.doc0, this.doc0, this.doc0]
- this.body = JSON.stringify(this.docs)
- this.request.post = sinon
- .stub()
- .callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.getProjectDocsIfMatch(
- this.project_id,
- this.project_state_hash,
- this.callback
- )
- })
- it('should get the documents from the document updater', function () {
- const url = `${this.settings.apis.documentupdater.url}/project/${this.project_id}/get_and_flush_if_old?state=${this.project_state_hash}`
- this.request.post.calledWith(url).should.equal(true)
- })
- it('should call the callback with the documents', function () {
- this.callback.calledWithExactly(null, this.docs).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.post = sinon
- .stub()
- .callsArgWith(1, new Error('something went wrong'), null, null)
- this.handler.getProjectDocsIfMatch(
- this.project_id,
- this.project_state_hash,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a conflict error code', function () {
- beforeEach(function () {
- this.request.post = sinon
- .stub()
- .callsArgWith(1, null, { statusCode: 409 }, 'Conflict')
- this.handler.getProjectDocsIfMatch(
- this.project_id,
- this.project_state_hash,
- this.callback
- )
- })
- it('should return the callback with no documents', function () {
- this.callback.alwaysCalledWithExactly().should.equal(true)
- })
- })
- })
- describe('clearProjectState', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 200 })
- this.handler.clearProjectState(this.project_id, this.callback)
- })
- it('should clear the project state from the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/clearState`,
- method: 'POST',
- })
- .should.equal(true)
- })
- it('should call the callback', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.clearProjectState(this.project_id, this.callback)
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns an error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, null)
- this.handler.clearProjectState(this.project_id, this.callback)
- })
- it('should return the callback with no documents', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('acceptChanges', function () {
- beforeEach(function () {
- this.change_id = 'mock-change-id-1'
- })
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.acceptChanges(
- this.project_id,
- this.doc_id,
- [this.change_id],
- this.callback
- )
- })
- it('should accept the change in the document updater', function () {
- this.request
- .calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/change/accept`,
- json: {
- change_ids: [this.change_id],
- },
- method: 'POST',
- timeout: 30 * 1000,
- })
- .should.equal(true)
- })
- it('should call the callback', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.acceptChanges(
- this.project_id,
- this.doc_id,
- [this.change_id],
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.acceptChanges(
- this.project_id,
- this.doc_id,
- [this.change_id],
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('deleteThread', function () {
- beforeEach(function () {
- this.thread_id = 'mock-thread-id-1'
- })
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.deleteThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should delete the thread in the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/comment/${this.thread_id}`,
- method: 'DELETE',
- })
- .should.equal(true)
- })
- it('should call the callback', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.deleteThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.deleteThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('resolveThread', function () {
- beforeEach(function () {
- this.thread_id = 'mock-thread-id-1'
- })
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.resolveThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should resolve the thread in the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/comment/${this.thread_id}/resolve`,
- method: 'POST',
- })
- .should.equal(true)
- })
- it('should call the callback', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.resolveThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.resolveThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('reopenThread', function () {
- beforeEach(function () {
- this.thread_id = 'mock-thread-id-1'
- })
- describe('successfully', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.reopenThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should reopen the thread in the document updater', function () {
- this.request
- .calledWithMatch({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/comment/${this.thread_id}/reopen`,
- method: 'POST',
- })
- .should.equal(true)
- })
- it('should call the callback', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.reopenThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.reopenThread(
- this.project_id,
- this.doc_id,
- this.thread_id,
- this.user_id,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- describe('updateProjectStructure ', function () {
- beforeEach(function () {
- this.user_id = 1234
- this.version = 999
- })
- describe('with project history disabled', function () {
- beforeEach(function () {
- this.settings.apis.project_history.sendProjectStructureOps = false
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- {},
- this.source,
- this.callback
- )
- })
- it('does not make a web request', function () {
- this.request.called.should.equal(false)
- })
- it('calls the callback', function () {
- this.callback.called.should.equal(true)
- })
- })
- describe('with project history enabled', function () {
- beforeEach(function () {
- this.settings.apis.project_history.sendProjectStructureOps = true
- this.url = `${this.settings.apis.documentupdater.url}/project/${this.project_id}`
- this.request.callsArgWith(1, null, { statusCode: 204 }, '')
- })
- describe('when an entity has changed name', function () {
- it('should send the structure update to the document updater', function (done) {
- this.docIdA = new ObjectId()
- this.docIdB = new ObjectId()
- this.changes = {
- oldDocs: [
- { path: '/old_a', doc: { _id: this.docIdA } },
- { path: '/old_b', doc: { _id: this.docIdB } },
- ],
- // create new instances of the same ObjectIds so that == doesn't pass
- newDocs: [
- {
- path: '/old_a',
- doc: { _id: new ObjectId(this.docIdA.toString()) },
- },
- {
- path: '/new_b',
- doc: { _id: new ObjectId(this.docIdB.toString()) },
- },
- ],
- newProject: { version: this.version },
- }
- const updates = [
- {
- type: 'rename-doc',
- id: this.docIdB.toString(),
- pathname: '/old_b',
- newPathname: '/new_b',
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request
- .calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- .should.equal(true)
- done()
- }
- )
- })
- })
- describe('when a doc has been added', function () {
- it('should send the structure update to the document updater', function (done) {
- this.docId = new ObjectId()
- this.changes = {
- newDocs: [
- { path: '/foo', docLines: 'a\nb', doc: { _id: this.docId } },
- ],
- newProject: { version: this.version },
- }
- const updates = [
- {
- type: 'add-doc',
- id: this.docId.toString(),
- pathname: '/foo',
- docLines: 'a\nb',
- historyRangesSupport: false,
- url: undefined,
- hash: undefined,
- ranges: undefined,
- metadata: undefined,
- createdBlob: false,
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- })
- describe('when a file has been added', function () {
- it('should send the structure update to the document updater', function (done) {
- this.fileId = new ObjectId()
- this.changes = {
- newFiles: [
- {
- path: '/bar',
- url: 'filestore.example.com/file',
- file: { _id: this.fileId, hash: '12345' },
- },
- ],
- newProject: { version: this.version },
- }
- const updates = [
- {
- type: 'add-file',
- id: this.fileId.toString(),
- pathname: '/bar',
- url: 'filestore.example.com/file',
- docLines: undefined,
- historyRangesSupport: false,
- hash: '12345',
- ranges: undefined,
- createdBlob: false,
- metadata: undefined,
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- })
- describe('when an entity has been deleted', function () {
- it('should end the structure update to the document updater', function (done) {
- this.docId = new ObjectId()
- this.changes = {
- oldDocs: [
- { path: '/foo', docLines: 'a\nb', doc: { _id: this.docId } },
- ],
- newProject: { version: this.version },
- }
- const updates = [
- {
- type: 'rename-doc',
- id: this.docId.toString(),
- pathname: '/foo',
- newPathname: '',
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- })
- describe('when a file is converted to a doc', function () {
- it('should send the delete first', function (done) {
- this.docId = new ObjectId()
- this.fileId = new ObjectId()
- this.changes = {
- oldFiles: [
- {
- path: '/foo.doc',
- url: 'filestore.example.com/file',
- file: { _id: this.fileId },
- },
- ],
- newDocs: [
- {
- path: '/foo.doc',
- docLines: 'hello there',
- doc: { _id: this.docId },
- },
- ],
- newProject: { version: this.version },
- }
- const updates = [
- {
- type: 'rename-file',
- id: this.fileId.toString(),
- pathname: '/foo.doc',
- newPathname: '',
- },
- {
- type: 'add-doc',
- id: this.docId.toString(),
- pathname: '/foo.doc',
- docLines: 'hello there',
- historyRangesSupport: false,
- url: undefined,
- hash: undefined,
- ranges: undefined,
- metadata: undefined,
- createdBlob: false,
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- })
- describe('when the project version is missing', function () {
- it('should call the callback with an error', function () {
- this.docId = new ObjectId()
- this.changes = {
- oldDocs: [
- { path: '/foo', docLines: 'a\nb', doc: { _id: this.docId } },
- ],
- }
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- this.callback
- )
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- const firstCallArgs = this.callback.args[0]
- firstCallArgs[0].message.should.equal(
- 'did not receive project version in changes'
- )
- })
- })
- describe('when ranges are present', function () {
- beforeEach(function () {
- this.docId = new ObjectId()
- this.ranges = {
- changes: [
- {
- op: { p: 0, i: 'foo' },
- metadata: { ts: '2024-01-01T00:00:00.000Z', user_id: 'user-1' },
- },
- {
- op: { p: 7, d: ' baz' },
- metadata: { ts: '2024-02-01T00:00:00.000Z', user_id: 'user-1' },
- },
- ],
- comments: [
- {
- op: { p: 4, c: 'bar', t: 'comment-1' },
- metadata: { resolved: false },
- },
- ],
- }
- this.changes = {
- newDocs: [
- {
- path: '/foo',
- docLines: 'foo\nbar',
- doc: { _id: this.docId },
- ranges: this.ranges,
- },
- ],
- newProject: { version: this.version },
- }
- })
- it('should forward ranges', function (done) {
- const updates = [
- {
- type: 'add-doc',
- id: this.docId.toString(),
- pathname: '/foo',
- docLines: 'foo\nbar',
- historyRangesSupport: false,
- url: undefined,
- hash: undefined,
- ranges: this.ranges,
- metadata: undefined,
- createdBlob: false,
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- it('should include flag when history ranges support is enabled', function (done) {
- this.ProjectGetter.getProjectWithoutLock
- .withArgs(this.project_id)
- .yields(null, {
- _id: this.project_id,
- overleaf: { history: { rangesSupportEnabled: true } },
- })
- const updates = [
- {
- type: 'add-doc',
- id: this.docId.toString(),
- pathname: '/foo',
- docLines: 'foo\nbar',
- historyRangesSupport: true,
- url: undefined,
- hash: undefined,
- ranges: this.ranges,
- metadata: undefined,
- createdBlob: false,
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- })
- describe('with filestore disabled', function () {
- beforeEach(function () {
- this.settings.disableFilestore = true
- })
- it('should add files without URL and with createdBlob', function (done) {
- this.fileId = new ObjectId()
- this.changes = {
- newFiles: [
- {
- path: '/bar',
- url: 'filestore.example.com/file',
- file: { _id: this.fileId, hash: '12345' },
- },
- ],
- newProject: { version: this.version },
- }
- const updates = [
- {
- type: 'add-file',
- id: this.fileId.toString(),
- pathname: '/bar',
- docLines: undefined,
- historyRangesSupport: false,
- url: undefined,
- hash: '12345',
- ranges: undefined,
- createdBlob: true,
- metadata: undefined,
- },
- ]
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- () => {
- this.request.should.have.been.calledWith({
- url: this.url,
- method: 'POST',
- json: {
- updates,
- userId: this.user_id,
- version: this.version,
- projectHistoryId: this.projectHistoryId,
- source: this.source,
- },
- timeout: 30 * 1000,
- })
- done()
- }
- )
- })
- it('should flag files without hash', function (done) {
- this.fileId = new ObjectId()
- this.changes = {
- newFiles: [
- {
- path: '/bar',
- url: 'filestore.example.com/file',
- file: { _id: this.fileId },
- },
- ],
- newProject: { version: this.version },
- }
- this.handler.updateProjectStructure(
- this.project_id,
- this.projectHistoryId,
- this.user_id,
- this.changes,
- this.source,
- err => {
- err.should.match(/found file with missing hash/)
- this.request.should.not.have.been.called
- done()
- }
- )
- })
- })
- })
- })
- describe('resyncProjectHistory', function () {
- it('should add docs', function (done) {
- const docId1 = new ObjectId()
- const docId2 = new ObjectId()
- const docs = [
- { doc: { _id: docId1 }, path: 'main.tex' },
- { doc: { _id: docId2 }, path: 'references.bib' },
- ]
- const files = []
- this.request.yields(null, { statusCode: 200 })
- const projectId = new ObjectId()
- const projectHistoryId = 99
- this.handler.resyncProjectHistory(
- projectId,
- projectHistoryId,
- docs,
- files,
- {},
- () => {
- this.request.should.have.been.calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${projectId}/history/resync`,
- method: 'POST',
- json: {
- docs: [
- { doc: docId1, path: 'main.tex' },
- { doc: docId2, path: 'references.bib' },
- ],
- files: [],
- projectHistoryId,
- },
- timeout: 6 * 60 * 1000,
- })
- done()
- }
- )
- })
- it('should add files', function (done) {
- const fileId1 = new ObjectId()
- const fileId2 = new ObjectId()
- const fileId3 = new ObjectId()
- const fileCreated2 = new Date()
- const fileCreated3 = new Date()
- const otherProjectId = new ObjectId().toString()
- const files = [
- { file: { _id: fileId1, hash: '42' }, path: '1.png' },
- {
- file: {
- _id: fileId2,
- hash: '1337',
- created: fileCreated2,
- linkedFileData: {
- provider: 'references-provider',
- },
- },
- path: '1.bib',
- },
- {
- file: {
- _id: fileId3,
- hash: '21',
- created: fileCreated3,
- linkedFileData: {
- provider: 'project_output_file',
- build_id: '1234-abc',
- clsiServerId: 'server-1',
- source_project_id: otherProjectId,
- source_output_file_path: 'foo/bar.txt',
- },
- },
- path: 'bar.txt',
- },
- ]
- const docs = []
- this.request.yields(null, { statusCode: 200 })
- const projectId = new ObjectId()
- const projectHistoryId = 99
- this.handler.resyncProjectHistory(
- projectId,
- projectHistoryId,
- docs,
- files,
- {},
- () => {
- this.request.should.have.been.calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${projectId}/history/resync`,
- method: 'POST',
- json: {
- docs: [],
- files: [
- {
- file: fileId1,
- _hash: '42',
- path: '1.png',
- url: `http://filestore/project/${projectId}/file/${fileId1}`,
- createdBlob: false,
- metadata: undefined,
- },
- {
- file: fileId2,
- _hash: '1337',
- path: '1.bib',
- url: `http://filestore/project/${projectId}/file/${fileId2}`,
- createdBlob: false,
- metadata: {
- importedAt: fileCreated2,
- provider: 'references-provider',
- },
- },
- {
- file: fileId3,
- _hash: '21',
- path: 'bar.txt',
- url: `http://filestore/project/${projectId}/file/${fileId3}`,
- createdBlob: false,
- metadata: {
- importedAt: fileCreated3,
- provider: 'project_output_file',
- source_project_id: otherProjectId,
- source_output_file_path: 'foo/bar.txt',
- // build_id and clsiServerId are omitted
- },
- },
- ],
- projectHistoryId,
- },
- timeout: 6 * 60 * 1000,
- })
- done()
- }
- )
- })
- describe('with filestore disabled', function () {
- beforeEach(function () {
- this.settings.disableFilestore = true
- })
- it('should add files without URL', function (done) {
- const fileId1 = new ObjectId()
- const fileId2 = new ObjectId()
- const fileId3 = new ObjectId()
- const fileCreated2 = new Date()
- const fileCreated3 = new Date()
- const otherProjectId = new ObjectId().toString()
- const files = [
- { file: { _id: fileId1, hash: '42' }, path: '1.png' },
- {
- file: {
- _id: fileId2,
- hash: '1337',
- created: fileCreated2,
- linkedFileData: {
- provider: 'references-provider',
- },
- },
- path: '1.bib',
- },
- {
- file: {
- _id: fileId3,
- hash: '21',
- created: fileCreated3,
- linkedFileData: {
- provider: 'project_output_file',
- build_id: '1234-abc',
- clsiServerId: 'server-1',
- source_project_id: otherProjectId,
- source_output_file_path: 'foo/bar.txt',
- },
- },
- path: 'bar.txt',
- },
- ]
- const docs = []
- this.request.yields(null, { statusCode: 200 })
- const projectId = new ObjectId()
- const projectHistoryId = 99
- this.handler.resyncProjectHistory(
- projectId,
- projectHistoryId,
- docs,
- files,
- {},
- () => {
- this.request.should.have.been.calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${projectId}/history/resync`,
- method: 'POST',
- json: {
- docs: [],
- files: [
- {
- file: fileId1,
- _hash: '42',
- path: '1.png',
- url: undefined,
- createdBlob: true,
- metadata: undefined,
- },
- {
- file: fileId2,
- _hash: '1337',
- path: '1.bib',
- url: undefined,
- createdBlob: true,
- metadata: {
- importedAt: fileCreated2,
- provider: 'references-provider',
- },
- },
- {
- file: fileId3,
- _hash: '21',
- path: 'bar.txt',
- url: undefined,
- createdBlob: true,
- metadata: {
- importedAt: fileCreated3,
- provider: 'project_output_file',
- source_project_id: otherProjectId,
- source_output_file_path: 'foo/bar.txt',
- // build_id and clsiServerId are omitted
- },
- },
- ],
- projectHistoryId,
- },
- timeout: 6 * 60 * 1000,
- })
- done()
- }
- )
- })
- it('should flag files with missing hashes', function (done) {
- const fileId1 = new ObjectId()
- const fileId2 = new ObjectId()
- const fileId3 = new ObjectId()
- const fileCreated2 = new Date()
- const fileCreated3 = new Date()
- const otherProjectId = new ObjectId().toString()
- const files = [
- { file: { _id: fileId1, hash: '42' }, path: '1.png' },
- {
- file: {
- _id: fileId2,
- created: fileCreated2,
- linkedFileData: {
- provider: 'references-provider',
- },
- },
- path: '1.bib',
- },
- {
- file: {
- _id: fileId3,
- hash: '21',
- created: fileCreated3,
- linkedFileData: {
- provider: 'project_output_file',
- build_id: '1234-abc',
- clsiServerId: 'server-1',
- source_project_id: otherProjectId,
- source_output_file_path: 'foo/bar.txt',
- },
- },
- path: 'bar.txt',
- },
- ]
- const docs = []
- this.request.yields(null, { statusCode: 200 })
- const projectId = new ObjectId()
- const projectHistoryId = 99
- this.handler.resyncProjectHistory(
- projectId,
- projectHistoryId,
- docs,
- files,
- {},
- err => {
- err.should.match(/found file with missing hash/)
- this.request.should.not.have.been.called
- done()
- }
- )
- })
- })
- })
- describe('appendToDocument', function () {
- describe('successfully', function () {
- beforeEach(function () {
- this.body = {
- rev: 1,
- }
- this.request.callsArgWith(1, null, { statusCode: 200 }, this.body)
- this.handler.appendToDocument(
- this.project_id,
- this.doc_id,
- this.user_id,
- this.lines,
- this.source,
- this.callback
- )
- })
- it('should append to the document in the document updater', function () {
- this.request
- .calledWith({
- url: `${this.settings.apis.documentupdater.url}/project/${this.project_id}/doc/${this.doc_id}/append`,
- json: {
- lines: this.lines,
- source: this.source,
- user_id: this.user_id,
- },
- method: 'POST',
- timeout: 30 * 1000,
- })
- .should.equal(true)
- })
- it('should call the callback with no error', function () {
- this.callback.calledWith(null).should.equal(true)
- })
- })
- describe('when the document updater API returns an error', function () {
- beforeEach(function () {
- this.request.callsArgWith(
- 1,
- new Error('something went wrong'),
- null,
- null
- )
- this.handler.appendToDocument(
- this.project_id,
- this.doc_id,
- this.user_id,
- this.lines,
- this.source,
- this.callback
- )
- })
- it('should return an error to the callback', function () {
- this.callback
- .calledWith(sinon.match.instanceOf(Error))
- .should.equal(true)
- })
- })
- describe('when the document updater returns a failure error code', function () {
- beforeEach(function () {
- this.request.callsArgWith(1, null, { statusCode: 500 }, '')
- this.handler.appendToDocument(
- this.project_id,
- this.doc_id,
- this.user_id,
- this.lines,
- this.source,
- this.callback
- )
- })
- it('should return the callback with an error', function () {
- this.callback
- .calledWith(
- sinon.match
- .instanceOf(Error)
- .and(
- sinon.match.has(
- 'message',
- 'document updater returned a failure status code: 500'
- )
- )
- )
- .should.equal(true)
- })
- })
- })
- })
|