On Wed, Oct 27, 2021 at 10:27:18AM +0530, Bharath Rupireddy wrote:
> +1 and the above statement looks good. Apart from that I have one more
> suggestion that I earlier made: have the function produce the error
> when no permissions were granted (just to ensure we have the error
> message covered) including has_function_privilege() case.
has_function_privilege() makes sure of the same thing, so I think that
what Jeff has done for this part is just but fine. There is no need
for more duplication in the tests.
--
Michael