test: add regression test against #533

meliorence · Nov 3, 2021 · 8f98f8a · 8f98f8a
1 parent 4bfb0e4
commit 8f98f8a
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 17 deletions.
diff --git a/packages/render-html/src/__mocks__/react-native.js b/packages/render-html/src/__mocks__/react-native.js
@@ -2,6 +2,25 @@ import { Image as RNImage } from 'react-native';
 
 export * from 'react-native';
 
+function getSizeWithHeaders(uri, headers, success, failure) {
+  setTimeout(() => {
+    let dimensions = [0, 0];
+    if (typeof uri === 'string') {
+      if (uri === 'error') {
+        failure?.apply(null, new Error('Could not fetch image dimensions'));
+        return;
+      }
+      const match = /(\d+)x(\d+)/gi.exec(uri);
+      if (!match) {
+        dimensions = [640, 360];
+      } else {
+        dimensions = [parseInt(match[1], 10), parseInt(match[2], 10)];
+      }
+    }
+    success.apply(null, dimensions);
+  }, 50);
+}
+
 export class Image extends RNImage {
   /**
    * This mock function will parse the uri to find a dimension pattern
@@ -13,26 +32,11 @@ export class Image extends RNImage {
    * be extracted, it will use 640x360.
    */
   static getSizeWithHeaders(uri, headers, success, failure) {
-    setTimeout(() => {
-      let dimensions = [0, 0];
-      if (typeof uri === 'string') {
-        if (uri === 'error') {
-          failure?.apply(null, new Error('Could not fetch image dimensions'));
-          return;
-        }
-        const match = /(\d+)x(\d+)/gi.exec(uri);
-        if (!match) {
-          dimensions = [640, 360];
-        } else {
-          dimensions = [parseInt(match[1], 10), parseInt(match[2], 10)];
-        }
-      }
-      success.apply(null, dimensions);
-    }, 50);
+    getSizeWithHeaders(uri, headers, callback, failure);
   }
 
   static getSize(uri, callback, failure) {
-    Image.getSizeWithHeaders(uri, undefined, callback, failure);
+    getSizeWithHeaders(uri, undefined, callback, failure);
   }
 }
 

diff --git a/packages/render-html/src/__tests__/component.render-html.test.tsx b/packages/render-html/src/__tests__/component.render-html.test.tsx
@@ -836,6 +836,7 @@ describe('RenderHTML', () => {
       const headers = {
         Authorization: 'Bearer XXX'
       };
+      const getSizeWithHeaders = jest.spyOn(Image, 'getSizeWithHeaders');
       function provideEmbeddedHeaders(uri: string, tag: string) {
         expect(tag).toBe('img');
         return headers;
@@ -853,6 +854,10 @@ describe('RenderHTML', () => {
       await findByTestId('image-success');
       const image = UNSAFE_getByType(Image);
       expect(image.props.source.headers).toBe(headers);
+      expect(getSizeWithHeaders).toHaveBeenCalledWith(
+        'https://custom.domain/',
+        headers
+      );
     });
   });
   describe('regarding enableExperimentalBRCollapsing', () => {