)]}'
{"jenkins/__init__.py":[{"author":{"_account_id":6987,"name":"Khai Do","email":"zaro0508@gmail.com","username":"zaro0508"},"change_message_id":"48431b7cee9ac1d405e25c56667740f2414d648f","unresolved":false,"context_lines":[{"line_number":351,"context_line":"                self._build_url(JOB_INFO, locals())"},{"line_number":352,"context_line":"            ))"},{"line_number":353,"context_line":"            if response:"},{"line_number":354,"context_line":"                return self._add_missing_builds(json.loads(response))"},{"line_number":355,"context_line":"            else:"},{"line_number":356,"context_line":"                raise JenkinsException(\u0027job[%s] does not exist\u0027 % name)"},{"line_number":357,"context_line":"        except HTTPError:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_75f6c4d7","line":354,"updated":"2016-04-29 01:44:24.000000000","message":"I\u0027m wondering about performance.  My concern here is that it might become slow to load if there are many builds.  I don\u0027t have a setup to test that but if you do would you be able to give us some ballpark metrics?  If it\u0027s slow it might be good to pass in an option to add the missing builds.","commit_id":"a8b57c0885874dd6fa94fa25ed14671e01d74dcf"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"ddc094a64b783068e6fa564c82f197764a3d11f7","unresolved":false,"context_lines":[{"line_number":351,"context_line":"                self._build_url(JOB_INFO, locals())"},{"line_number":352,"context_line":"            ))"},{"line_number":353,"context_line":"            if response:"},{"line_number":354,"context_line":"                return self._add_missing_builds(json.loads(response))"},{"line_number":355,"context_line":"            else:"},{"line_number":356,"context_line":"                raise JenkinsException(\u0027job[%s] does not exist\u0027 % name)"},{"line_number":357,"context_line":"        except HTTPError:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_c1d43048","line":354,"in_reply_to":"1a122d0e_75f6c4d7","updated":"2016-04-29 14:24:51.000000000","message":"Done","commit_id":"a8b57c0885874dd6fa94fa25ed14671e01d74dcf"}],"tests/jobs/test_info.py":[{"author":{"_account_id":6987,"name":"Khai Do","email":"zaro0508@gmail.com","username":"zaro0508"},"change_message_id":"48431b7cee9ac1d405e25c56667740f2414d648f","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        self._check_requests(jenkins_mock.call_args_list)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @patch.object(jenkins.Jenkins, \u0027jenkins_open\u0027)"},{"line_number":29,"context_line":"    def test_all_builds(self, jenkins_mock):"},{"line_number":30,"context_line":"        job_info_to_return \u003d {"},{"line_number":31,"context_line":"            u\u0027building\u0027: False,"},{"line_number":32,"context_line":"            u\u0027msg\u0027: u\u0027test\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_357d6c6d","line":29,"updated":"2016-04-29 01:44:24.000000000","message":"I think a better home for this test is in \u0027tests/test_build.py\u0027","commit_id":"a8b57c0885874dd6fa94fa25ed14671e01d74dcf"},{"author":{"_account_id":6987,"name":"Khai Do","email":"zaro0508@gmail.com","username":"zaro0508"},"change_message_id":"8b8040263a8c97394e6ff1c3d3ceb2cd8953f4b8","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        self._check_requests(jenkins_mock.call_args_list)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @patch.object(jenkins.Jenkins, \u0027jenkins_open\u0027)"},{"line_number":29,"context_line":"    def test_all_builds(self, jenkins_mock):"},{"line_number":30,"context_line":"        job_info_to_return \u003d {"},{"line_number":31,"context_line":"            u\u0027building\u0027: False,"},{"line_number":32,"context_line":"            u\u0027msg\u0027: u\u0027test\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_1de117ff","line":29,"in_reply_to":"1a122d0e_018468d0","updated":"2016-05-19 05:19:51.000000000","message":"you are correct. sorry about that.","commit_id":"a8b57c0885874dd6fa94fa25ed14671e01d74dcf"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"ddc094a64b783068e6fa564c82f197764a3d11f7","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        self._check_requests(jenkins_mock.call_args_list)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @patch.object(jenkins.Jenkins, \u0027jenkins_open\u0027)"},{"line_number":29,"context_line":"    def test_all_builds(self, jenkins_mock):"},{"line_number":30,"context_line":"        job_info_to_return \u003d {"},{"line_number":31,"context_line":"            u\u0027building\u0027: False,"},{"line_number":32,"context_line":"            u\u0027msg\u0027: u\u0027test\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_018468d0","line":29,"in_reply_to":"1a122d0e_357d6c6d","updated":"2016-04-29 14:24:51.000000000","message":"I disagree; this is testing the get_job_info() call, and should go with the other tests for that unit.","commit_id":"a8b57c0885874dd6fa94fa25ed14671e01d74dcf"}]}