No signal is emitted in testing for signals and slots

shravan_121

test.cpp:
QTest::keyClick(ccalculator->plusButton,Qt::Key_Plus);
QSignalSpy spy(ccalculator,&Calculator::addbuttonClicked);
QTest::keyClick(ccalculator->plusButton,Qt::Key_Plus);
QCOMPARE(spy.count(),1);
the result shows the actual value is 0 and expected value is 1.

calculator.cpp:
plusButton = createButton(tr("+"), SLOT(additiveOperatorClicked()this));
connect(plusButton,SIGNAL(addbuttonClicked()),this,SLOT(additiveOperatorClicked()));
additiveOperatorClicked()
{
//some codes
emit addbuttonClicked(tr("+"));
}
can anyone help me to solve this problem.

VRonin

make sure all the connections were successful. My recommendation is to use Qt5 connection syntax but if you stick with Qt4 version then wrap the connect statement in a Q_ASSUME() (so it asserts if the connection fails)

Q_ASSUME(connect(plusButton,SIGNAL(addbuttonClicked()),this,SLOT(additiveOperatorClicked())));

shravan_121

It shows me error ::Qobject::connect:no such signal Button::additivieoperatorClickedsignal()

J.Hilk

@shravan_121
pls post your whole connect statement

shravan_121

calculator.h:
class Calculator : public QWidget
{
Q_OBJECT

public:
Button *plusButton;
Button *minusButton;
Button *timesButton;
Button *divisionButton;
Calculator(QWidget *parent = 0);
public slots:
void digitClicked();
void additiveOperatorClicked();
void multiplicativeOperatorClicked();
signals:
void addbuttonClicked(const QString &data);
void multiplybuttonClicked(const QString &data);
void digitclickedsignal(const QString &data);

calculator.cpp:
Calculator::Calculator(QWidget *parent)
: QWidget(parent)
{
plusButton = createButton(tr("+"), SLOT(additiveOperatorClicked()this));

connect(plusButton,SIGNAL(addbuttonClicked()),this,SLOT(additiveOperatorClicked()));

void Calculator::additiveOperatorClicked()
{
Button *clickedButton = qobject_cast<Button *>(sender());
QString clickedOperator = clickedButton->text();
double operand = display->text().toDouble();
if (!pendingMultiplicativeOperator.isEmpty()) {
//! [12] //! [13]
if (!calculate(operand, pendingMultiplicativeOperator)) {
abortOperation();
return;
}
display->setText(QString::number(factorSoFar));
operand = factorSoFar;
factorSoFar = 0.0;
pendingMultiplicativeOperator.clear();

}

if (!pendingAdditiveOperator.isEmpty()) {
    if (!calculate(operand, pendingAdditiveOperator)) {
        abortOperation();
        return;
    }
    display->setText(QString::number(sumSoFar));
} else {
    sumSoFar = operand;
}
pendingAdditiveOperator = clickedOperator;
waitingForOperand = true;

 emit addbuttonClicked(tr("+"));

Bondrusiek

@shravan_121 On first view you have bad syntax. Look on signal emit addbuttonClicked(tr("+")); which has one argument QString. In connect function you use signal without argument ```
connect(plusButton,SIGNAL(addbuttonClicked()),this,SLOT(additiveOperatorClicked()));.
Try to replace signal on
void addbuttonClicked()
or change function slot with one argument ie

void additiveOperatorClicked(QString)

and use this in connect

connect(plusButton,SIGNAL(addbuttonClicked(QString)),this,SLOT(additiveOperatorClicked(QString)));

shravan_121

But after that i am getting same error: no such signal additiveoperatorclicked(QString)
no such slot addbuttonclicked(QString)

VRonin

@shravan_121 said in No signal is emitted in testing for signals and slots:

no such signal Button::additivieoperatorClicked

And this is the problem additivieoperatorClicked is a signal of Calculator not of Button. You are trying to connect with a receiver being a pointer to Button

shravan_121

@VRonin I dont know the actual solution can you please suggest a possible solution for the error.

shravan_121

This post is deleted!

shravan_121

I don't know the actual solution can you please suggest a possible solution for the error.

VRonin

@shravan_121 said in No signal is emitted in testing for signals and slots:

connect(plusButton,SIGNAL(addbuttonClicked()),

The problem is here. addbuttonClicked is a signal of Calculator but plusButton is of type Button.
So either the sender (the first argument) or the signal (the second) must be changed to make sense