SCAL-255099 added utility to evaluate response status correctly

Author: shivam-kumar-ts

src/utils.spec.ts

@@ -742,23 +742,12 @@ describe('checkInternetConnection', () => {
     });
 
     it('should return true when internet connection is available', async () => {
-        jest.spyOn(window, 'fetch').mockImplementation(() => Promise.resolve({
-            ok: true,
-            status: 200,
-            statusText: 'OK',
-        }));
         const result = await checkInternetConnection(maxRetries , baseDelay , thoughtSpotHost);
         expect(result).toBe(true);
         expect(fetchSpy).toHaveBeenCalledTimes(1);
     });
 
     it('should return false when internet connection is not available', async () => {
-        jest.spyOn(window, 'fetch').mockImplementation(() => Promise.resolve({
-            ok: false,
-            status: 404,
-            statusText: 'Not Found',
-        }));
-        
         const result = await checkInternetConnection(maxRetries , baseDelay , thoughtSpotHost);
         expect(result).toBe(false);
         expect(fetchSpy).toHaveBeenCalledTimes(maxRetries + 1);

src/utils.ts

@@ -18,6 +18,7 @@ import {
 } from './types';
 import { logger } from './utils/logger';
 import { EndPoints } from './utils/authService';
+import { isActiveService } from './utils/authService/tokenizedAuthService';
 
 /**
  * Construct a runtime filters query string from the given filters.
@@ -526,26 +527,16 @@ export const checkInternetConnection = async (
 
     for (let attempt = 0; attempt <= maxRetries; attempt++) {
         try {
-            const controller = new AbortController();
-            const timeoutId = setTimeout(() => controller.abort(), 5000);
-
-            const response = await fetch(pingUrl, {
-                method: 'HEAD',
-                cache: 'no-store',
-                signal: controller.signal,
-            });
-
-            clearTimeout(timeoutId);
-
-            if (response.status >= 200 && response.status < 300) {
+            const response = await isActiveService(thoughtSpotHost);
+            logger.debug(`Pinging ${pingUrl} failed with response ${response}`);
+            if (response) {
                 if (attempt > 0) {
                     logger.info('Internet connection restored');
                 }
                 return true;
             }
-            
-            logger.debug(`Pinging ${pingUrl} failed with HTTP ${response.status}: ${response.statusText}`);
-            throw new Error(`Server responded with status ${response.status}. Error: ${response.statusText}`);
+            logger.debug(`Pinging ${pingUrl} failed with response ${response}`);
+            throw new Error(`Server responded with status ${response}.`);
         } catch (error) {
             const isLastAttempt = attempt === maxRetries;
             const delayMs = baseDelay * Math.pow(2, attempt);