Skip to content

Commit 9ca3c89

Browse files
Sebastian KellerVexu
authored andcommitted
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 parent f506810 commit 9ca3c89

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

lib/std/special/test_runner.zig

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ pub fn main() void {
7070
.blocking => {
7171
skip_count += 1;
7272
test_node.end();
73-
progress.log("{s}... SKIP (async test)\n", .{test_fn.name});
73+
progress.log("SKIP (async test)\n", .{});
7474
if (!have_tty) std.debug.print("SKIP (async test)\n", .{});
7575
continue;
7676
},
@@ -82,13 +82,13 @@ pub fn main() void {
8282
} else |err| switch (err) {
8383
error.SkipZigTest => {
8484
skip_count += 1;
85-
progress.log("{s}... SKIP\n", .{test_fn.name});
85+
progress.log("SKIP\n", .{});
8686
if (!have_tty) std.debug.print("SKIP\n", .{});
8787
test_node.end();
8888
},
8989
else => {
9090
fail_count += 1;
91-
progress.log("{s}... FAIL ({s})\n", .{ test_fn.name, @errorName(err) });
91+
progress.log("FAIL ({s})\n", .{@errorName(err)});
9292
if (!have_tty) std.debug.print("FAIL ({s})\n", .{@errorName(err)});
9393
if (@errorReturnTrace()) |trace| {
9494
std.debug.dumpStackTrace(trace.*);

0 commit comments

Comments
 (0)