)]}'
{"tests/test_script.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ac23ec4a0ca6b43287b1e80c511dd84d2425e369","unresolved":false,"context_lines":[{"line_number":24,"context_line":"            jenkins_mock.call_args[0][0].get_full_url(),"},{"line_number":25,"context_line":"            self.make_url(\u0027scriptText\u0027))"},{"line_number":26,"context_line":"        self.assertTrue(quote(\u0027\u0026\u0026\u0027) in jenkins_mock.call_args[0][0].data.decode(\u0027utf8\u0027),"},{"line_number":27,"context_line":"                        jenkins_mock.call_args[0][0].data)"},{"line_number":28,"context_line":"        self._check_requests(jenkins_mock.call_args_list)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @patch.object(jenkins.Jenkins, \u0027jenkins_open\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a6eadb0_de8f7921","line":27,"updated":"2016-12-13 17:43:02.000000000","message":"This doesn\u0027t smell right; if the assert fails, it outputs .data as the error message, which, while perhaps useful, feels like it could be a bit confusing.  Also, there\u0027s the whole \u0027in\u0027 test inside an assertTrue(); with the way we\u0027re constructing JenkinsTestBase, we should have access to the assertIn() method here, which would probably be the superior way to go.","commit_id":"4cecde2b83fd7609908cd9fd4f43504e842643fb"}]}