move code into sync_finish(), fix requester failure injection points
add provider failure injection points
yes-i-really-mean-it for force-sync
inline sync_clear_store_state()

there is 1 caller, and this matches the actions in sync_start().
not needed