From 97ce66a9e7ff3b97b029db062b16d8b267c76e0b Mon Sep 17 00:00:00 2001 From: Ersun Warncke Date: Mon, 6 Apr 2020 11:09:46 -0400 Subject: [PATCH] use user_id for tpds queue name GitOrigin-RevId: e61440efd22ef9e858b2862b1a6fd022c6eb70a4 --- .../Features/ThirdPartyDataStore/TpdsUpdateSender.js | 6 +++--- .../src/ThirdPartyDataStore/TpdsUpdateSenderTests.js | 10 +++++----- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/services/web/app/src/Features/ThirdPartyDataStore/TpdsUpdateSender.js b/services/web/app/src/Features/ThirdPartyDataStore/TpdsUpdateSender.js index 73d1383303..9d7b2a8099 100644 --- a/services/web/app/src/Features/ThirdPartyDataStore/TpdsUpdateSender.js +++ b/services/web/app/src/Features/ThirdPartyDataStore/TpdsUpdateSender.js @@ -112,7 +112,7 @@ const TpdsUpdateSender = { streamOrigin: options.streamOrigin } return TpdsUpdateSender._enqueue( - options.project_id, + user_id, 'pipeStreamFrom', postOptions, function(err) { @@ -204,7 +204,7 @@ const TpdsUpdateSender = { } } return TpdsUpdateSender._enqueue( - options.project_id, + user_id, 'standardHttpRequest', moveOptions, callback @@ -237,7 +237,7 @@ const TpdsUpdateSender = { sl_all_user_ids: JSON.stringify(allUserIds) } return TpdsUpdateSender._enqueue( - options.project_id, + user_id, 'standardHttpRequest', deleteOptions, callback diff --git a/services/web/test/unit/src/ThirdPartyDataStore/TpdsUpdateSenderTests.js b/services/web/test/unit/src/ThirdPartyDataStore/TpdsUpdateSenderTests.js index e59fa95721..dde0e37c0b 100644 --- a/services/web/test/unit/src/ThirdPartyDataStore/TpdsUpdateSenderTests.js +++ b/services/web/test/unit/src/ThirdPartyDataStore/TpdsUpdateSenderTests.js @@ -109,7 +109,7 @@ describe('TpdsUpdateSender', function() { const file_id = '4545345' const path = '/some/path/here.jpg' this.updateSender._enqueue = function(uid, method, job, callback) { - uid.should.equal(project_id) + uid.should.equal(user_id) job.method.should.equal('post') job.streamOrigin.should.equal( `${filestoreUrl}/project/${project_id}/file/${file_id}` @@ -135,7 +135,7 @@ describe('TpdsUpdateSender', function() { const lines = ['line1', 'line2', 'line3'] this.updateSender._enqueue = (uid, method, job, callback) => { - uid.should.equal(project_id) + uid.should.equal(user_id) job.method.should.equal('post') const expectedUrl = `${thirdPartyDataStoreApiUrl}/user/${user_id}/entity/${encodeURIComponent( project_name @@ -161,7 +161,7 @@ describe('TpdsUpdateSender', function() { it('deleting entity', function(done) { const path = '/path/here/t.tex' this.updateSender._enqueue = function(uid, method, job, callback) { - uid.should.equal(project_id) + uid.should.equal(user_id) job.method.should.equal('DELETE') const expectedUrl = `${thirdPartyDataStoreApiUrl}/user/${user_id}/entity/${encodeURIComponent( project_name @@ -179,7 +179,7 @@ describe('TpdsUpdateSender', function() { const startPath = 'staring/here/file.tex' const endPath = 'ending/here/file.tex' this.updateSender._enqueue = function(uid, method, job, callback) { - uid.should.equal(project_id) + uid.should.equal(user_id) job.method.should.equal('put') job.uri.should.equal( `${thirdPartyDataStoreApiUrl}/user/${user_id}/entity` @@ -203,7 +203,7 @@ describe('TpdsUpdateSender', function() { const oldProjectName = '/oldProjectName/' const newProjectName = '/newProjectName/' this.updateSender._enqueue = function(uid, method, job, callback) { - uid.should.equal(project_id) + uid.should.equal(user_id) job.method.should.equal('put') job.uri.should.equal( `${thirdPartyDataStoreApiUrl}/user/${user_id}/entity`