Thanks to changes made in this patch, they should no longer be called.  Specifically:
- if we define the operation with the localStatus set to 'done', we won't try and run the local op
- if we define the operation with the serverStatus set to 'n/a', we won't try and run the server op

The server 'n/a' thing is new, the local trick existed already in some places.

We've been hookup the callbacks like this previously because at least one time I screwed up and didn't have one there and the call ended up being fatal, whereas here they're reasonable harmless or at least won't make things worse.  Logging sounds reasonable, but at least in the case of unit tests, because we log the specific operation variants we are running, we should already fail if we have marked the account as active during the step since we will then throw/fail if it generates log entries we don't expect.