adjust tests to use actual Response object instead of object approxim…

…ation
MichaelDeBoey · Mar 19, 2020 · 6909e24 · 6909e24
1 parent 0e3d84c
commit 6909e24
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 11 deletions.
diff --git a/src/__tests__/transformers/GIPHY.js b/src/__tests__/transformers/GIPHY.js
@@ -11,12 +11,13 @@ import {
 
 import { cache, getMarkdownASTForFile, parseASTToMarkdown } from '../helpers';
 
+const { Response } = jest.requireActual('node-fetch');
 jest.mock('node-fetch', () => jest.fn());
 
 const mockFetch = ({ height, width }) =>
-  fetchMock.mockResolvedValue({
-    json: () => Promise.resolve({ height, width }),
-  });
+  fetchMock.mockImplementation(() =>
+    Promise.resolve(new Response(JSON.stringify({ height, width })))
+  );
 
 beforeEach(() => {
   fetchMock.mockClear();

diff --git a/src/__tests__/transformers/Instagram.js b/src/__tests__/transformers/Instagram.js
@@ -6,10 +6,13 @@ import { getHTML, shouldTransform } from '../../transformers/Instagram';
 
 import { cache, getMarkdownASTForFile, parseASTToMarkdown } from '../helpers';
 
+const { Response } = jest.requireActual('node-fetch');
 jest.mock('node-fetch', () => jest.fn());
 
 const mockFetch = html =>
-  fetchMock.mockResolvedValue({ json: () => Promise.resolve({ html }) });
+  fetchMock.mockImplementation(() =>
+    Promise.resolve(new Response(JSON.stringify({ html })))
+  );
 
 beforeEach(() => {
   fetchMock.mockClear();

diff --git a/src/__tests__/transformers/Streamable.js b/src/__tests__/transformers/Streamable.js
@@ -10,10 +10,13 @@ import {
 
 import { cache, getMarkdownASTForFile, parseASTToMarkdown } from '../helpers';
 
+const { Response } = jest.requireActual('node-fetch');
 jest.mock('node-fetch', () => jest.fn());
 
 const mockFetch = html =>
-  fetchMock.mockResolvedValue({ json: () => Promise.resolve({ html }) });
+  fetchMock.mockImplementation(() =>
+    Promise.resolve(new Response(JSON.stringify({ html })))
+  );
 
 beforeEach(() => {
   fetchMock.mockClear();

diff --git a/src/__tests__/transformers/Twitter.js b/src/__tests__/transformers/Twitter.js
@@ -6,16 +6,17 @@ import { getHTML, shouldTransform } from '../../transformers/Twitter';
 
 import { cache, getMarkdownASTForFile, parseASTToMarkdown } from '../helpers';
 
+const { Response } = jest.requireActual('node-fetch');
 jest.mock('node-fetch', () => jest.fn());
 
 const mockFetch = (status, moment) =>
   fetchMock
-    .mockResolvedValueOnce({ json: () => Promise.resolve({ html: status }) })
-    .mockResolvedValueOnce({ json: () => Promise.resolve({ html: status }) })
-    .mockResolvedValueOnce({ json: () => Promise.resolve({ html: moment }) })
-    .mockResolvedValueOnce({ json: () => Promise.resolve({ html: moment }) })
-    .mockResolvedValueOnce({ json: () => Promise.resolve({ html: moment }) })
-    .mockResolvedValueOnce({ json: () => Promise.resolve({ html: moment }) });
+    .mockResolvedValueOnce(new Response(JSON.stringify({ html: status })))
+    .mockResolvedValueOnce(new Response(JSON.stringify({ html: status })))
+    .mockResolvedValueOnce(new Response(JSON.stringify({ html: moment })))
+    .mockResolvedValueOnce(new Response(JSON.stringify({ html: moment })))
+    .mockResolvedValueOnce(new Response(JSON.stringify({ html: moment })))
+    .mockResolvedValueOnce(new Response(JSON.stringify({ html: moment })));
 
 beforeEach(() => {
   fetchMock.mockReset();