diff --git a/services/docstore/test/acceptance/js/helpers/DocstoreClient.js b/services/docstore/test/acceptance/js/helpers/DocstoreClient.js index 7bd6bd6618..e2c2706fbc 100644 --- a/services/docstore/test/acceptance/js/helpers/DocstoreClient.js +++ b/services/docstore/test/acceptance/js/helpers/DocstoreClient.js @@ -1,6 +1,3 @@ -/* eslint-disable - camelcase, -*/ let DocstoreClient const request = require('request').defaults({ jar: false }) const settings = require('@overleaf/settings') @@ -22,10 +19,10 @@ async function getStringFromPersistor(persistor, bucket, key) { } module.exports = DocstoreClient = { - createDoc(project_id, doc_id, lines, version, ranges, callback) { + createDoc(projectId, docId, lines, version, ranges, callback) { return DocstoreClient.updateDoc( - project_id, - doc_id, + projectId, + docId, lines, version, ranges, @@ -33,10 +30,10 @@ module.exports = DocstoreClient = { ) }, - getDoc(project_id, doc_id, qs, callback) { + getDoc(projectId, docId, qs, callback) { request.get( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc/${doc_id}`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc/${docId}`, json: true, qs, }, @@ -44,10 +41,10 @@ module.exports = DocstoreClient = { ) }, - peekDoc(project_id, doc_id, qs, callback) { + peekDoc(projectId, docId, qs, callback) { request.get( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc/${doc_id}/peek`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc/${docId}/peek`, json: true, qs, }, @@ -55,20 +52,20 @@ module.exports = DocstoreClient = { ) }, - isDocDeleted(project_id, doc_id, callback) { + isDocDeleted(projectId, docId, callback) { request.get( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc/${doc_id}/deleted`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc/${docId}/deleted`, json: true, }, callback ) }, - getAllDocs(project_id, callback) { + getAllDocs(projectId, callback) { request.get( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc`, json: true, }, (req, res, body) => { @@ -77,10 +74,10 @@ module.exports = DocstoreClient = { ) }, - getAllDeletedDocs(project_id, callback) { + getAllDeletedDocs(projectId, callback) { request.get( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc-deleted`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc-deleted`, json: true, }, (error, res, body) => { @@ -93,20 +90,20 @@ module.exports = DocstoreClient = { ) }, - getAllRanges(project_id, callback) { + getAllRanges(projectId, callback) { request.get( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/ranges`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/ranges`, json: true, }, callback ) }, - updateDoc(project_id, doc_id, lines, version, ranges, callback) { + updateDoc(projectId, docId, lines, version, ranges, callback) { return request.post( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc/${doc_id}`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc/${docId}`, json: { lines, version, @@ -117,78 +114,78 @@ module.exports = DocstoreClient = { ) }, - deleteDoc(project_id, doc_id, callback) { + deleteDoc(projectId, docId, callback) { DocstoreClient.deleteDocWithDateAndName( - project_id, - doc_id, + projectId, + docId, new Date(), 'main.tex', callback ) }, - deleteDocWithDate(project_id, doc_id, date, callback) { + deleteDocWithDate(projectId, docId, date, callback) { DocstoreClient.deleteDocWithDateAndName( - project_id, - doc_id, + projectId, + docId, date, 'main.tex', callback ) }, - deleteDocWithName(project_id, doc_id, name, callback) { + deleteDocWithName(projectId, docId, name, callback) { DocstoreClient.deleteDocWithDateAndName( - project_id, - doc_id, + projectId, + docId, new Date(), name, callback ) }, - deleteDocWithDateAndName(project_id, doc_id, deletedAt, name, callback) { + deleteDocWithDateAndName(projectId, docId, deletedAt, name, callback) { request.patch( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc/${doc_id}`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc/${docId}`, json: { name, deleted: true, deletedAt }, }, callback ) }, - archiveAllDoc(project_id, callback) { + archiveAllDoc(projectId, callback) { request.post( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/archive`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/archive`, }, callback ) }, - archiveDocById(project_id, doc_id, callback) { + archiveDocById(projectId, docId, callback) { request.post( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/doc/${doc_id}/archive`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/doc/${docId}/archive`, }, callback ) }, - destroyAllDoc(project_id, callback) { + destroyAllDoc(projectId, callback) { request.post( { - url: `http://localhost:${settings.internal.docstore.port}/project/${project_id}/destroy`, + url: `http://localhost:${settings.internal.docstore.port}/project/${projectId}/destroy`, }, callback ) }, - getS3Doc(project_id, doc_id, callback) { + getS3Doc(projectId, docId, callback) { getStringFromPersistor( Persistor, settings.docstore.bucket, - `${project_id}/${doc_id}` + `${projectId}/${docId}` ) .then(data => { callback(null, JSON.parse(data))