Remove `confirmation_solicitor & election` circular dependency

[pwojcikdev]

confirmation_solicitor was accessing election variables under implicit mutex because it was called from inside one of the election methods. It was a bit hard to reason about, and prevented better encapsulation of election class. This PR cleans this part up.