60

I heard there is a possibility to enable google-test TestCase classes friends to my classes, thus enabling tests to access my private/protected members.

How to accomplish that?

3
  • Not an answer to this specific question, but a (controversial, to say the least) workaround for the same problem: one can always #define private public before including the definition of the class being tested.
    – jacquev6
    Jul 21, 2021 at 11:49
  • 1
    @jacquev6 the #define private public solution may not always be applicable as you may link against the actual unit (i.e. built without this re-#definition) possibly leading to missing linker symbols. In this case the FRIEND_TEST solution can help. All in all I would also recommend restricting the use of either solution to legacy code that can't / shouldn't be refactored. New code shall be directly designed for testability. Jan 20, 2022 at 9:40
  • "Free your functions! by Klaus Iglberger" introduces a different way of thinking. It helped me a lot. Dec 29, 2023 at 15:20

4 Answers 4

62

Try this (straight from Google Test docs...):

FRIEND_TEST(TestCaseName, TestName);

For example:

// foo.h
#include <gtest/gtest_prod.h>

// Defines FRIEND_TEST.
class Foo {
  ...
 private:
  FRIEND_TEST(FooTest, BarReturnsZeroOnNull);
  int Bar(void* x);
};

// foo_test.cc
...
TEST(FooTest, BarReturnsZeroOnNull) {
  Foo foo;
  EXPECT_EQ(0, foo.Bar(NULL));
  // Uses Foo's private member Bar().
}
6
  • 1
    What about if I have another test for instance BarReturnsOneOnSth. Do I have to add another FRIEND_TEST declaration for that test too?
    – pajton
    Mar 7, 2010 at 13:41
  • 2
    Yes. Each test is technically a class, and you need to befriend them one at a time.
    – hobbit
    Mar 7, 2010 at 13:48
  • 35
    How can I do it in a way which does not force me to include googletest header files in the header file with class Foo?
    – quant_dev
    Dec 12, 2011 at 17:07
  • 1
    @quant_dev: Refer to Ralfizzle's answer for doing it without including it the gtest header. Oct 7, 2015 at 0:10
  • 3
    Note that test and to be tested class have to be in the same namespace, as explained in googletest gtest_prod.h
    – zeeMonkeez
    Oct 21, 2018 at 21:40
43

I know this is old but I was searching for the same answer today. "gtest_prod.h" just introduces a simple macro to reference test classes.

#define FRIEND_TEST(test_case_name, test_name)\
friend class test_case_name##_##test_name##_Test

So FRIEND_TEST(FooTest, BarReturnsZeroOnNull); is equivalent to:

friend class FooTest_BarReturnsZeroOnNull_Test;

This works because each test is its own class as mentioned in the previous answer.

0
10

A far better strategy is to not allow friend tests among your unit tests.

Allowing friend tests accessing private members will lead to a code base that is hard to maintain. Tests that break whenever a component's inner implementation details are refactored is not what you want. If extra effort is instead put into getting a design where components can be tested through their public interface, you will get tests that only need updating whenever the public interface of a component is updated.

Tests relying on gtest/gtest_prod.h should be seen as a sign of poor design.

6
  • 4
    I understand that this is controversial (hopefully it earned you some kind of "controversial answer" badge <grin>), but I am glad someone brought up this viewpoint. Many agree with @Martin on this! dzone.com/articles/principles-creating Jul 27, 2017 at 23:15
  • 5
    This is true in general, but for UI testing (thinking of QT) you'll often want to get at child widgets that wouldn't normally be exposed. For example I need to check that when a button is pressed a widget becomes visible. The widget normally would not be exposed publicly. Jun 12, 2018 at 18:05
  • 8
    There are tradeoffs along various axes in developing testing strategies. The advantage you point to is valid, but that doesn't mean using friend for testing is always a bad idea. Oct 19, 2020 at 21:58
  • 5
    I think this is very application dependent, in some cases I can see your point. If you, however, consider for example HPC/scientific computing applications this would not be a sign of poor design. In many cases, the interface to a solver is made very simple - .solve(), but relies on complicated mathematics and solving of governing equations. These equations will be broken into smaller private functions, which you would definitely want to unit test. Sep 2, 2021 at 10:24
  • @BevanJones, that sounds like bad design to me. Anyways, of course we often have a system that has both good and bad sides to it. And often it's not reasonable to suggest that it should be re-written from scratch to make it more testable or whatever. That's reality, but it's still a sign of poor design choices. Not necessarily yours.
    – Martin G
    Sep 2, 2021 at 10:45
10

When your tested class and your test class are in a different namespace (e.g. your tests are in the global namespace), you may need to forward-declare your test class and add your namespace prefix in FRIEND_TEST:

// foo.h
#include <gtest/gtest_prod.h>

// forward-declaration of test class
class FooTest_BarReturnsZeroOnNull_Test; 

// Defines FRIEND_TEST.
class my_namespace::Foo {
    ...
private:
    // Specify the global namespace (`::`) in the `FRIEND_TEST()` usage
    FRIEND_TEST(::FooTest, BarReturnsZeroOnNull);
    int Bar(void* x);
};

// forward-declaration of this namespace from foo_test.cc
using namespace my_namespace;

...
TEST(FooTest, BarReturnsZeroOnNull) {
    Foo foo;
    EXPECT_EQ(0, foo.Bar(NULL));
    // Uses Foo's private member Bar().
}

I know that friend unit tests (or the friendliness in C++ in general) and white-box testing are a controversial subject, but when you work on complex, scientific algorithms, each step of which you need to test and validate, but that you don't want to expose in public (or even protected) interfaces, friend tests appear to me as a simple and pragmatic solution, especially in a test-driven development approach. It is always possible to refactor the code later (or to completely remove white-box tests) if it's against one's religion to use the friendliness or white-box testing.

2
  • 2
    This answer points one important thing other answers missed and save my world: the namespace matters quite a lot when using friend class.
    – Kir Chou
    May 6, 2021 at 2:44
  • Forward declaration might not be enough. Jan 8 at 17:28

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.