Explicitly check the array length (#21)

Author: yqiu24

javascript/dist/queue/BaseRunner.js

@@ -66,7 +66,9 @@ class BaseRunner {
         try {
             const failedTests = await this.client.hGetAll(this.retriedBuildKey('error-reports'));
             console.log(`[ci-queue] Failed tests`, failedTests);
-            const failedTestGroups = Object.values(failedTests).map(test => JSON.parse(test).test_group);
+            const failedTestGroups = Object.keys(failedTests).length > 0
+                ? Object.values(failedTests).map(test => JSON.parse(test).test_group)
+                : [];
             return failedTestGroups;
         }
         catch (e) {

javascript/src/queue/BaseRunner.ts

@@ -84,7 +84,9 @@ export class BaseRunner {
     try {
       const failedTests = await this.client.hGetAll(this.retriedBuildKey('error-reports'));
       console.log(`[ci-queue] Failed tests`, failedTests);
-      const failedTestGroups = Object.values(failedTests).map(test => JSON.parse(test).test_group);
+      const failedTestGroups = Object.keys(failedTests).length > 0 
+        ? Object.values(failedTests).map(test => JSON.parse(test).test_group)
+        : [];
       return failedTestGroups;
     } catch (e) {
       // If the previous build is still in-progress, there may not be any failed tests