From 5ed21863955149a5a877a53d7d5045b6919090ed Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20Bostr=C3=B6m?= Date: Sun, 27 Oct 2024 23:56:39 -0700 Subject: [PATCH] Use FAIL() in GTEST_SKIP() documentation This replaces EXPECT_EQ(0, 1) which would fail if hit. PiperOrigin-RevId: 690491467 Change-Id: Ieff4e616348254f29200e0ba1d9a6900a2eea130 --- docs/advanced.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/advanced.md b/docs/advanced.md index 240588a83b..7e998cc2cb 100644 --- a/docs/advanced.md +++ b/docs/advanced.md @@ -286,7 +286,7 @@ For example: ```c++ TEST(SkipTest, DoesSkip) { GTEST_SKIP() << "Skipping single test"; - EXPECT_EQ(0, 1); // Won't fail; it won't be executed + FAIL(); // Won't fail; it won't be executed } class SkipFixture : public ::testing::Test { @@ -298,7 +298,7 @@ class SkipFixture : public ::testing::Test { // Tests for SkipFixture won't be executed. TEST_F(SkipFixture, SkipsOneTest) { - EXPECT_EQ(5, 7); // Won't fail + FAIL(); // Won't fail; it won't be executed } ```