From a8f7a2823ea756a88af2c32a8df84cff6844eac8 Mon Sep 17 00:00:00 2001 From: Viktar Donich Date: Sat, 30 Sep 2017 11:56:32 +0100 Subject: [PATCH] Add a plugin.put() method Because it exists in GWT implementation of Gerrit JS API Change-Id: I7fa2501c88de9ce80402595bcff5112c70e07c99 --- .../gr-js-api-interface/gr-js-api-interface_test.html | 10 ++++++++++ .../shared/gr-js-api-interface/gr-public-js-api.js | 4 ++++ 2 files changed, 14 insertions(+) diff --git a/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-js-api-interface_test.html b/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-js-api-interface_test.html index f88371a9ce..03cc0f8d94 100644 --- a/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-js-api-interface_test.html +++ b/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-js-api-interface_test.html @@ -120,6 +120,16 @@ limitations under the License. }); }); + test('put', () => { + const payload = {foo: 'foo'}; + const response = {bar: 'bar'}; + getResponseObjectStub.returns(Promise.resolve(response)); + return plugin.put('/url', payload, r => { + assert.isTrue(sendStub.calledWith('PUT', '/url', payload)); + assert.strictEqual(r, response); + }); + }); + test('delete works', () => { const response = {status: 204}; sendStub.returns(Promise.resolve(response)); diff --git a/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-public-js-api.js b/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-public-js-api.js index 8f78dd3768..da5b8f3f7c 100644 --- a/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-public-js-api.js +++ b/polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-public-js-api.js @@ -157,6 +157,10 @@ return this._send('POST', url, opt_callback, payload); }, + Plugin.prototype.put = function(url, payload, opt_callback) { + return this._send('PUT', url, opt_callback, payload); + }, + Plugin.prototype.delete = function(url, opt_callback) { return getRestAPI().send('DELETE', url).then(response => { if (response.status !== 204) {