fix: send filter parameters to responses

Author: jepser

src/responses.js

@@ -20,14 +20,16 @@ const getResponses = (
   return http.request({
     method: 'get',
     url: `/forms/${uid}/responses`,
-    page_size: pageSize,
-    since,
-    until,
-    after,
-    before,
-    completed,
-    sort,
-    query,
-    fields
+    params: {
+      page_size: pageSize,
+      since,
+      until,
+      after,
+      before,
+      completed,
+      sort,
+      query,
+      fields
+    }
   })
 }

tests/unit/responses.test.js

@@ -16,3 +16,14 @@ test('List responses has the correct path and method', () => {
   expect(fetch.mock.calls[0][1].method).toBe('get')
   expect(fetch.mock.calls[0][0]).toBe(`${API_BASE_URL}/forms/2/responses`)
 })
+
+test('List responses with the given filters', () => {
+  const http = clientConstructor({
+    token: '123',
+  })
+  const responsesRequest = responses(http)
+  responsesRequest.list({ uid: 2, pageSize: 15, after: '12345' })
+  const params = (new URL(fetch.mock.calls[0][0])).searchParams
+  expect(params.get('page_size')).toBe('15')
+  expect(params.get('after')).toBe('12345')
+})

tests/unit/themes.test.js

@@ -30,8 +30,9 @@ test('Get themes has the correct path', () => {
 
 test('Get themes has the correct parameters', () => {
   themesRequest.list({ page: 3, pageSize: 15 })
-  expect(fetch.mock.calls[0][1].params.page).toBe(3)
-  expect(fetch.mock.calls[0][1].params.page_size).toBe(15)
+  const params = (new URL(fetch.mock.calls[0][0])).searchParams
+  expect(params.get('page')).toBe('3')
+  expect(params.get('page_size')).toBe('15')
 })
 
 test('Get themes has the correct path', () => {