zig
zig copied to clipboard
Reduce usage of expectEqual in behavior tests
Does not fully close https://github.com/ziglang/zig/issues/24361 (this is only a first, simple change set)
Summary:
- where the type of the operands allows, change
expectEqual(x, y)toexpect(x == y)for files intest/behavior/**
# without these changes (master branch)
$ git grep expectEqual test/behavior | wc -l
1018
# with these changes
$ git grep expectEqual test/behavior | wc -l
314
My editor was configured to re-order imports on save. Hope that isn't a huge issue.
Verification evidence:
- the CI should pass, no new tests are introduced, only changes to existing tests
note: I had closed my previous PR to prevent unintended CI runs during rebase etc https://github.com/ziglang/zig/pull/24389
note to self: use this for testing if you have to rebase again:
zig build test-behavior -Dskip-release -Dskip-non-native -Dskip-llvm=true -Dskip-libc -Dskip-single-threaded --watch
Looks like windows CI failure is https://github.com/ziglang/zig/issues/22510 ?