tree 8ed736ef426d0af87c7393fdcbce1cd380a1ce50
parent 5f74dfcc6ccd21d19e67baa5586a4fe2df063333
author Austin Foxley <afoxley@pigweed.infra.roller.google.com> 1747368983 -0700
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1747368983 -0700

roll: pigweed pw_unit_test: If SetUp fails skip test body

https://google.github.io/googletest/faq.html#CtorVsSetUp says this:

In the body of a constructor (or destructor), it’s not possible to use
the ASSERT_xx macros. Therefore, if the set-up operation could cause a
fatal test failure that should prevent the test from running, it’s
necessary to use abort and abort the whole test executable, or to use
SetUp() instead of a constructor.

This implies that we should check for *_ASSERT failure after SetUp runs
and stop running further tests. Due to how this is currently
implemented, we will also skip further tests if an *_EXPECT fails since
we don't differentiate between the two failure types in the framework
state.

Original-Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/292043
Tests-Not-Needed: Austin Foxley <afoxley@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Original-Revision: e0318cbf853bd2838ef0be854ae747593c445093

Rolled-Repo: https://pigweed.googlesource.com/pigweed/pigweed
Rolled-Commits: a1e03da2c0a45d..e0318cbf853bd2
Roll-Count: 1
Roller-URL: https://cr-buildbucket.appspot.com/build/8714745315423763153
GitWatcher: ignore
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: If0668caa6469cf7753cb7363cbe6cf2f1df9048d
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/showcase/sense/+/292044
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>