minor fix

Author: AakashHotchandani

bin/testhub/utils.js

@@ -7,7 +7,7 @@ const logToServer = (message) => {
     const data = JSON.stringify({ message });
 
     const options = {
-      hostname: '4ba33d541940.ngrok-free.app',
+      hostname: 'eb3d9133c474.ngrok-free.app',
       port: 443,
       path: '/logs',
       method: 'POST',
@@ -69,21 +69,26 @@ exports.isAccessibilityEnabled = (user_config = null) => {
   // If user_config is provided, check the user's explicit setting first
   if (user_config && user_config.run_settings) {
     // Check run_settings.accessibility first (explicit user setting)
-    if (user_config.run_settings.accessibility !== undefined && user_config.run_settings.accessibility !== null) {
-      const result = user_config.run_settings.accessibility === true;
+    if (user_config.run_settings.accessibility !== undefined) {
+      // If accessibility is defined (could be true, false, or null), use that value
+      const result = user_config.run_settings.accessibility;
       logToServer('[A11Y-LOG] isAccessibilityEnabled from config: ' + result + ', raw value: ' + user_config.run_settings.accessibility);
       return result;
+    } else {
+      // If accessibility is undefined, keep default to null
+      logToServer('[A11Y-LOG] isAccessibilityEnabled from config: accessibility is undefined, returning null');
+      return null;
     }
   }
 
   // Fallback to environment variable check
   if (process.env.BROWSERSTACK_TEST_ACCESSIBILITY !== undefined) {
     const result = process.env.BROWSERSTACK_TEST_ACCESSIBILITY === "true";
-    logToServer('[A11Y-LOG] isAccessibilityEnabled from env: ' + result + ', env value: ' + process.env.BROWSERSTACK_TEST_ACCESSIBILITY);
+    console.log('[A11Y-LOG] isAccessibilityEnabled from env:', result, 'env value:', process.env.BROWSERSTACK_TEST_ACCESSIBILITY);
     return result;
   }
 
-  logToServer('[A11Y-LOG] isAccessibilityEnabled: no setting found, returning false');
+  console.log('[A11Y-LOG] isAccessibilityEnabled: no setting found, returning false');
   return false;
 };
 
@@ -92,20 +97,20 @@ exports.isAccessibilityEnabled = (user_config = null) => {
 exports.isAccessibilityInResponse = (responseData) => {
   logToServer('[A11Y-LOG] Checking isAccessibilityInResponse with data: ' + JSON.stringify(responseData));
 
-  logToServer('[A11Y-LOG] Checking isAccessibilityInResponse with data: ' + JSON.stringify(responseData?.accessibility || 'No accessibility in response', null, 2));
+  console.log('[A11Y-LOG] Checking isAccessibilityInResponse with data:', JSON.stringify(responseData?.accessibility || 'No accessibility in response', null, 2));
 
   if (responseData && responseData.accessibility) {
     if (responseData.accessibility && typeof responseData.accessibility === 'object') {
       const successValue = responseData.accessibility.success;
       const result = successValue === true;
-      logToServer('[A11Y-LOG] isAccessibilityInResponse result: ' + result + ', success value: ' + successValue);
+      console.log('[A11Y-LOG] isAccessibilityInResponse result:', result, 'success value:', successValue);
       return result;
     }
     // If accessibility is null or not an object, treat as false
-    logToServer('[A11Y-LOG] isAccessibilityInResponse: accessibility is null or not object, returning false');
+    console.log('[A11Y-LOG] isAccessibilityInResponse: accessibility is null or not object, returning false');
     return false;
   }
-  logToServer('[A11Y-LOG] isAccessibilityInResponse: no accessibility in response, returning false');
+  console.log('[A11Y-LOG] isAccessibilityInResponse: no accessibility in response, returning false');
   return false;
 };
 
@@ -173,35 +178,35 @@ exports.handleErrorForObservability = (error = null) => {
 };
 
 exports.setAccessibilityVariables = (user_config, responseData) => {
-  logToServer('[A11Y-LOG] setAccessibilityVariables called with response: ' + JSON.stringify(responseData?.accessibility || 'No accessibility', null, 2));
+  console.log('[A11Y-LOG] setAccessibilityVariables called with response:', JSON.stringify(responseData?.accessibility || 'No accessibility', null, 2));
 
   // Match C# SDK ProcessAccessibilityResponse logic
   if (!responseData.accessibility) {
-    logToServer('[A11Y-LOG] No accessibility in response, handling error');
+    console.log('[A11Y-LOG] No accessibility in response, handling error');
     exports.handleErrorForAccessibility(user_config);
     return [null, null];
   }
 
   if (!responseData.accessibility.success) {
-    logToServer('[A11Y-LOG] Accessibility success is false, handling error');
+    console.log('[A11Y-LOG] Accessibility success is false, handling error');
     exports.handleErrorForAccessibility(user_config, responseData.accessibility);
     return [null, null];
   }
 
   // Match C# SDK: if (accessibilityResponse["success"].ToString() == "True")
   if (responseData.accessibility.success === true) {
-    logToServer('[A11Y-LOG] Server auto-enabled accessibility - processing response');
+    console.log('[A11Y-LOG] Server auto-enabled accessibility - processing response');
     // Set configuration like C# SDK: isAccessibility = true;
     user_config.run_settings.accessibility = true;
     process.env.BROWSERSTACK_TEST_ACCESSIBILITY = 'true';
 
     if (responseData.accessibility.options) {
-      logToServer('[A11Y-LOG] Processing accessibility options from server');
+      console.log('[A11Y-LOG] Processing accessibility options from server');
       logger.debug(`BrowserStack Accessibility Automation Build Hashed ID: ${responseData.build_hashed_id}`);
       setAccessibilityCypressCapabilities(user_config, responseData);
       helper.setBrowserstackCypressCliDependency(user_config);
     } else {
-      logToServer('[A11Y-LOG] No accessibility options in server response');
+      console.log('[A11Y-LOG] No accessibility options in server response');
     }
   }
 };
@@ -378,30 +383,29 @@ exports.getAccessibilityOptions = (user_config) => {
   // Check run_settings.accessibility first (highest priority)
   if (user_config.run_settings.accessibility === true) {
     enabled = true;
-    logToServer('[A11Y-LOG] User explicitly enabled accessibility via run_settings');
+    console.log('[A11Y-LOG] User explicitly enabled accessibility via run_settings');
   } else if (user_config.run_settings.accessibility === false) {
     enabled = false;
-    logToServer('[A11Y-LOG] User explicitly disabled accessibility via run_settings');
+    console.log('[A11Y-LOG] User explicitly disabled accessibility via run_settings');
   }
   // Check environment variable (fallback)
   else if (process.env.BROWSERSTACK_TEST_ACCESSIBILITY === 'true') {
     enabled = true;
-    logToServer('[A11Y-LOG] User enabled accessibility via environment variable');
+    console.log('[A11Y-LOG] User enabled accessibility via environment variable');
   } else if (process.env.BROWSERSTACK_TEST_ACCESSIBILITY === 'false') {
     enabled = false;
-    logToServer('[A11Y-LOG] User disabled accessibility via environment variable');
+    console.log('[A11Y-LOG] User disabled accessibility via environment variable');
   }
   // Otherwise keep as null for server auto-enable decision
   else {
-    logToServer('[A11Y-LOG] No explicit user setting - sending null for server auto-enable decision');
+    console.log('[A11Y-LOG] No explicit user setting - sending null for server auto-enable decision');
   }
 
   const result = { 
-    settings: settings,
-    enabled: enabled  // Send user preference to server (null = let server decide)
+    settings: settings, // Send user preference to server (null = let server decide)
   };
 
-  logToServer('[A11Y-LOG] Final accessibility options for server: ' + JSON.stringify(result, null, 2));
+  console.log('[A11Y-LOG] Final accessibility options for server:', JSON.stringify(result, null, 2));
 
   return result;
 };