Optimized getBranches to use parallel requests

Author: edgardmessias

src/svnRepository.ts

@@ -83,40 +83,54 @@ export class Repository {
   async getBranches() {
     const repoUrl = await this.getRepoUrl();
 
-    const branches = [];
+    let branches:string[] = [];
 
-    let trunkExists = await this.svn.exec("", [
-      "ls",
-      repoUrl + "/trunk",
-      "--depth",
-      "empty"
-    ]);
+    let promises = [];
 
-    if (trunkExists.exitCode === 0) {
-      branches.push("trunk");
-    }
+    promises.push(new Promise<string[]>(async resolve => {
+      let trunkExists = await this.svn.exec("", [
+        "ls",
+        repoUrl + "/trunk",
+        "--depth",
+        "empty"
+      ]);
+
+      if (trunkExists.exitCode === 0) {
+        resolve(["trunk"]);
+        return;
+      }
+      resolve([]);
+    }));
 
     const trees = ["branches", "tags"];
 
     for (let index in trees) {
-      const tree = trees[index];
-      const branchUrl = repoUrl + "/" + tree;
+      promises.push(new Promise<string[]>(async resolve => {
+        const tree = trees[index];
+        const branchUrl = repoUrl + "/" + tree;
 
-      const result = await this.svn.list(branchUrl);
+        const result = await this.svn.list(branchUrl);
 
-      if (result.exitCode !== 0) {
-        continue;
-      }
+        if (result.exitCode !== 0) {
+          resolve([]);
+          return;
+        }
 
-      const list = result.stdout
-        .trim()
-        .replace(/\/|\\/g, "")
-        .split(/[\r\n]+/)
-        .map((i: string) => tree + "/" + i);
+        const list = result.stdout
+          .trim()
+          .replace(/\/|\\/g, "")
+          .split(/[\r\n]+/)
+          .map((i: string) => tree + "/" + i);
 
-      branches.push(...list);
+        resolve(list);
+      }));
     }
 
+    const all = await Promise.all<any>(promises);
+    all.forEach(list => {
+      branches.push(...list);
+    });
+
     return branches;
   }