• Home
  • Line#
  • Scopes#
  • Navigate#
  • Raw
  • Download
1# Googletest FAQ
2
3## Why should test suite names and test names not contain underscore?
4
5{: .callout .note}
6Note: Googletest reserves underscore (`_`) for special purpose keywords, such as
7[the `DISABLED_` prefix](advanced.md#temporarily-disabling-tests), in addition
8to the following rationale.
9
10Underscore (`_`) is special, as C++ reserves the following to be used by the
11compiler and the standard library:
12
131.  any identifier that starts with an `_` followed by an upper-case letter, and
142.  any identifier that contains two consecutive underscores (i.e. `__`)
15    *anywhere* in its name.
16
17User code is *prohibited* from using such identifiers.
18
19Now let's look at what this means for `TEST` and `TEST_F`.
20
21Currently `TEST(TestSuiteName, TestName)` generates a class named
22`TestSuiteName_TestName_Test`. What happens if `TestSuiteName` or `TestName`
23contains `_`?
24
251.  If `TestSuiteName` starts with an `_` followed by an upper-case letter (say,
26    `_Foo`), we end up with `_Foo_TestName_Test`, which is reserved and thus
27    invalid.
282.  If `TestSuiteName` ends with an `_` (say, `Foo_`), we get
29    `Foo__TestName_Test`, which is invalid.
303.  If `TestName` starts with an `_` (say, `_Bar`), we get
31    `TestSuiteName__Bar_Test`, which is invalid.
324.  If `TestName` ends with an `_` (say, `Bar_`), we get
33    `TestSuiteName_Bar__Test`, which is invalid.
34
35So clearly `TestSuiteName` and `TestName` cannot start or end with `_`
36(Actually, `TestSuiteName` can start with `_` -- as long as the `_` isn't
37followed by an upper-case letter. But that's getting complicated. So for
38simplicity we just say that it cannot start with `_`.).
39
40It may seem fine for `TestSuiteName` and `TestName` to contain `_` in the
41middle. However, consider this:
42
43```c++
44TEST(Time, Flies_Like_An_Arrow) { ... }
45TEST(Time_Flies, Like_An_Arrow) { ... }
46```
47
48Now, the two `TEST`s will both generate the same class
49(`Time_Flies_Like_An_Arrow_Test`). That's not good.
50
51So for simplicity, we just ask the users to avoid `_` in `TestSuiteName` and
52`TestName`. The rule is more constraining than necessary, but it's simple and
53easy to remember. It also gives googletest some wiggle room in case its
54implementation needs to change in the future.
55
56If you violate the rule, there may not be immediate consequences, but your test
57may (just may) break with a new compiler (or a new version of the compiler you
58are using) or with a new version of googletest. Therefore it's best to follow
59the rule.
60
61## Why does googletest support `EXPECT_EQ(NULL, ptr)` and `ASSERT_EQ(NULL, ptr)` but not `EXPECT_NE(NULL, ptr)` and `ASSERT_NE(NULL, ptr)`?
62
63First of all, you can use `nullptr` with each of these macros, e.g.
64`EXPECT_EQ(ptr, nullptr)`, `EXPECT_NE(ptr, nullptr)`, `ASSERT_EQ(ptr, nullptr)`,
65`ASSERT_NE(ptr, nullptr)`. This is the preferred syntax in the style guide
66because `nullptr` does not have the type problems that `NULL` does.
67
68Due to some peculiarity of C++, it requires some non-trivial template meta
69programming tricks to support using `NULL` as an argument of the `EXPECT_XX()`
70and `ASSERT_XX()` macros. Therefore we only do it where it's most needed
71(otherwise we make the implementation of googletest harder to maintain and more
72error-prone than necessary).
73
74Historically, the `EXPECT_EQ()` macro took the *expected* value as its first
75argument and the *actual* value as the second, though this argument order is now
76discouraged. It was reasonable that someone wanted
77to write `EXPECT_EQ(NULL, some_expression)`, and this indeed was requested
78several times. Therefore we implemented it.
79
80The need for `EXPECT_NE(NULL, ptr)` wasn't nearly as strong. When the assertion
81fails, you already know that `ptr` must be `NULL`, so it doesn't add any
82information to print `ptr` in this case. That means `EXPECT_TRUE(ptr != NULL)`
83works just as well.
84
85If we were to support `EXPECT_NE(NULL, ptr)`, for consistency we'd have to
86support `EXPECT_NE(ptr, NULL)` as well. This means using the template meta
87programming tricks twice in the implementation, making it even harder to
88understand and maintain. We believe the benefit doesn't justify the cost.
89
90Finally, with the growth of the gMock matcher library, we are encouraging people
91to use the unified `EXPECT_THAT(value, matcher)` syntax more often in tests. One
92significant advantage of the matcher approach is that matchers can be easily
93combined to form new matchers, while the `EXPECT_NE`, etc, macros cannot be
94easily combined. Therefore we want to invest more in the matchers than in the
95`EXPECT_XX()` macros.
96
97## I need to test that different implementations of an interface satisfy some common requirements. Should I use typed tests or value-parameterized tests?
98
99For testing various implementations of the same interface, either typed tests or
100value-parameterized tests can get it done. It's really up to you the user to
101decide which is more convenient for you, depending on your particular case. Some
102rough guidelines:
103
104*   Typed tests can be easier to write if instances of the different
105    implementations can be created the same way, modulo the type. For example,
106    if all these implementations have a public default constructor (such that
107    you can write `new TypeParam`), or if their factory functions have the same
108    form (e.g. `CreateInstance<TypeParam>()`).
109*   Value-parameterized tests can be easier to write if you need different code
110    patterns to create different implementations' instances, e.g. `new Foo` vs
111    `new Bar(5)`. To accommodate for the differences, you can write factory
112    function wrappers and pass these function pointers to the tests as their
113    parameters.
114*   When a typed test fails, the default output includes the name of the type,
115    which can help you quickly identify which implementation is wrong.
116    Value-parameterized tests only show the number of the failed iteration by
117    default. You will need to define a function that returns the iteration name
118    and pass it as the third parameter to INSTANTIATE_TEST_SUITE_P to have more
119    useful output.
120*   When using typed tests, you need to make sure you are testing against the
121    interface type, not the concrete types (in other words, you want to make
122    sure `implicit_cast<MyInterface*>(my_concrete_impl)` works, not just that
123    `my_concrete_impl` works). It's less likely to make mistakes in this area
124    when using value-parameterized tests.
125
126I hope I didn't confuse you more. :-) If you don't mind, I'd suggest you to give
127both approaches a try. Practice is a much better way to grasp the subtle
128differences between the two tools. Once you have some concrete experience, you
129can much more easily decide which one to use the next time.
130
131## I got some run-time errors about invalid proto descriptors when using `ProtocolMessageEquals`. Help!
132
133{: .callout .note}
134**Note:** `ProtocolMessageEquals` and `ProtocolMessageEquiv` are *deprecated*
135now. Please use `EqualsProto`, etc instead.
136
137`ProtocolMessageEquals` and `ProtocolMessageEquiv` were redefined recently and
138are now less tolerant of invalid protocol buffer definitions. In particular, if
139you have a `foo.proto` that doesn't fully qualify the type of a protocol message
140it references (e.g. `message<Bar>` where it should be `message<blah.Bar>`), you
141will now get run-time errors like:
142
143```
144... descriptor.cc:...] Invalid proto descriptor for file "path/to/foo.proto":
145... descriptor.cc:...]  blah.MyMessage.my_field: ".Bar" is not defined.
146```
147
148If you see this, your `.proto` file is broken and needs to be fixed by making
149the types fully qualified. The new definition of `ProtocolMessageEquals` and
150`ProtocolMessageEquiv` just happen to reveal your bug.
151
152## My death test modifies some state, but the change seems lost after the death test finishes. Why?
153
154Death tests (`EXPECT_DEATH`, etc) are executed in a sub-process s.t. the
155expected crash won't kill the test program (i.e. the parent process). As a
156result, any in-memory side effects they incur are observable in their respective
157sub-processes, but not in the parent process. You can think of them as running
158in a parallel universe, more or less.
159
160In particular, if you use mocking and the death test statement invokes some mock
161methods, the parent process will think the calls have never occurred. Therefore,
162you may want to move your `EXPECT_CALL` statements inside the `EXPECT_DEATH`
163macro.
164
165## EXPECT_EQ(htonl(blah), blah_blah) generates weird compiler errors in opt mode. Is this a googletest bug?
166
167Actually, the bug is in `htonl()`.
168
169According to `'man htonl'`, `htonl()` is a *function*, which means it's valid to
170use `htonl` as a function pointer. However, in opt mode `htonl()` is defined as
171a *macro*, which breaks this usage.
172
173Worse, the macro definition of `htonl()` uses a `gcc` extension and is *not*
174standard C++. That hacky implementation has some ad hoc limitations. In
175particular, it prevents you from writing `Foo<sizeof(htonl(x))>()`, where `Foo`
176is a template that has an integral argument.
177
178The implementation of `EXPECT_EQ(a, b)` uses `sizeof(... a ...)` inside a
179template argument, and thus doesn't compile in opt mode when `a` contains a call
180to `htonl()`. It is difficult to make `EXPECT_EQ` bypass the `htonl()` bug, as
181the solution must work with different compilers on various platforms.
182
183## The compiler complains about "undefined references" to some static const member variables, but I did define them in the class body. What's wrong?
184
185If your class has a static data member:
186
187```c++
188// foo.h
189class Foo {
190  ...
191  static const int kBar = 100;
192};
193```
194
195You also need to define it *outside* of the class body in `foo.cc`:
196
197```c++
198const int Foo::kBar;  // No initializer here.
199```
200
201Otherwise your code is **invalid C++**, and may break in unexpected ways. In
202particular, using it in googletest comparison assertions (`EXPECT_EQ`, etc) will
203generate an "undefined reference" linker error. The fact that "it used to work"
204doesn't mean it's valid. It just means that you were lucky. :-)
205
206If the declaration of the static data member is `constexpr` then it is
207implicitly an `inline` definition, and a separate definition in `foo.cc` is not
208needed:
209
210```c++
211// foo.h
212class Foo {
213  ...
214  static constexpr int kBar = 100;  // Defines kBar, no need to do it in foo.cc.
215};
216```
217
218## Can I derive a test fixture from another?
219
220Yes.
221
222Each test fixture has a corresponding and same named test suite. This means only
223one test suite can use a particular fixture. Sometimes, however, multiple test
224cases may want to use the same or slightly different fixtures. For example, you
225may want to make sure that all of a GUI library's test suites don't leak
226important system resources like fonts and brushes.
227
228In googletest, you share a fixture among test suites by putting the shared logic
229in a base test fixture, then deriving from that base a separate fixture for each
230test suite that wants to use this common logic. You then use `TEST_F()` to write
231tests using each derived fixture.
232
233Typically, your code looks like this:
234
235```c++
236// Defines a base test fixture.
237class BaseTest : public ::testing::Test {
238 protected:
239  ...
240};
241
242// Derives a fixture FooTest from BaseTest.
243class FooTest : public BaseTest {
244 protected:
245  void SetUp() override {
246    BaseTest::SetUp();  // Sets up the base fixture first.
247    ... additional set-up work ...
248  }
249
250  void TearDown() override {
251    ... clean-up work for FooTest ...
252    BaseTest::TearDown();  // Remember to tear down the base fixture
253                           // after cleaning up FooTest!
254  }
255
256  ... functions and variables for FooTest ...
257};
258
259// Tests that use the fixture FooTest.
260TEST_F(FooTest, Bar) { ... }
261TEST_F(FooTest, Baz) { ... }
262
263... additional fixtures derived from BaseTest ...
264```
265
266If necessary, you can continue to derive test fixtures from a derived fixture.
267googletest has no limit on how deep the hierarchy can be.
268
269For a complete example using derived test fixtures, see
270[sample5_unittest.cc](https://github.com/google/googletest/blob/master/googletest/samples/sample5_unittest.cc).
271
272## My compiler complains "void value not ignored as it ought to be." What does this mean?
273
274You're probably using an `ASSERT_*()` in a function that doesn't return `void`.
275`ASSERT_*()` can only be used in `void` functions, due to exceptions being
276disabled by our build system. Please see more details
277[here](advanced.md#assertion-placement).
278
279## My death test hangs (or seg-faults). How do I fix it?
280
281In googletest, death tests are run in a child process and the way they work is
282delicate. To write death tests you really need to understand how they work.
283Please make sure you have read [this](advanced.md#how-it-works).
284
285In particular, death tests don't like having multiple threads in the parent
286process. So the first thing you can try is to eliminate creating threads outside
287of `EXPECT_DEATH()`. For example, you may want to use mocks or fake objects
288instead of real ones in your tests.
289
290Sometimes this is impossible as some library you must use may be creating
291threads before `main()` is even reached. In this case, you can try to minimize
292the chance of conflicts by either moving as many activities as possible inside
293`EXPECT_DEATH()` (in the extreme case, you want to move everything inside), or
294leaving as few things as possible in it. Also, you can try to set the death test
295style to `"threadsafe"`, which is safer but slower, and see if it helps.
296
297If you go with thread-safe death tests, remember that they rerun the test
298program from the beginning in the child process. Therefore make sure your
299program can run side-by-side with itself and is deterministic.
300
301In the end, this boils down to good concurrent programming. You have to make
302sure that there are no race conditions or deadlocks in your program. No silver
303bullet - sorry!
304
305## Should I use the constructor/destructor of the test fixture or SetUp()/TearDown()? {#CtorVsSetUp}
306
307The first thing to remember is that googletest does **not** reuse the same test
308fixture object across multiple tests. For each `TEST_F`, googletest will create
309a **fresh** test fixture object, immediately call `SetUp()`, run the test body,
310call `TearDown()`, and then delete the test fixture object.
311
312When you need to write per-test set-up and tear-down logic, you have the choice
313between using the test fixture constructor/destructor or `SetUp()/TearDown()`.
314The former is usually preferred, as it has the following benefits:
315
316*   By initializing a member variable in the constructor, we have the option to
317    make it `const`, which helps prevent accidental changes to its value and
318    makes the tests more obviously correct.
319*   In case we need to subclass the test fixture class, the subclass'
320    constructor is guaranteed to call the base class' constructor *first*, and
321    the subclass' destructor is guaranteed to call the base class' destructor
322    *afterward*. With `SetUp()/TearDown()`, a subclass may make the mistake of
323    forgetting to call the base class' `SetUp()/TearDown()` or call them at the
324    wrong time.
325
326You may still want to use `SetUp()/TearDown()` in the following cases:
327
328*   C++ does not allow virtual function calls in constructors and destructors.
329    You can call a method declared as virtual, but it will not use dynamic
330    dispatch, it will use the definition from the class the constructor of which
331    is currently executing. This is because calling a virtual method before the
332    derived class constructor has a chance to run is very dangerous - the
333    virtual method might operate on uninitialized data. Therefore, if you need
334    to call a method that will be overridden in a derived class, you have to use
335    `SetUp()/TearDown()`.
336*   In the body of a constructor (or destructor), it's not possible to use the
337    `ASSERT_xx` macros. Therefore, if the set-up operation could cause a fatal
338    test failure that should prevent the test from running, it's necessary to
339    use `abort` and abort the whole test
340    executable, or to use `SetUp()` instead of a constructor.
341*   If the tear-down operation could throw an exception, you must use
342    `TearDown()` as opposed to the destructor, as throwing in a destructor leads
343    to undefined behavior and usually will kill your program right away. Note
344    that many standard libraries (like STL) may throw when exceptions are
345    enabled in the compiler. Therefore you should prefer `TearDown()` if you
346    want to write portable tests that work with or without exceptions.
347*   The googletest team is considering making the assertion macros throw on
348    platforms where exceptions are enabled (e.g. Windows, Mac OS, and Linux
349    client-side), which will eliminate the need for the user to propagate
350    failures from a subroutine to its caller. Therefore, you shouldn't use
351    googletest assertions in a destructor if your code could run on such a
352    platform.
353
354## The compiler complains "no matching function to call" when I use ASSERT_PRED*. How do I fix it?
355
356If the predicate function you use in `ASSERT_PRED*` or `EXPECT_PRED*` is
357overloaded or a template, the compiler will have trouble figuring out which
358overloaded version it should use. `ASSERT_PRED_FORMAT*` and
359`EXPECT_PRED_FORMAT*` don't have this problem.
360
361If you see this error, you might want to switch to
362`(ASSERT|EXPECT)_PRED_FORMAT*`, which will also give you a better failure
363message. If, however, that is not an option, you can resolve the problem by
364explicitly telling the compiler which version to pick.
365
366For example, suppose you have
367
368```c++
369bool IsPositive(int n) {
370  return n > 0;
371}
372
373bool IsPositive(double x) {
374  return x > 0;
375}
376```
377
378you will get a compiler error if you write
379
380```c++
381EXPECT_PRED1(IsPositive, 5);
382```
383
384However, this will work:
385
386```c++
387EXPECT_PRED1(static_cast<bool (*)(int)>(IsPositive), 5);
388```
389
390(The stuff inside the angled brackets for the `static_cast` operator is the type
391of the function pointer for the `int`-version of `IsPositive()`.)
392
393As another example, when you have a template function
394
395```c++
396template <typename T>
397bool IsNegative(T x) {
398  return x < 0;
399}
400```
401
402you can use it in a predicate assertion like this:
403
404```c++
405ASSERT_PRED1(IsNegative<int>, -5);
406```
407
408Things are more interesting if your template has more than one parameter. The
409following won't compile:
410
411```c++
412ASSERT_PRED2(GreaterThan<int, int>, 5, 0);
413```
414
415as the C++ pre-processor thinks you are giving `ASSERT_PRED2` 4 arguments, which
416is one more than expected. The workaround is to wrap the predicate function in
417parentheses:
418
419```c++
420ASSERT_PRED2((GreaterThan<int, int>), 5, 0);
421```
422
423## My compiler complains about "ignoring return value" when I call RUN_ALL_TESTS(). Why?
424
425Some people had been ignoring the return value of `RUN_ALL_TESTS()`. That is,
426instead of
427
428```c++
429  return RUN_ALL_TESTS();
430```
431
432they write
433
434```c++
435  RUN_ALL_TESTS();
436```
437
438This is **wrong and dangerous**. The testing services needs to see the return
439value of `RUN_ALL_TESTS()` in order to determine if a test has passed. If your
440`main()` function ignores it, your test will be considered successful even if it
441has a googletest assertion failure. Very bad.
442
443We have decided to fix this (thanks to Michael Chastain for the idea). Now, your
444code will no longer be able to ignore `RUN_ALL_TESTS()` when compiled with
445`gcc`. If you do so, you'll get a compiler error.
446
447If you see the compiler complaining about you ignoring the return value of
448`RUN_ALL_TESTS()`, the fix is simple: just make sure its value is used as the
449return value of `main()`.
450
451But how could we introduce a change that breaks existing tests? Well, in this
452case, the code was already broken in the first place, so we didn't break it. :-)
453
454## My compiler complains that a constructor (or destructor) cannot return a value. What's going on?
455
456Due to a peculiarity of C++, in order to support the syntax for streaming
457messages to an `ASSERT_*`, e.g.
458
459```c++
460  ASSERT_EQ(1, Foo()) << "blah blah" << foo;
461```
462
463we had to give up using `ASSERT*` and `FAIL*` (but not `EXPECT*` and
464`ADD_FAILURE*`) in constructors and destructors. The workaround is to move the
465content of your constructor/destructor to a private void member function, or
466switch to `EXPECT_*()` if that works. This
467[section](advanced.md#assertion-placement) in the user's guide explains it.
468
469## My SetUp() function is not called. Why?
470
471C++ is case-sensitive. Did you spell it as `Setup()`?
472
473Similarly, sometimes people spell `SetUpTestSuite()` as `SetupTestSuite()` and
474wonder why it's never called.
475
476
477## I have several test suites which share the same test fixture logic, do I have to define a new test fixture class for each of them? This seems pretty tedious.
478
479You don't have to. Instead of
480
481```c++
482class FooTest : public BaseTest {};
483
484TEST_F(FooTest, Abc) { ... }
485TEST_F(FooTest, Def) { ... }
486
487class BarTest : public BaseTest {};
488
489TEST_F(BarTest, Abc) { ... }
490TEST_F(BarTest, Def) { ... }
491```
492
493you can simply `typedef` the test fixtures:
494
495```c++
496typedef BaseTest FooTest;
497
498TEST_F(FooTest, Abc) { ... }
499TEST_F(FooTest, Def) { ... }
500
501typedef BaseTest BarTest;
502
503TEST_F(BarTest, Abc) { ... }
504TEST_F(BarTest, Def) { ... }
505```
506
507## googletest output is buried in a whole bunch of LOG messages. What do I do?
508
509The googletest output is meant to be a concise and human-friendly report. If
510your test generates textual output itself, it will mix with the googletest
511output, making it hard to read. However, there is an easy solution to this
512problem.
513
514Since `LOG` messages go to stderr, we decided to let googletest output go to
515stdout. This way, you can easily separate the two using redirection. For
516example:
517
518```shell
519$ ./my_test > gtest_output.txt
520```
521
522## Why should I prefer test fixtures over global variables?
523
524There are several good reasons:
525
5261.  It's likely your test needs to change the states of its global variables.
527    This makes it difficult to keep side effects from escaping one test and
528    contaminating others, making debugging difficult. By using fixtures, each
529    test has a fresh set of variables that's different (but with the same
530    names). Thus, tests are kept independent of each other.
5312.  Global variables pollute the global namespace.
5323.  Test fixtures can be reused via subclassing, which cannot be done easily
533    with global variables. This is useful if many test suites have something in
534    common.
535
536## What can the statement argument in ASSERT_DEATH() be?
537
538`ASSERT_DEATH(statement, matcher)` (or any death assertion macro) can be used
539wherever *`statement`* is valid. So basically *`statement`* can be any C++
540statement that makes sense in the current context. In particular, it can
541reference global and/or local variables, and can be:
542
543*   a simple function call (often the case),
544*   a complex expression, or
545*   a compound statement.
546
547Some examples are shown here:
548
549```c++
550// A death test can be a simple function call.
551TEST(MyDeathTest, FunctionCall) {
552  ASSERT_DEATH(Xyz(5), "Xyz failed");
553}
554
555// Or a complex expression that references variables and functions.
556TEST(MyDeathTest, ComplexExpression) {
557  const bool c = Condition();
558  ASSERT_DEATH((c ? Func1(0) : object2.Method("test")),
559               "(Func1|Method) failed");
560}
561
562// Death assertions can be used anywhere in a function.  In
563// particular, they can be inside a loop.
564TEST(MyDeathTest, InsideLoop) {
565  // Verifies that Foo(0), Foo(1), ..., and Foo(4) all die.
566  for (int i = 0; i < 5; i++) {
567    EXPECT_DEATH_M(Foo(i), "Foo has \\d+ errors",
568                   ::testing::Message() << "where i is " << i);
569  }
570}
571
572// A death assertion can contain a compound statement.
573TEST(MyDeathTest, CompoundStatement) {
574  // Verifies that at lease one of Bar(0), Bar(1), ..., and
575  // Bar(4) dies.
576  ASSERT_DEATH({
577    for (int i = 0; i < 5; i++) {
578      Bar(i);
579    }
580  },
581  "Bar has \\d+ errors");
582}
583```
584
585## I have a fixture class `FooTest`, but `TEST_F(FooTest, Bar)` gives me error ``"no matching function for call to `FooTest::FooTest()'"``. Why?
586
587Googletest needs to be able to create objects of your test fixture class, so it
588must have a default constructor. Normally the compiler will define one for you.
589However, there are cases where you have to define your own:
590
591*   If you explicitly declare a non-default constructor for class `FooTest`
592    (`DISALLOW_EVIL_CONSTRUCTORS()` does this), then you need to define a
593    default constructor, even if it would be empty.
594*   If `FooTest` has a const non-static data member, then you have to define the
595    default constructor *and* initialize the const member in the initializer
596    list of the constructor. (Early versions of `gcc` doesn't force you to
597    initialize the const member. It's a bug that has been fixed in `gcc 4`.)
598
599## Why does ASSERT_DEATH complain about previous threads that were already joined?
600
601With the Linux pthread library, there is no turning back once you cross the line
602from a single thread to multiple threads. The first time you create a thread, a
603manager thread is created in addition, so you get 3, not 2, threads. Later when
604the thread you create joins the main thread, the thread count decrements by 1,
605but the manager thread will never be killed, so you still have 2 threads, which
606means you cannot safely run a death test.
607
608The new NPTL thread library doesn't suffer from this problem, as it doesn't
609create a manager thread. However, if you don't control which machine your test
610runs on, you shouldn't depend on this.
611
612## Why does googletest require the entire test suite, instead of individual tests, to be named *DeathTest when it uses ASSERT_DEATH?
613
614googletest does not interleave tests from different test suites. That is, it
615runs all tests in one test suite first, and then runs all tests in the next test
616suite, and so on. googletest does this because it needs to set up a test suite
617before the first test in it is run, and tear it down afterwards. Splitting up
618the test case would require multiple set-up and tear-down processes, which is
619inefficient and makes the semantics unclean.
620
621If we were to determine the order of tests based on test name instead of test
622case name, then we would have a problem with the following situation:
623
624```c++
625TEST_F(FooTest, AbcDeathTest) { ... }
626TEST_F(FooTest, Uvw) { ... }
627
628TEST_F(BarTest, DefDeathTest) { ... }
629TEST_F(BarTest, Xyz) { ... }
630```
631
632Since `FooTest.AbcDeathTest` needs to run before `BarTest.Xyz`, and we don't
633interleave tests from different test suites, we need to run all tests in the
634`FooTest` case before running any test in the `BarTest` case. This contradicts
635with the requirement to run `BarTest.DefDeathTest` before `FooTest.Uvw`.
636
637## But I don't like calling my entire test suite \*DeathTest when it contains both death tests and non-death tests. What do I do?
638
639You don't have to, but if you like, you may split up the test suite into
640`FooTest` and `FooDeathTest`, where the names make it clear that they are
641related:
642
643```c++
644class FooTest : public ::testing::Test { ... };
645
646TEST_F(FooTest, Abc) { ... }
647TEST_F(FooTest, Def) { ... }
648
649using FooDeathTest = FooTest;
650
651TEST_F(FooDeathTest, Uvw) { ... EXPECT_DEATH(...) ... }
652TEST_F(FooDeathTest, Xyz) { ... ASSERT_DEATH(...) ... }
653```
654
655## googletest prints the LOG messages in a death test's child process only when the test fails. How can I see the LOG messages when the death test succeeds?
656
657Printing the LOG messages generated by the statement inside `EXPECT_DEATH()`
658makes it harder to search for real problems in the parent's log. Therefore,
659googletest only prints them when the death test has failed.
660
661If you really need to see such LOG messages, a workaround is to temporarily
662break the death test (e.g. by changing the regex pattern it is expected to
663match). Admittedly, this is a hack. We'll consider a more permanent solution
664after the fork-and-exec-style death tests are implemented.
665
666## The compiler complains about `no match for 'operator<<'` when I use an assertion. What gives?
667
668If you use a user-defined type `FooType` in an assertion, you must make sure
669there is an `std::ostream& operator<<(std::ostream&, const FooType&)` function
670defined such that we can print a value of `FooType`.
671
672In addition, if `FooType` is declared in a name space, the `<<` operator also
673needs to be defined in the *same* name space. See
674[Tip of the Week #49](http://abseil.io/tips/49) for details.
675
676## How do I suppress the memory leak messages on Windows?
677
678Since the statically initialized googletest singleton requires allocations on
679the heap, the Visual C++ memory leak detector will report memory leaks at the
680end of the program run. The easiest way to avoid this is to use the
681`_CrtMemCheckpoint` and `_CrtMemDumpAllObjectsSince` calls to not report any
682statically initialized heap objects. See MSDN for more details and additional
683heap check/debug routines.
684
685## How can my code detect if it is running in a test?
686
687If you write code that sniffs whether it's running in a test and does different
688things accordingly, you are leaking test-only logic into production code and
689there is no easy way to ensure that the test-only code paths aren't run by
690mistake in production. Such cleverness also leads to
691[Heisenbugs](https://en.wikipedia.org/wiki/Heisenbug). Therefore we strongly
692advise against the practice, and googletest doesn't provide a way to do it.
693
694In general, the recommended way to cause the code to behave differently under
695test is [Dependency Injection](http://en.wikipedia.org/wiki/Dependency_injection). You can inject
696different functionality from the test and from the production code. Since your
697production code doesn't link in the for-test logic at all (the
698[`testonly`](http://docs.bazel.build/versions/master/be/common-definitions.html#common.testonly) attribute for BUILD targets helps to ensure
699that), there is no danger in accidentally running it.
700
701However, if you *really*, *really*, *really* have no choice, and if you follow
702the rule of ending your test program names with `_test`, you can use the
703*horrible* hack of sniffing your executable name (`argv[0]` in `main()`) to know
704whether the code is under test.
705
706## How do I temporarily disable a test?
707
708If you have a broken test that you cannot fix right away, you can add the
709`DISABLED_` prefix to its name. This will exclude it from execution. This is
710better than commenting out the code or using `#if 0`, as disabled tests are
711still compiled (and thus won't rot).
712
713To include disabled tests in test execution, just invoke the test program with
714the `--gtest_also_run_disabled_tests` flag.
715
716## Is it OK if I have two separate `TEST(Foo, Bar)` test methods defined in different namespaces?
717
718Yes.
719
720The rule is **all test methods in the same test suite must use the same fixture
721class.** This means that the following is **allowed** because both tests use the
722same fixture class (`::testing::Test`).
723
724```c++
725namespace foo {
726TEST(CoolTest, DoSomething) {
727  SUCCEED();
728}
729}  // namespace foo
730
731namespace bar {
732TEST(CoolTest, DoSomething) {
733  SUCCEED();
734}
735}  // namespace bar
736```
737
738However, the following code is **not allowed** and will produce a runtime error
739from googletest because the test methods are using different test fixture
740classes with the same test suite name.
741
742```c++
743namespace foo {
744class CoolTest : public ::testing::Test {};  // Fixture foo::CoolTest
745TEST_F(CoolTest, DoSomething) {
746  SUCCEED();
747}
748}  // namespace foo
749
750namespace bar {
751class CoolTest : public ::testing::Test {};  // Fixture: bar::CoolTest
752TEST_F(CoolTest, DoSomething) {
753  SUCCEED();
754}
755}  // namespace bar
756```
757