)]}'
{"keystone/tests/test_v3_auth.py":[{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"200246fd8b941f13658ba0ad2e89ed0524d6adb5","unresolved":false,"context_lines":[{"line_number":3019,"context_line":""},{"line_number":3020,"context_line":"    def test_trust_chained(self):"},{"line_number":3021,"context_line":"        # Test that a trust token can\u0027t be used to execute another trust.  To"},{"line_number":3022,"context_line":"        # to this, we create an A-\u003eB-\u003eC hierarchy of trusts, then attempt to"},{"line_number":3023,"context_line":"        # execute the trusts in series (C-\u003eB-\u003eA)."},{"line_number":3024,"context_line":""},{"line_number":3025,"context_line":"        # create a sub-trustee user"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_76dfcc3a","line":3022,"updated":"2014-07-23 23:06:57.000000000","message":"nit: s/to/do/","commit_id":"f83a8ce9eca018b9c31c48d602e0822752ad217c"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"02e14eb914f3ff5c33165b5bcb29f2b305a51590","unresolved":false,"context_lines":[{"line_number":3019,"context_line":""},{"line_number":3020,"context_line":"    def test_trust_chained(self):"},{"line_number":3021,"context_line":"        # Test that a trust token can\u0027t be used to execute another trust.  To"},{"line_number":3022,"context_line":"        # to this, we create an A-\u003eB-\u003eC hierarchy of trusts, then attempt to"},{"line_number":3023,"context_line":"        # execute the trusts in series (C-\u003eB-\u003eA)."},{"line_number":3024,"context_line":""},{"line_number":3025,"context_line":"        # create a sub-trustee user"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_23dc3546","line":3022,"in_reply_to":"baada198_76dfcc3a","updated":"2014-07-24 00:20:29.000000000","message":"Done","commit_id":"f83a8ce9eca018b9c31c48d602e0822752ad217c"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"200246fd8b941f13658ba0ad2e89ed0524d6adb5","unresolved":false,"context_lines":[{"line_number":3020,"context_line":"    def test_trust_chained(self):"},{"line_number":3021,"context_line":"        # Test that a trust token can\u0027t be used to execute another trust.  To"},{"line_number":3022,"context_line":"        # to this, we create an A-\u003eB-\u003eC hierarchy of trusts, then attempt to"},{"line_number":3023,"context_line":"        # execute the trusts in series (C-\u003eB-\u003eA)."},{"line_number":3024,"context_line":""},{"line_number":3025,"context_line":"        # create a sub-trustee user"},{"line_number":3026,"context_line":"        sub_trustee_user \u003d self.new_user_ref("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_b6e084f9","line":3023,"updated":"2014-07-23 23:06:57.000000000","message":"Maybe this paragraph would be better as s docstring?","commit_id":"f83a8ce9eca018b9c31c48d602e0822752ad217c"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"02e14eb914f3ff5c33165b5bcb29f2b305a51590","unresolved":false,"context_lines":[{"line_number":3020,"context_line":"    def test_trust_chained(self):"},{"line_number":3021,"context_line":"        # Test that a trust token can\u0027t be used to execute another trust.  To"},{"line_number":3022,"context_line":"        # to this, we create an A-\u003eB-\u003eC hierarchy of trusts, then attempt to"},{"line_number":3023,"context_line":"        # execute the trusts in series (C-\u003eB-\u003eA)."},{"line_number":3024,"context_line":""},{"line_number":3025,"context_line":"        # create a sub-trustee user"},{"line_number":3026,"context_line":"        sub_trustee_user \u003d self.new_user_ref("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_63e54d09","line":3023,"in_reply_to":"baada198_b6e084f9","updated":"2014-07-24 00:20:29.000000000","message":"Done","commit_id":"f83a8ce9eca018b9c31c48d602e0822752ad217c"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"a5610e7fbcc590800a15693eabd8309e7baf2c23","unresolved":false,"context_lines":[{"line_number":2982,"context_line":""},{"line_number":2983,"context_line":"        # assign a new role"},{"line_number":2984,"context_line":"        self.put("},{"line_number":2985,"context_line":"            \u0027/projects/%(project_id)s/users/%(user_id)s/roles/%(role_id)s\u0027 % {"},{"line_number":2986,"context_line":"                \u0027project_id\u0027: self.project_id,"},{"line_number":2987,"context_line":"                \u0027user_id\u0027: self.user_id,"},{"line_number":2988,"context_line":"                \u0027role_id\u0027: role[\u0027id\u0027]})"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_179e8007","line":2985,"updated":"2014-07-24 08:50:38.000000000","message":"Here is the same with \"grant_url\", we can use \"grant_url\" instead.","commit_id":"9a40c92a3f561470dea13ce844133604a630df87"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"006e23cfb24c8820851ac841507986977448c456","unresolved":false,"context_lines":[{"line_number":2982,"context_line":""},{"line_number":2983,"context_line":"        # assign a new role"},{"line_number":2984,"context_line":"        self.put("},{"line_number":2985,"context_line":"            \u0027/projects/%(project_id)s/users/%(user_id)s/roles/%(role_id)s\u0027 % {"},{"line_number":2986,"context_line":"                \u0027project_id\u0027: self.project_id,"},{"line_number":2987,"context_line":"                \u0027user_id\u0027: self.user_id,"},{"line_number":2988,"context_line":"                \u0027role_id\u0027: role[\u0027id\u0027]})"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_c35a6e1d","line":2985,"in_reply_to":"baada198_179e8007","updated":"2014-07-24 21:45:50.000000000","message":"Good catch.  Will update...","commit_id":"9a40c92a3f561470dea13ce844133604a630df87"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"a5610e7fbcc590800a15693eabd8309e7baf2c23","unresolved":false,"context_lines":[{"line_number":3060,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3061,"context_line":"            project_id\u003dself.project_id)"},{"line_number":3062,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data)"},{"line_number":3063,"context_line":"        token \u003d r.headers.get(\u0027X-Subject-Token\u0027)"},{"line_number":3064,"context_line":""},{"line_number":3065,"context_line":"        # create a trust from trustee -\u003e sub-trustee"},{"line_number":3066,"context_line":"        ref \u003d self.new_trust_ref("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_7705fc13","line":3063,"updated":"2014-07-24 08:50:38.000000000","message":"We can use token \u003d self.get_requested_token(auth_data) instead.","commit_id":"9a40c92a3f561470dea13ce844133604a630df87"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"006e23cfb24c8820851ac841507986977448c456","unresolved":false,"context_lines":[{"line_number":3060,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3061,"context_line":"            project_id\u003dself.project_id)"},{"line_number":3062,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data)"},{"line_number":3063,"context_line":"        token \u003d r.headers.get(\u0027X-Subject-Token\u0027)"},{"line_number":3064,"context_line":""},{"line_number":3065,"context_line":"        # create a trust from trustee -\u003e sub-trustee"},{"line_number":3066,"context_line":"        ref \u003d self.new_trust_ref("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_030aa6d1","line":3063,"in_reply_to":"baada198_7705fc13","updated":"2014-07-24 21:45:50.000000000","message":"Done","commit_id":"9a40c92a3f561470dea13ce844133604a630df87"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"a5610e7fbcc590800a15693eabd8309e7baf2c23","unresolved":false,"context_lines":[{"line_number":3080,"context_line":"            password\u003dsub_trustee_user[\u0027password\u0027],"},{"line_number":3081,"context_line":"            trust_id\u003dtrust2[\u0027id\u0027])"},{"line_number":3082,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data)"},{"line_number":3083,"context_line":"        trust_token \u003d r.headers[\u0027X-Subject-Token\u0027]"},{"line_number":3084,"context_line":""},{"line_number":3085,"context_line":"        # attempt to get the second trust using a trust token"},{"line_number":3086,"context_line":"        auth_data \u003d self.build_authentication_request("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_220b4425","line":3083,"updated":"2014-07-24 08:50:38.000000000","message":"Same comment as above.","commit_id":"9a40c92a3f561470dea13ce844133604a630df87"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"006e23cfb24c8820851ac841507986977448c456","unresolved":false,"context_lines":[{"line_number":3080,"context_line":"            password\u003dsub_trustee_user[\u0027password\u0027],"},{"line_number":3081,"context_line":"            trust_id\u003dtrust2[\u0027id\u0027])"},{"line_number":3082,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data)"},{"line_number":3083,"context_line":"        trust_token \u003d r.headers[\u0027X-Subject-Token\u0027]"},{"line_number":3084,"context_line":""},{"line_number":3085,"context_line":"        # attempt to get the second trust using a trust token"},{"line_number":3086,"context_line":"        auth_data \u003d self.build_authentication_request("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_230b2ad6","line":3083,"in_reply_to":"baada198_220b4425","updated":"2014-07-24 21:45:50.000000000","message":"Done","commit_id":"9a40c92a3f561470dea13ce844133604a630df87"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"bfa6733d0973ecbd3a586234c6513863b2fc6db3","unresolved":false,"context_lines":[{"line_number":3012,"context_line":"            user_id\u003dself.trustee_user[\u0027id\u0027],"},{"line_number":3013,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3014,"context_line":"            trust_id\u003dtrust[\u0027id\u0027])"},{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_a8a199aa","line":3015,"updated":"2014-07-28 02:10:53.000000000","message":"I think you should need to rebase and use v3_authenticate_token method here. Please refer to https://bugs.launchpad.net/keystone/+bug/1317261.","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"1c0d583aa14a637b6d262433d59f0156093be097","unresolved":false,"context_lines":[{"line_number":3012,"context_line":"            user_id\u003dself.trustee_user[\u0027id\u0027],"},{"line_number":3013,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3014,"context_line":"            trust_id\u003dtrust[\u0027id\u0027])"},{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_5b3dc1dc","line":3015,"in_reply_to":"baada198_8490fc43","updated":"2014-07-30 07:38:12.000000000","message":"(the link had a trailing \u0027.\u0027 - see https://review.openstack.org/#/c/101861 )\n\nI agree, although self.post as written works, it is better to use the new v3_authenticate_token method()","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"d19d6ea5eebfa3f8dd378a49d7664fa5837bf01a","unresolved":false,"context_lines":[{"line_number":3012,"context_line":"            user_id\u003dself.trustee_user[\u0027id\u0027],"},{"line_number":3013,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3014,"context_line":"            trust_id\u003dtrust[\u0027id\u0027])"},{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_d0f52b52","line":3015,"in_reply_to":"baada198_8490fc43","updated":"2014-07-30 08:36:59.000000000","message":"I am sorry:)...It is here: https://bugs.launchpad.net/keystone/+bug/1317261","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"87a43e1b068d5e14182c855786e3c7136502d923","unresolved":false,"context_lines":[{"line_number":3012,"context_line":"            user_id\u003dself.trustee_user[\u0027id\u0027],"},{"line_number":3013,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3014,"context_line":"            trust_id\u003dtrust[\u0027id\u0027])"},{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_be749dd0","line":3015,"in_reply_to":"baada198_a8a199aa","updated":"2014-08-01 17:35:49.000000000","message":"Done","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"c47a1232fa9ecf6f9505a6d84fb49df34971fa8f","unresolved":false,"context_lines":[{"line_number":3012,"context_line":"            user_id\u003dself.trustee_user[\u0027id\u0027],"},{"line_number":3013,"context_line":"            password\u003dself.trustee_user[\u0027password\u0027],"},{"line_number":3014,"context_line":"            trust_id\u003dtrust[\u0027id\u0027])"},{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_8490fc43","line":3015,"in_reply_to":"baada198_a8a199aa","updated":"2014-07-29 14:33:49.000000000","message":"This link isn\u0027t working for me, it is correct?","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"543a4dd50c92d9cb4a35b39a6e67e46fbbe38aee","unresolved":false,"context_lines":[{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"},{"line_number":3019,"context_line":"        do this, we create an A-\u003eB-\u003eC hierarchy of trusts, then attempt to"},{"line_number":3020,"context_line":"        execute the trusts in series (C-\u003eB-\u003eA)."},{"line_number":3021,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_abc51a3e","line":3018,"updated":"2014-07-27 10:32:44.000000000","message":"Thanks for making this a doctoring, although the recommended format is a summary line, followed by other info in a paragraph...the overall recommendations for docstrings are:\n\n- [H401] Docstrings should not start with a space.\n- [H403] Multi line docstrings should end on a new line.\n- [H404] Multi line docstrings should start without a leading new line.\n- [H405] Multi line docstrings should start with a one line summary followed\n  by an empty line.","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"87a43e1b068d5e14182c855786e3c7136502d923","unresolved":false,"context_lines":[{"line_number":3015,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3016,"context_line":""},{"line_number":3017,"context_line":"    def test_trust_chained(self):"},{"line_number":3018,"context_line":"        \"\"\"Test that a trust token can\u0027t be used to execute another trust.  To"},{"line_number":3019,"context_line":"        do this, we create an A-\u003eB-\u003eC hierarchy of trusts, then attempt to"},{"line_number":3020,"context_line":"        execute the trusts in series (C-\u003eB-\u003eA)."},{"line_number":3021,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_1ea1093a","line":3018,"in_reply_to":"baada198_abc51a3e","updated":"2014-08-01 17:35:49.000000000","message":"Done","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"bfa6733d0973ecbd3a586234c6513863b2fc6db3","unresolved":false,"context_lines":[{"line_number":3081,"context_line":"        auth_data \u003d self.build_authentication_request("},{"line_number":3082,"context_line":"            token\u003dtrust_token,"},{"line_number":3083,"context_line":"            trust_id\u003dtrust1[\u0027id\u0027])"},{"line_number":3084,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"    def assertTrustTokensRevoked(self, trust_id):"},{"line_number":3087,"context_line":"        revocation_response \u003d self.get(\u0027/OS-REVOKE/events\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_689b0173","line":3084,"updated":"2014-07-28 02:10:53.000000000","message":"The same comment above.","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"87a43e1b068d5e14182c855786e3c7136502d923","unresolved":false,"context_lines":[{"line_number":3081,"context_line":"        auth_data \u003d self.build_authentication_request("},{"line_number":3082,"context_line":"            token\u003dtrust_token,"},{"line_number":3083,"context_line":"            trust_id\u003dtrust1[\u0027id\u0027])"},{"line_number":3084,"context_line":"        r \u003d self.post(\u0027/auth/tokens\u0027, body\u003dauth_data, expected_status\u003d403)"},{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"    def assertTrustTokensRevoked(self, trust_id):"},{"line_number":3087,"context_line":"        revocation_response \u003d self.get(\u0027/OS-REVOKE/events\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_de7151df","line":3084,"in_reply_to":"baada198_689b0173","updated":"2014-08-01 17:35:49.000000000","message":"Done","commit_id":"391f794fdd5470c066e2d29a9b4a5e0c69f4e907"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"e15fb8ef1e8dc81a49d1397c95ebb75038e4c4c9","unresolved":false,"context_lines":[{"line_number":2941,"context_line":"                  token\u003dtrust_token,"},{"line_number":2942,"context_line":"                  expected_status\u003d403)"},{"line_number":2943,"context_line":""},{"line_number":2944,"context_line":"    def test_trust_deleted_grant(self):"},{"line_number":2945,"context_line":"        # create a new role"},{"line_number":2946,"context_line":"        role \u003d self.new_role_ref()"},{"line_number":2947,"context_line":"        self.assignment_api.create_role(role[\u0027id\u0027], role)"}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_25693cf8","line":2944,"updated":"2014-08-04 00:21:40.000000000","message":"these tests should be up in TestTokenRevokeById. Then they\u0027ll be tested using both revocation events and revocation by ID.","commit_id":"23b2c8476051eacf3c4f08fbe32667886c7aa234"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"75d03d64baca10fb74a0e484800d68eda993387a","unresolved":false,"context_lines":[{"line_number":2941,"context_line":"                  token\u003dtrust_token,"},{"line_number":2942,"context_line":"                  expected_status\u003d403)"},{"line_number":2943,"context_line":""},{"line_number":2944,"context_line":"    def test_trust_deleted_grant(self):"},{"line_number":2945,"context_line":"        # create a new role"},{"line_number":2946,"context_line":"        role \u003d self.new_role_ref()"},{"line_number":2947,"context_line":"        self.assignment_api.create_role(role[\u0027id\u0027], role)"}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_24e75410","line":2944,"in_reply_to":"baada198_1edb7fea","updated":"2014-08-04 23:38:56.000000000","message":"ok","commit_id":"23b2c8476051eacf3c4f08fbe32667886c7aa234"},{"author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"change_message_id":"5a11344b7858775894483eb43ab0aefb5b5ab77b","unresolved":false,"context_lines":[{"line_number":2941,"context_line":"                  token\u003dtrust_token,"},{"line_number":2942,"context_line":"                  expected_status\u003d403)"},{"line_number":2943,"context_line":""},{"line_number":2944,"context_line":"    def test_trust_deleted_grant(self):"},{"line_number":2945,"context_line":"        # create a new role"},{"line_number":2946,"context_line":"        role \u003d self.new_role_ref()"},{"line_number":2947,"context_line":"        self.assignment_api.create_role(role[\u0027id\u0027], role)"}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_1edb7fea","line":2944,"in_reply_to":"baada198_25693cf8","updated":"2014-08-04 22:40:56.000000000","message":"These new tests are testing trusts, not revocation.  As such, they are a part of the TestTrustAuth class.  Given this, I don\u0027t think your comment is applicable.","commit_id":"23b2c8476051eacf3c4f08fbe32667886c7aa234"}]}