removed unnecessary use of then catch and used ctx logger

Author: parthlambdatest

src/lib/httpClient.ts

@@ -69,16 +69,8 @@ export default class httpClient {
             url: '/screenshot',
             method: 'GET',
             params: { buildId, baseline }
-        }, log)
-            .then(response => {
-                log.debug(`Fetched screenshot data successfully for buildId: ${buildId}`);
-                return response;
-            })
-            .catch(error => {
-                log.error(`Error fetching screenshot data for buildId: ${buildId}. Error: ${error.message}`);
-                throw error;
-            });
-    }
+        }, log);
+    }    
 
     finalizeBuild(buildId: string, totalSnapshots: number, log: Logger) {
         let params: Record<string, string | number> = {buildId};

src/lib/utils.ts

@@ -229,7 +229,7 @@ export async function startPolling(ctx: Context, task: any): Promise<void> {
             const resp = await ctx.client.getScreenshotData(ctx.build.id, ctx.build.baseline, ctx.log);
 
             if (!resp.build) {
-                console.error("Error: Build data is null.");
+                ctx.log.info("Error: Build data is null.");
                 clearInterval(intervalId);
                 isPollingActive = false;
             }
@@ -279,7 +279,7 @@ export async function startPolling(ctx: Context, task: any): Promise<void> {
                 );                
             }
         } catch (error: any) {
-            console.error(`Error fetching screenshot data: ${error.message}`);
+            ctx.log.error(`Error fetching screenshot data: ${error.message}`);
             clearInterval(intervalId);
             isPollingActive = false;
         }