Review Request #2791 — Created March 27, 2013 and submitted
Creates initial tests for misconfiguration more to be added, but I'd like to get feedback on my approach. The real meat of the patch is EndToEndTestContext which is a class which can be shared amongst the e2e tests to reduce code duplication and make things easier.
Entire patch is tests.
Revision 2 (+249 -20)
Brock - Thanks for doing this! I've some comments on the EndToEndTestContext.
I'd split the test setup and getconnection into separate methods. The test setup can be shared across many tests, but each one of the tests needs to obtain a separate connection. There are also a sufficiently large number of tests that require multiple connections i.e., tests that exercise the privilege model across different users.
Most tests require more than one user to connect. Need a way to specify the username and password from the test.
Updated based on feedback. Adds additional tests which are mostly implemented.
Revision 3 (+425 -31)