make the build runner and test runner talk to each other

std.Build.addTest creates a CompileStep as before, however, this kind of
step no longer actually runs the unit tests. Instead it only compiles
it, and one must additionally create a RunStep from the CompileStep in
order to actually run the tests.

RunStep gains integration with the default test runner, which now
supports the standard --listen=- argument in order to communicate over
stdin and stdout. It also reports test statistics; how many passed,
failed, and leaked, as well as directly associating the relevant stderr
with the particular test name that failed.

This separation of CompileStep and RunStep means that
`CompileStep.Kind.test_exe` is no longer needed, and therefore has been
removed in this commit.

 * build runner: show unit test statistics in build summary
 * added Step.writeManifest since many steps want to treat it as a
   warning and emit the same message if it fails.
 * RunStep: fixed error message that prints the failed command printing
   the original argv and not the adjusted argv in case an interpreter
   was used.
 * RunStep: fixed not passing the command line arguments to the
   interpreter.
 * move src/Server.zig to std.zig.Server so that the default test runner
   can use it.
 * the simpler test runner function which is used by work-in-progress
   backends now no longer prints to stderr, which is necessary in order
   for the build runner to not print the stderr as a warning message.

1 files changed, 5 insertions, 16 deletions

diff --git a/src/main.zig b/src/main.zig
index e0283143d0..9602a5cd31 100644
--- a/src/main.zig
+++ b/src/main.zig
@@ -10,6 +10,7 @@ const ArrayList = std.ArrayList;
 const Ast = std.zig.Ast;
 const warn = std.log.warn;
 const ThreadPool = std.Thread.Pool;
+const cleanExit = std.process.cleanExit;
 
 const tracy = @import("tracy.zig");
 const Compilation = @import("Compilation.zig");
@@ -26,7 +27,7 @@ const target_util = @import("target.zig");
 const crash_report = @import("crash_report.zig");
 const Module = @import("Module.zig");
 const AstGen = @import("AstGen.zig");
-const Server = @import("Server.zig");
+const Server = std.zig.Server;
 
 pub const std_options = struct {
     pub const wasiCwd = wasi_cwd;
@@ -3545,6 +3546,7 @@ fn serve(
         .gpa = gpa,
         .in = in,
         .out = out,
+        .zig_version = build_options.version,
     });
     defer server.deinit();
 
@@ -3656,8 +3658,8 @@ fn serve(
                     );
                 }
             },
-            _ => {
-                @panic("TODO unrecognized message from client");
+            else => {
+                fatal("unrecognized message from client: 0x{x}", .{@enumToInt(hdr.tag)});
             },
         }
     }
@@ -5624,19 +5626,6 @@ fn detectNativeTargetInfo(cross_target: std.zig.CrossTarget) !std.zig.system.Nat
     return std.zig.system.NativeTargetInfo.detect(cross_target);
 }
 
-/// Indicate that we are now terminating with a successful exit code.
-/// In debug builds, this is a no-op, so that the calling code's
-/// cleanup mechanisms are tested and so that external tools that
-/// check for resource leaks can be accurate. In release builds, this
-/// calls exit(0), and does not return.
-pub fn cleanExit() void {
-    if (builtin.mode == .Debug) {
-        return;
-    } else {
-        process.exit(0);
-    }
-}
-
 const usage_ast_check =
     \\Usage: zig ast-check [file]
     \\