Re: Add regression coverage for REVOKE ADMIN OPTION - Mailing list pgsql-hackers

From Mark Dilger
Subject Re: Add regression coverage for REVOKE ADMIN OPTION
Date
Msg-id 0DE9BC11-13C4-4570-9199-7DD02CCEC055@enterprisedb.com
Whole thread Raw
In response to Re: Add regression coverage for REVOKE ADMIN OPTION  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Add regression coverage for REVOKE ADMIN OPTION
List pgsql-hackers

> On Nov 16, 2021, at 6:31 AM, Daniel Gustafsson <daniel@yesql.se> wrote:
>
>> On 16 Nov 2021, at 00:58, Mark Dilger <mark.dilger@enterprisedb.com> wrote:
>
>> While working on a fix for dangling references to dropped roles in the pg_auth_members.grantor field, I happened to
noticewe entirely lack regression test coverage of the REVOKE ADMIN OPTION FOR form of the RevokeRoleStmt.  I am
unawareof any bugs in the current implementation, but future work on roles may benefit if we close the testing gap. 
>
> LGTM.  Reading this I realized that the GRANTED BY keyword for RevokeRoleStmt
> isn't working as documented, it's not checking the role at all.  I've sent a
> diff for that with tests on the relevant thread, but I think it would be a good
> to get this in too to boost coverage.

Thanks for the review!

—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company






pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Add regression coverage for REVOKE ADMIN OPTION
Next
From: Robert Haas
Date:
Subject: Re: RecoveryInProgress() has critical side effects