Reduce verbosity and fix serving assets from test directory.

lib/web_ui/dev/steps/compile_bundle_step.dart

@@ -18,10 +18,12 @@ import '../utils.dart' show AnsiColors, FilePath, ProcessManager, cleanup, getBu
 class CompileBundleStep implements PipelineStep {
   CompileBundleStep({
     required this.bundle,
+    required this.isVerbose,
     this.testFiles,
   });
 
   final TestBundle bundle;
+  final bool isVerbose;
   final Set<FilePath>? testFiles;
 
   // Maximum number of concurrent compile processes to use.
@@ -66,12 +68,14 @@ class CompileBundleStep implements PipelineStep {
           testSetDirectory,
           outputBundleDirectory,
           renderer: bundle.compileConfig.renderer,
+          isVerbose: isVerbose,
         );
       case Compiler.dart2wasm:
         return Dart2WasmCompiler(
           testSetDirectory,
           outputBundleDirectory,
           renderer: bundle.compileConfig.renderer,
+          isVerbose: isVerbose,
         );
     }
   }
@@ -141,12 +145,16 @@ abstract class TestCompiler {
   TestCompiler(
     this.inputTestSetDirectory,
     this.outputTestBundleDirectory,
-    {required this.renderer}
+    {
+      required this.renderer,
+      required this.isVerbose,
+    }
   );
 
   final io.Directory inputTestSetDirectory;
   final io.Directory outputTestBundleDirectory;
   final Renderer renderer;
+  final bool isVerbose;
 
   Future<bool> compileTest(FilePath input);
 }
@@ -155,7 +163,10 @@ class Dart2JSCompiler extends TestCompiler {
   Dart2JSCompiler(
     super.inputTestSetDirectory,
     super.outputTestBundleDirectory,
-    {required super.renderer}
+    {
+      required super.renderer,
+      required super.isVerbose,
+    }
   );
 
   @override
@@ -199,7 +210,7 @@ class Dart2JSCompiler extends TestCompiler {
       environment.dartExecutable,
       arguments,
       workingDirectory: inputTestSetDirectory.path,
-      evalOutput: true,
+      evalOutput: !isVerbose,
     );
     final int exitCode = await process.wait();
     if (exitCode != 0) {
@@ -216,7 +227,10 @@ class Dart2WasmCompiler extends TestCompiler {
   Dart2WasmCompiler(
     super.inputTestSetDirectory,
     super.outputTestBundleDirectory,
-    {required super.renderer}
+    {
+      required super.renderer,
+      required super.isVerbose,
+    }
   );
 
   @override

lib/web_ui/dev/steps/run_suite_step.dart

@@ -30,6 +30,7 @@ import '../utils.dart';
 class RunSuiteStep implements PipelineStep {
   RunSuiteStep(this.suite, {
     required this.isDebug,
+    required this.isVerbose,
     required this.doUpdateScreenshotGoldens,
     required this.requireSkiaGold,
     this.testFiles,
@@ -39,6 +40,7 @@ class RunSuiteStep implements PipelineStep {
   final TestSuite suite;
   final Set<FilePath>? testFiles;
   final bool isDebug;
+  final bool isVerbose;
   final bool doUpdateScreenshotGoldens;
   final String? overridePathToCanvasKit;
 
@@ -92,6 +94,7 @@ class RunSuiteStep implements PipelineStep {
         skiaClient: skiaClient,
         overridePathToCanvasKit: overridePathToCanvasKit,
         isWasm: isWasm,
+        isVerbose: isVerbose,
       );
     });
 

lib/web_ui/dev/test_platform.dart

@@ -52,6 +52,7 @@ class BrowserPlatform extends PlatformPlugin {
     required this.browserEnvironment,
     required this.server,
     required this.isDebug,
+    required this.isVerbose,
     required this.doUpdateScreenshotGoldens,
     required this.packageConfig,
     required this.skiaClient,
@@ -123,6 +124,7 @@ class BrowserPlatform extends PlatformPlugin {
     required SkiaGoldClient? skiaClient,
     required String? overridePathToCanvasKit,
     required bool isWasm,
+    required bool isVerbose,
   }) async {
     final shelf_io.IOServer server =
         shelf_io.IOServer(await HttpMultiServer.loopback(0));
@@ -131,6 +133,7 @@ class BrowserPlatform extends PlatformPlugin {
       browserEnvironment: browserEnvironment,
       server: server,
       isDebug: Configuration.current.pauseAfterLoad,
+      isVerbose: isVerbose,
       doUpdateScreenshotGoldens: doUpdateScreenshotGoldens,
       packageConfig: await loadPackageConfigUri((await Isolate.packageConfig)!),
       skiaClient: skiaClient,
@@ -145,6 +148,8 @@ class BrowserPlatform extends PlatformPlugin {
   /// breakpoints in the code.
   final bool isDebug;
 
+  final bool isVerbose;
+
   /// The underlying server.
   final shelf.Server server;
 
@@ -255,7 +260,9 @@ class BrowserPlatform extends PlatformPlugin {
   }
 
   Future<shelf.Response> _fileNotFoundCatcher(shelf.Request request) async {
-    print('HTTP 404: ${request.url}');
+    if (isVerbose) {
+      print('HTTP 404: ${request.url}');
+    }
     return shelf.Response.notFound('File not found');
   }
 
@@ -386,10 +393,12 @@ class BrowserPlatform extends PlatformPlugin {
     final String filename = requestData['filename'] as String;
 
     if (!(await browserManager).supportsScreenshots) {
-      print(
-        'Skipping screenshot check for $filename. Current browser/OS '
-        'combination does not support screenshots.',
-      );
+      if (isVerbose) {
+        print(
+          'Skipping screenshot check for $filename. Current browser/OS '
+          'combination does not support screenshots.',
+        );
+      }
       return shelf.Response.ok(json.encode('OK'));
     }
 

lib/web_ui/dev/test_runner.dart

@@ -237,12 +237,16 @@ class TestCommand extends Command<bool> with ArgUtils<bool> {
       CopyArtifactsStep(artifacts),
       if (shouldCompile)
         for (final TestBundle bundle in bundles)
-          CompileBundleStep(bundle: bundle),
+          CompileBundleStep(
+            bundle: bundle,
+            isVerbose: isVerbose
+          ),
       if (shouldRun)
         for (final TestSuite suite in filteredSuites)
           RunSuiteStep(
             suite, 
             isDebug: isDebug,
+            isVerbose: isVerbose,
             doUpdateScreenshotGoldens: doUpdateScreenshotGoldens,
             requireSkiaGold: requireSkiaGold,
             overridePathToCanvasKit: overridePathToCanvasKit,

lib/web_ui/dev/utils.dart

@@ -372,7 +372,7 @@ Future<void> cleanup() async {
 io.Directory getTestSetDirectory(TestSet testSet) {
   return io.Directory(
     path.join(
-      environment.webUiBuildDir.path,
+      environment.webUiTestDir.path,
       testSet.directory,
     )
   );