Merge pull request #641 from browserstack/OBS_1750_change_defaults

Change defaults w.r.t build artifacts

Author: Archish Thakkar

bin/testObservability/helper/helper.js

@@ -741,6 +741,9 @@ exports.setTestObservabilityFlags = (bsConfig) => {
   /* testObservability */
   let isTestObservabilitySession = false;
   try {
+    /* set default again but under try catch in case of wrong config */
+    isTestObservabilitySession = utils.nonEmptyArray(bsConfig.run_settings.downloads) ? false : true;
+
     if(!utils.isUndefined(bsConfig["testObservability"])) isTestObservabilitySession = ( bsConfig["testObservability"] == true || bsConfig["testObservability"] == 1 );
     if(!utils.isUndefined(process.env.BROWSERSTACK_TEST_OBSERVABILITY)) isTestObservabilitySession = ( process.env.BROWSERSTACK_TEST_OBSERVABILITY == "true" || process.env.BROWSERSTACK_TEST_OBSERVABILITY == "1" );
     if(process.argv.includes('--disable-test-observability')) isTestObservabilitySession = false;
@@ -750,7 +753,6 @@ exports.setTestObservabilityFlags = (bsConfig) => {
     exports.debug(`EXCEPTION while parsing testObservability capability with error ${e}`, true, e);
   }
 
-
   /* browserstackAutomation */
   let isBrowserstackInfra = true;
   try {
@@ -762,6 +764,7 @@ exports.setTestObservabilityFlags = (bsConfig) => {
     exports.debug(`EXCEPTION while parsing browserstackAutomation capability with error ${e}`, true, e);
   }
 
+  if(isTestObservabilitySession) logger.warn("testObservability is set to true. Other test reporters you are using will be automatically disabled. Learn more at browserstack.com/docs/test-observability/overview/what-is-test-observability");
 
   process.env.BROWSERSTACK_TEST_OBSERVABILITY = isTestObservabilitySession;
   process.env.BROWSERSTACK_AUTOMATION = isBrowserstackInfra;