)]}'
{"keystone/tests/unit/test_v3_auth.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"379ddda0fb70e1c0c91a8f23a55150272a72e60d","unresolved":false,"context_lines":[{"line_number":3583,"context_line":"    # Trust chain scenarios"},{"line_number":3584,"context_line":""},{"line_number":3585,"context_line":"    def test_delete_trust_cascade(self):"},{"line_number":3586,"context_line":"        self.setup_trust_chain()"},{"line_number":3587,"context_line":"        self.assert_user_authenticate(self.user_list[0])"},{"line_number":3588,"context_line":"        self.delete(\u0027/OS-TRUST/trusts/%(trust_id)s\u0027 % {"},{"line_number":3589,"context_line":"            \u0027trust_id\u0027: self.trust_chain[0][\u0027id\u0027]})"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a57f1b5_00026c0b","line":3586,"range":{"start_line":3586,"start_character":0,"end_line":3586,"end_character":32},"updated":"2016-02-12 01:30:02.000000000","message":"And then these methods have now 2 setup calls: the one from this class which is ran by the test framework and this explicit call.\n\nI am not sure how much it is buying us to put these tests together","commit_id":"c35d7691cbacb642c0544b7195a02405d5d9dbc9"},{"author":{"_account_id":13152,"name":"Jorge Munoz","email":"elkidster@gmail.com","username":"jmunoz"},"change_message_id":"437d13f28b083a52afa3443ab1be6e91a37b8782","unresolved":false,"context_lines":[{"line_number":3583,"context_line":"    # Trust chain scenarios"},{"line_number":3584,"context_line":""},{"line_number":3585,"context_line":"    def test_delete_trust_cascade(self):"},{"line_number":3586,"context_line":"        self.setup_trust_chain()"},{"line_number":3587,"context_line":"        self.assert_user_authenticate(self.user_list[0])"},{"line_number":3588,"context_line":"        self.delete(\u0027/OS-TRUST/trusts/%(trust_id)s\u0027 % {"},{"line_number":3589,"context_line":"            \u0027trust_id\u0027: self.trust_chain[0][\u0027id\u0027]})"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a57f1b5_3b3e928c","line":3586,"range":{"start_line":3586,"start_character":0,"end_line":3586,"end_character":32},"in_reply_to":"3a57f1b5_00026c0b","updated":"2016-02-12 17:01:29.000000000","message":"The purpose of the patch is to simplify the testing for the fernet token provider and have trust redelegation/impersonation/chained workflow test coverage which is not in the TestFernetProvider trust test cases. I do agree that having two setup is not ideal, so what I can do is refactor the TestTrustAuth into the TrustAPIBehavior and leave the TestTrustChain as its own class which has an expensive setup.","commit_id":"c35d7691cbacb642c0544b7195a02405d5d9dbc9"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"379ddda0fb70e1c0c91a8f23a55150272a72e60d","unresolved":false,"context_lines":[{"line_number":3665,"context_line":"                      expected_status\u003dhttp_client.FORBIDDEN)"},{"line_number":3666,"context_line":""},{"line_number":3667,"context_line":""},{"line_number":3668,"context_line":"class TestTrustAuth(test_v3.RestfulTestCase):"},{"line_number":3669,"context_line":""},{"line_number":3670,"context_line":"    def config_overrides(self):"},{"line_number":3671,"context_line":"        super(TestTrustAuth, self).config_overrides()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a57f1b5_9b280d88","line":3668,"range":{"start_line":3668,"start_character":0,"end_line":3668,"end_character":45},"updated":"2016-02-12 01:30:02.000000000","message":"Also, there are still other separate classes","commit_id":"c35d7691cbacb642c0544b7195a02405d5d9dbc9"}]}