From ea35db0fe5387ed9cca2f8a8fabeaea10f89de67 Mon Sep 17 00:00:00 2001 From: Lucas Di Pentima Date: Tue, 21 Jan 2020 12:06:00 -0300 Subject: [PATCH] 15672: Adds test for params being just under the limit of needing POST. Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima --- .../common-service/common-resource-service.test.ts | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/src/services/common-service/common-resource-service.test.ts b/src/services/common-service/common-resource-service.test.ts index 943325b7..2a18ce23 100644 --- a/src/services/common-service/common-resource-service.test.ts +++ b/src/services/common-service/common-resource-service.test.ts @@ -124,4 +124,16 @@ describe("CommonResourceService", () => { expect(axiosMock.history.post[0].data.get('filters')).toBe(`[${tooBig}]`); expect(axiosMock.history.post[0].params._method).toBe('GET'); }); + + it("#list using GET when query string is not too big", async () => { + axiosMock + .onAny("/resource") + .reply(200); + const notTooBig = 'x'.repeat(1480); + const commonResourceService = new CommonResourceService(axiosInstance, "resource", actions); + await commonResourceService.list({ filters: notTooBig }); + expect(axiosMock.history.post.length).toBe(0); + expect(axiosMock.history.get.length).toBe(1); + expect(axiosMock.history.get[0].params.filters).toBe(`[${notTooBig}]`); + }); }); -- 2.30.2