Handle GitHub errors

[jhaff]

Moved to #431

---

Description

Fixes #409

This PR consolidates error handling logic for the case when a user's account is suspended(Octokit::AccountSuspended) and when a user's account is missing from Github (GithubPullRequestService::UserNotFoundOnGithubError). This sits in a new service object which gets passed a user and a block, GithubErrorHandler.

For both of these errors, we handle by deactivating the user, putting them in the inactive state (a new state on the state machine specifically for these users no longer accounted for in GitHub) and deleting their associated UserStat, if they have one.

Test process

• Test that a user causing this error gets deactivated and out in the inactive state
• Test that that user's UserStat is deleted, if they had one
• Test that if an unhandled error is passed to the GithubErrorHandler, it raises the error

Requirements to merge

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code in hard-to-understand areas
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[jhaff]

Rebased and added tests, ready for eyes

Tests passing locally:

[johndbritton]

Looking good. Need @mkcode to approve.

Also, update the PR body description.

[johndbritton]

It might be worth storing the state the user went inactive from when we transition them.

[mkcode]

Also - we need to add the handler to the most important case - the UserTransitionJob

[mkcode]

Ok. Let's address @johndbritton's concern as well. Let's add a state_before_inactive column which logs the previous state in the same context as when we set the last_error

[jhaff]

Force push for rebase

[mkcode]

Still missed adding this error handling to the TryUserTransitionJob

[jhaff]

****PR moved to #431