diff --git a/services/web/app/src/Features/History/HistoryManager.js b/services/web/app/src/Features/History/HistoryManager.js index 4debeb6179..304fff6f4b 100644 --- a/services/web/app/src/Features/History/HistoryManager.js +++ b/services/web/app/src/Features/History/HistoryManager.js @@ -3,7 +3,7 @@ const { fetchJson, fetchNothing } = require('@overleaf/fetch-utils') const settings = require('@overleaf/settings') const OError = require('@overleaf/o-error') const UserGetter = require('../User/UserGetter') -const { Project } = require('../../models/Project') +const ProjectGetter = require('../Project/ProjectGetter') async function initializeProject(projectId) { const body = await fetchJson(`${settings.apis.project_history.url}/project`, { @@ -115,7 +115,9 @@ async function _deleteProjectInFullProjectHistory(historyId) { * @returns Promise */ async function getCurrentContent(projectId) { - const project = await Project.findById(projectId).exec() + const project = await ProjectGetter.promises.getProject(projectId, { + overleaf: true, + }) const historyId = project?.overleaf?.history?.id if (!historyId) { throw new OError('project does not have a history id', { projectId }) diff --git a/services/web/test/unit/src/History/HistoryManagerTests.js b/services/web/test/unit/src/History/HistoryManagerTests.js index 461e66c904..5b04e51063 100644 --- a/services/web/test/unit/src/History/HistoryManagerTests.js +++ b/services/web/test/unit/src/History/HistoryManagerTests.js @@ -48,9 +48,9 @@ describe('HistoryManager', function () { }, } - this.ProjectModel = { - Project: { - findById: sinon.stub().returns(sinon.stub().resolves(this.project)), + this.ProjectGetter = { + promises: { + getProject: sinon.stub().resolves(this.project), }, } @@ -59,7 +59,7 @@ describe('HistoryManager', function () { '@overleaf/fetch-utils': this.FetchUtils, '@overleaf/settings': this.settings, '../User/UserGetter': this.UserGetter, - '../../models/Project': this.ProjectModel, + '../Project/ProjectGetter': this.ProjectGetter, }, }) })