)]}'
{"cinderclient/tests/functional/base.py":[{"author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"change_message_id":"d15a2b457487ab57591213f965f6163cfc797141","unresolved":false,"context_lines":[{"line_number":171,"context_line":"        self.wait_for_object_status(object_name, object[\u0027id\u0027], \u0027available\u0027)"},{"line_number":172,"context_line":"        return object"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    def object_delete(self, object_name, object_id):"},{"line_number":175,"context_line":"        \"\"\"Delete specified object by ID."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        :param object_name: object name"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_16744666","line":174,"range":{"start_line":174,"start_character":50,"end_line":174,"end_character":52},"updated":"2016-07-19 15:21:25.000000000","message":"To cover case with multiple objects just replace object_id with, let\u0027s say, list_objects_id. Then insert in line 181:\n\nif list_objects_id:\n    for object_id in list_objects_id:\n        try:\n            self.cinder(cmd_delete, params\u003dobject_id)\n        except exceptions.CommandFailed:\n            pass\nelse:\n    self.fails(\"error: too few argumets\")","commit_id":"53b98451d1ea9b6f0f47ab8022ee01d93bec584c"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"751813e69a07b7d1e1c4342af824312a41833f53","unresolved":false,"context_lines":[{"line_number":171,"context_line":"        self.wait_for_object_status(object_name, object[\u0027id\u0027], \u0027available\u0027)"},{"line_number":172,"context_line":"        return object"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    def object_delete(self, object_name, object_id):"},{"line_number":175,"context_line":"        \"\"\"Delete specified object by ID."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        :param object_name: object name"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_2328dc85","line":174,"range":{"start_line":174,"start_character":50,"end_line":174,"end_character":52},"in_reply_to":"dada55a8_16744666","updated":"2016-07-20 20:11:30.000000000","message":"I don\u0027t like this idea. It make code more complicated w/o any benefits","commit_id":"53b98451d1ea9b6f0f47ab8022ee01d93bec584c"},{"author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"change_message_id":"201ae9ef9806a291188f6f6b5f59a99e6b9b7339","unresolved":false,"context_lines":[{"line_number":171,"context_line":"        self.wait_for_object_status(object_name, object[\u0027id\u0027], \u0027available\u0027)"},{"line_number":172,"context_line":"        return object"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    def object_delete(self, object_name, object_id):"},{"line_number":175,"context_line":"        \"\"\"Delete specified object by ID."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        :param object_name: object name"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_15f44a85","line":174,"range":{"start_line":174,"start_character":50,"end_line":174,"end_character":52},"in_reply_to":"dada55a8_2328dc85","updated":"2016-08-09 15:14:20.000000000","message":"Otherwise, this case doesn\u0027t cover the case with multiple objects to delete. (e.g. $ cinder delete vol_id1 vol_id2 ...)","commit_id":"53b98451d1ea9b6f0f47ab8022ee01d93bec584c"}],"cinderclient/tests/functional/test_volume_delete_cli.py":[{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"d057c7fbcd178a5392bb2aeb177d34adc8ffb6dd","unresolved":false,"context_lines":[{"line_number":22,"context_line":"class CinderVolumeDeleteNegativeTests(base.ClientTestBase):"},{"line_number":23,"context_line":"    \"\"\"Check of cinder volume delete command.\"\"\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    @ddt.data("},{"line_number":26,"context_line":"        (\u0027\u0027, (r\u0027too few arguments\u0027)),"},{"line_number":27,"context_line":"        (\u00271234-1234-1234\u0027, (r\u0027No volume with a name or ID of\u0027)),"},{"line_number":28,"context_line":"        (\u0027my_volume\u0027, (r\u0027No volume with a name or ID of\u0027)),"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_6f740acf","line":25,"range":{"start_line":25,"start_character":9,"end_line":25,"end_character":13},"updated":"2016-07-18 12:57:14.000000000","message":"Try to cover case with multiple arguments.","commit_id":"53b98451d1ea9b6f0f47ab8022ee01d93bec584c"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"d9ff4a545bc0e817c1ef4d096b674bacc9cb76b4","unresolved":false,"context_lines":[{"line_number":22,"context_line":"class CinderVolumeDeleteNegativeTests(base.ClientTestBase):"},{"line_number":23,"context_line":"    \"\"\"Check of cinder volume delete command.\"\"\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    @ddt.data("},{"line_number":26,"context_line":"        (\u0027\u0027, (r\u0027too few arguments\u0027)),"},{"line_number":27,"context_line":"        (\u00271234-1234-1234\u0027, (r\u0027No volume with a name or ID of\u0027)),"},{"line_number":28,"context_line":"        (\u0027my_volume\u0027, (r\u0027No volume with a name or ID of\u0027)),"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_aa55501e","line":25,"range":{"start_line":25,"start_character":9,"end_line":25,"end_character":13},"in_reply_to":"dada55a8_0ab61cdc","updated":"2016-07-18 13:20:14.000000000","message":"Looks like it is. \nAlso interesting to check when one volume ID is valid and another is not.","commit_id":"53b98451d1ea9b6f0f47ab8022ee01d93bec584c"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"a336b42fe558283f0eb0686640c93a5c41b2d668","unresolved":false,"context_lines":[{"line_number":22,"context_line":"class CinderVolumeDeleteNegativeTests(base.ClientTestBase):"},{"line_number":23,"context_line":"    \"\"\"Check of cinder volume delete command.\"\"\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    @ddt.data("},{"line_number":26,"context_line":"        (\u0027\u0027, (r\u0027too few arguments\u0027)),"},{"line_number":27,"context_line":"        (\u00271234-1234-1234\u0027, (r\u0027No volume with a name or ID of\u0027)),"},{"line_number":28,"context_line":"        (\u0027my_volume\u0027, (r\u0027No volume with a name or ID of\u0027)),"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_0ab61cdc","line":25,"range":{"start_line":25,"start_character":9,"end_line":25,"end_character":13},"in_reply_to":"dada55a8_6f740acf","updated":"2016-07-18 13:11:07.000000000","message":"Did you mean this?\n(\u00271234 1234\u0027, (r\u0027No volume with a name or ID of\u0027))","commit_id":"53b98451d1ea9b6f0f47ab8022ee01d93bec584c"}]}