[terra-functional-testing] Run tests in all browsers provided by the BROWSERS env variable.

packages/terra-functional-testing/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Unreleased
 
+* Fixed
+  * Run tests in all browsers provided by the `BROWSERS` env variable.
+
 ## 2.0.0 - (July 16, 2021)
 
 * Breaking

packages/terra-functional-testing/src/terra-cli/wdio/index.js

@@ -28,6 +28,9 @@ const cli = {
 
             // Split the list of BROWSERS into an array if it is set in the string form like 'chrome,firefox,ie'.
             if (browsers.includes(',')) {
+              // There could be empty spaces between each browser name. For example: 'chrome, firefox,     ie').
+              // Remove all empty spaces in each browser name to avoid adding them with the empty space. For example: ' firefox'.
+              browsers = browsers.replace(/ /g, '');
               return browsers.split(',');
             }
 

packages/terra-functional-testing/tests/jest/terra-cli/wdio/index.test.js

@@ -26,6 +26,30 @@ describe('index', () => {
     expect(helpOutput).toMatchSnapshot();
   });
 
+  it('should correctly parse BROWSERS env variable when there are empty spaces', async () => {
+    process.env.BROWSERS = '[chrome, firefox,      ie]';
+    const parser = yargs.command(command).scriptName('terra');
+    const options = await new Promise((resolve) => {
+      parser.parse('wdio', (_err, _argv) => {
+        resolve(_argv);
+      });
+    });
+
+    expect(options.browsers).toEqual(['chrome', 'firefox', 'ie']);
+  });
+
+  it('should correctly parse BROWSERS env variable when there are no empty spaces', async () => {
+    process.env.BROWSERS = '[chrome,firefox,ie]';
+    const parser = yargs.command(command).scriptName('terra');
+    const options = await new Promise((resolve) => {
+      parser.parse('wdio', (_err, _argv) => {
+        resolve(_argv);
+      });
+    });
+
+    expect(options.browsers).toEqual(['chrome', 'firefox', 'ie']);
+  });
+
   it('should invoke the test runner with the default command line arguments', async () => {
     const { handler } = command;