)]}'
{"ironicclient/tests/unit/common/test_http.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"633b2d009e37c916c4b564f35d2c2def5093f48c","unresolved":false,"context_lines":[{"line_number":217,"context_line":"                                  client.json_request,"},{"line_number":218,"context_line":"                                  \u0027GET\u0027, \u0027/v1/resources\u0027)"},{"line_number":219,"context_line":"        self.assertEqual((\u0027Internal Server Error (HTTP %s)\u0027"},{"line_number":220,"context_line":"                          % http_client.INTERNAL_SERVER_ERROR), str(error))"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def test_server_exception_msg_only(self):"},{"line_number":223,"context_line":"        error_msg \u003d \u0027test error msg\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_91c40a60","line":220,"updated":"2016-07-08 08:28:39.000000000","message":"I wonder if we should just check for:\n\nself.assertIn(\u0027Internal Server Error\u0027, str(error))","commit_id":"73b91f5b342a91b3a778d72f7b85482045446dfd"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"268304ae5b6d13eb1f252478d4343fdc4cc434d4","unresolved":false,"context_lines":[{"line_number":217,"context_line":"                                  client.json_request,"},{"line_number":218,"context_line":"                                  \u0027GET\u0027, \u0027/v1/resources\u0027)"},{"line_number":219,"context_line":"        self.assertEqual((\u0027Internal Server Error (HTTP %s)\u0027"},{"line_number":220,"context_line":"                          % http_client.INTERNAL_SERVER_ERROR), str(error))"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def test_server_exception_msg_only(self):"},{"line_number":223,"context_line":"        error_msg \u003d \u0027test error msg\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_acb8cd2b","line":220,"in_reply_to":"1aa78d24_71887ec7","updated":"2016-07-08 08:43:12.000000000","message":"sure, will do","commit_id":"73b91f5b342a91b3a778d72f7b85482045446dfd"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"98494707449390caf16dfac1a148e46d5f731482","unresolved":false,"context_lines":[{"line_number":217,"context_line":"                                  client.json_request,"},{"line_number":218,"context_line":"                                  \u0027GET\u0027, \u0027/v1/resources\u0027)"},{"line_number":219,"context_line":"        self.assertEqual((\u0027Internal Server Error (HTTP %s)\u0027"},{"line_number":220,"context_line":"                          % http_client.INTERNAL_SERVER_ERROR), str(error))"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def test_server_exception_msg_only(self):"},{"line_number":223,"context_line":"        error_msg \u003d \u0027test error msg\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_b1766604","line":220,"in_reply_to":"1aa78d24_91c40a60","updated":"2016-07-08 08:34:09.000000000","message":"maybe, I\u0027m not a fun of checking error\u0027 strings at all, If I had my way I would remove them :)\n\nexception raised check is pretty enough","commit_id":"73b91f5b342a91b3a778d72f7b85482045446dfd"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"331246234f5a8fe0426b470b2458755af08414a1","unresolved":false,"context_lines":[{"line_number":217,"context_line":"                                  client.json_request,"},{"line_number":218,"context_line":"                                  \u0027GET\u0027, \u0027/v1/resources\u0027)"},{"line_number":219,"context_line":"        self.assertEqual((\u0027Internal Server Error (HTTP %s)\u0027"},{"line_number":220,"context_line":"                          % http_client.INTERNAL_SERVER_ERROR), str(error))"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def test_server_exception_msg_only(self):"},{"line_number":223,"context_line":"        error_msg \u003d \u0027test error msg\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_71887ec7","line":220,"in_reply_to":"1aa78d24_b1766604","updated":"2016-07-08 08:36:58.000000000","message":"Agreed, let\u0027s remove it then?","commit_id":"73b91f5b342a91b3a778d72f7b85482045446dfd"}]}