)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"57a7d5cbf6359f6352dec9734027c2d641f09989","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mike Durnosvistov \u003cmdurnosvistov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-10-03 16:55:51 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace custom patching in `setUp` on class-mock"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It would be better if we get rid of `setUp` method in which happens"},{"line_number":10,"context_line":"custom patching and replace it on class-mock."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa201ad_c296d24b","line":7,"updated":"2014-10-06 12:21:05.000000000","message":"I like the idea but please change the commit msg in order to be more explicit.\n\nI was thinking you were modifying all *classes* having setUp methods in there, not only one testcase in the API namespace.","commit_id":"7f5a58d1348705692c05b5a27949a9984ecfe7d1"},{"author":{"_account_id":8412,"name":"Mike Durnosvistov","email":"glacierr.dev@gmail.com","username":"mdurnosvistov"},"change_message_id":"cc33e98babdcb0c778a5baa30133afcbe4e275a1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mike Durnosvistov \u003cmdurnosvistov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-10-03 16:55:51 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace custom patching in `setUp` on class-mock"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It would be better if we get rid of `setUp` method in which happens"},{"line_number":10,"context_line":"custom patching and replace it on class-mock."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa201ad_82cf6a28","line":7,"in_reply_to":"baa201ad_c296d24b","updated":"2014-10-06 12:29:55.000000000","message":"Hmmm... I understood! I can add in commit message where I do it, but I think I cannot add it in the first line due to limit in 50 character.","commit_id":"7f5a58d1348705692c05b5a27949a9984ecfe7d1"}],"nova/tests/integrated/test_api_samples.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"e4d155507c91a51ec5b3b862ce21e881ea161083","unresolved":false,"context_lines":[{"line_number":3675,"context_line":"        super(HypervisorsSampleJsonTests, self).setUp()"},{"line_number":3676,"context_line":"        mock.patch(\"nova.servicegroup.API.service_is_up\","},{"line_number":3677,"context_line":"                   return_value\u003dTrue).start()"},{"line_number":3678,"context_line":"        self.addCleanup(mock.patch.stopall)"},{"line_number":3679,"context_line":""},{"line_number":3680,"context_line":"    def test_hypervisors_list(self):"},{"line_number":3681,"context_line":"        response \u003d self._do_get(\u0027os-hypervisors\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_3a27e1cb","side":"PARENT","line":3678,"updated":"2014-10-03 16:08:25.000000000","message":"what about this line?","commit_id":"42a8a48fd3671c6c8cf4085a112268722ed65a80"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"913b6bbf84227c86d295f032b460fd4eedebc5c0","unresolved":false,"context_lines":[{"line_number":3675,"context_line":"        super(HypervisorsSampleJsonTests, self).setUp()"},{"line_number":3676,"context_line":"        mock.patch(\"nova.servicegroup.API.service_is_up\","},{"line_number":3677,"context_line":"                   return_value\u003dTrue).start()"},{"line_number":3678,"context_line":"        self.addCleanup(mock.patch.stopall)"},{"line_number":3679,"context_line":""},{"line_number":3680,"context_line":"    def test_hypervisors_list(self):"},{"line_number":3681,"context_line":"        response \u003d self._do_get(\u0027os-hypervisors\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_5c1f296c","side":"PARENT","line":3678,"in_reply_to":"baa201ad_2a395041","updated":"2014-10-06 11:23:18.000000000","message":"My mistake.","commit_id":"42a8a48fd3671c6c8cf4085a112268722ed65a80"},{"author":{"_account_id":8412,"name":"Mike Durnosvistov","email":"glacierr.dev@gmail.com","username":"mdurnosvistov"},"change_message_id":"1ffad003fd7d7c24f3a69b75fe860926c3f9ab13","unresolved":false,"context_lines":[{"line_number":3675,"context_line":"        super(HypervisorsSampleJsonTests, self).setUp()"},{"line_number":3676,"context_line":"        mock.patch(\"nova.servicegroup.API.service_is_up\","},{"line_number":3677,"context_line":"                   return_value\u003dTrue).start()"},{"line_number":3678,"context_line":"        self.addCleanup(mock.patch.stopall)"},{"line_number":3679,"context_line":""},{"line_number":3680,"context_line":"    def test_hypervisors_list(self):"},{"line_number":3681,"context_line":"        response \u003d self._do_get(\u0027os-hypervisors\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_4afa1476","side":"PARENT","line":3678,"in_reply_to":"baa201ad_3a27e1cb","updated":"2014-10-03 18:02:23.000000000","message":"AFAIK mock after each test case itself clears variables... Or what you mean for this line?","commit_id":"42a8a48fd3671c6c8cf4085a112268722ed65a80"},{"author":{"_account_id":8412,"name":"Mike Durnosvistov","email":"glacierr.dev@gmail.com","username":"mdurnosvistov"},"change_message_id":"4000f33af06daabdc387cd505dc3456cd49d8032","unresolved":false,"context_lines":[{"line_number":3675,"context_line":"        super(HypervisorsSampleJsonTests, self).setUp()"},{"line_number":3676,"context_line":"        mock.patch(\"nova.servicegroup.API.service_is_up\","},{"line_number":3677,"context_line":"                   return_value\u003dTrue).start()"},{"line_number":3678,"context_line":"        self.addCleanup(mock.patch.stopall)"},{"line_number":3679,"context_line":""},{"line_number":3680,"context_line":"    def test_hypervisors_list(self):"},{"line_number":3681,"context_line":"        response \u003d self._do_get(\u0027os-hypervisors\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_2a395041","side":"PARENT","line":3678,"in_reply_to":"baa201ad_4afa1476","updated":"2014-10-03 18:03:20.000000000","message":"mock.patch!","commit_id":"42a8a48fd3671c6c8cf4085a112268722ed65a80"}]}