QTest a connection from a private member

[Axel Spoerl]

@wasawi2
You have to make the private member a private slot. Then it should work.

[wasawi2]

Thank you for your answer @Axel-Spoerl !
Could you please elaborate your answer?
I'm very sorry but I don´t understand what you mean...

Are you talking about modifying the code to test?
I would like not to have to modify the code to test. Although a minor modification might be possible...

[wasawi2]

Ideally i would like to test that the connection has been successful with something like this:
QVERIFY( false == QObject::connect(&myMainWindow_instance.aPrivateMember, aSignal, &aMainwindow_instance, aSlot, Qt::UniqueConnection) );

[Axel Spoerl]

Can you show the declaration of the method you want to test?

Probably it’s in the

private

Section of the header. A connection won’t work here. You have to put it in a

private slots

section.

[wasawi2]

the method is in the "public slots:"

[wasawi2]

but of course the so called "aPrivateMember" is under the "private:" section

[wasawi2]

Sorry actually the method I'm testing is under the public: section... maybe that is the problem...

[Axel Spoerl]

It needs to be a slot, no matter if private or public.

[wasawi2]

ok, let me see if I understand well.
so the following code does not work because the object "aPrivateMember" is actually private. if it was public the code actually works.

the test code

void MainWindowTest::init ( ){}
void MainWindowTest::cleanup ( ){}
void MainWindowTest::test_case ( )
{
	qwidget_instance = new QWidget(  );
	mainWindow_instance = new myMainWindow( qwidget_instance );

	mainWindow_instance->aFunctionToTest( );
	QCOMPARE( (bool) QObject::connect(mainWindow_instance->aPrivateMember, &aClass::aSignal, mainWindow_instance, &myMainWindow::aSlot, Qt::UniqueConnection), false  );
// the test passes if, and only if aPrivateMember was declared in the public section.
// if aPrivateMember is private the following compile error occurs:
// error: 'aClass* myMainWindow::aPrivateMember' is private within this context
}

QTEST_MAIN( MainWindowTest )

the code to test

.h
class myMainWindow: public QMainWindow
{
Q_OBJECT

public slots:
void aFunctionToTest ( );
void aSlot ( );

private:
aClass aPrivateMember;
};


.cpp
void myMainWindow::aFunctionToTest( )
{
	connect( aPrivateMember, &aClass::aSignal, this, &myMainWindow::aSlot);
}

So my question stays the same. Is it possible to test the success of the connection done in aFunctionToTest given that aPrivateMember is actually private?

Thank you so much for the patience!
I tried to be as clear as possible now.

And I'm pretty sure I must be missing something obvious.. but I cant see what it is...

[Axel Spoerl]

Ah, now I understand better.
The private member can’t be tested (called) in an external test class normally. It also can’t be invoked / connected, because it isn’t a slot.
So if you want to connect it, it has to be a slot. If you want to keep it private, you could befriend the test class.

[wasawi2]

I see.
So If i understand correctly without modifying the code to test there is no way to test the connection.

[jsulm]

@wasawi2 @Axel-Spoerl Actually with the Qt5 connect syntax you can connect methods and lambdas which are not declared as slots. So, it should be possible without code modification. You just need to make the test class friend, so it can access non-public members of the class being tested.

[JonB]

@jsulm said in QTest a connection from a private member:

You just need to make the test class friend

So as the OP is asking this does require the code being tested to be changed!

@wasawi2
Whichever way up you look at it, you will have to make a change to allow a private member to be accessed outside. Which is what private means (i.e. cannot access outside, unless you change that or put in friend) :)

[jsulm]

@JonB said in QTest a connection from a private member:

So as the OP is asking this does require the code being tested to be changed!

Well, yes. But it does not require to change the visibility of the class members/methods.

[JonB]

@jsulm
I absolutely concur, hence my:

[private] cannot access outside, unless you change that or put in friend

The point being to answer OP's

So If i understand correctly without modifying the code to test there is no way to test the connection.

They will have to make some change, which doubtless they were attempting to avoid. Just a heads-up.