Drop utils/fs.getSubDirectories().

Author: ericsnowcurrently

src/client/common/utils/fs.ts

@@ -3,32 +3,8 @@
 
 'use strict';
 
-import * as fs from 'fs';
-import * as path from 'path';
 import * as tmp from 'tmp';
 
-export function getSubDirectories(rootDir: string): Promise<string[]> {
-    return new Promise<string[]>(resolve => {
-        fs.readdir(rootDir, (error, files) => {
-            if (error) {
-                return resolve([]);
-            }
-            const subDirs: string[] = [];
-            files.forEach(name => {
-                const fullPath = path.join(rootDir, name);
-                try {
-                    if (fs.statSync(fullPath).isDirectory()) {
-                        subDirs.push(fullPath);
-                    }
-                }
-                // tslint:disable-next-line:no-empty one-line
-                catch (ex) { }
-            });
-            resolve(subDirs);
-        });
-    });
-}
-
 export function createTemporaryFile(extension: string, temporaryDirectory?: string): Promise<{ filePath: string; cleanupCallback: Function }> {
     // tslint:disable-next-line:no-any
     const options: any = { postfix: extension };

src/client/testing/common/managers/testConfigurationManager.ts

@@ -1,9 +1,9 @@
 import * as path from 'path';
 import { OutputChannel, QuickPickItem, Uri } from 'vscode';
 import { IApplicationShell } from '../../../common/application/types';
+import { IFileSystem } from '../../../common/platform/types';
 import { IInstaller, ILogger, IOutputChannel } from '../../../common/types';
 import { createDeferred } from '../../../common/utils/async';
-import { getSubDirectories } from '../../../common/utils/fs';
 import { IServiceContainer } from '../../../ioc/types';
 import { ITestConfigSettingsService, ITestConfigurationManager } from '../../types';
 import { TEST_OUTPUT_CHANNEL, UNIT_TEST_PRODUCTS } from '../constants';
@@ -101,7 +101,8 @@ export abstract class TestConfigurationManager implements ITestConfigurationMana
         return def.promise;
     }
     protected getTestDirs(rootDir: string): Promise<string[]> {
-        return getSubDirectories(rootDir).then(subDirs => {
+        const fs = this.serviceContainer.get<IFileSystem>(IFileSystem);
+        return fs.getSubDirectories(rootDir).then(subDirs => {
             subDirs.sort();
 
             // Find out if there are any dirs with the name test and place them on the top.