Better error message for partial create success

Author: kaibolay

src/commands/appdistribution-testcases-import.ts

@@ -6,6 +6,7 @@ import { AppDistributionClient } from "../appdistribution/client";
 import { getAppName, ensureFileExists } from "../appdistribution/options-parser-util";
 import { TestCase } from "../appdistribution/types";
 import * as utils from "../utils";
+import { FirebaseError } from "../error";
 
 export const command = new Command("appdistribution:testcases:import <test-cases-yaml-file>")
   .description("import test cases from YAML file")
@@ -19,9 +20,19 @@ export const command = new Command("appdistribution:testcases:import <test-cases
     const testCasesWithoutName = testCases.filter((tc) => !tc.name);
     // nothing can depend on these test cases yet, since they don't have
     // resource names
-    await Promise.all(
+    const creationResults = await Promise.allSettled(
       testCasesWithoutName.map((tc) => appDistroClient.createTestCase(appName, tc)),
     );
+    const failed = creationResults.filter((r) => r.status === "rejected");
+    if (failed.length > 0) {
+      for (const f of failed) {
+        utils.logWarning((f as any).reason);
+      }
+      const succeeded = creationResults.length - failed.length;
+      throw new FirebaseError(
+        `Created ${succeeded} test case(s), but failed to create ${failed.length}.`,
+      );
+    }
     // any test case with a resource name can be referenced by any other new or
     // existing test case, so we need to batch the upsert.
     const testCasesWithName = testCases.filter((tc) => !!tc.name);