Nick Campbell

I'm aware that we could use `Coach::Handler` to construct a full request handler object, but that seems like overkill in specs.

I'm not sure. Apologies, we seem to have bungled the 3.0 release a bit - I've just pushed the tag for it. Here's the diff between the versions: https://github.com/gocardless/activerecord-safer_migrations/compare/v2.0.0...v3.0.0 From...

OK so it turns out there were two separate issues here! Your first code was broken as I expected, because `statement_timeout == nil` when the `set_statement_timeout` line evaluates. The exception...

Gonna mark this issue as closed but please reopen if you encounter further problems!