)]}'
{"cinder/tests/unit/volume/test_rpcapi.py":[{"author":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"change_message_id":"b6c9cf1a09b7b68c4ac37c3ff36508d6a3875a36","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        super(VolumeRPCAPITestCase, self).setUp()"},{"line_number":39,"context_line":"        self.rpcapi \u003d volume_rpcapi.VolumeAPI"},{"line_number":40,"context_line":"        self.base_version \u003d \u00273.0\u0027"},{"line_number":41,"context_line":"        vol \u003d {}"},{"line_number":42,"context_line":"        vol[\u0027host\u0027] \u003d \u0027fake_host\u0027"},{"line_number":43,"context_line":"        vol[\u0027availability_zone\u0027] \u003d CONF.storage_availability_zone"},{"line_number":44,"context_line":"        vol[\u0027status\u0027] \u003d \"available\""},{"line_number":45,"context_line":"        vol[\u0027attach_status\u0027] \u003d \"detached\""},{"line_number":46,"context_line":"        vol[\u0027metadata\u0027] \u003d {\"test_key\": \"test_val\"}"},{"line_number":47,"context_line":"        vol[\u0027size\u0027] \u003d 1"},{"line_number":48,"context_line":"        volume \u003d db.volume_create(self.context, vol)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_14a9897b","line":47,"range":{"start_line":41,"start_character":8,"end_line":47,"end_character":23},"updated":"2016-10-14 12:29:09.000000000","message":"nit: why not use fake_volume.fake_db_volume?","commit_id":"96fd87fc66c5cfa844f21be6959f181c328bf32d"},{"author":{"_account_id":12825,"name":"Szymon Wróblewski","email":"szymon.wroblewski@ovhcloud.com","username":"bluex"},"change_message_id":"b6c9cf1a09b7b68c4ac37c3ff36508d6a3875a36","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        vol[\u0027size\u0027] \u003d 1"},{"line_number":48,"context_line":"        volume \u003d db.volume_create(self.context, vol)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        kwargs \u003d {"},{"line_number":51,"context_line":"            \u0027status\u0027: fields.SnapshotStatus.CREATING,"},{"line_number":52,"context_line":"            \u0027progress\u0027: \u00270%\u0027,"},{"line_number":53,"context_line":"            \u0027display_name\u0027: \u0027fake_name\u0027,"},{"line_number":54,"context_line":"            \u0027display_description\u0027: \u0027fake_description\u0027}"},{"line_number":55,"context_line":"        snapshot \u003d tests_utils.create_snapshot(self.context, vol[\u0027id\u0027],"},{"line_number":56,"context_line":"                                               **kwargs)"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_1447c94b","line":54,"range":{"start_line":50,"start_character":8,"end_line":54,"end_character":54},"updated":"2016-10-14 12:29:09.000000000","message":"nit: why not simply pass named arguments to create_snapshot","commit_id":"96fd87fc66c5cfa844f21be6959f181c328bf32d"}]}