Merge pull request #243 from pactumjs/242-support-duplicate-query-params

feat: support duplicate query params

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pactum",
-  "version": "3.2.4",
+  "version": "3.3.0",
   "description": "REST API Testing Tool for all levels in a Test Pyramid",
   "main": "./src/index.js",
   "types": "./src/index.d.ts",

src/helpers/helper.js

@@ -24,22 +24,21 @@ const helper = {
     return (typeof value === 'string' && value);
   },
 
-  getPlainQuery(query) {
-    let plain_query = '';
-    if (typeof query === 'object') {
-      for (const key in query) {
-        if (plain_query !== '') {
-          plain_query = plain_query + '&';
-        }
-        const value = query[key];
-        if (typeof value === 'undefined') {
-          plain_query = plain_query + `${key}`;
-        } else {
-          plain_query = plain_query + `${key}=${query[key]}`;
+  getPlainQuery(query = {}) {
+    const values = [];
+    for (const key in query) {
+      const value = query[key];
+      if (typeof value === 'undefined') {
+        values.push(key);
+      } else if (Array.isArray(value)) {
+        for (const current_value of value) {
+          values.push(`${key}=${current_value}`);
         }
+      } else {
+        values.push(`${key}=${value}`);
       }
     }
-    return plain_query;
+    return values.join('&');
   },
 
   isValidObject(value) {

src/models/Spec.js

@@ -148,7 +148,14 @@ class Spec {
       if (!helper.isValidString(key)) {
         throw new PactumRequestError('`key` is required');
       }
-      this._request.queryParams[key] = value;
+      if (Object.keys(this._request.queryParams).includes(key)) {
+       if (!Array.isArray(this._request.queryParams[key])) {
+        this._request.queryParams[key] = [this._request.queryParams[key]];
+       }
+       this._request.queryParams[key].push(value);
+      } else {
+        this._request.queryParams[key] = value;
+      }
     } else {
       if (!helper.isValidObject(key) || Object.keys(key).length === 0) {
         throw new PactumRequestError('`params` are required');

test/component/withQueryParams.spec.js

@@ -64,6 +64,24 @@ describe('withQueryParams', () => {
         .withQueryParams('id', '1')
         .expectStatus(200);
     });
+
+    it('with duplicate query params', async () => {
+      await pactum.spec()
+        .useInteraction({
+          strict: false,
+          request: {
+            method: 'GET',
+            path: '/api/query'
+          },
+          response: {
+            status: 200
+          }
+        })
+        .get('http://localhost:9393/api/query')
+        .withQueryParams('filters', 'name=test')
+        .withQueryParams('filters', 'total>10')
+        .expectStatus(200);
+    });
 
   });