The test scenarios look fine. The main comment I have is that for positive test cases, we shouldn't call verifyAuthzException(). If this statement fails with auth error then the test should fail. This method is created for negative test where we want to verify that the expected is auth specific.
ACCESS-72 Implement test cases regarding TRANSFORM query
Review Request #2897 - Created April 18, 2013 and submitted
|arvind, brock, jarcec, prasadm, shreepadma|
test related to privilege TRANSFORM: query include TRANSFORM required privilege transform@server or all@server to work
right now test fail, looks like it doesn't work
Thanks for Prasad's comments, here is the new patch
Revision 2 (+196)
Looks good. Just one suggestion around closing connections.
This should be context.close() which will close the statement and connection. If you don't close the context then the connection and statement references will remains in the context and the cleanup could result into problems.
same as previous.