openstack/zaqar
Code Issues Proposed changes

Use pop in message_delete_many

Browse Source 
message_delete_many used by websocket incorrected made reference to the
pop_limit argument, instead of using pop.

Change-Id: I1c94ad82c3b20cc331045c19d4a9987a701b081b
Closes-Bug: #1653525
This commit is contained in:
Thomas Herve 2017-01-02 16:41:47 +01:00
parent 828bab9d66
commit 39452c9a1c
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
zaqar/api/v2/endpoints.py
View File

@ -563,7 +563,7 @@ class Endpoints(object):
project_id = req._headers.get('X-Project-ID')
queue_name = req._body.get('queue_name')
message_ids = req._body.get('message_ids')
pop_limit = req._body.get('pop_limit')
pop_limit = req._body.get('pop')
LOG.debug(u'Messages collection DELETE - queue: %(queue)s,'
u'project: %(project)s, messages: %(message_ids)s',

3
zaqar/tests/unit/transport/websocket/v2/test_messages.py
View File

@ -522,8 +522,7 @@ class MessagesBaseTest(base.V2Base):
self._post_messages("kitkat", repeat=5)
action = consts.MESSAGE_DELETE_MANY
body = {"queue_name": "kitkat",
"pop_limit": 2}
body = {"queue_name": "kitkat", "pop": 2}
send_mock = mock.Mock()
self.protocol.sendMessage = send_mock