Skip to content

Commit 6c03dde

Browse files
log changes
1 parent dd25780 commit 6c03dde

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

bin/testhub/testhubHandler.js

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,10 @@ class TestHubHandler {
3939
const data = await this.generateBuildUpstreamData(user_config);
4040
const config = this.getConfig(obsUserName, obsAccessKey);
4141

42-
console.log('[A11Y] Making TestHub API request to:', TESTHUB_CONSTANTS.TESTHUB_BUILD_API);
42+
logger.debug('[A11Y] Making TestHub API request to:', TESTHUB_CONSTANTS.TESTHUB_BUILD_API);
4343
const response = await nodeRequest( "POST", TESTHUB_CONSTANTS.TESTHUB_BUILD_API, data, config);
4444

45-
console.log('[A11Y] TestHub API response received:', JSON.stringify(response.data?.accessibility || 'No accessibility in response', null, 2));
45+
logger.debug('[A11Y] TestHub API response received:', JSON.stringify(response.data?.accessibility || 'No accessibility in response', null, 2));
4646
const launchData = this.extractDataFromResponse(user_config, data, response, config);
4747
} catch (error) {
4848
console.log(error);
@@ -60,7 +60,7 @@ class TestHubHandler {
6060
const accessibilityOptions = testhubUtils.getAccessibilityOptions(user_config);
6161

6262
// Log what accessibility data is being sent to server
63-
console.log('[A11Y] Sending accessibility options to TestHub server:', JSON.stringify(accessibilityOptions, null, 2));
63+
logger.debug('[A11Y] Sending accessibility options to TestHub server:', JSON.stringify(accessibilityOptions, null, 2));
6464

6565
const data = {
6666
project_name: projectName,
@@ -117,20 +117,14 @@ class TestHubHandler {
117117
const userAccessibilityEnabled = testhubUtils.isAccessibilityEnabled(user_config);
118118
const serverAutoEnabled = testhubUtils.isAccessibilityInResponse(response.data);
119119

120-
console.log('[A11Y] C# SDK pattern check:', {
121-
'isAccessibilityEnabled': userAccessibilityEnabled,
122-
'isAccessibilityInResponse': serverAutoEnabled,
123-
'final_decision': userAccessibilityEnabled || serverAutoEnabled
124-
});
125-
126120
if (userAccessibilityEnabled || serverAutoEnabled) {
127121
// Match C# SDK: bsConfig.accessibility = true; accessibilityAutomation.ProcessAccessibilityResponse(buildCreationResponse);
128-
console.log('[A11Y] Enabling accessibility - either user enabled or server auto-enabled');
122+
logger.debug('[A11Y] Enabling accessibility - either user enabled or server auto-enabled');
129123
user_config.run_settings.accessibility = true;
130124
testhubUtils.setAccessibilityVariables(user_config, response.data);
131125
} else {
132126
// Accessibility not enabled by user and not auto-enabled by server
133-
console.log('[A11Y] Accessibility not enabled - neither user enabled nor server auto-enabled');
127+
logger.debug('[A11Y] Accessibility not enabled - neither user enabled nor server auto-enabled');
134128
process.env.BROWSERSTACK_TEST_ACCESSIBILITY = 'false';
135129
testhubUtils.checkAndSetAccessibility(user_config, false);
136130
}

0 commit comments

Comments
 (0)