Code review comment for lp:~rockstar/launchpad/recipe-security

We've always used transaction.commit instead of Store.flush. transaction.commit apparently has some hooks in it.

I'd be curious to know where you found tests pass that don't need transaction.commit. I'm pretty sure it's only in places where I needed to get the tests to pass.