[roll] Roll third_party/fuchsia-infra-bazel-rules [roll] Roll third_party/googletest Remove short-circuiting from AllOf, for better failure messages
For `EXPECT_THAT` matcher usage, showing only the first failure meant
that users would sometimes have to make a fix and run the test again
only to notice that there's another failure. It's better to show more
failures so that the user can fix several issues in one go.
In practice, very little code actually wants the short circuiting here,
only a handful of cases with custom matchers used like
`AllOf(BoundsCheck(), UncheckedAccess())`. These cases are fixable by
refactoring `UncheckedAccess()` to instead also apply a bounds check to
fail the matcher rather than crash. Notably, this change doesn't affect
`AnyOf`, so another workaround is to change `AllOf(m1, m2, ...)` into
`Not(AnyOf(Not(m1), Not(m2), ...))`.
PiperOrigin-RevId: 826316273
Original-Revision: 17d335d7c7f15d989516255471c3d7f5d204308d
Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia-infra-bazel-rules/+/1409004
Original-Revision: 1074a3f8df48c78e40ee8c99dc57e795ff878859
Roller-URL: https://cr-buildbucket.appspot.com/build/8699515001466045953
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: Iaca917a637a7b8376c397c608479131304aee45a
Reviewed-on: https://fuchsia-review.googlesource.com/c/sdk-samples/getting-started/+/1409143
Commit-Queue: GI Roller <global-integration-roller@fuchsia-infra.iam.gserviceaccount.com>
diff --git a/third_party/fuchsia-infra-bazel-rules b/third_party/fuchsia-infra-bazel-rules
index 77e703b..1074a3f 160000
--- a/third_party/fuchsia-infra-bazel-rules
+++ b/third_party/fuchsia-infra-bazel-rules
@@ -1 +1 @@
-Subproject commit 77e703b0a8dc1f69c26d81844741ad882b084aed
+Subproject commit 1074a3f8df48c78e40ee8c99dc57e795ff878859
