Просмотр исходного кода

Fix ProjectEntityHandler.getAllEntities callback in _findExistingFileType (#9108)

GitOrigin-RevId: fe29488b44fa5e738c23f2939297057fb2832758
Alf Eaton 4 лет назад
Родитель
Сommit
19db9c2583

+ 0 - 1
services/web/app/src/Features/Project/ProjectEntityHandler.js

@@ -232,7 +232,6 @@ module.exports = ProjectEntityHandler
 module.exports.promises = promisifyAll(ProjectEntityHandler, {
   without: ['getAllEntitiesFromProject'],
   multiResult: {
-    getAllEntities: ['docs', 'files'],
     getDoc: ['lines', 'rev', 'version', 'ranges'],
   },
 })

+ 1 - 1
services/web/app/src/Features/ThirdPartyDataStore/UpdateMerger.js

@@ -55,7 +55,7 @@ module.exports = UpdateMerger = {
   _findExistingFileType(project_id, path, callback) {
     ProjectEntityHandler.getAllEntities(
       project_id,
-      function (err, docs, files) {
+      function (err, { docs, files }) {
         if (err != null) {
           return callback(err)
         }

+ 4 - 1
services/web/test/unit/src/ThirdPartyDataStore/UpdateMergerTests.js

@@ -33,7 +33,10 @@ describe('UpdateMerger :', function () {
     this.existingFiles = [{ path: '/figure.pdf' }, { path: '/folder/fig1.pdf' }]
     this.ProjectEntityHandler.getAllEntities = sinon
       .stub()
-      .callsArgWith(1, null, this.existingDocs, this.existingFiles)
+      .callsArgWith(1, null, {
+        docs: this.existingDocs,
+        files: this.existingFiles,
+      })
 
     this.fsPath = '/tmp/file/path'
     this.source = 'dropbox'