)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"6a460cc15cf3797f3a915e0e792ad56a0c8a1a99","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Also, added a test to check the branch where _() was being"},{"line_number":10,"context_line":"used:"},{"line_number":11,"context_line":"https://github.com/openstack/keystone/blob/master/keystone/contrib/federation/controllers.py#L279"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id5b957ad6d8d3a85a614d1bd55e2c0dadd7dbd04"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a890539_86dfa0d5","line":11,"updated":"2014-11-21 17:50:21.000000000","message":"It\u0027s nice to have a clickable link, but this one is too long :)","commit_id":"32da628b35adfcf415f173d837fc8c72eb5b1529"}],"keystone/tests/test_v3_federation.py":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"20457d82e08b7a207addb50a08389cc584d645c3","unresolved":false,"context_lines":[{"line_number":1830,"context_line":"        not scoped by project."},{"line_number":1831,"context_line":""},{"line_number":1832,"context_line":"        \"\"\""},{"line_number":1833,"context_line":"        CONF.saml.idp_entity_id \u003d self.ISSUER"},{"line_number":1834,"context_line":"        region_id \u003d self._create_region_with_url()"},{"line_number":1835,"context_line":"        token_id \u003d self._fetch_domain_scoped_token()"},{"line_number":1836,"context_line":"        body \u003d self._create_generate_saml_request(token_id, region_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_d909942d","line":1833,"updated":"2014-11-21 14:05:17.000000000","message":"Seems like you would want to do this with the config fixture.","commit_id":"32da628b35adfcf415f173d837fc8c72eb5b1529"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"b803f153a9a6089117a3e7bdbd43640325d0661f","unresolved":false,"context_lines":[{"line_number":1830,"context_line":"        not scoped by project."},{"line_number":1831,"context_line":""},{"line_number":1832,"context_line":"        \"\"\""},{"line_number":1833,"context_line":"        CONF.saml.idp_entity_id \u003d self.ISSUER"},{"line_number":1834,"context_line":"        region_id \u003d self._create_region_with_url()"},{"line_number":1835,"context_line":"        token_id \u003d self._fetch_domain_scoped_token()"},{"line_number":1836,"context_line":"        body \u003d self._create_generate_saml_request(token_id, region_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_aa1b847e","line":1833,"in_reply_to":"5a890539_79b42035","updated":"2014-11-21 15:38:18.000000000","message":"Yes, these need to be changed to use config fixture... I\u0027m fine with putting that in a separate commit.","commit_id":"32da628b35adfcf415f173d837fc8c72eb5b1529"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"edc48ddcbd5f3b8ed603c58a7811b830a4b49921","unresolved":false,"context_lines":[{"line_number":1830,"context_line":"        not scoped by project."},{"line_number":1831,"context_line":""},{"line_number":1832,"context_line":"        \"\"\""},{"line_number":1833,"context_line":"        CONF.saml.idp_entity_id \u003d self.ISSUER"},{"line_number":1834,"context_line":"        region_id \u003d self._create_region_with_url()"},{"line_number":1835,"context_line":"        token_id \u003d self._fetch_domain_scoped_token()"},{"line_number":1836,"context_line":"        body \u003d self._create_generate_saml_request(token_id, region_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_054bab2b","line":1833,"in_reply_to":"5a890539_aa1b847e","updated":"2014-11-21 15:46:24.000000000","message":"Actually, there\u0027s only 2 other tests in this class that don\u0027t use the config fixture properly. Here\u0027s the fix: https://review.openstack.org/136399","commit_id":"32da628b35adfcf415f173d837fc8c72eb5b1529"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"24bdfae4a1a1f42cfc30fa7be46eb5f35b70489d","unresolved":false,"context_lines":[{"line_number":1830,"context_line":"        not scoped by project."},{"line_number":1831,"context_line":""},{"line_number":1832,"context_line":"        \"\"\""},{"line_number":1833,"context_line":"        CONF.saml.idp_entity_id \u003d self.ISSUER"},{"line_number":1834,"context_line":"        region_id \u003d self._create_region_with_url()"},{"line_number":1835,"context_line":"        token_id \u003d self._fetch_domain_scoped_token()"},{"line_number":1836,"context_line":"        body \u003d self._create_generate_saml_request(token_id, region_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_79b42035","line":1833,"in_reply_to":"5a890539_d909942d","updated":"2014-11-21 14:09:50.000000000","message":"yeah, I think it would be the ideal, but just followed the pattern from all the other tests in this class.","commit_id":"32da628b35adfcf415f173d837fc8c72eb5b1529"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"b803f153a9a6089117a3e7bdbd43640325d0661f","unresolved":false,"context_lines":[{"line_number":1838,"context_line":"        with mock.patch.object(keystone_idp, \u0027_sign_assertion\u0027,"},{"line_number":1839,"context_line":"                               return_value\u003dself.signed_assertion):"},{"line_number":1840,"context_line":"            self.post(self.SAML_GENERATION_ROUTE, body\u003dbody,"},{"line_number":1841,"context_line":"                      expected_status\u003d500)"},{"line_number":1842,"context_line":""},{"line_number":1843,"context_line":"    def test_generate_saml_route(self):"},{"line_number":1844,"context_line":"        \"\"\"Test that the SAML generation endpoint produces XML."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_e50cdf3d","line":1841,"updated":"2014-11-21 15:38:18.000000000","message":"The server shouldn\u0027t return a 500 response for anything that the client does. Add a bug # so we can track it.","commit_id":"32da628b35adfcf415f173d837fc8c72eb5b1529"}]}