mirror of
https://github.com/scylladb/scylladb.git
synced 2026-06-02 04:56:58 +00:00
d50f048279
As a result of the preceding patches, permissions on a function are now granted to its creator. As a result, some permissions may appear which we did not expect before. In the test_udf_permissions_serialization, we create a function as the superuser, and as a result, when we compare the permissions we specifically granted to the ones read from the LIST PERMISSIONS result, we get more than expected - this is fixed by granting permissions explicitly to a new user and only checking this user's permissions list. In the test_grant_revoke_udf_permissions case, we test whether the DROP permission in enforced on a function that we have previously created as the same user - as a result we have the DROP permission even without granting it directly. We fix this by testing the DROP permission on a function created by a different user. In the test_grant_revoke_alter_udf_permissions case, we previously tested that we require both ALTER and CREATE permissions when executing a CREATE OR REPLACE FUNCTION statement. The new permissions required for this statement now depend on whether we actually CREATE or REPLACE a function, so now we test that the ALTER permission is required when REPLACING a function, and the CREATE permission is required when CREATING a function. After the changes, the case no longer needs to be arfitifially extracted from the previous one, so they are merged now. Analogous adjustments are made in the test case test_grant_revoke_alter_uda_permissions.