better result location semantics with optionals and return locations

somewhere along this branch, #1901 has been fixed.

1 files changed, 8 insertions, 14 deletions

diff --git a/std/special/test_runner.zig b/std/special/test_runner.zig
index 001b26ebb0..db01293059 100644
--- a/std/special/test_runner.zig
+++ b/std/special/test_runner.zig
@@ -2,34 +2,28 @@ const std = @import("std");
 const io = std.io;
 const builtin = @import("builtin");
 const test_fn_list = builtin.test_functions;
+const warn = std.debug.warn;
 
-pub fn main() void {
-    const stderr = io.getStdErr() catch std.process.abort();
-
+pub fn main() !void {
     var ok_count: usize = 0;
     var skip_count: usize = 0;
     for (test_fn_list) |test_fn, i| {
-        stderr.write("test ") catch std.process.abort();
-        stderr.write(test_fn.name) catch std.process.abort();
+        warn("{}/{} {}...", i + 1, test_fn_list.len, test_fn.name);
 
         if (test_fn.func()) |_| {
             ok_count += 1;
-            stderr.write("...OK\n") catch std.process.abort();
+            warn("OK\n");
         } else |err| switch (err) {
             error.SkipZigTest => {
                 skip_count += 1;
-                stderr.write("...SKIP\n") catch std.process.abort();
-            },
-            else => {
-                stderr.write("error: ") catch std.process.abort();
-                stderr.write(@errorName(err)) catch std.process.abort();
-                std.process.abort();
+                warn("SKIP\n");
             },
+            else => return err,
         }
     }
     if (ok_count == test_fn_list.len) {
-        stderr.write("All tests passed.\n") catch std.process.abort();
+        warn("All tests passed.\n");
     } else {
-        stderr.write("Some tests skipped.\n") catch std.process.abort();
+        warn("{} passed; {} skipped.\n", ok_count, skip_count);
     }
 }