)]}'
{"glance/tests/unit/v2/test_v2_policy.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"131ebbb797ef1fa97c0e802400d5e96e0bb08b41","unresolved":true,"context_lines":[{"line_number":62,"context_line":"        # Admin always wins"},{"line_number":63,"context_line":"        context.is_admin \u003d True"},{"line_number":64,"context_line":"        context.owner \u003d \u0027someuser\u0027"},{"line_number":65,"context_line":"        policy.check_is_image_mutable(context, image)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        # Image has no owner is never mutable by non-admins"},{"line_number":68,"context_line":"        context.is_admin \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"a1cd08a7_97d76dba","line":65,"range":{"start_line":65,"start_character":0,"end_line":65,"end_character":53},"updated":"2021-08-07 00:28:43.000000000","message":"I know the function is simple enough and this is only temporary code anyway, but can we use self.assertIsNone here?","commit_id":"bff5d342dc4dbc98b8385dd7ee9abdb9ec0c0ce7"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"5615adbe293626a22e02d717f3ffd07f3b69d2e5","unresolved":true,"context_lines":[{"line_number":62,"context_line":"        # Admin always wins"},{"line_number":63,"context_line":"        context.is_admin \u003d True"},{"line_number":64,"context_line":"        context.owner \u003d \u0027someuser\u0027"},{"line_number":65,"context_line":"        policy.check_is_image_mutable(context, image)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        # Image has no owner is never mutable by non-admins"},{"line_number":68,"context_line":"        context.is_admin \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"907a50e1_b1c7e697","line":65,"range":{"start_line":65,"start_character":0,"end_line":65,"end_character":53},"in_reply_to":"271b65b0_cf269332","updated":"2021-08-10 16:05:45.000000000","message":"Oh, OK, I see your point. It makes sense, I won\u0027t bother you with that in upcoming patch reviews then :)","commit_id":"bff5d342dc4dbc98b8385dd7ee9abdb9ec0c0ce7"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"53454fbefc7bbc3465f79707ff40cbd6c7606b5c","unresolved":true,"context_lines":[{"line_number":62,"context_line":"        # Admin always wins"},{"line_number":63,"context_line":"        context.is_admin \u003d True"},{"line_number":64,"context_line":"        context.owner \u003d \u0027someuser\u0027"},{"line_number":65,"context_line":"        policy.check_is_image_mutable(context, image)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        # Image has no owner is never mutable by non-admins"},{"line_number":68,"context_line":"        context.is_admin \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"271b65b0_cf269332","line":65,"range":{"start_line":65,"start_character":0,"end_line":65,"end_character":53},"in_reply_to":"a1cd08a7_97d76dba","updated":"2021-08-10 14:46:37.000000000","message":"Sure? I don\u0027t normally assert things that aren\u0027t defined. We\u0027re not returning None in some cases and non-None in others, so asserting that None is returned isn\u0027t really validating anything, other than the byproduct of Python assuming None if there was no return statement.\n\nBut, I\u0027ll do this and add Abhi\u0027s suggested TODO.","commit_id":"bff5d342dc4dbc98b8385dd7ee9abdb9ec0c0ce7"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"131ebbb797ef1fa97c0e802400d5e96e0bb08b41","unresolved":true,"context_lines":[{"line_number":87,"context_line":"        # Context matches image owner is mutable"},{"line_number":88,"context_line":"        image.owner \u003d \u0027someuser\u0027"},{"line_number":89,"context_line":"        context.owner \u003d \u0027someuser\u0027"},{"line_number":90,"context_line":"        policy.check_is_image_mutable(context, image)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class APIImagePolicy(APIPolicyBase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3cb97d94_30e3fd0b","line":90,"range":{"start_line":90,"start_character":0,"end_line":90,"end_character":53},"updated":"2021-08-07 00:28:43.000000000","message":"Ditto","commit_id":"bff5d342dc4dbc98b8385dd7ee9abdb9ec0c0ce7"}]}