)]}'
{"oslo/messaging/_cmd/zmq_receiver.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"1e094a1752fc0a568a55fa54178644a7b3116403","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"baada198_f0a1c8f8","updated":"2014-07-05 16:34:05.000000000","message":"Unrelated change, should be a separate patch","commit_id":"a42f5d6e0b665693dcf86f5e56d90baf1ca6acb5"}],"tests/drivers/test_impl_rabbit.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"1e094a1752fc0a568a55fa54178644a7b3116403","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        target \u003d messaging.Target(topic\u003d\u0027testtopic\u0027)"},{"line_number":269,"context_line":"        listener \u003d driver.listen(target)"},{"line_number":270,"context_line":"        received \u003d listener.poll(timeout\u003d0.050)"},{"line_number":271,"context_line":"        self.assertIsNone(received)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"TestSendReceive.generate_scenarios()"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_d09e84b8","line":271,"updated":"2014-07-05 16:34:05.000000000","message":"This should be a separate test case. The way you\u0027ve done it, the exact same test gets executed many times - one for each of the test scenarios\n\nAlso, would be good to have a qpid test","commit_id":"a42f5d6e0b665693dcf86f5e56d90baf1ca6acb5"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"3bde76d6dddfc227daf32a110f8e1d191e51014f","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        target \u003d messaging.Target(topic\u003d\u0027testtopic\u0027)"},{"line_number":269,"context_line":"        listener \u003d driver.listen(target)"},{"line_number":270,"context_line":"        received \u003d listener.poll(timeout\u003d0.050)"},{"line_number":271,"context_line":"        self.assertIsNone(received)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"TestSendReceive.generate_scenarios()"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_fd57c6c7","line":271,"in_reply_to":"baada198_d09e84b8","updated":"2014-07-07 13:51:35.000000000","message":"Did you miss this comment?","commit_id":"a42f5d6e0b665693dcf86f5e56d90baf1ca6acb5"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"47b6c8efc5d5fe1166fdbd7dfe44598d848ad8ce","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        target \u003d messaging.Target(topic\u003d\u0027testtopic\u0027)"},{"line_number":269,"context_line":"        listener \u003d driver.listen(target)"},{"line_number":270,"context_line":"        received \u003d listener.poll(timeout\u003d0.050)"},{"line_number":271,"context_line":"        self.assertIsNone(received)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"TestSendReceive.generate_scenarios()"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_f8fd146f","line":271,"in_reply_to":"baada198_fd57c6c7","updated":"2014-07-07 14:31:53.000000000","message":"Sorry, I had troubles to revert the change on oslo/messaging/_cmd/zmq_receiver.py. I didn\u0027t touch zmq_receiver.py. The change maybe come from a rebase or a mistake with git.\n\nI created a new test case in the patch set 3.","commit_id":"a42f5d6e0b665693dcf86f5e56d90baf1ca6acb5"}]}
