close #379

src/image/helpers.test.ts

@@ -112,6 +112,6 @@ describe('resolveSourceFromImgAttributes', () => {
  sizes: `(max-width: 600px) 480px, 800px`,
  src: 'elva-fairy.jpg',
  }).uri
- ).toThrow()
+ ).not.toThrow()
  })
 })

src/router/parse-next-path.test.ts

@@ -0,0 +1,25 @@
+import { parseNextPath } from './parse-next-path'
+
+describe('parseNextPath', () => {
+ it('should handle arrays', () => {
+ expect(
+ parseNextPath({
+ pathname: '/',
+ query: {
+ ids: [1, 2],
+ },
+ })
+ ).toEqual('/?ids=1&ids=2')
+ })
+
+ it('supports dynamic routes', () => {
+ expect(
+ parseNextPath({
+ pathname: '/[id]',
+ query: {
+ id: 1,
+ },
+ })
+ ).toEqual('/1')
+ })
+})

src/router/parse-next-path.ts

@@ -9,6 +9,8 @@ const parseNextPath = (from: Parameters<NextRouter['push']>[0]) => {
  // but I can't see why you would use this with RN + Next.js
  if (typeof from == 'object' && from.query && typeof from.query == 'object') {
  const query = { ...from.query }
+ // replace dynamic routes
+ // and [...param] syntax
  for (const key in query) {
  if (path.includes(`[${key}]`)) {
  path = path.replace(`[${key}]`, `${query[key] ?? ''}`)
@@ -21,11 +23,18 @@ const parseNextPath = (from: Parameters<NextRouter['push']>[0]) => {
  }
  }
  }
+
  if (Object.keys(query).length) {
+ // add query param separator
  path += '?'
  for (const key in query) {
- if (query[key] != null) {
- path += `${key}=${query[key]}&`
+ const value = query[key]
+ if (Array.isArray(value)) {
+ value.forEach((item) => {
+ path += `${key}=${item}&`
+ })
+ } else if (value != null) {
+ path += `${key}=${value}&`
  }
  }
  if (path.endsWith('&') || path.endsWith('?')) {

tsconfig.json

@@ -12,7 +12,7 @@
  "jsx": "react-jsx"
  },
  "include": ["./src"],
- "exclude": ["**/__mocks__/*", "**/__tests__/*"],
+ "exclude": ["**/__mocks__/*", "**/__tests__/*", "**.*.test.ts"],
  "ts-node": {
  // these options are overrides used only by ts-node
  "compilerOptions": {