)]}'
{"cinderclient/tests/unit/v2/test_capabilities.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"cacb24dc2f486df866f83e7fb8836964cd9a4d74","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        self.assertEqual(FAKE_CAPABILITY, capabilities._info)"},{"line_number":41,"context_line":"        self._assert_request_id(capabilities)"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def test___repr__(self):"},{"line_number":44,"context_line":"        \"\"\""},{"line_number":45,"context_line":"        Unit test for Capabilities.__repr__"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_747847f9","line":43,"updated":"2016-05-24 18:37:19.000000000","message":"Does any actual cinderclient code or test code ever call repr() on this object?\n\nI agree that repr() should work, but this is a rather odd test to add IMO.","commit_id":"87e3ad9057a4aedf2dbfb958d2a70d5bafd49a8a"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"14058acd7b9630c5a296f00c5e4ae90ccbddc007","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        \u0027compression\u0027: {"},{"line_number":30,"context_line":"            \u0027title\u0027: \u0027Compression\u0027,"},{"line_number":31,"context_line":"            \u0027description\u0027: \u0027Enables compression.\u0027,"},{"line_number":32,"context_line":"            \u0027type\u0027: \u0027boolean\u0027}}}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class CapabilitiesTest(utils.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_6b350fd9","line":32,"range":{"start_line":32,"start_character":12,"end_line":32,"end_character":32},"updated":"2016-06-22 11:10:47.000000000","message":"nit: It\u0027s usually better to end this like\n\n             \u0027type\u0027: \u0027boolean\u0027,\n         },\n     },\n }\n\nThat way if something gets added the diff is limited to what gets added.","commit_id":"f71af7c809483716f744dab42db11849bafa8371"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"14058acd7b9630c5a296f00c5e4ae90ccbddc007","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        \"\"\""},{"line_number":45,"context_line":"        Unit test for Capabilities.__repr__"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        Verify that one Capabilities object can be printed."},{"line_number":48,"context_line":"        \"\"\""},{"line_number":49,"context_line":"        cap \u003d Capabilities(None, FAKE_CAPABILITY)"},{"line_number":50,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_a8189d85","line":47,"range":{"start_line":47,"start_character":20,"end_line":47,"end_character":23},"updated":"2016-06-22 11:10:47.000000000","message":"nit: Remove \"one\"","commit_id":"f71af7c809483716f744dab42db11849bafa8371"}],"cinderclient/v3/capabilities.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"316b37ed0d587b5113b27e456db182b4ca5e0625","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class Capabilities(base.Resource):"},{"line_number":23,"context_line":"    NAME_ATTR \u003d \u0027name\u0027"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def __repr__(self):"},{"line_number":26,"context_line":"        return \"\u003cCapabilities\u003e\""}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_77e2ed30","line":23,"range":{"start_line":23,"start_character":4,"end_line":23,"end_character":22},"updated":"2016-05-25 09:09:28.000000000","message":"This should be removed as well.","commit_id":"87e3ad9057a4aedf2dbfb958d2a70d5bafd49a8a"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"316b37ed0d587b5113b27e456db182b4ca5e0625","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    NAME_ATTR \u003d \u0027name\u0027"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    def __repr__(self):"},{"line_number":26,"context_line":"        return \"\u003cCapabilities\u003e\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class CapabilitiesManager(base.Manager):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_97e7913f","line":26,"range":{"start_line":26,"start_character":8,"end_line":26,"end_character":31},"updated":"2016-05-25 09:09:28.000000000","message":"I\u0027d rather we actually used display_name as mentioned in my general comment, but if we are going to return this we should just remove the method.","commit_id":"87e3ad9057a4aedf2dbfb958d2a70d5bafd49a8a"}]}