)]}'
{"keystoneauth1/tests/unit/extras/saml2/test_auth_saml2.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"020d751e23eda6aa640c4fc2c0c67a992f60ec92","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        # Since we have included a random Accept header, the plugin should have"},{"line_number":70,"context_line":"        # added the PAOS_HEADER to it using the correct media type separator"},{"line_number":71,"context_line":"        accept_header \u003d plugin_headers[\u0027Accept\u0027]"},{"line_number":72,"context_line":"        self.assertIn(self.HEADER_MEDIA_TYPE_SEPARATOR, accept_header)"},{"line_number":73,"context_line":"        self.assertIn(random_header,"},{"line_number":74,"context_line":"                      accept_header.split(self.HEADER_MEDIA_TYPE_SEPARATOR))"},{"line_number":75,"context_line":"        self.assertIn(PAOS_HEADER,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a3c09a3_d388df73","line":72,"updated":"2017-01-17 15:27:25.000000000","message":"The only suggestion I can make on this patch would be to build the expected output from the sum of the parts that we have [0]. I think this makes it a bit easier to understand from a reviewer perspective and when the test fails [1].\n\nOtherwise I think this is great!\n\n[0] http://cdn.pasteraw.com/brsu7spxxoqky9991yo7s558yv54p2x\n[1] http://cdn.pasteraw.com/561j1919smema8xbh0q814zfnxyz6h0","commit_id":"01e0122a14c9a61a50038df67c008455f6cffd90"}]}