)]}'
{"cinder/tests/unit/test_blockbridge.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"43ee222a5b9053c8e728329c6f695480b02fb94b","unresolved":false,"context_lines":[{"line_number":249,"context_line":""},{"line_number":250,"context_line":"        full_url \u003d \"/api/cinder\" + url"},{"line_number":251,"context_line":"        json_body \u003d test_utils.CompareJSON(create_params)"},{"line_number":252,"context_line":"        tsk_header \u003d \"ext_auth\u003dkeystone/%(project_id)s/%(user_id)s\" % kwargs"},{"line_number":253,"context_line":"        authz_header \u003d \"Bearer %s\" % self.cfg.blockbridge_auth_token"},{"line_number":254,"context_line":"        headers \u003d {"},{"line_number":255,"context_line":"            \u0027X-Blockbridge-Task\u0027: tsk_header,"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_6917d451","line":252,"range":{"start_line":252,"start_character":39,"end_line":252,"end_character":42},"updated":"2016-02-15 14:32:20.000000000","message":"Now, this is no longer a JSON body, it is a CompareJSON object, right?\n\nHow can it be passed to mock_conn.request? Is it the right type? Is this the right variable name?","commit_id":"ded541d30b99c764ec4a499cc724d2e43c69cfb1"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"5465e09b17ec7062f5d8e61e2deee5d3974bd55a","unresolved":false,"context_lines":[{"line_number":249,"context_line":""},{"line_number":250,"context_line":"        full_url \u003d \"/api/cinder\" + url"},{"line_number":251,"context_line":"        json_body \u003d test_utils.CompareJSON(create_params)"},{"line_number":252,"context_line":"        tsk_header \u003d \"ext_auth\u003dkeystone/%(project_id)s/%(user_id)s\" % kwargs"},{"line_number":253,"context_line":"        authz_header \u003d \"Bearer %s\" % self.cfg.blockbridge_auth_token"},{"line_number":254,"context_line":"        headers \u003d {"},{"line_number":255,"context_line":"            \u0027X-Blockbridge-Task\u0027: tsk_header,"}],"source_content_type":"text/x-python","patch_set":1,"id":"dae33548_d5a3139d","line":252,"range":{"start_line":252,"start_character":39,"end_line":252,"end_character":42},"in_reply_to":"dae33548_6917d451","updated":"2016-02-15 15:40:23.000000000","message":"It is still the HTTP body and the HTTP body is expected to be serialized by JSON.\n\n\u003e How can it be passed to mock_conn.request?\n\nCompareJSON is an helper to compare the deserialized JSON object, rather than the serialized string, since dictionary items have a random order in the serialized JSON.\n\nMock.assert_called_once_with(expected) checked arg \u003d\u003d expected, so CompareJSON implements __eq__().\n\n\n\u003e Is this the right variable name?\n\n\"json_body\" means that it\u0027s an HTTP body which is expected to be JSON. I don\u0027t really care of the variable name :-)","commit_id":"ded541d30b99c764ec4a499cc724d2e43c69cfb1"}]}