authorgravatar for sebastiankeller@fastmail.netSebastian Keller <sebastiankeller@fastmail.net> 2022-02-13 21:58:41+01:00
committergravatar for git@vexu.euVeikka Tuominen <git@vexu.eu> 2022-02-14 12:09:24+02:00
log9ca3c897ec546bc08eefe53471c995deefc9f36d
treef86759dde542eef9c33c03fbcc0e8dcffdcf317e
parentf5068107cdf54f143a150feb5d500962ba98e963

test_runner.zig: Do not log test name twice

In #10859 I moved the `test_node.end()` call after everything else has been logged. Now the `test_fn.name` is printed by `Progress` itself, making the additional log obsolete.

1 files changed, 3 insertions(+), 3 deletions(-)

lib/std/special/test_runner.zig+3-3
...@@ -70,7 +70,7 @@ pub fn main() void {...@@ -70,7 +70,7 @@ pub fn main() void {
70 .blocking => {70 .blocking => {
71 skip_count += 1;71 skip_count += 1;
72 test_node.end();72 test_node.end();
73 progress.log("{s}... SKIP (async test)\n", .{test_fn.name});73 progress.log("SKIP (async test)\n", .{});
74 if (!have_tty) std.debug.print("SKIP (async test)\n", .{});74 if (!have_tty) std.debug.print("SKIP (async test)\n", .{});
75 continue;75 continue;
76 },76 },
...@@ -82,13 +82,13 @@ pub fn main() void {...@@ -82,13 +82,13 @@ pub fn main() void {
82 } else |err| switch (err) {82 } else |err| switch (err) {
83 error.SkipZigTest => {83 error.SkipZigTest => {
84 skip_count += 1;84 skip_count += 1;
85 progress.log("{s}... SKIP\n", .{test_fn.name});85 progress.log("SKIP\n", .{});
86 if (!have_tty) std.debug.print("SKIP\n", .{});86 if (!have_tty) std.debug.print("SKIP\n", .{});
87 test_node.end();87 test_node.end();
88 },88 },
89 else => {89 else => {
90 fail_count += 1;90 fail_count += 1;
91 progress.log("{s}... FAIL ({s})\n", .{ test_fn.name, @errorName(err) });91 progress.log("FAIL ({s})\n", .{@errorName(err)});
92 if (!have_tty) std.debug.print("FAIL ({s})\n", .{@errorName(err)});92 if (!have_tty) std.debug.print("FAIL ({s})\n", .{@errorName(err)});
93 if (@errorReturnTrace()) |trace| {93 if (@errorReturnTrace()) |trace| {
94 std.debug.dumpStackTrace(trace.*);94 std.debug.dumpStackTrace(trace.*);