On Mon, 14 Apr 2025 13:25:20 GMT, Axel Boldt-Christmas <abold...@openjdk.org> wrote:
> `GTEST_SKIPPED` tests count is accounted in gtests run. But when we calculate > the number of GTEST errors we take `ERROR = RUN - PASSED - FAILED`. We need > parse and account for the skipped as well. Currently we only account for > `DISABLED` gtests (which are not counted as run tests). > > This patch adds support for both `DISABLED` and `GTEST_SKIPPED` tests. > > So with this patch we end up with: > `ERROR = RUN - PASSED - FAILED - GTEST_SKIPPED` > `SKIPPED = GTEST_SKIPPED + GTEST_DISABLED ` > > Verified that this works locally on MacOS and Linux. Currently running > through testing and GHA. > > _The old `match` expression was problematic with some awk, not sure if there > was some other reason it was used, rather than the style used elsewhere > `maybe set, if unset set=0`_ Marked as reviewed by erikj (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/24622#pullrequestreview-2764439775
