死のテストで奇妙なヒープチェッカーのエラー

Question

モックオブジェクトと死のテストを使用して私のgoogletest unittestsの1つに問題があります。これは、問題を示して最小限のコードサンプルです：

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

using namespace ::testing; 

class MockA { 
    public: 
    MockA() {}; 
    virtual ~MockA() {}; 

    MOCK_METHOD1(bla,int(int)); 
}; 


class B { 
    public: 
    B(MockA * a) 
      : a_(a) {}; 
    void kill() { 
     exit(1); 
    } 
    MockA * a_; 
}; 

TEST(BDeathTest,BDies) { 
    MockA * a = new MockA(); 
    ON_CALL(*a,bla(_)).WillByDefault(Return(1)); 
    B * b = new B(a); 
    EXPECT_DEATH(b->kill(),""); 
    delete a; 
} 

int main(int argc, char **argv) { 
    ::testing::InitGoogleTest(&argc, argv); 
    return RUN_ALL_TESTS(); 
} 

出力：

[==========] Running 1 test from 1 test case. 
[----------] Global test environment set-up. 
[----------] 1 test from BDeathTest 
[ RUN  ] BDeathTest.BDies 

gtest.cc:27: ERROR: this mock object (used in test BDeathTest.BDies) should be deleted but never is. Its address is @0x7fe453c00ec0. 
ERROR: 1 leaked mock object found at program exit. 
[  OK ] BDeathTest.BDies (2 ms) 
[----------] 1 test from BDeathTest (2 ms total) 

[----------] Global test environment tear-down 
[==========] 1 test from 1 test case ran. (2 ms total) 
[ PASSED ] 1 test. 

それは右EXPECT_DEATH表明した後、ヒープ上のモックオブジェクトの上に左用googlemockチェックのように思えるが、aを削除呼び出される関数でaが使用される可能性があるため、マクロを呼び出す前には明らかに良い解決策ではありません。実際には、テストスイートの解体の終わりにチェックが行われると思います。私は何が欠けていますか？

Answer 1

aが漏洩されますので、あなたはこのことについてgmockを指示する必要があります：あなたはまた、すべてのgmockリーク検出をオフにする::testing::FLAGS_gmock_catch_leaked_mocks = false;を使用することができますが、これはおそらくに入るために悪い習慣です

TEST(BDeathTest,BDies) { 
    MockA * a = new MockA; 
    ON_CALL(*a,bla(_)).WillByDefault(Return(1)); 
    B * b = new B(a); 
    Mock::AllowLeak(a); // <=== Self-explanatory addition 
    EXPECT_DEATH(b->kill(),""); 
    delete a; 
    delete b; 
} 

。しかし、exit()と呼ぶときにたくさんの疑似オブジェクトがある場合は、この場合には適切かもしれません。テストを続行する場合は、EXPECT_DEATHの直後に再度スイッチを入れる価値があります（上の例では、それを元に戻すのは無意味です）。

TEST(BDeathTest,BDies) { 
    MockA * a = new MockA; 
    ON_CALL(*a,bla(_)).WillByDefault(Return(1)); 
    B * b = new B(a); 
    FLAGS_gmock_catch_leaked_mocks = false; // <=== Switch off mock leak checking 
    EXPECT_DEATH(b->kill(),""); 
    FLAGS_gmock_catch_leaked_mocks = true; // <=== Re-enable mock leak checking 
              //  in case the test is refactored 
    delete a; 
    delete b; 
} 

最後に、この特別な場合を処理するための第三の方法は、それがリークすることを可能にするのではなくB::kill()でdelete a_;です。

class B { 
    ... 
    void kill() { 
     delete a_; 
     exit(1); 
    } 
    MockA * a_; 
}; 

TEST(BDeathTest,BDies) { 
    MockA * a = new MockA; 
    ON_CALL(*a,bla(_)).WillByDefault(Return(1)); 
    B * b = new B(a); 
    EXPECT_DEATH(b->kill(),""); 
    delete a; 
    delete b; 
} 

GTESTは死のテストを実行するための新しいプロセスを生成しますので、あなたはまた、テスト・フィクスチャ内aを削除しながら、ちょうどexit ING前a_を削除しても安全です。

しかし、どのようにgtestの死のテストが動作しているかわからない人には、同じ変数が2回削除されるように見え、混乱を招く可能性があります。