| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530 |
- const SandboxedModule = require('sandboxed-module')
- const sinon = require('sinon')
- const modulePath = require('path').join(
- __dirname,
- '../../../app/js/MongoManager'
- )
- const { ObjectId } = require('mongodb')
- const { assert, expect } = require('chai')
- const Errors = require('../../../app/js/Errors')
- describe('MongoManager', function () {
- beforeEach(function () {
- this.db = {
- docs: {
- updateOne: sinon.stub().yields(null, { matchedCount: 1 }),
- insertOne: sinon.stub().yields(null),
- },
- docOps: {},
- }
- this.MongoManager = SandboxedModule.require(modulePath, {
- requires: {
- './mongodb': {
- db: this.db,
- ObjectId,
- },
- '@overleaf/metrics': { timeAsyncMethod: sinon.stub() },
- '@overleaf/settings': {
- max_deleted_docs: 42,
- docstore: { archivingLockDurationMs: 5000 },
- },
- './Errors': Errors,
- },
- })
- this.projectId = ObjectId().toString()
- this.docId = ObjectId().toString()
- this.rev = 42
- this.callback = sinon.stub()
- this.stubbedErr = new Error('hello world')
- })
- describe('findDoc', function () {
- beforeEach(function () {
- this.doc = { name: 'mock-doc' }
- this.db.docs.findOne = sinon.stub().callsArgWith(2, null, this.doc)
- this.filter = { lines: true }
- this.MongoManager.findDoc(
- this.projectId,
- this.docId,
- this.filter,
- this.callback
- )
- })
- it('should find the doc', function () {
- this.db.docs.findOne
- .calledWith(
- {
- _id: ObjectId(this.docId),
- project_id: ObjectId(this.projectId),
- },
- {
- projection: this.filter,
- }
- )
- .should.equal(true)
- })
- it('should call the callback with the doc', function () {
- this.callback.calledWith(null, this.doc).should.equal(true)
- })
- })
- describe('patchDoc', function () {
- beforeEach(function (done) {
- this.meta = { name: 'foo.tex' }
- this.callback.callsFake(done)
- this.MongoManager.patchDoc(
- this.projectId,
- this.docId,
- this.meta,
- this.callback
- )
- })
- it('should pass the parameter along', function () {
- this.db.docs.updateOne.should.have.been.calledWith(
- {
- _id: ObjectId(this.docId),
- project_id: ObjectId(this.projectId),
- },
- {
- $set: this.meta,
- },
- this.callback
- )
- })
- })
- describe('getProjectsDocs', function () {
- beforeEach(function () {
- this.filter = { lines: true }
- this.doc1 = { name: 'mock-doc1' }
- this.doc2 = { name: 'mock-doc2' }
- this.doc3 = { name: 'mock-doc3' }
- this.doc4 = { name: 'mock-doc4' }
- this.db.docs.find = sinon.stub().returns({
- toArray: sinon
- .stub()
- .callsArgWith(0, null, [this.doc, this.doc3, this.doc4]),
- })
- })
- describe('with included_deleted = false', function () {
- beforeEach(function () {
- this.MongoManager.getProjectsDocs(
- this.projectId,
- { include_deleted: false },
- this.filter,
- this.callback
- )
- })
- it('should find the non-deleted docs via the project_id', function () {
- this.db.docs.find
- .calledWith(
- {
- project_id: ObjectId(this.projectId),
- deleted: { $ne: true },
- },
- {
- projection: this.filter,
- }
- )
- .should.equal(true)
- })
- it('should call the callback with the docs', function () {
- this.callback
- .calledWith(null, [this.doc, this.doc3, this.doc4])
- .should.equal(true)
- })
- })
- describe('with included_deleted = true', function () {
- beforeEach(function () {
- this.MongoManager.getProjectsDocs(
- this.projectId,
- { include_deleted: true },
- this.filter,
- this.callback
- )
- })
- it('should find all via the project_id', function () {
- this.db.docs.find
- .calledWith(
- {
- project_id: ObjectId(this.projectId),
- },
- {
- projection: this.filter,
- }
- )
- .should.equal(true)
- })
- it('should call the callback with the docs', function () {
- this.callback
- .calledWith(null, [this.doc, this.doc3, this.doc4])
- .should.equal(true)
- })
- })
- })
- describe('getProjectsDeletedDocs', function () {
- beforeEach(function (done) {
- this.filter = { name: true }
- this.doc1 = { _id: '1', name: 'mock-doc1.tex' }
- this.doc2 = { _id: '2', name: 'mock-doc2.tex' }
- this.doc3 = { _id: '3', name: 'mock-doc3.tex' }
- this.db.docs.find = sinon.stub().returns({
- toArray: sinon.stub().yields(null, [this.doc1, this.doc2, this.doc3]),
- })
- this.callback.callsFake(done)
- this.MongoManager.getProjectsDeletedDocs(
- this.projectId,
- this.filter,
- this.callback
- )
- })
- it('should find the deleted docs via the project_id', function () {
- this.db.docs.find
- .calledWith({
- project_id: ObjectId(this.projectId),
- deleted: true,
- })
- .should.equal(true)
- })
- it('should filter, sort by deletedAt and limit', function () {
- this.db.docs.find
- .calledWith(sinon.match.any, {
- projection: this.filter,
- sort: { deletedAt: -1 },
- limit: 42,
- })
- .should.equal(true)
- })
- it('should call the callback with the docs', function () {
- this.callback
- .calledWith(null, [this.doc1, this.doc2, this.doc3])
- .should.equal(true)
- })
- })
- describe('upsertIntoDocCollection', function () {
- beforeEach(function () {
- this.oldRev = 77
- })
- it('should upsert the document', function (done) {
- this.MongoManager.upsertIntoDocCollection(
- this.projectId,
- this.docId,
- this.oldRev,
- { lines: this.lines },
- err => {
- assert.equal(err, null)
- const args = this.db.docs.updateOne.args[0]
- assert.deepEqual(args[0], {
- _id: ObjectId(this.docId),
- project_id: ObjectId(this.projectId),
- rev: this.oldRev,
- })
- assert.equal(args[1].$set.lines, this.lines)
- assert.equal(args[1].$inc.rev, 1)
- done()
- }
- )
- })
- it('should handle update error', function (done) {
- this.db.docs.updateOne.yields(this.stubbedErr)
- this.MongoManager.upsertIntoDocCollection(
- this.projectId,
- this.docId,
- this.rev,
- { lines: this.lines },
- err => {
- err.should.equal(this.stubbedErr)
- done()
- }
- )
- })
- it('should insert without a previous rev', function (done) {
- this.MongoManager.upsertIntoDocCollection(
- this.projectId,
- this.docId,
- null,
- { lines: this.lines, ranges: this.ranges },
- err => {
- expect(this.db.docs.insertOne).to.have.been.calledWith({
- _id: ObjectId(this.docId),
- project_id: ObjectId(this.projectId),
- rev: 1,
- lines: this.lines,
- ranges: this.ranges,
- })
- expect(err).to.not.exist
- done()
- }
- )
- })
- it('should handle generic insert error', function (done) {
- this.db.docs.insertOne.yields(this.stubbedErr)
- this.MongoManager.upsertIntoDocCollection(
- this.projectId,
- this.docId,
- null,
- { lines: this.lines, ranges: this.ranges },
- err => {
- expect(err).to.equal(this.stubbedErr)
- done()
- }
- )
- })
- it('should handle duplicate insert error', function (done) {
- this.db.docs.insertOne.yields({ code: 11000 })
- this.MongoManager.upsertIntoDocCollection(
- this.projectId,
- this.docId,
- null,
- { lines: this.lines, ranges: this.ranges },
- err => {
- expect(err).to.be.instanceof(Errors.DocRevValueError)
- done()
- }
- )
- })
- })
- describe('destroyProject', function () {
- beforeEach(function (done) {
- this.projectId = ObjectId()
- this.docIds = [ObjectId(), ObjectId()]
- this.db.docs.deleteMany = sinon.stub().yields()
- this.db.docOps.deleteMany = sinon.stub().yields()
- this.db.docs.find = sinon
- .stub()
- .withArgs({ project_id: this.projectId })
- .returns({
- toArray: sinon.stub().yields(
- null,
- this.docIds.map(id => ({
- _id: id,
- }))
- ),
- })
- this.MongoManager.destroyProject(this.projectId, done)
- })
- it('should destroy all docs', function () {
- sinon.assert.calledWith(this.db.docs.deleteMany, {
- project_id: this.projectId,
- })
- })
- it('should destroy the docOps', function () {
- sinon.assert.calledWith(this.db.docOps.deleteMany, {
- doc_id: { $in: this.docIds },
- })
- })
- })
- describe('getDocVersion', function () {
- describe('when the doc exists', function () {
- beforeEach(function () {
- this.doc = { version: (this.version = 42) }
- this.db.docOps.findOne = sinon.stub().callsArgWith(2, null, this.doc)
- this.MongoManager.getDocVersion(this.docId, this.callback)
- })
- it('should look for the doc in the database', function () {
- this.db.docOps.findOne
- .calledWith(
- { doc_id: ObjectId(this.docId) },
- {
- projection: { version: 1 },
- }
- )
- .should.equal(true)
- })
- it('should call the callback with the version', function () {
- this.callback.calledWith(null, this.version).should.equal(true)
- })
- })
- describe("when the doc doesn't exist", function () {
- beforeEach(function () {
- this.db.docOps.findOne = sinon.stub().callsArgWith(2, null, null)
- this.MongoManager.getDocVersion(this.docId, this.callback)
- })
- it('should call the callback with 0', function () {
- this.callback.calledWith(null, 0).should.equal(true)
- })
- })
- })
- describe('setDocVersion', function () {
- beforeEach(function () {
- this.version = 42
- this.db.docOps.updateOne = sinon.stub().callsArg(3)
- this.MongoManager.setDocVersion(this.docId, this.version, this.callback)
- })
- it('should update the doc version', function () {
- this.db.docOps.updateOne
- .calledWith(
- {
- doc_id: ObjectId(this.docId),
- },
- {
- $set: {
- version: this.version,
- },
- },
- {
- upsert: true,
- }
- )
- .should.equal(true)
- })
- it('should call the callback', function () {
- this.callback.called.should.equal(true)
- })
- })
- describe('checkRevUnchanged', function () {
- this.beforeEach(function () {
- this.doc = { _id: ObjectId(), name: 'mock-doc', rev: 1 }
- })
- it('should call the callback when the rev has not changed', function (done) {
- this.db.docs.findOne = sinon.stub().callsArgWith(2, null, { rev: 1 })
- this.MongoManager.checkRevUnchanged(this.doc, err => {
- assert.isUndefined(err)
- done()
- })
- })
- it('should return an error when the rev has changed', function (done) {
- this.db.docs.findOne = sinon.stub().callsArgWith(2, null, { rev: 2 })
- this.MongoManager.checkRevUnchanged(this.doc, err => {
- err.should.be.instanceof(Errors.DocModifiedError)
- done()
- })
- })
- it('should return a value error if incoming rev is NaN', function (done) {
- this.db.docs.findOne = sinon.stub().callsArgWith(2, null, { rev: 2 })
- this.doc = { _id: ObjectId(), name: 'mock-doc', rev: NaN }
- this.MongoManager.checkRevUnchanged(this.doc, err => {
- err.should.be.instanceof(Errors.DocRevValueError)
- done()
- })
- })
- it('should return a value error if checked doc rev is NaN', function (done) {
- this.db.docs.findOne = sinon.stub().callsArgWith(2, null, { rev: NaN })
- this.MongoManager.checkRevUnchanged(this.doc, err => {
- err.should.be.instanceof(Errors.DocRevValueError)
- done()
- })
- })
- })
- describe('restoreArchivedDoc', function () {
- beforeEach(function () {
- this.archivedDoc = {
- lines: ['a', 'b', 'c'],
- ranges: { some: 'ranges' },
- rev: 2,
- }
- })
- describe('complete doc', function () {
- beforeEach(function (done) {
- this.MongoManager.restoreArchivedDoc(
- this.projectId,
- this.docId,
- this.archivedDoc,
- done
- )
- })
- it('updates Mongo', function () {
- expect(this.db.docs.updateOne).to.have.been.calledWith(
- {
- _id: ObjectId(this.docId),
- project_id: ObjectId(this.projectId),
- rev: this.archivedDoc.rev,
- },
- {
- $set: {
- lines: this.archivedDoc.lines,
- ranges: this.archivedDoc.ranges,
- },
- $unset: {
- inS3: true,
- },
- }
- )
- })
- })
- describe('without ranges', function () {
- beforeEach(function (done) {
- delete this.archivedDoc.ranges
- this.MongoManager.restoreArchivedDoc(
- this.projectId,
- this.docId,
- this.archivedDoc,
- done
- )
- })
- it('sets ranges to an empty object', function () {
- expect(this.db.docs.updateOne).to.have.been.calledWith(
- {
- _id: ObjectId(this.docId),
- project_id: ObjectId(this.projectId),
- rev: this.archivedDoc.rev,
- },
- {
- $set: {
- lines: this.archivedDoc.lines,
- ranges: {},
- },
- $unset: {
- inS3: true,
- },
- }
- )
- })
- })
- describe("when the update doesn't succeed", function () {
- it('throws a DocRevValueError', function (done) {
- this.db.docs.updateOne.yields(null, { matchedCount: 0 })
- this.MongoManager.restoreArchivedDoc(
- this.projectId,
- this.docId,
- this.archivedDoc,
- err => {
- expect(err).to.be.instanceof(Errors.DocRevValueError)
- done()
- }
- )
- })
- })
- })
- })
|