fix: assert is unlocked after acquiring the mutex

[mikesposito]

Explanation

The #assertIsUnlocked assertion is placed in all controller method as first line. Though for methods that mutate the controller state and update the vault (through #persistOrRollback) or that acquire the controller mutex through #withRollback, the assertion should come after the mutex has been acquired.

References

Changelog

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
• I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes