update env name + removed unused env

Author: Tanmay-Bstack

bin/accessibility-automation/cypress/index.js

@@ -342,8 +342,7 @@ afterEach(() => {
             browserStackLog(`Cypress env browserstack testhub uuid: ${Cypress.env("BROWSERSTACK_TESTHUB_UUID")}`);
             browserStackLog(`Cypress env browserstack testhub jwt: ${Cypress.env("BROWSERSTACK_TESTHUB_JWT")}`);
             browserStackLog(`Cypress env http port: ${Cypress.env("BROWSERSTACK_TESTHUB_API_PORT")}`);
-            browserStackLog(`test env: ${Cypress.env("TEST_ENV")}`);
-            browserStackLog(`reporter api: ${Cypress.env("REPORTER_API")}`);
+            browserStackLog(`reporter api: ${Cypress.env("REPORTER_API_PORT_NO")}`);
 
             let testRunUuid = null;
             cy.task('get_test_run_uuid', { testIdentifier: attributes.title })

bin/accessibility-automation/plugin/index.js

@@ -17,14 +17,13 @@ const browserstackAccessibility = (on, config) => {
     },
     get_test_run_uuid({ testIdentifier, retries = 15, interval = 300 } = {}) {
       return new Promise((resolve) => {
-        console.log(`printing env variables take 6`);
+        console.log(`printing env variables take 7`);
         console.log(`Cypress env browserstack testhub uuid from plugin: ${config.env.BROWSERSTACK_TESTHUB_UUID}`);
-        console.log(`Cypress env http port: ${config.env.REPORTER_API}`);
-        console.log(`test env: ${config.env.TEST_ENV}`);
-        console.log(`reporter api from process: ${process.env.REPORTER_API}`);
+        console.log(`Cypress env http port: ${config.env.REPORTER_API_PORT_NO}`);
+        console.log(`reporter api from process: ${process.env.REPORTER_API_PORT_NO}`);
         console.log(`Fetching testRunUuid for testIdentifier: ${testIdentifier}`);
         if(!testIdentifier) return resolve(null);
-        const port = process.env.REPORTER_API;
+        const port = process.env.REPORTER_API_PORT_NO;
         let attempt = 0;
         const fetchUuid = () => {
           const options = {
@@ -105,7 +104,7 @@ const browserstackAccessibility = (on, config) => {
   config.env.BROWSERSTACK_TESTHUB_UUID = process.env.BROWSERSTACK_TESTHUB_UUID
   config.env.BROWSERSTACK_TESTHUB_JWT = process.env.BROWSERSTACK_TESTHUB_JWT
   config.env.BROWSERSTACK_TESTHUB_API_PORT = process.env.BROWSERSTACK_TESTHUB_API_PORT
-  config.env.REPORTER_API = process.env.REPORTER_API
+  config.env.REPORTER_API_PORT_NO = process.env.REPORTER_API_PORT_NO
 
   config.env.IS_ACCESSIBILITY_EXTENSION_LOADED = browser_validation.toString()
 

bin/commands/runs.js

@@ -117,7 +117,7 @@ module.exports = function run(args, rawArgs) {
 
     const preferredPort = 5348;
     const port = await findAvailablePort(preferredPort);
-    process.env.REPORTER_API = port
+    process.env.REPORTER_API_PORT_NO = port
 
     // Send build start to TEST REPORTING AND ANALYTICS
     if(shouldProcessEventForTesthub()) {

bin/testObservability/reporter/index.js

@@ -256,8 +256,7 @@ class MyReporter {
       }
     });
 
-    const port = process.env.REPORTER_API;
-    process.env.TEST_ENV = 'from-reporter';
+    const port = process.env.REPORTER_API_PORT_NO;
 
     this.httpServer.listen(port, '127.0.0.1', async () => {
       console.log(`Test Observability HTTP server listening on port ${port}`);

bin/testhub/utils.js

@@ -234,7 +234,7 @@ exports.setTestHubCommonMetaInfo = (user_config, responseData) => {
   process.env.BROWSERSTACK_TESTHUB_UUID = responseData.build_hashed_id;
   user_config.run_settings.system_env_vars.push(`BROWSERSTACK_TESTHUB_JWT`);
   user_config.run_settings.system_env_vars.push(`BROWSERSTACK_TESTHUB_UUID`);
-  user_config.run_settings.system_env_vars.push(`REPORTER_API`);
+  user_config.run_settings.system_env_vars.push(`REPORTER_API_PORT_NO`);
 };
 
 exports.checkIfAccessibilityIsSupported = (user_config, accessibilityFlag) => {