)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"eb7a6acc55d106bc7f0322bd0cebc8ed2256935b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Added missing test coverage for run_command_and_log function"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I5136a7efb187e23542516ea647cf04309977d777"},{"line_number":10,"context_line":"Co-Author: Kamil Sambor \u003cksambor@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_8313d366","line":10,"updated":"2018-08-06 12:34:04.000000000","message":"Co-Authored-By (see https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references)","commit_id":"856ca3c45ed9574968f7ae421aa685c25c891082"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"3867b2450c0b1cd17752916fadf30db7e1a20eac","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Added missing test coverage for run_command_and_log function"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I5136a7efb187e23542516ea647cf04309977d777"},{"line_number":10,"context_line":"Co-Author: Kamil Sambor \u003cksambor@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_c62799b6","line":10,"in_reply_to":"3f79a3b5_8313d366","updated":"2018-08-06 12:37:39.000000000","message":"ah, thanks, wasn\u0027t sure, I was lacking that page - had only another one in bookmarks, and it didn\u0027t mention it...","commit_id":"856ca3c45ed9574968f7ae421aa685c25c891082"}],"tripleoclient/tests/test_utils.py":[{"author":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"change_message_id":"7cf7a4a0eda1eec3df4ee008285fefe63a146ba6","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        self.mock_process.wait.side_effect \u003d [0]"},{"line_number":173,"context_line":"        self.mock_process.returncode \u003d 0"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    @mock.patch(\u0027subprocess.Popen\u0027)"},{"line_number":176,"context_line":"    def test_success_default(self, mock_popen):"},{"line_number":177,"context_line":"        mock_popen.return_value \u003d self.mock_process"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_5aaa8385","line":175,"updated":"2018-08-06 07:27:27.000000000","message":"This mock can be move to setUp","commit_id":"8ff583a6a4f7d5547b8a7ff0f884e7a10822a981"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"75b43e1d56f47f77361d54b17b88d880675993c8","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        self.mock_process.wait.side_effect \u003d [0]"},{"line_number":173,"context_line":"        self.mock_process.returncode \u003d 0"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    @mock.patch(\u0027subprocess.Popen\u0027)"},{"line_number":176,"context_line":"    def test_success_default(self, mock_popen):"},{"line_number":177,"context_line":"        mock_popen.return_value \u003d self.mock_process"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_5a34c34a","line":175,"in_reply_to":"3f79a3b5_5aaa8385","updated":"2018-08-06 07:38:46.000000000","message":"I tried it, but apparently that kind of prevented the \"self.mock_popen.assert_called_once_with()\" check :/. Maybe I was doing it wrong. Not an expert at all on that field.","commit_id":"8ff583a6a4f7d5547b8a7ff0f884e7a10822a981"}]}