)]}'
{"keystone/tests/unit/test_v3_federation.py":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"13ccd3fb316dde84eb78c09e621623bff973546c","unresolved":false,"context_lines":[{"line_number":4052,"context_line":"                         \"returned non-zero exit status %s. %s\\n\" %"},{"line_number":4053,"context_line":"                         (CONF.saml.xmlsec1_binary, sample_returncode,"},{"line_number":4054,"context_line":"                          sample_output))]  # \u003d\u003d py3.6"},{"line_number":4055,"context_line":"        self.assertIn(logger_fixture.output, expected_log)"},{"line_number":4056,"context_line":""},{"line_number":4057,"context_line":"    @mock.patch(\u0027oslo_utils.fileutils.write_to_tempfile\u0027)"},{"line_number":4058,"context_line":"    def test_sign_assertion_fileutils_exc(self, write_to_tempfile_mock):"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_5b394aef","line":4055,"updated":"2018-03-16 08:46:36.000000000","message":"Could we use a regex match to allow for \\.? instead of copying the whole line?","commit_id":"859dc3ed1d622a8668c37e3fb17c13eb210ce3f6"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"89e7280bfb3d84c6395f2d5a110dee31aea61e4f","unresolved":false,"context_lines":[{"line_number":4052,"context_line":"                         \"returned non-zero exit status %s. %s\\n\" %"},{"line_number":4053,"context_line":"                         (CONF.saml.xmlsec1_binary, sample_returncode,"},{"line_number":4054,"context_line":"                          sample_output))]  # \u003d\u003d py3.6"},{"line_number":4055,"context_line":"        self.assertIn(logger_fixture.output, expected_log)"},{"line_number":4056,"context_line":""},{"line_number":4057,"context_line":"    @mock.patch(\u0027oslo_utils.fileutils.write_to_tempfile\u0027)"},{"line_number":4058,"context_line":"    def test_sign_assertion_fileutils_exc(self, write_to_tempfile_mock):"}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_bb1ae56b","line":4055,"in_reply_to":"df7087c5_5b394aef","updated":"2018-03-16 08:50:24.000000000","message":"sure, it\u0027s better","commit_id":"859dc3ed1d622a8668c37e3fb17c13eb210ce3f6"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"371063dd1fcbc06f47d39c1829778aa1cabbd44f","unresolved":false,"context_lines":[{"line_number":4044,"context_line":"                          keystone_idp._sign_assertion,"},{"line_number":4045,"context_line":"                          self.signed_assertion)"},{"line_number":4046,"context_line":"        # The function __str__ in subprocess.CalledProcessError is different"},{"line_number":4047,"context_line":"        # between py3.6 and lower python version."},{"line_number":4048,"context_line":"        expected_log \u003d ("},{"line_number":4049,"context_line":"            \"Error when signing assertion, reason: Command \u0027%s\u0027 returned \""},{"line_number":4050,"context_line":"            \"non-zero exit status %s\\.? %s\\n\" %"}],"source_content_type":"text/x-python","patch_set":3,"id":"df7087c5_81a3f0b1","line":4047,"updated":"2018-03-16 10:21:32.000000000","message":"If there\u0027s a note in a changelog somewhere it might be nice to reference it here or in the commit message","commit_id":"4a1df081fd3868a75bfabe8099025069eb42248f"}]}