fix(bazel): failure messages accidentally printing [Object object] …

…for commands Since commands consist of `BazelExpandedValue` objects, the current failure messages incorrectly print the command.
angular · Nov 5, 2021 · baa69aa · baa69aa
1 parent 8c73f43
commit baa69aa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/bazel/integration/test_runner/process_utils.ts b/bazel/integration/test_runner/process_utils.ts
@@ -47,7 +47,7 @@ export function runCommandInChildProcess(
   workingDir: string,
   env: NodeJS.ProcessEnv,
 ): Promise<boolean> {
-  const humanReadableCommand = `${binary} ${args.join(' ')}`;
+  const humanReadableCommand = `${binary}${args.length ? ` ${args.join(' ')}` : ''}`;
 
   debug(`Executing command: ${humanReadableCommand} in ${workingDir}`);
 

diff --git a/bazel/integration/test_runner/runner.ts b/bazel/integration/test_runner/runner.ts
@@ -231,7 +231,7 @@ export class TestRunner {
 
       if (!success) {
         throw Error(
-          `Integration test command: \`${binary} ${evaluatedArgs.join(' ')}\` failed. ` +
+          `Integration test command: \`${binary.value} ${evaluatedArgs.join(' ')}\` failed. ` +
             `See error output above for details.`,
         );
       }