)]}'
{"jenkins/__init__.py":[{"author":{"_account_id":15362,"name":"fdegir","display_name":"Fatih Degirmenci","email":"fdegir@gmail.com","username":"fdegir"},"change_message_id":"1dd3a6c08994d9a00d6e06f6aee8d3c8b90b16e6","unresolved":false,"context_lines":[{"line_number":419,"context_line":"        Example::"},{"line_number":420,"context_line":"            \u003e\u003e\u003e j \u003d Jenkins()"},{"line_number":421,"context_line":"            \u003e\u003e\u003e last_build \u003d j.is_building(\u0027TestJob\u0027)"},{"line_number":422,"context_line":"        \u0027\u0027\u0027"},{"line_number":423,"context_line":"        return self.get_last_build_info(name)[\u0027building\u0027]"},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"    def get_queue_info(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a41bdd9_633fa324","line":422,"updated":"2015-07-14 22:47:19.000000000","message":"would be good to handle the exception here for the requests on the jobs without any builds.\n\nor perhaps this can be neglected if the exception(s) are handled in get_last_build_info() properly.","commit_id":"faf1580f79a303c21c36af58e5964c3a418083f4"},{"author":{"_account_id":6987,"name":"Khai Do","email":"zaro0508@gmail.com","username":"zaro0508"},"change_message_id":"21f8df956c7994ef9f9115024b65d7f6a9ca946d","unresolved":false,"context_lines":[{"line_number":419,"context_line":"        Example::"},{"line_number":420,"context_line":"            \u003e\u003e\u003e j \u003d Jenkins()"},{"line_number":421,"context_line":"            \u003e\u003e\u003e last_build \u003d j.is_building(\u0027TestJob\u0027)"},{"line_number":422,"context_line":"        \u0027\u0027\u0027"},{"line_number":423,"context_line":"        return self.get_last_build_info(name)[\u0027building\u0027]"},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"    def get_queue_info(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a41bdd9_ffe472ac","line":422,"in_reply_to":"9a41bdd9_633fa324","updated":"2015-07-15 17:31:37.000000000","message":"I believe all request exceptions are handled in get_last_build_info() and jenkins_open() methods.","commit_id":"faf1580f79a303c21c36af58e5964c3a418083f4"}],"tests/test_jenkins.py":[{"author":{"_account_id":15362,"name":"fdegir","display_name":"Fatih Degirmenci","email":"fdegir@gmail.com","username":"fdegir"},"change_message_id":"1dd3a6c08994d9a00d6e06f6aee8d3c8b90b16e6","unresolved":false,"context_lines":[{"line_number":828,"context_line":"    @patch.object(jenkins.Jenkins, \u0027jenkins_open\u0027)"},{"line_number":829,"context_line":"    def test_is_building(self, jenkins_mock):"},{"line_number":830,"context_line":"        build_info_to_return \u003d {"},{"line_number":831,"context_line":"            \"building\": False,"},{"line_number":832,"context_line":"        }"},{"line_number":833,"context_line":"        jenkins_mock.return_value \u003d json.dumps(build_info_to_return)"},{"line_number":834,"context_line":"        j \u003d jenkins.Jenkins(\u0027http://example.com/\u0027, \u0027test\u0027, \u0027test\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a41bdd9_6326c316","line":831,"updated":"2015-07-14 22:47:19.000000000","message":"would it be possible to test other case as well?","commit_id":"faf1580f79a303c21c36af58e5964c3a418083f4"},{"author":{"_account_id":6987,"name":"Khai Do","email":"zaro0508@gmail.com","username":"zaro0508"},"change_message_id":"21f8df956c7994ef9f9115024b65d7f6a9ca946d","unresolved":false,"context_lines":[{"line_number":828,"context_line":"    @patch.object(jenkins.Jenkins, \u0027jenkins_open\u0027)"},{"line_number":829,"context_line":"    def test_is_building(self, jenkins_mock):"},{"line_number":830,"context_line":"        build_info_to_return \u003d {"},{"line_number":831,"context_line":"            \"building\": False,"},{"line_number":832,"context_line":"        }"},{"line_number":833,"context_line":"        jenkins_mock.return_value \u003d json.dumps(build_info_to_return)"},{"line_number":834,"context_line":"        j \u003d jenkins.Jenkins(\u0027http://example.com/\u0027, \u0027test\u0027, \u0027test\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a41bdd9_3a99887c","line":831,"in_reply_to":"9a41bdd9_6326c316","updated":"2015-07-15 17:31:37.000000000","message":"I could be wrong but I don\u0027t believe that is necessary since the method only contains a single line.  That logic is in get_last_build_info() method and i believe there is already tests for that.","commit_id":"faf1580f79a303c21c36af58e5964c3a418083f4"}]}