)]}'
{"oslo_policy/tests/test_policy.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"361ec8f642f2600f7c65a277aa8dcbd6714a5623","unresolved":true,"context_lines":[{"line_number":885,"context_line":"    def test_map_context_attributes_populated_system(self):"},{"line_number":886,"context_line":"        request_context \u003d context.RequestContext(system_scope\u003d\u0027all\u0027)"},{"line_number":887,"context_line":"        expected_creds \u003d request_context.to_policy_values()"},{"line_number":888,"context_line":"        expected_creds[\u0027system\u0027] \u003d \u0027all\u0027"},{"line_number":889,"context_line":""},{"line_number":890,"context_line":"        creds \u003d self.enforcer._map_context_attributes_into_creds("},{"line_number":891,"context_line":"            request_context"}],"source_content_type":"text/x-python","patch_set":1,"id":"3d1af46c_ec00d9d5","side":"PARENT","line":888,"updated":"2021-08-05 21:24:25.000000000","message":"Don\u0027t we still want to test this case?","commit_id":"0bc8a2e70faaf8600a44c6e7e366d2296adc128b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"309ad620431d58965a875a58b2b2af07a180b1f0","unresolved":true,"context_lines":[{"line_number":885,"context_line":"    def test_map_context_attributes_populated_system(self):"},{"line_number":886,"context_line":"        request_context \u003d context.RequestContext(system_scope\u003d\u0027all\u0027)"},{"line_number":887,"context_line":"        expected_creds \u003d request_context.to_policy_values()"},{"line_number":888,"context_line":"        expected_creds[\u0027system\u0027] \u003d \u0027all\u0027"},{"line_number":889,"context_line":""},{"line_number":890,"context_line":"        creds \u003d self.enforcer._map_context_attributes_into_creds("},{"line_number":891,"context_line":"            request_context"}],"source_content_type":"text/x-python","patch_set":1,"id":"69a8ea70_71f7d6a5","side":"PARENT","line":888,"in_reply_to":"3d1af46c_ec00d9d5","updated":"2021-08-12 20:25:43.000000000","message":"No, _map_context_attributes_into_creds no longer converts \u0027system_scope\u0027 to \u0027system\u0027, I moved that conversion out of this method to the calling method so that the conversion will occur regardless if a RequestContext was passed in or a cred dictionary.\nThe test below, test_enforcer_understands_system_scope_creds_dict, will cover both code paths now.\nFor the other checks of the key/value mapping, the above test, test_enforcer_consolidates_context_attributes_with_creds, covers that case.","commit_id":"0bc8a2e70faaf8600a44c6e7e366d2296adc128b"}]}