Yeah I think this is OK.
But the original goal seems like it would be easier and better done with an immutable function which lies and calls elog to leak information. That's the actual attack this is supposed to protect against anyways.
That would make the tests more robust against other changes causing failures. Even things like changing explain output formatting for example.
--
greg
