)]}'
{"keystone/tests/unit/test_v3_protection.py":[{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"ec37959799e87363405331b76fa74f35153a585c","unresolved":false,"context_lines":[{"line_number":391,"context_line":"                                         user_id\u003dself.admin_user[\u0027id\u0027],"},{"line_number":392,"context_line":"                                         project_id\u003dself.project[\u0027id\u0027])"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"    def test_user_validate_same_token(self):"},{"line_number":395,"context_line":"        # Given a non-admin user token, the token can be used to validate"},{"line_number":396,"context_line":"        # itself."},{"line_number":397,"context_line":"        # This is GET /v3/auth/tokens, with X-Auth-Token \u003d\u003d X-Subject-Token"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_bda0ce42","line":394,"updated":"2015-03-16 10:47:37.000000000","message":"sugested use of  @utils.wip(\u0027Exposing bug #1421825\u0027)","commit_id":"d6c21618c2512b6a80909c2539b85fbbf6e60354"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"29b04af848b1c562a48b0f503b6497db23de4b28","unresolved":false,"context_lines":[{"line_number":391,"context_line":"                                         user_id\u003dself.admin_user[\u0027id\u0027],"},{"line_number":392,"context_line":"                                         project_id\u003dself.project[\u0027id\u0027])"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"    def test_user_validate_same_token(self):"},{"line_number":395,"context_line":"        # Given a non-admin user token, the token can be used to validate"},{"line_number":396,"context_line":"        # itself."},{"line_number":397,"context_line":"        # This is GET /v3/auth/tokens, with X-Auth-Token \u003d\u003d X-Subject-Token"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_e108dc2e","line":394,"in_reply_to":"9a80dd14_bda0ce42","updated":"2015-03-16 14:11:58.000000000","message":"I don\u0027t think we should be requiring developers to use the utils.wip method of indicating that a test is going to fail. The way this test is written now it makes it explicit that this isn\u0027t working correctly and in what way, whereas that will be hidden if using utils.wip.","commit_id":"d6c21618c2512b6a80909c2539b85fbbf6e60354"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d426c9bbc78a4703bd8c7b97ca2a9a4e5aae2f4c","unresolved":false,"context_lines":[{"line_number":407,"context_line":"        self.get(\u0027/auth/tokens\u0027, token\u003dtoken,"},{"line_number":408,"context_line":"                 headers\u003d{\u0027X-Subject-Token\u0027: token}, expected_status\u003d403)"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"    def test_user_validate_user_token(self):"},{"line_number":411,"context_line":"        # A user can validate one of their own tokens."},{"line_number":412,"context_line":"        # This is GET /v3/auth/tokens"},{"line_number":413,"context_line":"        # FIXME(blk-u): This test fails, a user can\u0027t validate their own token,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_bc8a0218","line":410,"updated":"2015-03-18 19:03:54.000000000","message":"All these test case dictate behavior that should be consistent regardless of the token provider. We should consider rolling them into a test module that we already have so that we can run the various providers we support against all the different test cases detailed here. \n\nOr we add test modules that inherit IdentityTestPolicySample but setup each provider accordingly.","commit_id":"d6c21618c2512b6a80909c2539b85fbbf6e60354"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"24329936d4349a62c0c18a6ec2be8bd89b1a6b36","unresolved":false,"context_lines":[{"line_number":407,"context_line":"        self.get(\u0027/auth/tokens\u0027, token\u003dtoken,"},{"line_number":408,"context_line":"                 headers\u003d{\u0027X-Subject-Token\u0027: token}, expected_status\u003d403)"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"    def test_user_validate_user_token(self):"},{"line_number":411,"context_line":"        # A user can validate one of their own tokens."},{"line_number":412,"context_line":"        # This is GET /v3/auth/tokens"},{"line_number":413,"context_line":"        # FIXME(blk-u): This test fails, a user can\u0027t validate their own token,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_3c71b2de","line":410,"in_reply_to":"9a80dd14_bc8a0218","updated":"2015-03-18 19:12:11.000000000","message":"What does the choice of token provider have to do with policy enforcement? If these 2 things are coupled like this then we should break the coupling.","commit_id":"d6c21618c2512b6a80909c2539b85fbbf6e60354"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"70444cceb6a5ed1d164d3e738296915405654f79","unresolved":false,"context_lines":[{"line_number":366,"context_line":"        password \u003d uuid.uuid4().hex"},{"line_number":367,"context_line":"        self.admin_user[\u0027password\u0027] \u003d password"},{"line_number":368,"context_line":"        self.admin_user \u003d self.identity_api.create_user(self.admin_user)"},{"line_number":369,"context_line":"        self.admin_user[\u0027password\u0027] \u003d password"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"        self.role \u003d self.new_role_ref()"},{"line_number":372,"context_line":"        self.role_api.create_role(self.role[\u0027id\u0027], self.role)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6ced46_11fdf43a","line":369,"updated":"2015-03-24 06:07:14.000000000","message":"I see repetitive code. Refactoring?\n\n self.just_a_user \u003d _create_user()\n self.another_user \u003d _create_user()\n ...","commit_id":"d57e6e3d6547a731522794cc9eecc5cb2c9a6df4"}]}