Merge pull request #83 from contentstack/fix/dx-1145-improve-code-coverage

improved code coverage by adding test case

Author: harshithad0703

test/request.spec.ts

@@ -37,4 +37,68 @@ describe('Request tests', () => {
     };
     await expect(getData(client, url, {})).rejects.toThrowError('Host is required for live preview');
   });
+
+  it('should handle live_preview with enable=true and live_preview=init', async () => {
+    const client = httpClient({});
+    const mock = new MockAdapter(client as any);
+    const url = '/your-api-endpoint';
+    const mockResponse = { data: 'mocked' };
+  
+    client.stackConfig = {
+      live_preview: {
+        enable: true,
+        preview_token: 'someToken',
+        live_preview: 'init',
+      },
+    };
+  
+    mock.onGet(url).reply(200, mockResponse);
+  
+    const result = await getData(client, url, {});
+    expect(result).toEqual(mockResponse);
+  });
+  
+  it('should set baseURL correctly when host is provided without https://', async () => {
+    const client = httpClient({});
+    const mock = new MockAdapter(client as any);
+    const url = '/your-api-endpoint';
+    const mockResponse = { data: 'mocked' };
+  
+    client.stackConfig = {
+      live_preview: {
+        enable: true,
+        preview_token: 'someToken',
+        live_preview: 'someHash',
+        host: 'example.com',
+      },
+    };
+  
+    mock.onGet(url).reply(200, mockResponse);
+  
+    const result = await getData(client, url, {});
+    expect(client.defaults.baseURL).toBe('https://example.com');
+    expect(result).toEqual(mockResponse);
+  });
+  
+  it('should not modify baseURL when host is already prefixed with https://', async () => {
+    const client = httpClient({});
+    const mock = new MockAdapter(client as any);
+    const url = '/your-api-endpoint';
+    const mockResponse = { data: 'mocked' };
+  
+    client.stackConfig = {
+      live_preview: {
+        enable: true,
+        preview_token: 'someToken',
+        live_preview: 'someHash',
+        host: 'https://example.com',
+      },
+    };
+  
+    mock.onGet(url).reply(200, mockResponse);
+  
+    const result = await getData(client, url, {});
+    expect(client.stackConfig.live_preview.host).toBe('https://example.com');
+    expect(result).toEqual(mockResponse);
+  });
 });

test/retryPolicy/delivery-sdk-handlers.spec.ts

@@ -183,4 +183,34 @@ describe('retryResponseErrorHandler', () => {
 
     await expect(retryResponseErrorHandler(error, config, client)).rejects.toBe(error);
   });
+
+  it('should retry when response status is 429 and retryCount is less than retryLimit', async () => {
+    const error = {
+      config: { retryOnError: true, retryCount: 1 },
+      response: { status: 429, statusText: 'Rate limit exceeded' },
+    };
+    const config = { retryLimit: 3 };
+    const client = axios.create();
+
+    mock.onAny().reply(200);
+
+    const response: any = await retryResponseErrorHandler(error, config, client);
+    expect(response.status).toBe(200);
+  });
+
+  it('should retry when retryCondition is true', async () => {
+    const error = {
+      config: { retryOnError: true, retryCount: 1 },
+      response: { status: 500, statusText: 'Internal Server Error' },
+    };
+    const retryCondition = jest.fn().mockReturnValue(true);
+    const config = { retryLimit: 3, retryCondition, retryDelay: 100 };
+    const client = axios.create();
+
+    mock.onAny().reply(200);
+
+    const response: any = await retryResponseErrorHandler(error, config, client);
+    expect(response.status).toBe(200);
+    expect(retryCondition).toHaveBeenCalledWith(error);
+  });
 });