GoogleTest: Mock object should be deleted but never is.

[SGaist]

Hi,

Why are you creating that object on the heap rather than the stack ?

[henrik2016]

@SGaist I have edited my post.

[jsulm]

@henrik2016 said in GoogleTest: Mock object should be deleted but never is.:

_selectedConfValid = new MockOpcConf();

Youre still creating it on the heap, why not simply:

TestOpcUa() 
    {
        MockOpcConf _selectedConfValid; // Allocate on the stack, no need to delete explicetly

[henrik2016]

@jsulm The function I would like to test need pointer.
I have got a Destructor in OpcConf. Is it possible to add a destructor in the Mock-Class MockOpcConf?

[jsulm]

@henrik2016 said in GoogleTest: Mock object should be deleted but never is.:

The function I would like to test need pointer.

And what is the problem? Just pass it the pointer then:

&_selectedConfValid

[henrik2016]

@jsulm But how can I modify the lambda function?

[jsulm]

@henrik2016 What exactly do you want to change?

[henrik2016]

@jsulm I create 19 Mock-Objects like this:

shared_ptr<MockOpcNode> deviceNo = make_shared<MockOpcNode>();

At the end of my test, I have to delete all objects. How can I do this?

[jsulm]

@henrik2016 said in GoogleTest: Mock object should be deleted but never is.:

At the end of my test, I have to delete all objects. How can I do this?

You don't if you do not allocate on the heap (don't use new) - that was the whole point of what @SGaist wrote.
So, why do you do it this way

_selectedConfValid = new MockOpcConf();

intead of

MockOpcConf _selectedConfValid;

?

[henrik2016]

@jsulm I already use MockOpcConf _selectedConfValid;

But I have got the other objects.