Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(221)

Side by Side Diff: base/test/gtest_util.h

Issue 2288473002: Implement Dump-on-DCHECK (via a new LogSeverity). (Closed)
Patch Set: Pull the Official Windows build default in, and rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « base/syslog_logging.cc ('k') | build/config/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BASE_TEST_GTEST_UTIL_H_ 5 #ifndef BASE_TEST_GTEST_UTIL_H_
6 #define BASE_TEST_GTEST_UTIL_H_ 6 #define BASE_TEST_GTEST_UTIL_H_
7 7
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
11 11
12 #include "base/compiler_specific.h" 12 #include "base/compiler_specific.h"
13 #include "base/logging.h" 13 #include "base/logging.h"
14 #include "build/build_config.h" 14 #include "build/build_config.h"
15 #include "testing/gtest/include/gtest/gtest.h" 15 #include "testing/gtest/include/gtest/gtest.h"
16 16
17 // EXPECT/ASSERT_DCHECK_DEATH is intended to replace EXPECT/ASSERT_DEBUG_DEATH 17 // EXPECT/ASSERT_DCHECK_DEATH is intended to replace EXPECT/ASSERT_DEBUG_DEATH
18 // when the death is expected to be caused by a DCHECK. Contrary to 18 // when the death is expected to be caused by a DCHECK. Contrary to
19 // EXPECT/ASSERT_DEBUG_DEATH however, it doesn't execute the statement in non- 19 // EXPECT/ASSERT_DEBUG_DEATH however, it doesn't execute the statement in non-
20 // dcheck builds as DCHECKs are intended to catch things that should never 20 // dcheck builds as DCHECKs are intended to catch things that should never
21 // happen and as such executing the statement results in undefined behavior 21 // happen and as such executing the statement results in undefined behavior
22 // (|statement| is compiled in unsupported configurations nonetheless). 22 // (|statement| is compiled in unsupported configurations nonetheless).
23 // Death tests misbehave on Android. 23 // Death tests misbehave on Android.
24 #if DCHECK_IS_ON() && defined(GTEST_HAS_DEATH_TEST) && !defined(OS_ANDROID) 24 #if DCHECK_IS_ON() && defined(GTEST_HAS_DEATH_TEST) && !defined(OS_ANDROID) && \
25 !defined(DCHECK_IS_DUMP_WITHOUT_CRASH)
25 26
26 // EXPECT/ASSERT_DCHECK_DEATH tests verify that a DCHECK is hit ("Check failed" 27 // EXPECT/ASSERT_DCHECK_DEATH tests verify that a DCHECK is hit ("Check failed"
27 // is part of the error message), but intentionally do not expose the gtest 28 // is part of the error message), but intentionally do not expose the gtest
28 // death test's full |regex| parameter to avoid users having to verify the exact 29 // death test's full |regex| parameter to avoid users having to verify the exact
29 // syntax of the error message produced by the DCHECK. 30 // syntax of the error message produced by the DCHECK.
30 #define EXPECT_DCHECK_DEATH(statement) EXPECT_DEATH(statement, "Check failed") 31 #define EXPECT_DCHECK_DEATH(statement) EXPECT_DEATH(statement, "Check failed")
31 #define ASSERT_DCHECK_DEATH(statement) ASSERT_DEATH(statement, "Check failed") 32 #define ASSERT_DCHECK_DEATH(statement) ASSERT_DEATH(statement, "Check failed")
32 33
33 #else 34 #else
34 // DCHECK_IS_ON() && defined(GTEST_HAS_DEATH_TEST) && !defined(OS_ANDROID) 35 // DCHECK_IS_ON() && defined(GTEST_HAS_DEATH_TEST) && !defined(OS_ANDROID)
35 36
36 // Macro copied from gtest-death-test-internal.h as it's (1) internal for now 37 // Macro copied from gtest-death-test-internal.h as it's (1) internal for now
37 // and (2) only defined if !GTEST_HAS_DEATH_TEST which is only a subset of the 38 // and (2) only defined if !GTEST_HAS_DEATH_TEST which is only a subset of the
38 // conditions in which it's needed here. 39 // conditions in which it's needed here.
39 // TODO(gab): Expose macro in upstream gtest repo for consumers like us that 40 // TODO(gab): Expose macro in upstream gtest repo for consumers like us that
40 // want more specific death tests and remove this hack. 41 // want more specific death tests and remove this hack.
41 # define GTEST_UNSUPPORTED_DEATH_TEST(statement, regex, terminator) \ 42 #define GTEST_UNSUPPORTED_DEATH_TEST(statement, regex, terminator) \
42 GTEST_AMBIGUOUS_ELSE_BLOCKER_ \ 43 GTEST_AMBIGUOUS_ELSE_BLOCKER_ \
43 if (::testing::internal::AlwaysTrue()) { \ 44 if (::testing::internal::AlwaysTrue()) { \
44 GTEST_LOG_(WARNING) \ 45 GTEST_LOG_(WARNING) \
45 << "Death tests are not supported on this platform.\n" \ 46 << "Death tests are not supported in this configuration.\n" \
46 << "Statement '" #statement "' cannot be verified."; \ 47 << "Statement '" #statement "' cannot be verified."; \
47 } else if (::testing::internal::AlwaysFalse()) { \ 48 } else if (::testing::internal::AlwaysFalse()) { \
48 ::testing::internal::RE::PartialMatch(".*", (regex)); \ 49 ::testing::internal::RE::PartialMatch(".*", (regex)); \
49 GTEST_SUPPRESS_UNREACHABLE_CODE_WARNING_BELOW_(statement); \ 50 GTEST_SUPPRESS_UNREACHABLE_CODE_WARNING_BELOW_(statement); \
50 terminator; \ 51 terminator; \
51 } else \ 52 } else \
52 ::testing::Message() 53 ::testing::Message()
53 54
54 #define EXPECT_DCHECK_DEATH(statement) \ 55 #define EXPECT_DCHECK_DEATH(statement) \
55 GTEST_UNSUPPORTED_DEATH_TEST(statement, "Check failed", ) 56 GTEST_UNSUPPORTED_DEATH_TEST(statement, "Check failed", )
56 #define ASSERT_DCHECK_DEATH(statement) \ 57 #define ASSERT_DCHECK_DEATH(statement) \
57 GTEST_UNSUPPORTED_DEATH_TEST(statement, "Check failed", return) 58 GTEST_UNSUPPORTED_DEATH_TEST(statement, "Check failed", return)
58 59
59 #endif 60 #endif
60 // DCHECK_IS_ON() && defined(GTEST_HAS_DEATH_TEST) && !defined(OS_ANDROID) 61 // DCHECK_IS_ON() && defined(GTEST_HAS_DEATH_TEST) && !defined(OS_ANDROID)
61 62
62 namespace base { 63 namespace base {
(...skipping 30 matching lines...) Expand all
93 94
94 // Reads the list of gtest-based tests from |path| into |output|. 95 // Reads the list of gtest-based tests from |path| into |output|.
95 // Returns true on success. 96 // Returns true on success.
96 bool ReadTestNamesFromFile( 97 bool ReadTestNamesFromFile(
97 const FilePath& path, 98 const FilePath& path,
98 std::vector<TestIdentifier>* output) WARN_UNUSED_RESULT; 99 std::vector<TestIdentifier>* output) WARN_UNUSED_RESULT;
99 100
100 } // namespace base 101 } // namespace base
101 102
102 #endif // BASE_TEST_GTEST_UTIL_H_ 103 #endif // BASE_TEST_GTEST_UTIL_H_
OLDNEW
« no previous file with comments | « base/syslog_logging.cc ('k') | build/config/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698