Use forEach() w/ an accumulator over reduce() with an ignored return value (#14817)

Sonar reports this as a bug since it's ignoring a return value in a situation where you don't _have_ to use a reduction/fold.

Author: brettcannon

src/client/testing/common/testUtils.ts

@@ -143,7 +143,8 @@ export class TestsHelper implements ITestsHelper {
         folders.sort();
         const resource = Uri.file(workspaceFolder);
         folders.forEach((dir) => {
-            dir.split(path.sep).reduce((parentPath, currentName, _index, _values) => {
+            let parentPath = ''; // Accumulator
+            dir.split(path.sep).forEach((currentName) => {
                 let newPath = currentName;
                 let parentFolder: TestFolder | undefined;
                 if (parentPath.length > 0) {
@@ -175,8 +176,8 @@ export class TestsHelper implements ITestsHelper {
                         });
                     tests.testFolders.push(testFolder);
                 }
-                return newPath;
-            }, '');
+                parentPath = newPath;
+            });
         });
     }
     public parseTestName(name: string, rootDirectory: string, tests: Tests): TestsToRun | undefined {