From bcb12fa0f651f7de3a10f4535ed856e52b1c3f62 Mon Sep 17 00:00:00 2001 From: shiqian Date: Fri, 1 Aug 2008 19:04:48 +0000 Subject: Fixes the definition of GTEST_ATTRIBUTE_UNUSED and make the tests pass in opt mode. --- test/gtest-death-test_test.cc | 29 ++--------------------------- test/gtest_repeat_test.cc | 4 ++-- 2 files changed, 4 insertions(+), 29 deletions(-) (limited to 'test') diff --git a/test/gtest-death-test_test.cc b/test/gtest-death-test_test.cc index 4e4ca1a2..4bfd9d63 100644 --- a/test/gtest-death-test_test.cc +++ b/test/gtest-death-test_test.cc @@ -467,31 +467,6 @@ TEST_F(TestForDeathTest, TestExpectDebugDeath) { #endif } -// Tests that EXPECT_DEBUG_DEATH works as expected, -// that is, in debug mode, it: -// 1. Asserts on death. -// 2. Has no side effect. -// -// And in opt mode, it: -// 1. Has side effects and returns the expected value (12). -TEST_F(TestForDeathTest, TestExpectDebugDeathM) { - int sideeffect = 0; - EXPECT_DEBUG_DEATH({ // NOLINT - // Tests that the return value is 12 in opt mode. - EXPECT_EQ(12, DieInDebugElse12(&sideeffect)); - // Tests that the side effect occurrs in opt mode. - EXPECT_EQ(12, sideeffect); - }, "death.*DieInDebugElse12") << "In ExpectDebugDeathM"; - -#ifdef NDEBUG - // Checks that the assignment occurs in opt mode (sideeffect). - EXPECT_EQ(12, sideeffect); -#else - // Checks that the assignment does not occur in dbg mode (no sideeffect). - EXPECT_EQ(0, sideeffect); -#endif -} - // Tests that ASSERT_DEBUG_DEATH works as expected // In debug mode: // 1. Asserts on debug death. @@ -499,7 +474,7 @@ TEST_F(TestForDeathTest, TestExpectDebugDeathM) { // // In opt mode: // 1. Has side effects and returns the expected value (12). -TEST_F(TestForDeathTest, TestAssertDebugDeathM) { +TEST_F(TestForDeathTest, TestAssertDebugDeath) { int sideeffect = 0; ASSERT_DEBUG_DEATH({ // NOLINT @@ -507,7 +482,7 @@ TEST_F(TestForDeathTest, TestAssertDebugDeathM) { EXPECT_EQ(12, DieInDebugElse12(&sideeffect)); // Tests that the side effect occurred in opt mode. EXPECT_EQ(12, sideeffect); - }, "death.*DieInDebugElse12") << "In AssertDebugDeathM"; + }, "death.*DieInDebugElse12"); #ifdef NDEBUG // Checks that the assignment occurs in opt mode (sideeffect). diff --git a/test/gtest_repeat_test.cc b/test/gtest_repeat_test.cc index fa52442f..e2f03812 100644 --- a/test/gtest_repeat_test.cc +++ b/test/gtest_repeat_test.cc @@ -58,6 +58,8 @@ using testing::GTEST_FLAG(repeat); namespace { +// We need this when we are testing Google Test itself and therefore +// cannot use Google Test assertions. #define GTEST_CHECK_INT_EQ_(expected, actual) \ do {\ const int expected_val = (expected);\ @@ -130,8 +132,6 @@ void ResetCounts() { // Checks that the count for each test is expected. void CheckCounts(int expected) { - // We cannot use Google Test assertions here since we are testing Google Test - // itself. GTEST_CHECK_INT_EQ_(expected, g_environment_set_up_count); GTEST_CHECK_INT_EQ_(expected, g_environment_tear_down_count); GTEST_CHECK_INT_EQ_(expected, g_should_fail_count); -- cgit v1.2.3