fixup! feat(@angular/build): enhance build logs and output location for library builder

Author: alan-agius4

packages/angular/build/src/builders/library/builder.ts

@@ -350,12 +350,16 @@ function logBuildResult(
   outputPath?: string,
 ): void {
   const buildTime = Number(process.hrtime.bigint() - startTime) / 10 ** 9;
-
-  logger.info(
-    `Library bundle generation ${success ? 'complete' : 'failed'}. [${buildTime.toFixed(3)} seconds] - ${new Date().toISOString()}`,
-  );
-
-  if (success && outputPath) {
-    logger.info(`Output location: ${outputPath}\n`);
+  const message =
+    `Library bundle generation ${success ? 'complete' : 'failed'}.` +
+    ` [${buildTime.toFixed(3)} seconds] - ${new Date().toISOString()}`;
+
+  if (success) {
+    logger.info(message);
+    if (outputPath) {
+      logger.info(`Output location: ${outputPath}\n`);
+    }
+  } else {
+    logger.error(message);
   }
 }

packages/angular/build/src/builders/library/tests/behavior/build_spec.ts

@@ -162,5 +162,32 @@ describeLibraryBuilder(executeLibraryBuilder, LIBRARY_BUILDER_INFO, (harness) =>
         }),
       );
     });
+
+    it('should log failure on build error', async () => {
+      await harness.writeFile(
+        'projects/lib/src/public-api.ts',
+        `export const title: number = 'invalid type';`,
+      );
+
+      harness.useTarget('build', {
+        ...BASE_OPTIONS,
+      });
+
+      const { result, logs } = await harness.executeOnce();
+      expect(result?.success).toBeFalse();
+      expect(logs).toContain(
+        jasmine.objectContaining({
+          level: 'error',
+          message: jasmine.stringMatching(
+            /Library bundle generation failed\. \[\d+\.\d{3} seconds\] - \d{4}-\d{2}-\d{2}T/,
+          ),
+        }),
+      );
+      expect(logs).not.toContain(
+        jasmine.objectContaining({
+          message: jasmine.stringMatching(/Output location:/),
+        }),
+      );
+    });
   });
 });