errose28 commented on a change in pull request #2857:
URL: https://github.com/apache/ozone/pull/2857#discussion_r755588049



##########
File path: 
hadoop-ozone/dist/src/main/smoketest/security/ozone-secure-tenant.robot
##########
@@ -72,3 +76,12 @@ Secure Tenant Create Tenant Failure with Regular (non-admin) 
user
 Secure Tenant SetSecret Failure with Regular (non-admin) user
     ${rc}  ${output} =  Run And Return Rc And Output  ozone tenant user 
set-secret 'tenantone$bob' --secret=somesecret2 --export
                         Should contain   ${output}         Permission denied. 
Requested accessId
+
+Secure Tenant Revoke User AccessId Success
+    Run Keyword   Kinit test user     testuser     testuser.keytab
+    ${output} =         Execute          ozone tenant user revoke 
'tenantone$bob'
+                        Should contain   ${output}         Revoked accessId 
'tenantone$bob'.

Review comment:
       We should probably have a negative test case here too, since a bug in 
this would be bad. Try to use Bob's access ID and make sure it no longer works.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to