Add more tests

Author: marcospassos

src/components/Slot/index.test.tsx

@@ -18,7 +18,14 @@ describe('<Slot />', () => {
             fallback: {title: 'fallback'},
         };
 
-        const result = {content: {title: 'result'}} satisfies FetchResponse;
+        const result = {
+            metadata: {
+                version: '1.0',
+            },
+            content: {
+                title: 'result',
+            },
+        } satisfies FetchResponse;
 
         jest.mocked(useContent).mockReturnValue(result);
 

src/hooks/useContent.test.ts

@@ -34,14 +34,19 @@ describe('useContent (CSR)', () => {
     });
 
     it('should fetch the content', () => {
-        const fetch: Plug['fetch'] = jest.fn().mockResolvedValue({
-            content: {},
-        });
+        const fetch: Plug['fetch'] = jest.fn();
+
+        const response: FetchResponse<{title: string}> = {
+            metadata: {
+                version: '1.0',
+            },
+            content: {
+                title: 'foo',
+            },
+        };
 
         jest.mocked(useCroct).mockReturnValue({fetch: fetch} as Plug);
-        jest.mocked(useLoader).mockReturnValue({
-            title: 'foo',
-        });
+        jest.mocked(useLoader).mockReturnValue(response);
 
         const slotId = 'home-banner@1';
         const preferredLocale = 'en';
@@ -78,15 +83,15 @@ describe('useContent (CSR)', () => {
             attributes: attributes,
         });
 
-        expect(result.current).toEqual({title: 'foo'});
+        expect(result.current).toEqual(response);
     });
 
     it('should use the initial value when the cache key changes if the stale-while-loading flag is false', async () => {
         const key = {
             current: 'initial',
         };
 
-        const fetch: Plug['fetch'] = jest.fn().mockResolvedValue({content: {}});
+        const fetch: Plug['fetch'] = jest.fn();
 
         jest.mocked(useCroct).mockReturnValue({fetch: fetch} as Plug);
 
@@ -147,7 +152,7 @@ describe('useContent (CSR)', () => {
             current: 'initial',
         };
 
-        const fetch: Plug['fetch'] = jest.fn().mockResolvedValue({content: {}});
+        const fetch: Plug['fetch'] = jest.fn();
 
         jest.mocked(useCroct).mockReturnValue({fetch: fetch} as Plug);
 
@@ -283,9 +288,7 @@ describe('useContent (CSR)', () => {
         const slotId = 'slot-id';
         const preferredLocale = 'en';
 
-        const fetch: Plug['fetch'] = jest.fn().mockResolvedValue({
-            content: {},
-        });
+        const fetch: Plug['fetch'] = jest.fn();
 
         jest.mocked(useCroct).mockReturnValue({fetch: fetch} as Plug);
 
@@ -310,9 +313,7 @@ describe('useContent (CSR)', () => {
     });
 
     it('should normalize an empty preferred locale to undefined', () => {
-        const fetch: Plug['fetch'] = jest.fn().mockResolvedValue({
-            content: {},
-        });
+        const fetch: Plug['fetch'] = jest.fn();
 
         jest.mocked(useCroct).mockReturnValue({fetch: fetch} as Plug);
 
@@ -329,4 +330,35 @@ describe('useContent (CSR)', () => {
 
         expect(jest.mocked(fetch).mock.calls[0][1]).toStrictEqual({});
     });
+
+    it('should return the metadata along with the content', () => {
+        const fetch: Plug['fetch'] = jest.fn();
+
+        const response: FetchResponse<{title: string}> = {
+            metadata: {
+                version: '1.0',
+            },
+            content: {
+                title: 'foo',
+            },
+        };
+
+        jest.mocked(useCroct).mockReturnValue({fetch: fetch} as Plug);
+        jest.mocked(useLoader).mockReturnValue(response);
+
+        const slotId = 'home-banner@1';
+
+        const {result} = renderHook(
+            () => useContent<{title: string}>(slotId),
+        );
+
+        jest.mocked(useLoader)
+            .mock
+            .calls[0][0]
+            .loader();
+
+        expect(fetch).toHaveBeenCalledWith(slotId, {});
+
+        expect(result.current).toEqual(response);
+    });
 });