为什么 GoogleMock 会泄露我的 shared_ptr？

Question

我使用 GoogleMock/GoogleTest 进行测试，当匹配器将 shared_ptr 作为模拟参数并在同一个 shared_ptr 上调用 EXPECT 时，我看到了一些奇怪的行为。有问题的代码：

#include <gmock/gmock.h>
#include <gtest/gtest.h>

#include <boost/shared_ptr.hpp>
#include <boost/make_shared.hpp>
using namespace boost;
using namespace testing;

struct MyParameter
{
    virtual ~MyParameter() {}
    virtual void myMethod() = 0;
};

struct MyParameterMock : public MyParameter
{
    MOCK_METHOD0(myMethod, void());
};

struct MyClass
{
    virtual ~MyClass() {}
    virtual void myMethod(shared_ptr<MyParameter> p) {}
};

struct MyClassMock : public MyClass
{
    MOCK_METHOD1(myMethod, void(shared_ptr<MyParameter>));
};

TEST(LeakTest, GoogleMockLeaksMatchedPointer)
{
    shared_ptr<MyClassMock> c = make_shared<MyClassMock>();
    shared_ptr<MyParameterMock> p = make_shared<MyParameterMock>();
    {
        InSequence dummy;
        EXPECT_CALL(*c, myMethod(Eq(p)));
        EXPECT_CALL(*p, myMethod());
    }
    c->myMethod(p);
    p->myMethod();
}

当这个测试运行时，我得到

leak_ptr_mock.cpp:37: ERROR: this mock object (used in test LeakTest.GoogleMockLeaksMatchedPointer) should be deleted but never is. Its address is @0x9309544.
ERROR: 1 leaked mock object found at program exit.

知道为什么会发生这种情况吗？我宁愿不用Mock::AllowLeak。

Answer 1

这是将p 保留为shared_ptr 的结果，使用InSequence 和您声明期望的顺序。

当你打电话时

    EXPECT_CALL(*c, myMethod(Eq(p)));

您增加了p 的use_count。为了让泄漏检测通过，p 必须在TEST 结束时（或之前）销毁。

这里的问题是，在内部，gmock 通过持有指向前面期望的指针来维护所需的模拟调用序列的记录。所以当你调用EXPECT_CALL(*p, myMethod());时，它会得到一份指向之前期望的指针。

这会在TEST 结束时阻止对p 的析构函数的调用。

为了解决这个问题，我认为你最好的选择是跟注

    EXPECT_TRUE(Mock::VerifyAndClearExpectations(p.get()));

就在您退出 TEST 之前。这清除了对p 的期望，包括关键的先决条件期望，这反过来又允许正确调用p 的析构函数。

或者，如果模拟调用的顺序不重要，只需删除InSequence dummy; 也将允许p 的析构函数执行。

顺便说一句，您的代码有几个问题；

• 你的基础结构应该有虚拟析构函数
• MyClass::myMethod 应该是虚拟的，以允许 gmock 的函数覆盖它
• p->myMethod(p); 应该是 p->myMethod();