[WIP] Create function like TestOnBorrow that includes context

[MorrisLaw]

This is an initial PR to address issue #439

[stevenh]

I'm not sure this can ever be of use if I'm honest as this is setup on the pool so the ctx with a timeout, which would be the desired use, will expire and resulting in all future calls failing.

[stevenh]

Context should always be the first parameter, I would also drop the With so its just TestOnBorrowContext as per Dial and DailContext

[MorrisLaw]

the ctx with a timeout, which would be the desired use, will expire and resulting in all future calls failing.

I agree @stevenh , seems kind of pointless then... Should we just close this PR out?

[stevenh]

Closing based on timeout and other feedback.

[dcormier]

I'm not sure this can ever be of use if I'm honest as this is setup on the pool so the ctx with a timeout, which would be the desired use, will expire and resulting in all future calls failing.

But wouldn't it be called with the context.Context passed to GetContext? I haven't dug into the code, but I would assume that's when TestOnBorrow would be called, so the context used should be correct. And if the context is canceled or timed out, then it should still be correct.

[stevenh]

I take it back, @dcormier is correct, reopening.