chore(deps): bump query-string from 6.14.1 to 7.0.0 (#83)

* chore(deps): bump query-string from 6.14.1 to 7.0.0

Bumps [query-string](https://github.com/sindresorhus/query-string) from 6.14.1 to 7.0.0.
- [Release notes](https://github.com/sindresorhus/query-string/releases)
- [Commits](sindresorhus/query-string@v6.14.1...v7.0.0)

Signed-off-by: dependabot[bot] <[email protected]>

* fix: drop empty array values

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Dorian Maliszewski <[email protected]>

Author: dependabot[bot]

packages/use-query-params/package.json

@@ -27,7 +27,7 @@
   },
   "license": "MIT",
   "dependencies": {
-    "query-string": "^6.14.1",
+    "query-string": "^7.0.0",
     "react-router-dom": "^5.2.0"
   },
   "peerDependencies": {

packages/use-query-params/src/__tests__/index.js

@@ -1,7 +1,7 @@
 import { act, renderHook } from '@testing-library/react-hooks'
 import React from 'react'
-import { MemoryRouter } from 'react-router-dom'
-import useQueryParam from '../index'
+import { MemoryRouter, useHistory } from 'react-router-dom'
+import useQueryParam from '..'
 
 // eslint-disable-next-line react/prop-types
 const wrapper = ({ pathname = 'one', search }) => ({ children }) => (
@@ -206,4 +206,37 @@ describe('useQueryParam', () => {
       compagny: 'Scaleway',
     })
   })
+
+  test('should correctly set query params with array', () => {
+    jest.useFakeTimers()
+    const { result, rerender } = renderHook(
+      () => [useQueryParam(), useHistory()],
+      {
+        wrapper: wrapper({ search: '' }),
+      },
+    )
+
+    act(() => {
+      result.current[0].setQueryParams({
+        names: ['John', null, 'Jane', null, undefined, ''],
+      })
+    })
+    jest.runAllTimers()
+    expect(result.current[0].queryParams).toEqual({
+      names: ['John', null, 'Jane', null, undefined, ''],
+    })
+    expect(result.current[1].location.search).toEqual('?names=John,Jane')
+
+    rerender()
+  })
+
+  test('should correctly with existing array', () => {
+    const { result } = renderHook(() => useQueryParam(), {
+      wrapper: wrapper({ search: 'names=John,,Jane,,,' }),
+    })
+
+    expect(result.current.queryParams).toEqual({
+      names: ['John', '', 'Jane', '', '', ''],
+    })
+  })
 })

packages/use-query-params/src/index.js

@@ -22,6 +22,8 @@ const useQueryParams = () => {
     params =>
       stringify(params, {
         arrayFormat: 'comma',
+        skipEmptyString: true,
+        skipNull: true,
         sort: (a, b) => a.localeCompare(b),
       }),
     [],

yarn.lock

@@ -7492,10 +7492,10 @@ qs@~6.5.2:
   resolved "https://registry.yarnpkg.com/qs/-/qs-6.5.2.tgz#cb3ae806e8740444584ef154ce8ee98d403f3e36"
   integrity sha512-N5ZAX4/LxJmF+7wN74pUD6qAh9/wnvdQcjq9TZjevvXzSUo7bfmw91saqMjzGS2xq91/odN2dW/WOl7qQHNDGA==
 
-query-string@^6.14.1:
-  version "6.14.1"
-  resolved "https://registry.yarnpkg.com/query-string/-/query-string-6.14.1.tgz#7ac2dca46da7f309449ba0f86b1fd28255b0c86a"
-  integrity sha512-XDxAeVmpfu1/6IjyT/gXHOl+S0vQ9owggJ30hhWKdHAsNPOcasn5o9BW0eejZqL2e4vMjhAxoW3jVHcD6mbcYw==
+query-string@^7.0.0:
+  version "7.0.0"
+  resolved "https://registry.yarnpkg.com/query-string/-/query-string-7.0.0.tgz#aaad2c8d5c6a6d0c6afada877fecbd56af79e609"
+  integrity sha512-Iy7moLybliR5ZgrK/1R3vjrXq03S13Vz4Rbm5Jg3EFq1LUmQppto0qtXz4vqZ386MSRjZgnTSZ9QC+NZOSd/XA==
   dependencies:
     decode-uri-component "^0.2.0"
     filter-obj "^1.1.0"