review changes

Author: rev-doshi

bin/accessibility-automation/cypress/index.js

@@ -42,7 +42,7 @@ Cypress.on('test:after:run', (attributes, runnable) => {
           const excluded = excludeTagArray.some((exclude) => fullTestName.includes(exclude));
           const included = includeTagArray.length === 0 || includeTags.some((include) => fullTestName.includes(include));
           shouldScanTestForAccessibility = !excluded && included;
-        } catch (error){
+        } catch (error) {
           console.log("Error while validating test case for accessibility before scanning. Error : ", error);
         }
       }
@@ -52,14 +52,18 @@ Cypress.on('test:after:run', (attributes, runnable) => {
       } else {
         os_data = Cypress.platform === 'linux' ? 'mac' : "win"
       }
+      let filePath = '';
+      if (attributes.invocationDetails.relativeFile !== undefined) {
+        filePath = attributes.invocationDetails.relativeFile;
+      }
       const dataForExtension = {
         "saveResults": shouldScanTestForAccessibility,
         "testDetails": {
           "name": attributes.title,
           "testRunId": '5058', // variable not consumed, shouldn't matter what we send
-          "filePath": attributes.invocationDetails.relativeFile,
+          "filePath": filePath,
           "scopeList": [
-            attributes.invocationDetails.relativeFile,
+            filePath,
             attributes.title
           ]
         },

bin/accessibility-automation/helper.js

@@ -88,6 +88,7 @@ exports.createAccessibilityTestRun = async (user_config, framework) => {
     if (process.env.BS_A11Y_JWT) {
       process.env.BROWSERSTACK_TEST_ACCESSIBILITY = 'true';
     }
+    logger.debug(`BrowserStack Accessibility Automation Test Run ID: ${response.data.data.id}`);
 
     this.setAccessibilityCypressCapabilities(user_config, response.data);
     setAccessibilityEventListeners();
@@ -163,7 +164,6 @@ const getAccessibilityCypressCommandEventListener = () => {
 }
 
 const setAccessibilityEventListeners = () => {
-  logger.info("setAccessibilityEventListeners")
   try {
     const cypressCommandEventListener = getAccessibilityCypressCommandEventListener();
     glob(process.cwd() + '/cypress/support/*.js', {}, (err, files) => {

bin/helpers/capabilityHelper.js

@@ -149,8 +149,12 @@ const caps = (bsConfig, zip) => {
 const getAccessibilityPlatforms = (bsConfig) => {
   const browserList = bsConfig.browsers;
   const accessibilityPlatforms = Array(browserList.length).fill(false);
+  let rootLevelAccessibility = false;
+  if (bsConfig.run_settings.accessibility !== undefined) {
+    rootLevelAccessibility = bsConfig.run_settings.accessibility.toString() === 'true'
+  }
   browserList.forEach((browserDetails, idx) => {
-    accessibilityPlatforms[idx] = (browserDetails.accessibility === undefined) ? false : browserDetails.accessibility
+    accessibilityPlatforms[idx] = (browserDetails.accessibility === undefined) ? rootLevelAccessibility : browserDetails.accessibility
   });
   return accessibilityPlatforms;
 }