26

I am new to gmock, so I want to know how can I stub simple C function called in a function under test for Unit Testing.

Example:

int func(int a)
{
  boolean find;
  // Some code
  find = func_1();
  return find;
}

I have searched about gmock and in my understanding gmock does not provide functionality to stub simple C functions, therefore I want to ask does gmock provides functionality to mock or stub func_1?

If not how can I stub func_1 manually in my test code without changing source code? I am using google test framework for unit testing.

Thanks.

273K
  • 29,503
  • 10
  • 41
  • 64
SRehman
  • 389
  • 1
  • 3
  • 7

7 Answers7

29

I found myself in the same situation lately. I had to write unit tests for libraries written in C, which in turn had dependencies to other libraries also written in C. So I wanted to mock all function calls of dependencies using gmock. Let me explain my approach by an example.

Assume the code to be tested (library A) calls a function from another library, lib_x_function():

lib_a_function()
{
   ...
   retval = lib_x_function();
   ...
}

So, I want to mock the library X. Therefore I write an interface class and a mock class in a file lib_x_mock.h:

class LibXInterface {
public:
   virtual ~LibXInterface() {}
   virtual int lib_x_function() = 0;
}

class LibXMock : public LibXInterface {
public:
   virtual ~LibXMock() {}
   MOCK_METHOD0(lib_x_function, int());
}

Additionally I create a source file (say, lib_x_mock.cc), that defines a stub for the actual C function. This shall call the mock method. Note the extern reference to the mock object.

#include lib_x.h
#include lib_x_mock.h
extern LibXMock LibXMockObj;    /* This is just a declaration! The actual
                                   mock obj must be defined globally in your
                                   test file. */

int lib_x_function()
{
    return LibXMockObj.lib_x_function();
}

Now, in the test file, which tests the library A, I must define the mock object globally, so that it is both reachable within your tests and from lib_x_mock.cc. This is lib_a_tests.cc:

#include lib_x_mock.h

LibXMock LibXMockObj;  /* This is now the actual definition of the mock obj */

...
TEST_F(foo, bar)
{
   EXPECT_CALL(LibXMockObj, lib_x_function());
   ...
}

This approach works perfectly for me, and I have dozens of tests and several mocked libraries. However, I have a few doubts if it is ok to create a global mock object - I asked this in a separate question and still wait for answers. Besides this I'm happy with the solution.


UPDATE: The problem about the global object can be easily remedied by creating the object e.g. in the constructor of the test fixture, and just storing a pointer to that object in a global variable.

However, also note my alternative answer to this question, that I just posted.

Georg P.
  • 2,785
  • 2
  • 27
  • 53
  • I followed your procedure. Only difference is that there is no parameter in your given function. In contrast, two parameters in my function (one is struct and another one is int). In interface, `virtual DltReturnValue dlt_client_connect(DltClient *client, int verbose) = 0;` and in test file `TEST_F(DltLogClient_test, init) { DltClient *client = new DltClient(); EXPECT_CALL(MockXIFObj, dlt_client_init(client, 0)); EXPECT_EQ(0, dltLogclient->init()); }` I get a SEGFAULT. Where made I wrong? – Nibir Mar 06 '18 at 08:54
  • This crashes because the mock object is destroyed after the tests are done, but it should be destroyed as part of the test. So matthandi solution is correct, and this one is not. – Daid Braam Mar 18 '20 at 09:38
  • If this causes a crash, then it has something to do with your code. But like I wrote in the edit in the bottom, you can easily let the mock object be constructed and destructed together with the test fixture. That should solve your problem. – Georg P. Mar 18 '20 at 19:21
  • @Georg P. For me this approach also does not work. I keep getting undefined reference if I dont compile the source production code of lib_x_function(). It seems the lib_x_function version declared on lib_x_mock.cc file will have a different name on the object file (C++ compilation) after compilation, resulting on linking error – Edson Manoel Jul 17 '23 at 20:13
28

This is another answer of mine to this question. In the two years that passed since the first answer, I came to understand that GMock is simply the wrong framework for mocking C functions. In situations where you have a lot of functions to mock, my previously posted answer is simply too cumbersome. The reason is that GMock uses Object Seams to replace production code with mock code. This relies on polymorphic classes, which don't exist in C.

Instead, to mock C functions, you should use Link Seams, which replace the production code with the mock code at link time. Several frameworks exist for this purpose, but my favorite one is the Fake Function Framework (FFF). Check it out, it's a lot simpler than GMock. It also works perfectly well in C++ applications.

For the interested, here is a good article by Michael Feathers about the different seam types.

Georg P.
  • 2,785
  • 2
  • 27
  • 53
  • This should be the accepted answer, FFF is much easier to use than GMock. Note that you _can_ use GTest and FFF. With GTest+FFF, you can create stub functions as fixture methods (as long as they're static), which makes it very easy to derive the fixtures and specialize the stubs. Something like `library_x_fake.custom_fake = MyGTestFixture::library_x_fake;` (`MyGTestFixture::library_x_fake` must be `static`). – Leonardo Oct 12 '22 at 16:33
9

I was looking already a long time for a solution to mock legacy c-functions with googleMock without changing existing code and last days I found the following really great article: https://www.codeproject.com/articles/1040972/using-googletest-and-googlemock-frameworks-for-emb

Today I wrote my first unit test for c-functions using gmock and took as example two functions from the bcm2835.c library (http://www.airspayce.com/mikem/bcm2835/) for raspberry Pi programming: Here is my solution: I'm using the gcc 4.8.3. under Eclipse and Windows. Be Aware to set the Compiler option -std=gnu++11.

Here are my functions to be tested

int inits(void);
void pinMode(uint8_t pin, uint8_t mode);

int inits(){
    return bcm2835_init();
}

void pinMode(uint8_t pin, uint8_t mode){
    bcm2835_gpio_fsel(pin, mode);
}

Includes and defines for unit testing with googleTest / googleMock

// MOCKING C-Functions with GMOCK :)
#include <memory>
#include "gtest/gtest.h"
#include "gmock/gmock.h"
using namespace ::testing;
using ::testing::Return;

Mock BCM2835Lib functions

class BCM2835Lib_MOCK{
public:
    virtual ~BCM2835Lib_MOCK(){}

    // mock methods
    MOCK_METHOD0(bcm2835_init,int());
    MOCK_METHOD2(bcm2835_gpio_fsel,void(uint8_t,uint8_t));
};

Create a TestFixture

class TestFixture: public ::testing::Test{
public:
    TestFixture(){
        _bcm2835libMock.reset(new ::testing::NiceMock<BCM2835Lib_MOCK>());
    }
    ~TestFixture(){
        _bcm2835libMock.reset();
    }
    virtual void SetUp(){}
    virtual void TearDown(){}

    // pointer for accessing mocked library
    static std::unique_ptr<BCM2835Lib_MOCK> _bcm2835libMock;
};

Instantiate mocked lib functions

// instantiate mocked lib
std::unique_ptr<BCM2835Lib_MOCK> TestFixture::_bcm2835libMock;

Fake lib functions to connect Mocks with the c-functions

// fake lib functions
int  bcm2835_init(){return TestFixture::_bcm2835libMock->bcm2835_init();}
void bcm2835_gpio_fsel(uint8_t pin, uint8_t mode){TestFixture::_bcm2835libMock->bcm2835_gpio_fsel(pin,mode);}

Create unit testing class for BCM2835 from TestFixture

// create unit testing class for BCM2835 from TestFixture
class BCM2835LibUnitTest : public TestFixture{
public:
    BCM2835LibUnitTest(){
        // here you can put some initializations
    }
};

Write the Tests using googleTest and googleMock

TEST_F(BCM2835LibUnitTest,inits){
    EXPECT_CALL(*_bcm2835libMock,bcm2835_init()).Times(1).WillOnce(Return(1));

    EXPECT_EQ(1,inits()) << "init must return 1";
}

TEST_F(BCM2835LibUnitTest,pinModeTest){

    EXPECT_CALL(*_bcm2835libMock,bcm2835_gpio_fsel( (uint8_t)RPI_V2_GPIO_P1_18
                                                   ,(uint8_t)BCM2835_GPIO_FSEL_OUTP
                                                  )
               )
               .Times(1)
               ;

    pinMode((uint8_t)RPI_V2_GPIO_P1_18,(uint8_t)BCM2835_GPIO_FSEL_OUTP);
}

Results :)

[----------] 2 tests from BCM2835LibUnitTest
[ RUN      ] BCM2835LibUnitTest.inits
[       OK ] BCM2835LibUnitTest.inits (0 ms)
[ RUN      ] BCM2835LibUnitTest.pinModeTest
[       OK ] BCM2835LibUnitTest.pinModeTest (0 ms)
[----------] 2 tests from BCM2835LibUnitTest (0 ms total)

Hope it will help :) - for me this is a really working solution.

matthandi
  • 91
  • 1
  • 2
  • 8
    Where does the code you showed for "Instantiate mocked lib functions" live? Which .cpp file is that in? – bjornruffians May 15 '18 at 19:27
  • 1
    Could have been better written, showing all necessary files that need to be created and which code goes in which file. Provide example that is reproducible – Edson Manoel Jul 14 '23 at 14:45
  • Because the fake implementation is written in cpp, during linking time the function where the library is actually used will complain about undefined references or if the compilation of the original library is kept it will run into the original one, because that will be the object linked to the function call. For me this solution did not work, I may be doing something wrong perhaps – Edson Manoel Jul 14 '23 at 15:39
  • @matthandi Your approach still fails for me. Debugging the code to understand why the call expectation was failing, I noticed that the original production function implementation is the one called always (equivalent to calling original bcm2835_gpio_fsel on your example). Why does that happen? – Edson Manoel Jul 17 '23 at 14:24
2

You can use the Cutie library to mock C function GoogleMock style.
There's a full sample in the repo, but just a taste:

INSTALL_MOCK(close);
CUTIE_EXPECT_CALL(fclose, _).WillOnce(Return(i));
MrDor
  • 98
  • 5
1

I had a similar case in a project I was unit-testing. My solution was to create two make files, one for production and one for testing.

If the function func_1() is definded in the header a.h, and implemented in a.cpp, then for testing you can add a new source file a_testing.cpp, that will implement all the functions in a.h as a stub. For unittesting, just compile and link with a_testing.cpp instead of a.cpp and the tested code will call your stub.

In a_testing.cpp you can then forward the call to a gmock object that will set expectations and actions as usual based on the state and paramteres.

I know it's not perfect, but it works ans solve the problem without changing production code or interfaces at all.

amir
  • 41
  • 5
0

In each UT we are trying to verify a specific behavior.

You should fake something when it's very hard/impossible(we need to isolate our unit)/spend a lot of time(running time..) to simulate a specific behavior.

Using a 'C' function in the explicit way means that the function is apart of your unit(therefore you shouldn't mock it..). In this answer I explain the initiative to test the method as is(in the edit..). In my opinion you should call func with the parameters which cause func_1 to simulate the behavior you want to verify.

GMock is based on compilation fake(macros), therefore you cannot do such a thing. To fake 'C' methods you have to use a different tools such as Typemock Isolator++.

If you don't want to use Isolator++, then you should refactor your method; Change func to func(int a, <your pointer the function>) and then use the pointer instead of func_1.

My chart in this answer might help to decide the way to handle your case.

Old Fox
  • 8,629
  • 4
  • 34
  • 52
  • Your chart provides good a point of view regarding how much effort needs to put into Unit Testing. But I have certain question in mind. What if `func_1` points to many dependencies (functions) and then testing while implementing `func_1` i.e. Integration Testing could be very time consuming. And what if the source code `func` cannot be changed. Is there a way to tell compiler to point to a stub function instead of implementing real `func_1` without changing source code? – SRehman Aug 13 '15 at 15:22
  • @user3159610 there is no way to tell the compiler to stub the method. From your last comment seems that your method is a legacy code. The main reason to use code-weaving tools like `isolator++` is legacy code. If it is a legacy code then, I'd offer you to skip the tests or create an integration tests only for the main flows. I agree with you about the integration test's (time consuming), however if the code worth UT's you should get a refactoring task in the sprint or the money for tools.... – Old Fox Aug 13 '15 at 17:02
0

This might not totally fit your case, but if you find yourself writing C code that needs to be stubbed (e.g. you want to stub some I/O connection), you could use function pointers.

So let's say you have a header and source file with the following function declaration and definition:

some_file.h

// BEGIN SOME_FILE_H
#ifndef SOME_FILE_H
#define SOME_FILE_H

#include <stdbool.h>

bool func_1(void);

#endif // SOME_FILE_H
// END SOME_FILE_H

some_file.c

// BEGIN SOME_FILE_C
#include "some_file.h"

bool func_1(void) {
  return true;
}
// END SOME_FILE_C

Now, if you would like to stub this method, all you have to do is convert this method into a function pointer. You will also have to adjust the .c file, since we changed the function and made it a function pointer.

some_file.h

// BEGIN SOME_FILE_H
#ifndef SOME_FILE_H
#define SOME_FILE_H

#ifdef __cplusplus
extern "C" {
#endif

#include <stdbool.h>

extern bool (*func_1)(void);

#ifdef __cplusplus
}
#endif

#endif // SOME_FILE_H
// END SOME_FILE_H

some_file.c

// BEGIN SOME_FILE_C
#include "some_file.h"

// Make the method static, so that it cannot be accessed anywhere else except in this file
static bool func_1_Impl(void) {
    return true;
}
bool (*func_1)(void) = func_1_Impl;
// END SOME_FILE_C

You don't have to adjust the function calls anywhere else, since func_1 will simply be redirected to func_1_Impl.

Now for stubbing this method:

In your *_test.cc file (or whatever you call your test file), you can create a mock class with the same interface as some_file.h. You can then overwrite the function pointer with the defined mock function.

some_file_test.cc

#include <gtest/gtest.h>
#include <gmock/gmock.h>
#include "some_file.h"
#include "header_where_func_is_declared.h"

using ::testing::AtLeast;

class SomeFile {
public:
    virtual bool func1() = 0;
};

class MockSomeFile : SomeFile {
public:
    MOCK_METHOD(bool, func1, (), (override));
};

TEST(Func1Test, ShouldMockStuff) {
    // Arrange
    auto expected = 0; // or whatever the expected result is

    // Define the mock object to be used
    static MockSomeFile mock;

    // The important part: Overwrite the function pointer (with a lambda function)
    func_1 = []() { return mock.func1(); };

    // Define the call expectations
    EXPECT_CALL(mock, func1)
            .Times(AtLeast(1));

    // Act
    auto actual = func();

    // Assert
    EXPECT_EQ(expected, actual);
}

This test should pass, showing that the mocking worked. You can also check, whether the test will fail, if you change the EXPECT_CALL call, e.g. set .Times(AtLeast(2)).

Note: You might see that the adjusted test passed with AtLeast(2), although this is wrong. You should still see the correct error message in the console.

I hope this helps you and everyone else who has a similar problem!

Th2mas
  • 11
  • 1
  • 2