C ++ Google模拟/单元测试:未调用模拟方法,原始方法是

时间:2018-07-18 11:53:45

标签: c++ unit-testing googlemock

我正在嘲笑一个方法,但是我的GoogleTest似乎一直在调用原始实现。可能丢失了一些东西,但找不到这是什么?

我的课程是Postgres数据库访问器:

namespace project
{

class PostgresDb
{
public :
    // typedef the CursorPtr, this is just a smart pointer.
    // sth like 
    typedef shared_ptr<DbDataType> CursorPtr ;

    PostgresDb( const std::string & connString ) ;

    CursorPtr getRecords( const int id_1 ,
                          const int id_2 ) ;

protected :

    // this will be used by the MockPostgresDb in our unit tests.
    // it leaves the object uninitialized
     PostgresDb (){} ;

 }

}// end of namespace

这是我的模拟类:

namespace project_test
{

class MockPostgresDb : public project::PostgresDb
{
public :
    MockPostgresDb(){} ;
    MOCK_METHOD2( getRecords, CursorPtr*( const int , const int  ) ) ;
} ;

class MockCursor : public CursorPtr
{
    // ....
}

}

这是我正在测试的方法和测试:

void original_get_method( const int id_1, const int id_2 , PostgresDb db)
{
    // ....
    db.getRecords( id_1, id_2 ) ;
    // ....
}

// helper function
setupGetRecords( MockPostgresDb* db     ,
                 MockCursor*     cursor ) 
{
    EXPECT_CALL( *db, getRecords(_,_) )
        .Times( 1 )
        .WillRepeatedly(::testing::Return( cursor ) ) ;
}


TEST_F(....)
{
    MockPostgresDb db ;
    MockCursor     cursor ;
    // ....
    setupGetRecords( &db, &cursor ) ; 
    // ....
    // and then calling a functi
    original_method( id_1, id_2, db ) ;
}

所以在我看来,我正在呼叫original_method并传递一个mock_dbmock_db调用其方法getRecords,该方法返回MockCursor。这是应该让我模拟的地方,但我确实输入了db::getRecords

我试图找到不匹配的地方,但无法弄清楚。

编辑:

因此,正如所指出的-getRecords应该返回CursorPtr而不是CursorPtr*。所以这就是我所做的:

我也试图改变

MOCK_METHOD2( getRecords, CursorPtr*( const int , const int  ) ) ;`

MOCK_METHOD2( getRecords, CursorPtr( const int , const int  ) ) ;  // ( no * ) 

并将帮助程序更改为

// helper function
setupGetRecords( MockPostgresDb* db     ,
                 MockCursor     cursor ) 
{
    EXPECT_CALL( *db, getRecords(_,_) )
        .Times( 1 )
        .WillRepeatedly(::testing::Return( cursor ) ) ;
}

并得到一些不匹配的编译错误类型。什么地方出了错?谢谢。

1 个答案:

答案 0 :(得分:1)

这里的主要问题是在函数void original_get_method( const int id_1, const int id_2 , PostgresDb db)PostgresDb是通过值传递的,即将调用PostgresDb的副本构造函数。这意味着在original_get_method函数内部您没有MockPostgresDb对象,而是一个简单的PostgresDb,这就是为什么不调用模拟方法而是仅调用原始实现的原因。此外,getRecords函数不是虚函数,即类MockPostgresDb中的模拟方法仅遮盖了PostgresDb的函数(因此无法在运行时告诉程序哪个函数调用,即如果有一个PostgresDb对象或对该对象的引用,它将始终调用该类中的函数。

所以首先要改变的是

class PostgresDb
{
public :
    // typedef the CursorPtr, this is just a smart pointer.
    // sth like 
    typedef shared_ptr<DbDataType> CursorPtr ;

    PostgresDb( const std::string & connString ) ;

    // ===> make this method virtual <====
    virtual CursorPtr getRecords( const int id_1 ,
                          const int id_2 ) ;
}

// pass PostgresDb by reference
void original_get_method( const int id_1, const int id_2 , PostgresDb& db);

这两个更改将在original_get_method中调用模拟方法。


注释中指出了下一个问题:

MOCK_METHOD2( getRecords, CursorPtr*( const int , const int  ) ) ;

模拟方法具有不同的签名,然后基类->函数将不会在派生(模拟的)类中被覆盖(即,它们是两个完全不同的函数)。这就是您想要的:

MOCK_METHOD2( getRecords, CursorPtr( const int , const int  ) ) ;

下一个问题如下:

typedef shared_ptr<DbDataType> CursorPtr ;

您定义新类型CursorPtr,然后从中继承

class MockCursor : public CursorPtr {}

但是MockCursor不是从DbDataType继承而是从std::shared_ptr继承,即它们完全不同。相反,让MockDbDataTypeDbDataType继承并创建一个新的类型MockCursor

typedef std::shared_ptr<MockDbDataType> MockCursor;

然后应该可以正常编译:

setupGetRecords( MockPostgresDb* db     ,
                 MockCursor     cursor ) 
{
    EXPECT_CALL( *db, getRecords(_,_) )
        .Times( 1 )
        .WillRepeatedly(::testing::Return( cursor ) ) ;
}

我还没有测试这段代码,所以如果有任何问题让我知道。