Note: we previously didn't hit this because we did not use the database until the provider was actually used (after which the on_enable would have been called, meaning that init_idp had already been called a second time).
We do use the database now upon initialization when storing the scheme format, which is why we do hit it now.
LGTM