)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"aa7bb3896929c1005c77470b83ecd7f472ba486f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Harshada Mangesh Kakad \u003charshada.kakad@izeltech.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-11-21 01:28:31 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Set info \u003d _parse_driver_info(self.node) in setup rather than testcases"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In test_seamicro.py file almost all the testcases is using local"},{"line_number":10,"context_line":"variable info \u003d _parse_driver_info(self.node). Ideally it should"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a890539_ebdd9b43","line":7,"updated":"2014-11-21 10:51:00.000000000","message":"This is very confusing. Maybe something less \"cody\".\n\n\"Do not call _parse_driver_info on every test\"\n\nSomething like that","commit_id":"4d9fdeda543eea86c5554eee6fd1716f253925d1"},{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"0551884e0408afb386c5760314e9bbc96ec129d9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Avoid calling _parse_driver_info in every test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In test_seamicro.py file almost all the testcases is calling"},{"line_number":10,"context_line":"_parse_driver_info. Ideally it should be set in setup only once."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic7daa4e8daec891380b1ead6824214abff28fbb6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a890539_f9962cef","line":9,"updated":"2014-11-24 08:12:58.000000000","message":"s/is/are ?","commit_id":"728fa5d54eb89512ed65cf497108320b687ea870"}],"ironic/tests/drivers/test_seamicro.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"aa7bb3896929c1005c77470b83ecd7f472ba486f","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        self.freeSize \u003d freeSize"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"class SeaMicroValidateParametersTestCase(db_base.DbTestCase):"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def test__parse_driver_info_good(self):"},{"line_number":83,"context_line":"        # make sure we get back the expected things"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_8b4417b8","line":80,"updated":"2014-11-21 10:51:00.000000000","message":"good I see the _parse_driver_info being tested here","commit_id":"4d9fdeda543eea86c5554eee6fd1716f253925d1"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"aa7bb3896929c1005c77470b83ecd7f472ba486f","unresolved":false,"context_lines":[{"line_number":404,"context_line":"                             mock_info.call_count)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"    @mock.patch.object(seamicro, \u0027_parse_driver_info\u0027)"},{"line_number":407,"context_line":"    def test_vendor_passthru_validate_parse_driver_info_fail(self, mock_info):"},{"line_number":408,"context_line":"        mock_info.side_effect \u003d exception.InvalidParameterValue(\"bad\")"},{"line_number":409,"context_line":"        with task_manager.acquire(self.context, self.node[\u0027uuid\u0027],"},{"line_number":410,"context_line":"                                  shared\u003dTrue) as task:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_cb07ff7f","line":407,"updated":"2014-11-21 10:51:00.000000000","message":"was confused whether this would fail at SetUp() but seems fine.","commit_id":"4d9fdeda543eea86c5554eee6fd1716f253925d1"}]}