Added Mockito dependency
Changed using containerLog to getContainer().getLogger() for unit tests to work
Added mock Container context for testing Valve behaviour
Probably not too Thread safe initial tests for length of delays
Use new instance of DelayValve for each test to avoid potential threading issues
Parameterised min, max and default delays
Added assert comments and test case for missing delay attribute
Added elapsed time value to assert comments