Skip to content

Commit d526b0f

Browse files
LemonBoyandrewrk
authored andcommitted
std: Disable _only_ the flaky part of the ResetEvent test
The remaining part is working just fine as it's not time-dependent.
1 parent 8eaaa90 commit d526b0f

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

lib/std/reset_event.zig

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -363,12 +363,6 @@ const AtomicEvent = struct {
363363
};
364364

365365
test "ResetEvent" {
366-
if (true) {
367-
// I have now observed this fail on macOS, Windows, and Linux.
368-
// https://github.com/ziglang/zig/issues/7009
369-
return error.SkipZigTest;
370-
}
371-
372366
var event = ResetEvent.init();
373367
defer event.deinit();
374368

@@ -462,9 +456,13 @@ test "ResetEvent" {
462456
defer receiver.wait();
463457
context.sender();
464458

465-
var timed = Context.init();
466-
defer timed.deinit();
467-
const sleeper = try std.Thread.spawn(&timed, Context.sleeper);
468-
defer sleeper.wait();
469-
try timed.timedWaiter();
459+
if (false) {
460+
// I have now observed this fail on macOS, Windows, and Linux.
461+
// https://github.com/ziglang/zig/issues/7009
462+
var timed = Context.init();
463+
defer timed.deinit();
464+
const sleeper = try std.Thread.spawn(&timed, Context.sleeper);
465+
defer sleeper.wait();
466+
try timed.timedWaiter();
467+
}
470468
}

0 commit comments

Comments
 (0)