)]}'
{"nova/tests/unit/api/openstack/compute/contrib/test_hypervisors.py":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"74d41cb9d738c262e405a87f5cb7efeb567cc5c2","unresolved":false,"context_lines":[{"line_number":285,"context_line":"        ) as ("},{"line_number":286,"context_line":"            elevated_mock,"},{"line_number":287,"context_line":"            mock_service"},{"line_number":288,"context_line":"        ):"},{"line_number":289,"context_line":"            self.controller.show(req, self.TEST_HYPERS_OBJ[0].id)"},{"line_number":290,"context_line":"            mock_service.assert_called_once_with(elevated_context,"},{"line_number":291,"context_line":"                                                 self.TEST_HYPERS_OBJ[0].host)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_e6dc1c14","line":288,"updated":"2015-05-04 14:27:44.000000000","message":"Just a style note: this is really *ugly* indentation. Also, is there a reason you couldn\u0027t use the decorator mocks and avoid the nesting?","commit_id":"cbf606bdba621bbdf940fac7c8c3fbc40d0f12b8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"41ddc0718c6c0ef5d54c2fff482c83c20956e982","unresolved":false,"context_lines":[{"line_number":285,"context_line":"        ) as ("},{"line_number":286,"context_line":"            elevated_mock,"},{"line_number":287,"context_line":"            mock_service"},{"line_number":288,"context_line":"        ):"},{"line_number":289,"context_line":"            self.controller.show(req, self.TEST_HYPERS_OBJ[0].id)"},{"line_number":290,"context_line":"            mock_service.assert_called_once_with(elevated_context,"},{"line_number":291,"context_line":"                                                 self.TEST_HYPERS_OBJ[0].host)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aca2d91_37423069","line":288,"in_reply_to":"3acd31a7_e6dc1c14","updated":"2015-05-08 20:17:01.000000000","message":"Ed, this is a cherry pick to stable, this would have been a more useful comment on the change when it was under review on master.  We don\u0027t change the code in cherry picks though.","commit_id":"cbf606bdba621bbdf940fac7c8c3fbc40d0f12b8"}]}
