)]}'
{"nova/tests/volume/test_cinder.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"54d4aaaf0dc0e43358b12acd5d1d820b984e1cf6","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from nova import test"},{"line_number":23,"context_line":"from nova.volume import cinder"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"CTX \u003d context.get_admin_context()"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class FakeCinderClient(object):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOH%2F%2FuE8%3D","line":25,"updated":"2013-04-01 18:50:52.000000000","message":"Put this into a setUp() method in the test class and assign it to an instance variable.  I\u0027m worried about the test suite contending on this object, since it\u0027s a little more than a traditional constant…","commit_id":"2d0ea23a85c81fe1178f6b06d4b7089003c1d964"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"b37dcb6c6e7c7852e160e813d3064ea6b6006b62","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from nova import test"},{"line_number":23,"context_line":"from nova.volume import cinder"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"CTX \u003d context.get_admin_context()"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class FakeCinderClient(object):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOH%2F%2FtJE%3D","line":25,"in_reply_to":"AAAAOH%2F%2FuE8%3D","updated":"2013-04-02 08:30:23.000000000","message":"Agree, thanks","commit_id":"2d0ea23a85c81fe1178f6b06d4b7089003c1d964"},{"author":{"_account_id":191,"name":"Trey Morris","email":"trey.morris@rackspace.com","username":"tr3buchet"},"change_message_id":"bbc0e339439f5e976d110988e397e725dc553b75","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        self.assertEqual([\u0027id1\u0027, \u0027id2\u0027], self.api.get_all(CTX))"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_check_attach(self):"},{"line_number":101,"context_line":"        volume \u003d {\u0027status\u0027: \u0027error\u0027}"},{"line_number":102,"context_line":"        self.assertRaises(exception.InvalidVolume,"},{"line_number":103,"context_line":"                          self.api.check_attach, CTX, volume)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOH%2F%2FtkE%3D","line":100,"updated":"2013-04-01 23:37:03.000000000","message":"might want to break this test in multiple tests","commit_id":"2d0ea23a85c81fe1178f6b06d4b7089003c1d964"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"b37dcb6c6e7c7852e160e813d3064ea6b6006b62","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        self.assertEqual([\u0027id1\u0027, \u0027id2\u0027], self.api.get_all(CTX))"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_check_attach(self):"},{"line_number":101,"context_line":"        volume \u003d {\u0027status\u0027: \u0027error\u0027}"},{"line_number":102,"context_line":"        self.assertRaises(exception.InvalidVolume,"},{"line_number":103,"context_line":"                          self.api.check_attach, CTX, volume)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOH%2F%2FtI4%3D","line":100,"in_reply_to":"AAAAOH%2F%2FtkE%3D","updated":"2013-04-02 08:30:23.000000000","message":"Sure","commit_id":"2d0ea23a85c81fe1178f6b06d4b7089003c1d964"}]}