From b4ee30caef9a077def8a2010c32711d7fa853442 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?N=C3=ADcholas=20Oliveira?= Date: Mon, 13 May 2024 23:17:48 -0300 Subject: [PATCH] fix: tests --- package-lock.json | 7 +++-- packages/next/package.json | 2 +- .../src/handlers/__tests__/previewHandler.ts | 30 +++++++++++-------- .../handlers/__tests__/revalidateHandler.ts | 5 ++-- 4 files changed, 25 insertions(+), 19 deletions(-) diff --git a/package-lock.json b/package-lock.json index 64b73ab0c..5f68ea799 100644 --- a/package-lock.json +++ b/package-lock.json @@ -16160,9 +16160,10 @@ } }, "node_modules/next-router-mock": { - "version": "0.9.3", + "version": "0.9.13", + "resolved": "https://registry.npmjs.org/next-router-mock/-/next-router-mock-0.9.13.tgz", + "integrity": "sha512-906n2RRaE6Y28PfYJbaz5XZeJ6Tw8Xz1S6E31GGwZ0sXB6/XjldD1/2azn1ZmBmRk5PQRkzjg+n+RHZe5xQzWA==", "dev": true, - "license": "MIT", "peerDependencies": { "next": ">=10.0.0", "react": ">=17.0.0" @@ -21908,7 +21909,7 @@ "expect-type": "^0.15.0", "jest": "^29.0.3", "jest-fetch-mock": "^3.0.3", - "next-router-mock": "^0.9.1", + "next-router-mock": "^0.9.13", "node-mocks-http": "^1.14.1", "ts-jest": "^29.0.1", "tsc-esm-fix": "^2.20.27", diff --git a/packages/next/package.json b/packages/next/package.json index 0fe359b4e..25f199a61 100644 --- a/packages/next/package.json +++ b/packages/next/package.json @@ -62,7 +62,7 @@ "copy-webpack-plugin": "^10.2.4", "expect-type": "^0.15.0", "jest": "^29.0.3", - "next-router-mock": "^0.9.1", + "next-router-mock": "^0.9.13", "node-mocks-http": "^1.14.1", "ts-jest": "^29.0.1", "typescript": "^5.4.2", diff --git a/packages/next/src/handlers/__tests__/previewHandler.ts b/packages/next/src/handlers/__tests__/previewHandler.ts index cb6d2e95b..a12af6001 100644 --- a/packages/next/src/handlers/__tests__/previewHandler.ts +++ b/packages/next/src/handlers/__tests__/previewHandler.ts @@ -1,11 +1,12 @@ import { createMocks } from 'node-mocks-http'; import { DRAFT_POST_ID, VALID_AUTH_TOKEN } from '@headstartwp/core/test'; import { removeSourceUrl, setHeadstartWPConfig } from '@headstartwp/core'; +import { NextApiRequest, NextApiResponse } from 'next'; import { previewHandler } from '../previewHandler'; describe('previewHandler', () => { it('does not accepts POST requests', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'POST', query: {}, }); @@ -17,7 +18,7 @@ describe('previewHandler', () => { }); it('rejects requests missing params', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: 1 }, }); @@ -29,7 +30,7 @@ describe('previewHandler', () => { }); it('fails if a valid auth token is not provided', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: 'test', post_type: 'post' }, }); @@ -41,7 +42,7 @@ describe('previewHandler', () => { }); it('works if a valid auth token is provided', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'post' }, }); @@ -57,7 +58,7 @@ describe('previewHandler', () => { }); it('sets preview cookie path', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'post' }, }); @@ -94,7 +95,7 @@ describe('previewHandler', () => { ], }); - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, @@ -123,7 +124,10 @@ describe('previewHandler', () => { '/book/modi-qui-dignissimos-sed-assumenda-sint-iusto-preview=true', ); - const { req: reqWithLocale, res: resWithLocale } = createMocks({ + const { req: reqWithLocale, res: resWithLocale } = createMocks< + NextApiRequest, + NextApiResponse + >({ method: 'GET', query: { post_id: DRAFT_POST_ID, @@ -155,7 +159,7 @@ describe('previewHandler', () => { }); it('sets preview cookie path with locale', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, @@ -187,7 +191,7 @@ describe('previewHandler', () => { }); it('set preview cookie path to all paths if onRedirect is passed without getRedirectPath', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'post' }, }); @@ -213,7 +217,7 @@ describe('previewHandler', () => { }); it('set preview cookie path redirectPath if getRedirectPath is passed', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'post' }, }); @@ -239,7 +243,7 @@ describe('previewHandler', () => { }); it('correctly takes into account `options`', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'post' }, }); @@ -271,7 +275,7 @@ describe('previewHandler', () => { }); it('fails if post type is not defined', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'recipe' }, }); @@ -290,7 +294,7 @@ describe('previewHandler', () => { preview: { usePostLinkForRedirect: true }, }); - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: DRAFT_POST_ID, token: VALID_AUTH_TOKEN, post_type: 'post' }, }); diff --git a/packages/next/src/handlers/__tests__/revalidateHandler.ts b/packages/next/src/handlers/__tests__/revalidateHandler.ts index 5b0a639e4..5fcfd65fb 100644 --- a/packages/next/src/handlers/__tests__/revalidateHandler.ts +++ b/packages/next/src/handlers/__tests__/revalidateHandler.ts @@ -1,9 +1,10 @@ import { createMocks } from 'node-mocks-http'; +import { NextApiRequest, NextApiResponse } from 'next'; import { revalidateHandler } from '../revalidateHandler'; describe('revalidateHandler', () => { it('does not accepts POST requests', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'POST', query: {}, }); @@ -15,7 +16,7 @@ describe('revalidateHandler', () => { }); it('rejects requests missing params', async () => { - const { req, res } = createMocks({ + const { req, res } = createMocks({ method: 'GET', query: { post_id: 1 }, });