jessequ
/
qamqp
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

allow for creation of a queue with no name, added auto test for validation

This commit is contained in:
Matt Broadstone 2014-06-09 14:07:13 -04:00
parent 60167321a9
commit 50a588e678
2 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/amqp_queue.cpp
View File

@ -207,11 +207,6 @@ void QueuePrivate::deliver(const Frame::Method &frame)
void QueuePrivate::declare() void QueuePrivate::declare()
{ {
if (name.isEmpty()) {
qAmqpDebug() << Q_FUNC_INFO << "can't declare queue with no name";
return;
}
Frame::Method frame(Frame::fcQueue, QueuePrivate::miDeclare); Frame::Method frame(Frame::fcQueue, QueuePrivate::miDeclare);
frame.setChannel(channelNumber); frame.setChannel(channelNumber);

12
tests/auto/qamqpqueue/tst_qamqpqueue.cpp
View File

@ -19,6 +19,8 @@ private Q_SLOTS:
void standardExchanges_data(); void standardExchanges_data();
void standardExchanges(); void standardExchanges();
void unnamedQueue();
void remove(); void remove();
void removeIfUnused(); void removeIfUnused();
void unbind(); void unbind();
@ -90,6 +92,16 @@ void tst_QAMQPQueue::standardExchanges()
QCOMPARE(queue->getMessage().payload(), QByteArray("test message")); QCOMPARE(queue->getMessage().payload(), QByteArray("test message"));
} }
void tst_QAMQPQueue::unnamedQueue()
{
Queue *queue = client->createQueue();
queue->declare();
QVERIFY(waitForSignal(queue, SIGNAL(declared())));
queue->consume();
QVERIFY(!queue->name().isEmpty());
}
void tst_QAMQPQueue::remove() void tst_QAMQPQueue::remove()
{ {
Queue *queue = client->createQueue("test-remove"); Queue *queue = client->createQueue("test-remove");