)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"b7fdcf11253e3fa2240433fd38f7c2eef56a93e9","unresolved":false,"context_lines":[{"line_number":11,"context_line":"enforcement lookups. However, the API was not catching them for the"},{"line_number":12,"context_line":"show and list operations."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch adds the decorators to the two methods to catch any that"},{"line_number":15,"context_line":"may be emitted from the policy engine or whereever else."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: #1528031"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a5ae5dd_a4c29c32","line":14,"updated":"2016-02-05 04:30:46.000000000","message":"any exception","commit_id":"73f88708ce133f1e0e1cfab20ccd6dfd825cb5b9"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"7a3f0499e85d34c638867fd63c3e3fa4ce5e60b1","unresolved":false,"context_lines":[{"line_number":12,"context_line":"show and list operations."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch adds the decorators to the two methods to catch any that"},{"line_number":15,"context_line":"may be emitted from the policy engine or whereever else."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: #1528031"},{"line_number":18,"context_line":"Change-Id: If4aea5245cdbb2ea545e9a96d73386e3c21a3696"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a5ae5dd_2439ccf2","line":15,"updated":"2016-02-05 04:49:38.000000000","message":"Done","commit_id":"73f88708ce133f1e0e1cfab20ccd6dfd825cb5b9"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"b7fdcf11253e3fa2240433fd38f7c2eef56a93e9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"show and list operations."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch adds the decorators to the two methods to catch any that"},{"line_number":15,"context_line":"may be emitted from the policy engine or whereever else."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: #1528031"},{"line_number":18,"context_line":"Change-Id: If4aea5245cdbb2ea545e9a96d73386e3c21a3696"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a5ae5dd_c4a8f879","line":15,"updated":"2016-02-05 04:30:46.000000000","message":"nit: wherever :)","commit_id":"73f88708ce133f1e0e1cfab20ccd6dfd825cb5b9"}],"neutron/tests/unit/api/v2/test_base.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b2c3779da2a7641937f996ddb2e91646dc57298f","unresolved":false,"context_lines":[{"line_number":1118,"context_line":"        instance \u003d self.plugin.return_value"},{"line_number":1119,"context_line":"        instance.get_networks.side_effect \u003d [db_exc.RetryRequest(None), []]"},{"line_number":1120,"context_line":"        api \u003d webtest.TestApp(router.APIRouter())"},{"line_number":1121,"context_line":"        api.get(_get_path(\u0027networks\u0027, fmt\u003dself.fmt))"},{"line_number":1122,"context_line":""},{"line_number":1123,"context_line":"    def test_retry_on_show(self):"},{"line_number":1124,"context_line":"        instance \u003d self.plugin.return_value"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_264b30b2","line":1121,"updated":"2016-02-04 15:39:51.000000000","message":"You need some assert here, right?\n\ninstance.get_networks.assert_called_once_with(...)","commit_id":"94e90603e118d38efa06588679b35d7cda56b7ea"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"8a0e8ade44b84740b25b3a26845277d2e653b1db","unresolved":false,"context_lines":[{"line_number":1118,"context_line":"        instance \u003d self.plugin.return_value"},{"line_number":1119,"context_line":"        instance.get_networks.side_effect \u003d [db_exc.RetryRequest(None), []]"},{"line_number":1120,"context_line":"        api \u003d webtest.TestApp(router.APIRouter())"},{"line_number":1121,"context_line":"        api.get(_get_path(\u0027networks\u0027, fmt\u003dself.fmt))"},{"line_number":1122,"context_line":""},{"line_number":1123,"context_line":"    def test_retry_on_show(self):"},{"line_number":1124,"context_line":"        instance \u003d self.plugin.return_value"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_9ee6dab2","line":1121,"in_reply_to":"5a5ae5dd_264b30b2","updated":"2016-02-04 22:05:14.000000000","message":"This test is just making sure no exceptions are thrown, but an assert can help make sure that the side effect is being triggered.","commit_id":"94e90603e118d38efa06588679b35d7cda56b7ea"}]}