)]}'
{"keystone/tests/test_v3_auth.py":[{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"98d1cc299eb26a1d02012f2784e3382db61e0202","unresolved":false,"context_lines":[{"line_number":3507,"context_line":"        for identity_attr in auth.controllers.AuthContext.IDENTITY_ATTRIBUTES:"},{"line_number":3508,"context_line":"            self.auth_context[identity_attr] \u003d None"},{"line_number":3509,"context_line":""},{"line_number":3510,"context_line":"            if identity_attr \u003d\u003d \u0027expires_at\u0027:"},{"line_number":3511,"context_line":"                # \u0027expires_at\u0027 is a special case and is tested above."},{"line_number":3512,"context_line":"                self.auth_context[\u0027expires_at\u0027] \u003d uuid.uuid4().hex"},{"line_number":3513,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a961159_b514c99f","line":3510,"updated":"2015-01-05 16:00:54.000000000","message":"Why not check this before setting auth_context attribute at line 3508?\nThat assignment is useless if this expression results in True.","commit_id":"c30c0b9a3c28c600f40bf64c2588b9ca1eb65e63"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"9c93d85373a3e180110b4667103b4ca18ccb0d3e","unresolved":false,"context_lines":[{"line_number":3507,"context_line":"        for identity_attr in auth.controllers.AuthContext.IDENTITY_ATTRIBUTES:"},{"line_number":3508,"context_line":"            self.auth_context[identity_attr] \u003d None"},{"line_number":3509,"context_line":""},{"line_number":3510,"context_line":"            if identity_attr \u003d\u003d \u0027expires_at\u0027:"},{"line_number":3511,"context_line":"                # \u0027expires_at\u0027 is a special case and is tested above."},{"line_number":3512,"context_line":"                self.auth_context[\u0027expires_at\u0027] \u003d uuid.uuid4().hex"},{"line_number":3513,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a961159_39e7e813","line":3510,"in_reply_to":"3a961159_593754e3","updated":"2015-01-05 23:34:39.000000000","message":"I got it now.\nIn the case of \u0027expires_at\u0027, we can set it to None and after to something else without getting exception.Unauthorized.","commit_id":"c30c0b9a3c28c600f40bf64c2588b9ca1eb65e63"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"88fa41b6408edfd158f75428f3651281f8832ddf","unresolved":false,"context_lines":[{"line_number":3507,"context_line":"        for identity_attr in auth.controllers.AuthContext.IDENTITY_ATTRIBUTES:"},{"line_number":3508,"context_line":"            self.auth_context[identity_attr] \u003d None"},{"line_number":3509,"context_line":""},{"line_number":3510,"context_line":"            if identity_attr \u003d\u003d \u0027expires_at\u0027:"},{"line_number":3511,"context_line":"                # \u0027expires_at\u0027 is a special case and is tested above."},{"line_number":3512,"context_line":"                self.auth_context[\u0027expires_at\u0027] \u003d uuid.uuid4().hex"},{"line_number":3513,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a961159_593754e3","line":3510,"in_reply_to":"3a961159_b514c99f","updated":"2015-01-05 23:29:22.000000000","message":"this test is making sure that __setitem__ is implemented correctly, so we want to test that the attribute can be set to None.","commit_id":"c30c0b9a3c28c600f40bf64c2588b9ca1eb65e63"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"98d1cc299eb26a1d02012f2784e3382db61e0202","unresolved":false,"context_lines":[{"line_number":3509,"context_line":""},{"line_number":3510,"context_line":"            if identity_attr \u003d\u003d \u0027expires_at\u0027:"},{"line_number":3511,"context_line":"                # \u0027expires_at\u0027 is a special case and is tested above."},{"line_number":3512,"context_line":"                self.auth_context[\u0027expires_at\u0027] \u003d uuid.uuid4().hex"},{"line_number":3513,"context_line":"                continue"},{"line_number":3514,"context_line":""},{"line_number":3515,"context_line":"            self.assertRaises(exception.Unauthorized,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a961159_35c45940","line":3512,"updated":"2015-01-05 16:00:54.000000000","message":"Why not just skip this without setting anything? As we\u0027re doing at line 3499.","commit_id":"c30c0b9a3c28c600f40bf64c2588b9ca1eb65e63"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"88fa41b6408edfd158f75428f3651281f8832ddf","unresolved":false,"context_lines":[{"line_number":3509,"context_line":""},{"line_number":3510,"context_line":"            if identity_attr \u003d\u003d \u0027expires_at\u0027:"},{"line_number":3511,"context_line":"                # \u0027expires_at\u0027 is a special case and is tested above."},{"line_number":3512,"context_line":"                self.auth_context[\u0027expires_at\u0027] \u003d uuid.uuid4().hex"},{"line_number":3513,"context_line":"                continue"},{"line_number":3514,"context_line":""},{"line_number":3515,"context_line":"            self.assertRaises(exception.Unauthorized,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3a961159_39dd2871","line":3512,"in_reply_to":"3a961159_35c45940","updated":"2015-01-05 23:29:22.000000000","message":"expires_at works differently than other attributes in that you can set it after setting it to None. That\u0027s part of what this test is asserting which is different than the previous test.","commit_id":"c30c0b9a3c28c600f40bf64c2588b9ca1eb65e63"}]}